Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add permissions block for lock acquisition #270

Merged
merged 3 commits into from
Jan 14, 2025

Conversation

MarkAckert
Copy link
Member

Fixes build issue in ci-tests

Signed-off-by: MarkAckert <mark.ackert@broadcom.com>
Signed-off-by: MarkAckert <mark.ackert@broadcom.com>
@MarkAckert MarkAckert requested a review from pj892031 January 14, 2025 15:41
@MarkAckert
Copy link
Member Author

Test failure can be ignored for this PR - it came after the lock acquisition and will be fixed in another PR.

Signed-off-by: MarkAckert <mark.ackert@broadcom.com>
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@pj892031 pj892031 merged commit bd82a37 into v2.x/master Jan 14, 2025
6 of 7 checks passed
@pj892031 pj892031 deleted the user/markackert/update-workflow branch January 14, 2025 16:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants