-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix taskrun failing with duplicate unique image found #6260
fix taskrun failing with duplicate unique image found #6260
Conversation
/kind bug |
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
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.
@chengjoey we may want some unit tests to exercise that 👼🏼
cc @afrittoli @imjasonh
947f404
to
cb27814
Compare
cb27814
to
a0e8890
Compare
thanks @vdemeester , i added unknown platform test case for |
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
/assign |
/assign |
/assign |
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.
@chengjoey Thank you for fixing this issue! Just a few comments/clarifications on my end. Also, can you please update the commit message and PR description adding some context on why this change is needed and why having the runtime.GOOS and runtime.GOARCH not being known was causing a failure.
cc: @jerop
fix [issue-6257](tektoncd#6257) ignore unknown OS/architecture image entrypoint, because runtime.GOOS and runtime.GOARCH will not be unkonwn. all possible GOOS values are defined in src/go/build/syslist.go Signed-off-by: chengjoey <zchengjoey@gmail.com>
a0e8890
to
f7b0647
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
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.
Approved. Thanks @chengjoey!
@EmmaMunley Can we please get this merged? I believe only the |
|
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.
/lgtm
thank you @chengjoey
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: EmmaMunley, jerop, vdemeester The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Changes
fix issue-6257
ignore unknown OS/architecture image entrypoint, because runtime.GOOS and runtime.GOARCH will not be unkonwn. all possible GOOS values are defined in src/go/build/syslist.go
Signed-off-by: chengjoey zchengjoey@gmail.com
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes