-
Notifications
You must be signed in to change notification settings - Fork 193
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
[ART-7273] add modifications to support rhel-8-golang-1.17 #3356
[ART-7273] add modifications to support rhel-8-golang-1.17 #3356
Conversation
|
||
# FOD wrapper modification | ||
COPY go_wrapper.sh /tmp/go_wrapper.sh | ||
RUN /bin/bash -c 'GO_BIN_PATH=`which go`; mv $GO_BIN_PATH $GO_BIN_PATH.real; mv /tmp/go_wrapper.sh $GO_BIN_PATH; chmod +x $GO_BIN_PATH' |
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.
RUN
already is a shell. Not sure wrapping with bash -c '...'
is necessary.
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.
command updated
images/openshift-golang-builder.yml
Outdated
@@ -7,6 +7,11 @@ content: | |||
branch: | |||
target: rhel-8-golang-1.17 | |||
url: git@github.com:openshift/ocp-build-data.git | |||
modifications: | |||
- action: add | |||
source: target_go_wrapper.sh |
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.
doozer pr talks about doozerpath
?
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.
updated to doozer_source
2d8335f
to
03bd3bd
Compare
/hold cancel |
03bd3bd
to
68b37a3
Compare
images/openshift-golang-builder.yml
Outdated
@@ -25,6 +30,7 @@ name: openshift/golang-builder | |||
owners: | |||
- aos-team-art@redhat.com | |||
|
|||
# Don't add floating tag to FoD builder for now, until we confident CPASS can use it |
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.
comment is misleading
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.
removed
68b37a3
to
5b6f0ec
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: joepvd The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
9c5a8f2
into
openshift-eng:rhel-8-golang-1.17
add go wrapper during golang build process, the wrapper script will get from openshift-eng/doozer#811
rhel-8-golang-1.17 can built with FoD wrapper and don't update new builder to floating tag for now