ipm icon indicating copy to clipboard operation
ipm copied to clipboard

Need support to check proper IRIS license when installing a package

Open isc-mzhou opened this issue 2 years ago • 9 comments

As we plan to launch new supply chain product through IPM, we would like to check if the IRIS instance has the proper license for the installation. For example, IRIS instance without a proper IRIS for Supply Chain license should not be allowed to install package "isc-supply-chain". See Slide 7 in this ppt file for proposed changes in IRIS for Supply Chain license key, which can be used to verify if an IPM installation should be allowed. https://intersystemscorporation-my.sharepoint.com/:p:/g/personal/mzhou_intersystems_com/EdAvFHy0DHhNvI_ZWvE8pfQB-7fH0nzf26TGqCc1kpDEfQ?e=ZqsES8

isc-mzhou avatar Mar 17 '23 17:03 isc-mzhou

So, I suppose it just requires a specific feature flag in license. Then it can be implemented through the additional attribute in SystemRequirements tag

daimor avatar Mar 17 '23 17:03 daimor

Since this will be applicable beyond supply chain, I will leave it to IPM PM to decide the best way forward. I only provided one possible solution.

isc-mzhou avatar Mar 17 '23 17:03 isc-mzhou

I have no access to the slide, so, I just guessed

daimor avatar Mar 17 '23 17:03 daimor

Somehow I cannot find your name in sharepoint to give you access. so I am pasting the slide here: image

isc-mzhou avatar Mar 17 '23 17:03 isc-mzhou

Yeah, because I'm not part of the company

Well, using description part of the license does not look good, it should be through the Feature Flag

$system.License.GetFeature(X) https://docs.intersystems.com/irislatest/csp/documatic/%25CSP.Documatic.cls?LIBRARY=%25SYS&CLASSNAME=%25SYSTEM.License

On screenshot it shown as Extended Features

daimor avatar Mar 17 '23 18:03 daimor

Thanks for sharing the link. Looks like ideal solution is to make this work: $System.License.IsAppLicensed("isc-supply-chain") somehow. Please work with Bob on the proper solution, and supply chain will follow the same practice.

isc-mzhou avatar Mar 17 '23 18:03 isc-mzhou

I think this is a case for https://github.com/intersystems-community/zpm-registry/issues/66 rather than changes to IPM itself. cc @evshvarov and @isc-rkuszews

isc-tleavitt avatar Apr 04 '23 19:04 isc-tleavitt

I see two needs here:

  1. ZPM Server needs to know about your organization and licensed products to let you download only the right products. (As is the case with the WRC downloads and with the InterSystems Container Registry today.

  2. We might add a feature to the IPM client so that it checks the license key before attempting to install. Note that the code will need to check the license anyway, but we can produce a cleaner user experience if we check at install time

kuszewski avatar Apr 04 '23 19:04 kuszewski

@kuszewski agreed.

isc-tleavitt avatar Apr 04 '23 19:04 isc-tleavitt