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

Distribution Specs (v3) - Builder Distribution Spec #271

Merged
merged 10 commits into from
Feb 16, 2022

Conversation

jimil749
Copy link
Contributor

This PR consists the artifact format and the delivery mechanism for Buildpacks' Builder Image.

For the most part, the builder references the buildpacks components i.e. buildpackage, build-image and the lifecycle.

Signed-off-by: Jimil Desai [email protected]

PS: Links to lifecycle and build-image are broken, I'll fix that once those PRs are merged!

@jimil749 jimil749 requested a review from a team as a code owner November 15, 2021 16:43
@jimil749 jimil749 force-pushed the distribution-builder branch from e383f7e to a6c81c4 Compare November 24, 2021 15:40
@jimil749
Copy link
Contributor Author

Resolved the merge conflict. It'd be great to have a review for this.

distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
@jimil749 jimil749 force-pushed the distribution-builder branch from bd5ab82 to f6c99de Compare December 10, 2021 07:34
@jromero jromero requested a review from sclevine December 13, 2021 13:29
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
Signed-off-by: Jimil Desai <[email protected]>
distribution.md Outdated Show resolved Hide resolved
@nebhale nebhale self-requested a review January 5, 2022 19:13
Signed-off-by: Jimil Desai <[email protected]>
@jromero jromero requested a review from jabrown85 January 7, 2022 14:55
@hone hone requested a review from ekcasey January 12, 2022 19:11
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
distribution.md Outdated Show resolved Hide resolved
@hone
Copy link
Member

hone commented Jan 19, 2022

@jromero and @ekcasey to discuss offline.

- Rename `io.buildpacks.buildpackages` => `io.buildpacks.buildpack`
- Remove `io.buildpacks.builder.metadata`.`buildpacks`
    - Duplicate data from `io.buildpacks.buildpack.layers`
- Unset `io.buildpacks.buildpack.metadata` on builder
- Update TOC

Signed-off-by: Javier Romero <[email protected]>
@jromero
Copy link
Member

jromero commented Jan 31, 2022

@ekcasey can I get a re-review?

@jromero jromero added this to the Distribution 0.3 milestone Feb 14, 2022
@jromero
Copy link
Member

jromero commented Feb 16, 2022

@buildpacks/core-team - can someone hit the merge button?

@hone hone merged commit 44d1ee8 into buildpacks:distribution/0.3 Feb 16, 2022
@hone
Copy link
Member

hone commented Feb 16, 2022

@jromero done!

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.

9 participants