Skip to content

Conversation

kaovilai
Copy link
Member

@kaovilai kaovilai commented Jan 24, 2023

We want to make /go/ writable for all users to resolve CI permission issue

@kaovilai kaovilai force-pushed the oadp-1.1-build-ci-dockerfile-chmod branch from a3c44f9 to 1b6a1a0 Compare January 24, 2023 23:44
@kaovilai kaovilai force-pushed the oadp-1.1-build-ci-dockerfile-chmod branch from 1b6a1a0 to d074390 Compare January 24, 2023 23:50
@kaovilai kaovilai changed the title oadp-1.1: Use go 1.19 in build/ci-Dockerfile chmod -R 777 /go/ oadp-1.1: Use go 1.18 in build/ci-Dockerfile chmod -R 777 /go/ Jan 24, 2023
@weshayutin
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 25, 2023
@kaovilai
Copy link
Member Author

/retest

@openshift-ci
Copy link

openshift-ci bot commented Jan 25, 2023

@kaovilai: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@weshayutin
Copy link
Contributor

/lgtm

@weshayutin weshayutin merged commit 2487332 into openshift:oadp-1.1 Jan 25, 2023
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants