lf-edge / eve

EVE is Edge Virtualization Engine
https://www.lfedge.org/projects/eve/
Apache License 2.0
470 stars 159 forks source link

zedagent: fix agent name of topics which were moved from zedrouter to msrv #4007

Closed milan-zededa closed 3 months ago

milan-zededa commented 3 months ago

AppInstMetaData, PatchEnvelopeUsage and PatchEnvelopeInfo are now published by new service msrv.

@uncleDecart Can you please re-test patch envelopes? I guess they are also broken on the master but we do not have eden test to confirm (would be great to add something similar to metadata test). Also, since the persisted topic AppInstMetaData moved from one dir to another, we will have to add procedure to upgrade-converter to re-publish any messages from /persist/status/zedrouter/AppInstMetaData/ to /persist/status/msrv/AppInstMetaData/. This is needed to preserve already published app metadata after EVE upgrade. Pavel, could you please take care of it?

milan-zededa commented 3 months ago

Smoke tests are fine now. We may get some other failures though since we reached the docker pull limit for the day.

milan-zededa commented 3 months ago

I'm assuming the upgradeconverter will be covered in a separate PR

Yes, correct.