-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Add support for multi-arch releaseTar URL #41483
Conversation
common/scripts/report_build_info.sh
Outdated
@@ -44,3 +44,4 @@ echo "istio.io/pkg/version.buildGitRevision=${BUILD_GIT_REVISION}" | |||
echo "istio.io/pkg/version.buildStatus=${tree_status}" | |||
echo "istio.io/pkg/version.buildTag=${GIT_DESCRIBE_TAG}" | |||
echo "istio.io/pkg/version.buildHub=${HUB}" | |||
echo "istio.io/pkg/version.buildArch=${TARGET_ARCH}" |
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.
Files in the common
directory come from istio/common-files. This change would need to be made in that repo and when committed, it will propagate to the other repos like here.
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.
@ericvn Thanks
Mergemain
d182d9f
to
5187711
Compare
5187711
to
05c5359
Compare
/test lint |
@istio/wg-test-and-release-maintainers PTAL. Thanks |
Please provide a description of this PR:
The path shouldn't be fixed
amd64
anymore.The result on arm is like:
Please check any characteristics that apply to this pull request.