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 image_cve_to_work_on_push.md #227

Closed
wants to merge 1 commit into from
Closed

Create image_cve_to_work_on_push.md #227

wants to merge 1 commit into from

Conversation

arnaudclerc
Copy link

new feature proposal to granularly configure how to possibly fix CVE

new feature proposal to granularly configure how to possibl fix CVE

Signed-off-by: arnaudclerc <arnaud.clerc@epitech.eu>
@OrlinVasilev
Copy link
Member

@arnaudclerc - fantastic ! can you please add that proposal for next week agenda for community meeting if you can present it ?

@arnaudclerc
Copy link
Author

arnaudclerc commented Jun 7, 2023

@arnaudclerc - fantastic ! can you please add that proposal for next week agenda for community meeting if you can present it ?

Hello, i am not a code owner of Harbor, I don't think i have the capacity to go any further than exposing the need

@OrlinVasilev
Copy link
Member

@arnaudclerc we can add that into the list but we need resources to work on that! Thank you for contributing! @msha01 for visibility! :)

@arnaudclerc arnaudclerc closed this by deleting the head repository Jun 20, 2023
@OrlinVasilev
Copy link
Member

@arnaudclerc why did you close that one?

@arnaudclerc
Copy link
Author

in the issue discussion, we discovered that the behaviour wasn't happening when using Docker CLI, so it has been deducted that Harbor doesn't require this feature.
The feature can still be implemented but i don't see a real urgent need for it.

The bug was driven by Kaniko, I still don't know what tool's mechanisms make it so different than a regular push via Docker

#227

@OrlinVasilev
Copy link
Member

Thank you!!! @arnaudclerc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants