intersystems
Add support for OCI registries through ORAS
Introduce support for OCI registries using ORAS.
Mostly uses the ORAS Python client library via embedded Python, but augmented with ObjectScript and REST.
A docker container containing a simple OCI registry (ZOT) has been added to the docker-compose. To test out the containerized zot registry, in IPM configure an ORAS registry with http://oras:5000 as the URL.
It can then be published to and installed from like any other remote registry. Visit http://localhost:5001 to see the registry's UI.
@isc-kiyer, @isc-shuliu fixed the majority of the requested changes. The only issue is the backwards compatibility with embedded Python. Unfortunately it looks like removing all [ Language = python ] methods has not proven straightforward. For example, take this snippet of Python code
# library function that returns a dictionary
manifest = client.get_manifest(target)
for key in manifest['annotations']:
if key == "com.intersystems.ipm.module.v1+xml":
moduleXML = manifest['annotations'][key]
# remove whitespace
return "".join(moduleXML.split())
Unless there's something I'm missing (which could be very possible), iterating through the Python dictionary in Objectscript is incredibly painful. @isc-tleavitt Thoughts or suggestions?
@isc-dchui Hmm I checked and embedded python was introduced in 21.2 and the first EM release to include it was 22.1. If we want to say we don't support older versions of IRIS for IPM v1, we could include this embedded python (I am fine with that but would want @isc-tleavitt to confirm). That being said, we need to make sure we don't hit any bugs embedded python may have had in those older versions so would need to test this so am a bit conflicted.
In weekly meeting today, https://openexchange.intersystems.com/package/PyHelper came up as an option - specifically ArrayFrompyDict looks useful.
Discussed with @isc-shuliu today. Plan is:
- I've polled on DC to see if anyone really wants earlier version compatibility: https://community.intersystems.com/post/anyone-using-ipm-earlier-20221
- @isc-shuliu is going to spend some time kicking the tires on this, then we'll merge to v1-next and possibly take removal [ Language = python ] methods as a follow-up action. This will get ORAS out of a feature branch and into a branch that will also be getting related fixes that HS/TC will need.
@isc-shuliu thought on a simple approach the migration failures: you could make the new intermediate HTTP repo class [NoExtent] and match the old storage definition in the subclass.
@isc-shuliu thought on a simple approach the migration failures: you could make the new intermediate HTTP repo class [NoExtent] and match the old storage definition in the subclass.
This was silly; it's an intermediate class that already extends a class with an extent so it doesn't work. Alternative: just use the old storage def for the new intermediate class, which happens to work because all the properties moved to the intermediate class.