kezhenxu94 commented on pull request #7220: URL: https://github.com/apache/skywalking/pull/7220#issuecomment-872788005
> > > @wu-sheng @kezhenxu94 I noticed that Istio+ALS e2e needs skywalking-kubernetes to be updated within [apache/skywalking-kubernetes#74](https://github.com/apache/skywalking-kubernetes/pull/74), but it need this PR merged first. :-P > > > > > > Can we merge [apache/skywalking-kubernetes#74](https://github.com/apache/skywalking-kubernetes/pull/74) first? > > If skywalking-kubernetes does not dependency this PR's docker image, I think we could merge [apache/skywalking-kubernetes#74](https://github.com/apache/skywalking-kubernetes/pull/74) first. It doesn't, you can merge that first, and update https://github.com/apache/skywalking/blob/04bb6673b8e7d0df07ba6fdd09ba78ff3ac89148/.github/workflows/e2e.istio.yaml#L98 in this PR, after this PR gets merged, then update skywalking-kubernets https://github.com/apache/skywalking-kubernetes/blob/49f9e3cef4e995dde9b77fb3b24876dd2d9c6c12/.github/workflows/e2e.ci.yaml#L46 to verify -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
