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

clair: only run clair-action once and convert output #1615

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

Conversation

crozzy
Copy link

@crozzy crozzy commented Nov 18, 2024

By adding the convert command to the clair-action cli the clair-scan task can take advantage and only call the expensive reporting command once.

Before you complete this pull request ...

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

@zregvart
Copy link
Member

/ok-to-test

@crozzy
Copy link
Author

crozzy commented Nov 27, 2024

How does one go about being able to access the build-definitions-pull-request logs?

@zregvart
Copy link
Member

How does one go about being able to access the build-definitions-pull-request logs?

Unfortunately the logs are not stored anywhere publicly, you'd need cluster access to see them. I'll rebase this and have it run again and attach the logs here.

By adding the convert command to the clair-action cli the clair-scan
task can take advantage and only call the expensive reporting command
once.

Signed-off-by: crozzy <[email protected]>
@zregvart zregvart force-pushed the wip-convert-clair-output branch from e2d37d9 to 8fdbf22 Compare November 28, 2024 08:40
@zregvart
Copy link
Member

/ok-to-test

@zregvart
Copy link
Member

The logs from the e2e test don't tell much
e2e-tests.log

The span up PipelineRun had this TaskRun emit an error:
undefined.log

@crozzy
Copy link
Author

crozzy commented Dec 2, 2024

Nice, I guess that is to be expected until the clair version is updated

@crozzy
Copy link
Author

crozzy commented Dec 5, 2024

cc @p-rog

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.

2 participants