-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dump packages and platforms env variables #23994
Dump packages and platforms env variables #23994
Conversation
Jenkinsfile
Outdated
withEnv(["HOME=${env.WORKSPACE}",'PACKAGES=docker']) { | ||
retryWithSleep(retries: 2, seconds: 5){ sh(label: "Install Go ${env.GO_VERSION}", script: '.ci/scripts/install-go.sh') } | ||
sh(script: 'make release-manager-snapshot') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only for testing purposes
PLATFORMS = EnvOr("PLATFORMS", "") | ||
PACKAGES = EnvOr("PACKAGES", "") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, no side effects.
This reverts commit 07798ba.
❕ Build Aborted
Expand to view the summary
Build stats
Trends 🧪Log outputExpand to view the last 100 lines of log output
|
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
What does this PR do?
Expose the environment context about the platforms and packages
Why is it important?
to help with the debugging in the release process that happens with another tooling/environment/context
Tests
Given the 07798ba the output was