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

[release/0.6] Backport Bug fixes, Security patches and CI/CD updates for 0.6.2 #1286

Closed

Conversation

Shubhranshu153
Copy link
Contributor

@Shubhranshu153 Shubhranshu153 commented Jun 25, 2024

Issue #, if available:

Description of changes:

Commits cherry-picked from main to release/0.6:

Testing performed:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Shubhranshu153 Shubhranshu153 requested a review from a team as a code owner June 25, 2024 16:17
turan18 and others added 4 commits June 25, 2024 10:10
Signed-off-by: Yasin Turan <[email protected]>
Signed-off-by: David Son <[email protected]>
This was done to use a smaller base image which makes us less prone to
security issues.

Additionally, this commit switches to using raw image URLs instead of
inserting in the version via a variable, so that dependabot can track
new versions.

The Dockerfile line that pulls the registry  was moved up to allow
proper tagging when building locally instead of with Docker Compose.

Signed-off-by: Austin Vazquez <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Previously, in our CLI create command, we would create zTOCs, push them
to the content store, then label the index to refer to the zTOCs.
This created a problem where a user using the containerd content store
might have their zTOCs deleted by the containerd garbage collector
before it could be labeled. While a lease was used, it was only done
during the latter step, so it did not prevent zTOCs from being deleted
before being labeled.

This commit fixes this by using a lease on the entire process via a new
command, BuildAndWriteIndex, which is called by "soci create".

Signed-off-by: David Son <[email protected]>
@austinvazquez
Copy link
Contributor

Thanks @Shubhranshu153 for the efforts here. Closing this as no longer needed as v0.7.0 has been released.

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.

6 participants