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

Create a separate unit test specific container #4861

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

isarkis
Copy link
Member

@isarkis isarkis commented Feb 7, 2025

b/394976646

@isarkis isarkis requested a review from a team as a code owner February 7, 2025 02:51
@isarkis isarkis requested review from yell0wd0g and removed request for yell0wd0g February 7, 2025 02:51
@isarkis isarkis marked this pull request as draft February 7, 2025 02:52
if: needs.initialize.outputs.test_on_host == 'true'
needs: [initialize]
permissions:
packages: write

Check failure

Code scanning / Scorecard

Token-Permissions High

score is 0: jobLevel 'packages' permission set to 'write'
Remediation tip: Verify which permissions are needed and consider whether you can reduce them.
Click Remediation section below for further remediation help
runs-on: [self-hosted, chrobalt-linux-runner]
steps:
- name: Checkout
uses: actions/checkout@v4

Check warning

Code scanning / Scorecard

Pinned-Dependencies Medium

score is 0: GitHub-owned GitHubAction not pinned by hash
Click Remediation section below to solve this issue
@@ -0,0 +1,19 @@
# Use the official Ubuntu image as the base
FROM debian:12

Check warning

Code scanning / Scorecard

Pinned-Dependencies Medium

score is 0: containerImage not pinned by hash
Remediation tip: pin your Docker image by updating debian:12 to debian:12@sha256:4abf773f2a570e6873259c4e3ba16de6c6268fb571fd46ec80be7c67822823b3
Click Remediation section below for further remediation help
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant