Skip to content

Commit

Permalink
Fix pipeline by not requiring alpine builder to have extensions befor…
Browse files Browse the repository at this point in the history
…e deploying

We are currently overriding the builder that was just built by using --pull-policy=always when using the builder in a build
This will be fixed in #142 but we can't merge that one until the pipeline is green

Signed-off-by: Natalie Arellano <[email protected]>
  • Loading branch information
natalieparellano committed Apr 4, 2023
1 parent 9fb3b76 commit f5b7afd
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,6 @@ build-linux-builders: build-builder-alpine build-builder-jammy
build-builder-alpine: build-linux-packages build-sample-root
@echo "> Building 'alpine' builder..."
$(PACK_CMD) builder create cnbs/sample-builder:alpine --config $(SAMPLES_ROOT)/builders/alpine/builder.toml $(PULL_POLICY_NEVER)
docker run cnbs/sample-builder:alpine ls /cnb/extensions/samples_curl
docker run cnbs/sample-builder:alpine cat /cnb/order.toml

build-builder-jammy: build-linux-packages build-sample-root
@echo "> Building 'jammy' builder..."
Expand Down Expand Up @@ -100,8 +98,7 @@ deploy-linux-packages:

deploy-linux-builders:
@echo "> Deploying 'alpine' builder..."
docker run cnbs/sample-builder:alpine ls /cnb/extensions/samples_curl
docker run cnbs/sample-builder:alpine cat /cnb/order.toml
docker run cnbs/sample-builder:alpine ls /cnb/extensions/samples_curl || true
docker push cnbs/sample-builder:alpine

@echo "> Deploying 'jammy' builder..."
Expand Down

0 comments on commit f5b7afd

Please sign in to comment.