ipm icon indicating copy to clipboard operation
ipm copied to clipboard

v0.7.x to v0.9.x upgrade path and storage migration

Open isc-tleavitt opened this issue 1 year ago • 6 comments
trafficstars

Fixes #454

Upgrade Path zpm 0.7.1 (once released) will be able to install zpm 0.9.0 (once released).

Storage Migration Moves ^ZPM.Dev.ModuleD to ^IPM.Storage.ModuleD

In ^IPM.Storage.ModuleD(<ID>,"Defaults",n) maps classname to equivalent - %ZPM.PackageManager.Developer.ModuleSetting -> %IPM.Storage.ModuleSetting

Migrates %ZPM.PackageManager.Developer.Document.ModuleStream to %IPM.StudioDocument.ModuleStream

For custom lifecycle classes, we're less picky and allow the legacy naming too. (I'm probably the only one using one of these anyway, in https://github.com/intersystems/isc-ipm-js )

Also incorporates the final CI fixes for #451.

isc-tleavitt avatar Apr 25 '24 20:04 isc-tleavitt

Leaving this as a draft until CI is all passing. (I don't have time to run it all locally in act today, so let's see what happens!)

isc-tleavitt avatar Apr 25 '24 20:04 isc-tleavitt

Well, that was quick. Also trying to fix the last lingering task in #451 here and that's where the CI issue is currently.

isc-tleavitt avatar Apr 25 '24 20:04 isc-tleavitt

This looks like it's working. Note, it builds on #457 so review that one first.

isc-tleavitt avatar Apr 26 '24 18:04 isc-tleavitt

@timleavitt a few comments. a few questions. I don't entirely understand the GitHub workflow syntax so haven't reviewed that. Would be good to get a second set of eyes on that bit

@isc-shuliu I'd appreciate if you could review the GitHub workflow parts of this, since you've worked with it more than anyone else reviewing. (And probably more than I have!)

isc-tleavitt avatar Apr 29 '24 12:04 isc-tleavitt

This is ready for review (again).

isc-tleavitt avatar Apr 30 '24 16:04 isc-tleavitt

Bumping this back up - needs re-review.

@isc-kiyer @isc-bzhang @isc-eneil @isc-shuliu

isc-tleavitt avatar May 13 '24 13:05 isc-tleavitt

Need to update integration tests to interoperability-enable the VERIFY namespace.

isc-tleavitt avatar May 15 '24 17:05 isc-tleavitt

Once again, I think we're good here - @isc-kiyer @isc-shuliu could you take a hopefully-final look?

isc-tleavitt avatar May 15 '24 18:05 isc-tleavitt

Looks all good to me!

shuheng-liu avatar May 15 '24 21:05 shuheng-liu

@shuheng-liu FYI, you switched profiles ;)

isc-tleavitt avatar May 16 '24 12:05 isc-tleavitt