Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix upgrade errors from v0.9.x to v0.10.x #732

Merged
merged 3 commits into from
Feb 6, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- #700: Fix a bug due to incompatible conventions between SemVer and OCI tags
- #669: Work with a wider variety of ORAS repos (removes _catalog call)
- #726: Fixed a bug where loading a tarball doesn't install dependencies from `.modules` subfolder even when it's available
- #731: Issue upgrading from v0.9.x due to refactor of repo classes

### Security
-
Expand Down
3 changes: 3 additions & 0 deletions src/cls/IPM/Repo/Http/Definition.cls
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ Parameter MONIKER As String = "http";
/// Override this in subclasses to provide more padding.
Parameter MaxDisplayTabCount As INTEGER = 4;

/// Overridden to avoid upgrade issues because of the refactor from Remote.Definition -> Http.Definition
Parameter MANAGEDEXTENT = 0;

Property URL As %IPM.DataType.RepoLocation [ Required ];

Index URL On URL [ Unique ];
Expand Down
2 changes: 1 addition & 1 deletion src/cls/IPM/Utils/Migration.cls
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ ClassMethod RunAll(verbose As %Boolean = 1) As %Status
}
Try {
Set $Namespace = $Zstrip(ns,"<>WC")
Do ..MigrateReposFromIPM09(verbose) // Do this first; MigrateZPMToIPM will fail with <FRAMESTACK> otherwise.
Do ..MigrateZPMToIPM(verbose)
Do ..MigrateReposFromIPM09(verbose)
} Catch e {
Set sc = $$$ADDSC(sc, e.AsStatus())
}
Expand Down