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

Grant permissions required metal-toolbox workflow #390

Conversation

yuumasato
Copy link
Member

The release and release-latest GitHub Actions are failing with invalid error:
https://github.com/ComplianceAsCode/compliance-operator/actions/runs/5895677716

@openshift-ci openshift-ci bot requested review from rhmdnd and xiaojiey August 24, 2023 10:36
@yuumasato yuumasato force-pushed the grant-metal-toolbox-workflow-permissions branch 2 times, most recently from d9f5784 to caced3a Compare August 24, 2023 10:39
Update permissions on release and release-latest workflows.
@yuumasato yuumasato force-pushed the grant-metal-toolbox-workflow-permissions branch from caced3a to 3b8560d Compare August 24, 2023 10:42
@yuumasato
Copy link
Member Author

I added a temporary trigger for pull requests, and it ran: https://github.com/ComplianceAsCode/compliance-operator/actions/runs/5962659809
But fails because it cannot push from a PR from my fork, so this needs to be merged.

@Vincent056
Copy link

/lgtm
let's merge this one

@openshift-ci
Copy link

openshift-ci bot commented Aug 25, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Vincent056, yuumasato

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Vincent056
Copy link

added qe,doc,px approval label since it is an internal change

@Vincent056
Copy link

/retest

@Vincent056
Copy link

@yuumasato e2e is not relevant here, do you have the permission to force merge, we need this PR to get latest CO image and test ComplianceAsCode/content#10464

@yuumasato
Copy link
Member Author

@Vincent056 I can override the merge requirements.

@yuumasato yuumasato merged commit cd3744f into ComplianceAsCode:master Aug 25, 2023
5 checks passed
@yuumasato yuumasato deleted the grant-metal-toolbox-workflow-permissions branch August 25, 2023 08:19
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