Fix/improve installation edge cases#824
Merged
isc-tleavitt merged 14 commits intomainfrom Jun 4, 2025
Merged
Conversation
Contributor
isc-tleavitt
commented
May 23, 2025
isc-tleavitt
commented
- Install now works in containers with durable %SYS without jumping through extra hoops (Version 0.10.X installation error #809)
- Install behaves properly if Flexible Python Runtime is not configured, with descriptive runtime errors on ORAS use (Installation issues with 0.10.x in an environment without flexible python runtime configured #796)
- Installing in an environment with a separate broken IRISSYS install no longer fails (Intersystems Upgrade - Reinstall IPM Issue #811)
The copy of rpds.py now succeeds (it was trying to write to a read-only directory) For consistency, just always use our own rpds.py over the real Rust thing - we don't support direct install anyway
USER>zpm "install oras/testcoverage" ERROR! Flexible Python Runtime is not configured. You must configure it and reinstall IPM to use ORAS registries. > ERROR #5002: ObjectScript error: <OBJECT DISPATCH>GetClient+2^%IPM.Repo.Oras.PackageService.1 *Failed to Load Python: Check documentation and messages.log, Check CPF parameters:[PythonRuntimeLibrary,PythonRuntimeLibraryVersion], Check sys.path setup in: $INSTANCE/lib/python/iris_site.py
isc-tleavitt
requested review from
isc-egabhart,
isc-eneil,
isc-jili,
isc-jlechtne and
isc-kiyer
These might have broken something later in CI and weren't actually part of the solution for mapping misalignment.
isc-kiyer
requested changes
May 23, 2025
Collaborator
isc-kiyer
left a comment
isc-kiyer
left a comment
There was a problem hiding this comment.
@isc-tleavitt a few comments but looks good overall!
isc-kiyer
reviewed
May 27, 2025
Hit some compilation/other issues - this partially resolves them, but still not 100%. Committing progress to get a look at CI after making these changes.
isc-kiyer
reviewed
Jun 3, 2025
This was
linked to
issues
Jun 3, 2025
Closed
Contributor
Author
|
Should be good here. Note, I wasn't able to test the negative case (just the positive case) for the version check due to a really weird upstream IRIS issue (internal ref: DP-441981) |
isc-kiyer
approved these changes
Jun 4, 2025
This was
unlinked from
issues
Jun 4, 2025
Closed
This was
linked to
issues
Jun 4, 2025
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.