@liulanzheng not quite sure why the CI would use go 1.19 (as I did upgrade to 1.22)?
Is the env GO_VERSION set outside of the action files?

Thanks.

Rebased against latest main.

e2e still failing as they rely on go 1.19 apparently.

@yuchen0cc It seems that the CI was using the script from the main branch instead of the script modified in the PR. How should this situation be handled?

@apostasie After confirming with @yuchen0cc, to prevent potential attacks from CI, the CI runs the code from the target branch. Therefore, we have to merge the code first before running the new CI. Additionally, could we merge the zdfs first before merging this branch?

@apostasie After confirming with @yuchen0cc, to prevent potential attacks from CI, the CI runs the code from the target branch. Therefore, we have to merge the code first before running the new CI. Additionally, could we merge the zdfs first before merging this branch?

Thanks @liulanzheng !

For sure - we do need the zdfs PR merged first.
Akihiro just pinged someone over there, so, let's give it some time for them to react?

If you want, I can ping you back here once it is done and this one is updated to reflect that.

Hey @liulanzheng

zdfs PR got merged last night.

I just updated this here so that we are back on zdfs main instead of my fork (thanks to @AkihiroSuda help).

I guess this here is ready for review (or merge on a feature branch?)

Let me know if you need anything else here from me.

Thanks a lot!

All CI passed

Thanks a lot @liulanzheng !

Appreciate the help.