8000 ADR 17: Enable container autodiscovery to scan images from private repos by the-simmon · Pull Request #1374 · secureCodeBox/secureCodeBox · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@the-simmon
Copy link
Contributor

First draft, mostly brain storming, feel free to commit to this branch :) (this time with correct adr number)

@the-simmon the-simmon self-assigned this Sep 20, 2022
@the-simmon the-simmon added architecture Architecture changes auto-discovery labels Sep 20, 2022
@github-actions
Copy link
github-actions bot commented Sep 20, 2022

MegaLinter status: ✅ SUCCESS

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

Copy link
Contributor
@SebieF SebieF left a comment

Choose a reason for hiding this comment

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

Very nice work, thank you :) Only few minor comments from my side.

It might be worth to include the trivy operator and k8s scan functionality as an additional possibility? Other than that, I really like the sidecar proposal and that one is already very sophisticated thanks to your example!

@the-simmon
Copy link
Contributor Author

Very nice work, thank you :) Only few minor comments from my side.

It might be worth to include the trivy operator and k8s scan functionality as an additional possibility? Other than that, I really like the sidecar proposal and that one is already very sophisticated thanks to your example!

I don think it is possible to incorporate the trivy operator into the scb as it functions differently. I looked at the trivy docs and added the idea to use ownerReferences to delete the secret after the scan is done to this adr.

@the-simmon the-simmon marked this pull request as ready for review October 18, 2022 13:21
Copy link
Member
@Weltraumschaf Weltraumschaf left a comment

Choose a reason for hiding this comment

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

LGTM. Did some suggestions.

Weltraumschaf
Weltraumschaf previously approved these changes Dec 13, 2022
@Weltraumschaf
Copy link
Member

@the-simmon Can't rebase. You must rebase by your own.

@the-simmon the-simmon force-pushed the docs/adr17-autodiscovery-secrets branch 2 times, most recently from 7310228 to f874476 Compare December 16, 2022 12:32
Signed-off-by: Simon Hülkenberg <simon.huelkenberg@iteratec.com>
@the-simmon the-simmon force-pushed the docs/adr17-autodiscovery-secrets branch from f874476 to fc9dd48 Compare December 16, 2022 12:51
@the-simmon
Copy link
Contributor Author

@the-simmon Can't rebase. You must rebase by your own.

Done (I squashed all my commits into one because rebase did weird things because i renamed my file once)

@Weltraumschaf Weltraumschaf merged commit 61b3565 into main Dec 16, 2022
@Weltraumschaf Weltraumschaf deleted the docs/adr17-autodiscovery-secrets branch December 16, 2022 15:00
@rfelber rfelber added the enhancement New feature or request label Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

0