Skip to content

[build] Use golang 1.13 for builder - #7

Merged
aravindhp merged 1 commit into
openshift:masterfrom
aravindhp:dockerfile-ci-go.1.13
Feb 27, 2020
Merged

aravindhp merged 1 commit into
openshift:masterfrom
aravindhp:dockerfile-ci-go.1.13

Conversation

@aravindhp

@aravindhp aravindhp commented Feb 27, 2020

Copy link
Copy Markdown
Contributor

The operator requires golang 1.13 for building. Remove the hack that required us to $HOME/build/ as we are changing build images. Though to be accurate, the issue had nothing to do with the image but with podman-container-tools/buildah#2189.

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 27, 2020
@aravindhp

Copy link
Copy Markdown
Contributor Author

/cc @openshift/openshift-team-windows-containers

@openshift-ci-robot
openshift-ci-robot requested a review from a team February 27, 2020 20:22
@suhanime

Copy link
Copy Markdown

@aravindhp WORKDIR location is also changed from $HOME/build to /build. Please update your commit message to reflect so, otherwise lgtm

The operator requires golang 1.13 for building.
Remove the hack that required us to $HOME/build/ as we are changing
build images. Though to be accurate, the issue had nothing to do with
the image but with podman-container-tools/buildah#2189.
@aravindhp
aravindhp force-pushed the dockerfile-ci-go.1.13 branch from b412d9e to e218a26 Compare February 27, 2020 20:30
@PratikMahajan

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 27, 2020
@aravindhp

Copy link
Copy Markdown
Contributor Author

@aravindhp WORKDIR location is also changed from $HOME/build to /build. Please update your commit message to reflect so, otherwise lgtm

Fixed

@akhil-rane

Copy link
Copy Markdown
Contributor

/lgtm

@ravisantoshgudimetla ravisantoshgudimetla left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@aravindhp
aravindhp merged commit 625e2a2 into openshift:master Feb 27, 2020
@aravindhp
aravindhp deleted the dockerfile-ci-go.1.13 branch February 27, 2020 20:51
wgahnagl pushed a commit to wgahnagl/windows-machine-config-operator that referenced this pull request Mar 6, 2023
[git] modify gitignore to exclude IDE settings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants