Skip to content

Commit 732ac96

Browse files
committed
Add updates to scripts and makefile
1 parent 7977f30 commit 732ac96

File tree

3 files changed

+11
-9
lines changed

3 files changed

+11
-9
lines changed

tests/Makefile

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ build-test-runner-image: ## Build conformance test runner image
4444

4545
.PHONY: build-crossplane-image
4646
build-crossplane-image: ## Build the crossplane image
47-
docker build --platform linux/$(GOARCH) --build-arg NGINX_CONF_DIR=$(NGINX_CONF_DIR) -t nginx-crossplane:latest -f framework/crossplane/Dockerfile ..
47+
docker build --platform $(GOOS)/$(GOARCH) --build-arg NGINX_CONF_DIR=$(NGINX_CONF_DIR) -t nginx-crossplane:latest -f framework/crossplane/Dockerfile ..
4848

4949
.PHONY: run-conformance-tests
5050
run-conformance-tests: ## Run conformance tests
@@ -104,16 +104,10 @@ create-gke-router: ## Create a GKE router to allow egress traffic from private n
104104
sync-files-to-vm: ## Syncs your local NGF files with the NGF repo on the VM
105105
./scripts/sync-files-to-vm.sh
106106

107-
108-
# This is to access $(GKE_PROJECT)
109-
include scripts/vars.env
110-
export
111-
112107
.PHONY: nfr-test
113108
nfr-test: GOARCH=amd64
114109
nfr-test: check-for-plus-usage-endpoint build-crossplane-image ## Run the NFR tests on a GCP VM
115-
docker tag nginx-crossplane:latest gcr.io/$(GKE_PROJECT)/nginx-crossplane:latest
116-
docker push gcr.io/$(GKE_PROJECT)/nginx-crossplane:latest
110+
./scripts/push-crossplane-image.sh
117111
CI=$(CI) ./scripts/run-tests-gcp-vm.sh
118112

119113
.PHONY: start-longevity-test

tests/framework/resourcemanager.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -671,7 +671,7 @@ func (rm *ResourceManager) ScaleNginxDeployment(namespace, releaseName string, r
671671
ctx, cancel := context.WithTimeout(context.Background(), rm.TimeoutConfig.UpdateTimeout)
672672
defer cancel()
673673

674-
// If there is another NginxProxy which "overrides" the gateway class level one, then this won't work and
674+
// If there is another NginxProxy which "overrides" the gateway class one, then this won't work and
675675
// may need refactoring.
676676
proxy, err := rm.getGatewayClassNginxProxy(namespace, releaseName)
677677
if err != nil {
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
#!/usr/bin/env bash
2+
3+
set -eo pipefail
4+
5+
source scripts/vars.env
6+
7+
docker tag nginx-crossplane:latest gcr.io/$GKE_PROJECT/nginx-crossplane:latest
8+
docker push gcr.io/$GKE_PROJECT/nginx-crossplane:latest

0 commit comments

Comments
 (0)