You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2021/09/03 05:43:29 UTC

[GitHub] [apisix-ingress-controller] gxthrj commented on a change in pull request #660: fix: add initContainers to verify if apisix is ready

gxthrj commented on a change in pull request #660:
URL: https://github.com/apache/apisix-ingress-controller/pull/660#discussion_r701599207



##########
File path: test/e2e/scaffold/ingress.go
##########
@@ -217,7 +217,7 @@ spec:
       initContainers:
       - name: wait-apisix-admin
         image: localhost:5000/busybox:1.28
-        command: ['sh', '-c', "until nc -z apisix-service-e2e-test.%s.svc.cluster.local 9080 ; do echo waiting for apisix-admin; sleep 2; done;"]
+        command: ['sh', '-c', "until nc -z apisix-service-e2e-test.%s.svc.cluster.local 9180 ; do echo waiting for apisix-admin; sleep 2; done;"]

Review comment:
       https://github.com/apache/apisix-ingress-controller/pull/660/files#diff-37e06787f8794acb42268c2dde87b4907b54af48e47fdb3b4e2648f97c9fc65aR350 the %s is replaced here.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org