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

feat(admin)!: kustomizeable configmap for scan job default configuration #110

Merged
merged 1 commit into from
Jan 24, 2023

Conversation

erikgb
Copy link
Member

@erikgb erikgb commented Jan 20, 2023

Close #109

BREAKING CHANGE: This moves the configuration of Trivy server from the operator configmap to the new configmap for scan jobs. Any admins overriding the default value will is required to update their configuration.

@erikgb erikgb changed the title feat(admin): kustomizeable configmap for scan job default configuration feat(admin)!: kustomizeable configmap for scan job default configuration Jan 20, 2023
@erikgb erikgb force-pushed the scan-job-configmap branch from 22d7d0b to c9caa1e Compare January 20, 2023 11:56
@erikgb erikgb marked this pull request as ready for review January 20, 2023 11:56
@erikgb erikgb requested a review from a team as a code owner January 20, 2023 11:56
@erikgb erikgb force-pushed the scan-job-configmap branch 4 times, most recently from 5faf819 to baca084 Compare January 23, 2023 07:57
internal/trivy/scan_job.go Outdated Show resolved Hide resolved
bendikp
bendikp previously approved these changes Jan 23, 2023
internal/trivy/scan_job.go Outdated Show resolved Hide resolved
@erikgb erikgb force-pushed the scan-job-configmap branch 2 times, most recently from 18dde21 to 30918e2 Compare January 24, 2023 09:19
Copy link
Member

@bendikp bendikp left a comment

Choose a reason for hiding this comment

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

lgtm

@erikgb erikgb force-pushed the scan-job-configmap branch from 30918e2 to 14e22df Compare January 24, 2023 09:37
@erikgb erikgb enabled auto-merge (squash) January 24, 2023 09:37
@erikgb erikgb merged commit d51e2a9 into statnett:main Jan 24, 2023
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.

Use a configmap for scan job (default) configuration
2 participants