You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "Baoyuantop (via GitHub)" <gi...@apache.org> on 2023/02/08 01:17:34 UTC
[GitHub] [apisix-docker] Baoyuantop opened a new pull request, #427: fix: keep both curl and wget in apisix image
Baoyuantop opened a new pull request, #427:
URL: https://github.com/apache/apisix-docker/pull/427
To fix the problem with https://github.com/apache/apisix-ingress-controller/issues/1637 we decided to replace the `wget` in the apisix image with `curl` and made the change in
- https://github.com/apache/apisix-docker/pull/419
- https://github.com/apache/apisix-docker/pull/421
But we found that this caused problems for some users of build image base apisix: why did `wget` suddenly disappear?
Just like the changes in this PR https://github.com/apache/apisix-docker/pull/423
So we'd better keep both `wget` and `curl`.
--
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
[GitHub] [apisix-docker] Baoyuantop commented on pull request #427: fix: keep both curl and wget in apisix image
Posted by "Baoyuantop (via GitHub)" <gi...@apache.org>.
Baoyuantop commented on PR #427:
URL: https://github.com/apache/apisix-docker/pull/427#issuecomment-1421919612
> CI was failed @Baoyuantop
In `all-in-one` part it is built using `apache/apisix:latest`, which cannot be used with `wget` due to previous changes. Maybe I can cancel the changes to this section first.
--
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
[GitHub] [apisix-docker] tokers commented on pull request #427: fix: keep both curl and wget in apisix image
Posted by "tokers (via GitHub)" <gi...@apache.org>.
tokers commented on PR #427:
URL: https://github.com/apache/apisix-docker/pull/427#issuecomment-1421809880
CI was failed @Baoyuantop
--
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
[GitHub] [apisix-docker] soulbird merged pull request #427: fix: keep both curl and wget in apisix image
Posted by "soulbird (via GitHub)" <gi...@apache.org>.
soulbird merged PR #427:
URL: https://github.com/apache/apisix-docker/pull/427
--
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