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

Proposal : Building tarred bundles without being dependent on a OCI registry #737

Merged
merged 5 commits into from
May 7, 2024

Conversation

ashpect
Copy link
Contributor

@ashpect ashpect commented Mar 20, 2024

A step in the direction for resolving the issue 1277.
For discussion, see PR 692

Copy link

netlify bot commented Mar 20, 2024

Deploy Preview for carvel canceled.

Name Link
🔨 Latest commit e22e8e2
🔍 Latest deploy log https://app.netlify.com/sites/carvel/deploys/663a613e542b7c000833ec0b

Copy link
Member

@joaopapereira joaopapereira left a comment

Choose a reason for hiding this comment

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

I think that the Proposal looks good, the only thing that I would add to it is a paragraph to enfasize that imgpkg push --to-oci-tar will only create in disk the OCI image for the bundle itself. In the future we can have the inflate flag that might help with that use case.
I also added some answers for the open questions.

After we add this I think the proposal is ready to be approved.

I am sorry for the delay to get this proposal through.

proposals/repo-to-tar/README.md Outdated Show resolved Hide resolved
proposals/repo-to-tar/README.md Outdated Show resolved Hide resolved
Signed-off-by: ashpect <[email protected]>
Copy link
Member

@joaopapereira joaopapereira left a comment

Choose a reason for hiding this comment

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

As soon as we have no more open questions I think we can change the document to Review state, send the link again to the Carvel Slack channel, wait for any other review, or the consensus period expires, and we can Approve and merge the proposal.

proposals/repo-to-tar/README.md Outdated Show resolved Hide resolved
---
title: "Building tarred bundles without being dependent on a OCI registry"
authors: [ "Ashish Kumar <[email protected]>" ]
status: "draft"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
status: "draft"
status: "review"

Copy link
Member

@joaopapereira joaopapereira left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: ashpect <[email protected]>
Copy link
Member

@joaopapereira joaopapereira left a comment

Choose a reason for hiding this comment

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

LGTM

@joaopapereira joaopapereira merged commit 64364e4 into carvel-dev:develop May 7, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants