ci: gate Conan 2 install by runner.environment (fix self-hosted exit 127) - #593
Merged
Conversation
On self-hosted runners (VM905) setup-python + pip install lands conan in a user-site path the next step PATH does not see, shadowing the pre-provisioned /usr/local/bin/conan and breaking Detect Conan profile (exit 127). Mirror the existing system-deps gate: pip install only on github-hosted; on self-hosted verify the pre-provisioned conan. Covers all 10 Install Conan 2 steps across build.yml, coin-matrix.yml, codeql-analysis.yml, release.yml.
The Windows (windows-2022) Install-Conan-2 step uses a bash if/then/fi conditional but had no shell override, so it ran under the default PowerShell and died with a ParserError (exit 1). The 5 Linux/macOS Install-Conan steps default to bash and were unaffected. Pin shell: bash on the Windows step so the conditional parses. Last red leg on the keystone self-hosted-runner CI gate.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fleet-blocker: master CI red —
conan: command not found(exit 127) on self-hostedNot a code regression. The failing step is
Detect Conan profile, right afterInstall Conan 2, on the self-hosted runner (VM905). On self-hosted,setup-python+pip install conanlands conan in a user-site path the next step's PATH does not see, shadowing the pre-provisioned/usr/local/bin/conan. github-hosted jobs are unaffected. A runner restart does NOT fix this — the runners are up and conan 2.29.1 is already installed.Fix
Mirror the existing
Install system dependenciesgate (runner.environment): on github-hosted,pip installas before; on self-hosted, use the pre-provisioned conan and just verify it. Applied to all 10Install Conan 2steps:build.yml(6)coin-matrix.yml(1)codeql-analysis.yml(1) — keeps theexport PATHline in the github-hosted branchrelease.yml(2)Self-healing
This PR's own coin-matrix/CodeQL jobs run under the fix, so green CI here IS the validation.
No self-merge — flagging integrator for the merge tap once CI is green.