You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "tao12345666333 (via GitHub)" <gi...@apache.org> on 2023/03/06 07:36:54 UTC
[GitHub] [apisix-ingress-controller] tao12345666333 opened a new issue, #1710: CI: Add test case using new default ingress-class value `apisix-and-all` for Ingress Resources.
tao12345666333 opened a new issue, #1710:
URL: https://github.com/apache/apisix-ingress-controller/issues/1710
This PR is good overall.
I hope that some test cases using `apisix-and-all` can be added to the test cases of IngressClass to ensure that there will be no break change.
Of course, it can also be implemented in subsequent PRs, without blocking the merging of this PR
_Originally posted by @tao12345666333 in https://github.com/apache/apisix-ingress-controller/issues/1674#issuecomment-1453464349_
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1710: CI: Add test case using new default ingress-class value `apisix-and-all` for Ingress Resources.
Posted by "tao12345666333 (via GitHub)" <gi...@apache.org>.
tao12345666333 commented on issue #1710:
URL: https://github.com/apache/apisix-ingress-controller/issues/1710#issuecomment-1518434520
#1726 has been merged. I will close this one
--
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-ingress-controller] afzal442 commented on issue #1710: CI: Add test case using new default ingress-class value `apisix-and-all` for Ingress Resources.
Posted by "afzal442 (via GitHub)" <gi...@apache.org>.
afzal442 commented on issue #1710:
URL: https://github.com/apache/apisix-ingress-controller/issues/1710#issuecomment-1518307416
Hi @tao12345666333 you can close it, I think.
--
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-ingress-controller] tao12345666333 closed issue #1710: CI: Add test case using new default ingress-class value `apisix-and-all` for Ingress Resources.
Posted by "tao12345666333 (via GitHub)" <gi...@apache.org>.
tao12345666333 closed issue #1710: CI: Add test case using new default ingress-class value `apisix-and-all` for Ingress Resources.
URL: https://github.com/apache/apisix-ingress-controller/issues/1710
--
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