Skip to content

Conversation

@natalieparellano
Copy link
Member

No description provided.

Signed-off-by: Natalie Arellano <narellano@vmware.com>
@natalieparellano natalieparellano requested a review from a team as a code owner October 12, 2022 19:20
@natalieparellano
Copy link
Member Author

This should probably be blocked on buildpacks/pack#1478 as CI won't pass until pack has been updated to support builder creation with extensions

@AidanDelaney
Copy link
Member

Looks like we need to release a pack with support for Dockerfiles before these samples changes can land.

@natalieparellano natalieparellano linked an issue Nov 10, 2022 that may be closed by this pull request
Signed-off-by: Natalie Arellano <narellano@vmware.com>
In order to create a builder with image extensions

Signed-off-by: Natalie Arellano <narellano@vmware.com>
The alpine builder contains image extensions, and we require the pull policy to be 'always'
(so that we can get the manifest for the builder in the registry)

Signed-off-by: Natalie Arellano <narellano@vmware.com>
@natalieparellano
Copy link
Member Author

@AidanDelaney the pipeline is green! WDYT?

@@ -1,4 +1,4 @@
PACK_FLAGS?=--pull-policy=never
PULL_POLICY_NEVER?=--pull-policy=never
Copy link
Member Author

Choose a reason for hiding this comment

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

I have to confess, I'm not sure why this is needed.

build-buildpacks-alpine: build-sample-root
@echo "> Creating 'hello-moon' app using 'alpine' builder..."
$(PACK_CMD) build sample-hello-moon-app:alpine -v --builder cnbs/sample-builder:alpine --buildpack $(SAMPLES_ROOT)/buildpacks/hello-world --buildpack $(SAMPLES_ROOT)/buildpacks/hello-moon $(PACK_FLAGS) $(PACK_BUILD_FLAGS)
$(PACK_CMD) build sample-hello-moon-app:alpine -v --builder cnbs/sample-builder:alpine --buildpack $(SAMPLES_ROOT)/buildpacks/hello-world --buildpack $(SAMPLES_ROOT)/buildpacks/hello-moon $(PACK_BUILD_FLAGS)
Copy link
Member Author

Choose a reason for hiding this comment

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

Only changed the pull policy for alpine, as that's the only builder with extensions.

Copy link
Member

Choose a reason for hiding this comment

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

AFAIK it stops pack trying to look for cnbs/sample-builder on a registry.

Copy link
Member

@AidanDelaney AidanDelaney left a comment

Choose a reason for hiding this comment

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

PR provides an example where the image is switched to run-image-curl. Also demonstrates generate installing the a tree binary via apk. LGTM.

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.

Create sample apps with dockerfiles, hooks, etc

3 participants