Xiaopeng Han

Results 138 comments of Xiaopeng Han

@therealmitchconnors I think some of the analyzers/other messages may have the same issue. Instead of analyzing pods' workloads, is it feasible to create workloads' associated pods from the template at...

@therealmitchconnors I just tested old release branches. It looks like the problem is only related to api versions. Old releases only analyze `v1alpha3` so no messages are reported. If changing...

> @hanxiaop would be great to land this - shouldnt be too hard to fix up > > /wait @phlax Sorry I was too busy last week, I'm going to...

> @jmarantz Sorry for that. Thanks for the reminder!

> i can pick this one up again > > /retest @phlax I've tried different ways but still doesn't work. Could you please take a look? Thanks!

Force pushed since needs to fix DCO sign-off.

/test integ-pilot-istiodremote-tests_istio

https://github.com/istio/istio/blob/d2b96bb9572875fee561c7c34e1e42f58e04c88c/pilot/pkg/config/memory/store.go#L204-L212 Looked through the code, the error is due to the conflict in update. Set the newer config `ResourceVersion` to the annotation part can avoid of `ResourceVersion` conflicts.