Skip to content

[CFX-5788] Add MinimumVersion for Prerequisites#567

Merged
taras-pokornyy merged 2 commits into
datarobot-oss:mainfrom
taras-pokornyy:CFX-5788_MinimumVersions_for_Prerequisites
Jun 17, 2026
Merged

[CFX-5788] Add MinimumVersion for Prerequisites#567
taras-pokornyy merged 2 commits into
datarobot-oss:mainfrom
taras-pokornyy:CFX-5788_MinimumVersions_for_Prerequisites

Conversation

@taras-pokornyy

@taras-pokornyy taras-pokornyy commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

RATIONALE

Add MinimumVersion for Prerequisites and --version/version to each command

CHANGES

PR Automation

Comment-Commands: Trigger CI by commenting on the PR:

  • /trigger-smoke-test or /trigger-test-smoke - Run smoke tests
  • /trigger-install-test or /trigger-test-install - Run installation tests

Labels: Apply labels to trigger workflows:

  • run-smoke-tests or go - Run smoke tests on demand (only works for non-forked PRs)

Important

For Forked PRs: The run-smoke-tests label won't work. A required Smoke Tests check will block merge until a maintainer acts:

  • A maintainer uses /approve-smoke-tests to run smoke tests (results will set the check)
  • A maintainer uses /skip-smoke-tests to bypass the check without running tests

Please comment requesting a maintainer review if you need smoke tests to run.


Note

Low Risk
Only changes default prerequisite metadata used by local dependency checks; no runtime or security-critical logic is modified in this diff.

Overview
Default quickstart prerequisites now declare minimum versions and use version-printing commands instead of bare binary names in RequiredTools.

Python (3.9.6), uv (0.11.20), task (3.50.0), and pulumi (3.245.0) each get a MinimumVersion and a Command like python3 --version or pulumi version, so the existing CheckPrerequisites / isVersionInstalled path can reject installs that are present but too old when repo YAML does not override requirements.

Reviewed by Cursor Bugbot for commit 6da13f9. Configure here.

@taras-pokornyy taras-pokornyy self-assigned this Jun 11, 2026
@taras-pokornyy
taras-pokornyy requested a review from a team June 15, 2026 11:43
@ajalon1

ajalon1 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

/approve-smoke-tests

@github-actions

Copy link
Copy Markdown
Contributor

🔐 Fork PR smoke tests triggered by @ajalon1

⚠️ Security Notice: This will run tests with access to repository secrets.

What happens next:

  1. Security scans will run automatically (Trivy, gosec)
  2. If security scans pass, smoke tests will run
  3. Results will be posted as PR comments

⚠️ Important: Review the PR code carefully before approving!

@github-actions

Copy link
Copy Markdown
Contributor

🔐 Fork smoke tests started by maintainer

⏳ Security scans passed. Running smoke tests...

Commit: d91b87d1642874dac92d5680f5ddab41dad57a36
View run

@github-actions

Copy link
Copy Markdown
Contributor

All smoke tests passed! (Fork PR)

✅ Security Scan: success
✅ Linux: success
✅ Windows: success

View run details

@taras-pokornyy
taras-pokornyy merged commit 18f3823 into datarobot-oss:main Jun 17, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants