Skip to content

ADR 17: Enable container autodiscovery to scan images from private repos - #1374

Merged
Weltraumschaf merged 1 commit into
mainfrom
docs/adr17-autodiscovery-secrets
Dec 16, 2022
Merged

ADR 17: Enable container autodiscovery to scan images from private repos#1374
Weltraumschaf merged 1 commit into
mainfrom
docs/adr17-autodiscovery-secrets

Conversation

@the-simmon

Copy link
Copy Markdown
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

github-actions Bot commented Sep 20, 2022

Copy link
Copy Markdown

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

@SebieF SebieF left a comment

Copy link
Copy Markdown
Contributor

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!

Comment thread docs/adr/adr_0017.md Outdated
Comment thread docs/adr/adr_0017.md Outdated
Comment thread docs/adr/adr_0017.md Outdated
Comment thread docs/adr/adr_0017.md Outdated
@the-simmon

Copy link
Copy Markdown
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

@Weltraumschaf Weltraumschaf left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Did some suggestions.

Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Comment thread docs/adr/adr_0017.md
Weltraumschaf
Weltraumschaf previously approved these changes Dec 13, 2022
@Weltraumschaf

Copy link
Copy Markdown
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
Copy Markdown
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.

5 participants