Skip to content
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

job for openshift CI #197

Merged
merged 1 commit into from
Feb 20, 2024
Merged

job for openshift CI #197

merged 1 commit into from
Feb 20, 2024

Conversation

cooktheryan
Copy link
Collaborator

This job is needed for OpenShift CI to be able to push an image before doing cosign

Signed-off-by: Ryan Cook <[email protected]>
Copy link
Contributor

@Gregory-Pereira Gregory-Pereira left a comment

Choose a reason for hiding this comment

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

/lgtm

Not sure why we need privileged SC but im sure you have your reasons. Also not sure what is doing the signing here, does the openshift CI cluster have tekton chains configured?

Copy link

openshift-ci bot commented Feb 20, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cooktheryan, Gregory-Pereira

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:
  • OWNERS [Gregory-Pereira,cooktheryan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cooktheryan
Copy link
Collaborator Author

cooktheryan commented Feb 20, 2024

@Gregory-Pereira without priv we hit an error

➜  release git:(testing_securesign) kubectl logs -f trusted-artifact-signer-test-sign-verify-p94t5
Error: error writing "0 0 4294967295\n" to /proc/16/uid_map: write /proc/16/uid_map: operation not permitted
time="2024-02-20T16:29:09Z" level=error msg="writing \"0 0 4294967295\\n\" to /proc/16/uid_map: write /proc/16/uid_map: operation not permitted"
time="2024-02-20T16:29:09Z" level=error msg="(Unable to determine exit status)"
Error: error writing "0 0 4294967295\n" to /proc/26/uid_map: write /proc/26/uid_map: operation not permitted
time="2024-02-20T16:29:09Z" level=error msg="writing \"0 0 4294967295\\n\" to /proc/26/uid_map: write /proc/26/uid_map: operation not permitted"
time="2024-02-20T16:29:09Z" level=error msg="(Unable to determine exit status)"
time="2024-02-20T16:29:09Z" level=error msg="writing \"0 0 4294967295\\n\" to /proc/34/uid_map: write /proc/34/uid_map: operation not permitted"
Error: error writing "0 0 4294967295\n" to /proc/34/uid_map: write /proc/34/uid_map: operation not permitted
time="2024-02-20T16:29:09Z" level=error msg="(Unable to determine exit status)"

No tekton chains currently installed in the testing. It would be a good additional test but requires some more work

@cooktheryan cooktheryan merged commit 81015a8 into main Feb 20, 2024
4 of 7 checks passed
@cooktheryan cooktheryan deleted the ci_job branch February 20, 2024 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants