Skip to content

Add iriscli and ipm container utility scripts#1124

Merged
isc-dchui merged 5 commits intomainfrom
container-scripts
Apr 22, 2026
Merged

Add iriscli and ipm container utility scripts#1124
isc-dchui merged 5 commits intomainfrom
container-scripts

Conversation

@isc-dchui
Copy link
Copy Markdown
Collaborator

@isc-dchui isc-dchui commented Apr 20, 2026

Description

  • Resolves OS/Container utility scripts #1110
  • Adds two container utility scripts
    • iriscli launches an interactive IRIS terminal (essentially wrapper for iris session iris), e.g.docker compose exec -it iris iriscli
    • ipm allows running of IPM commands, e.g. docker compose exec iris ipm list
  • Auto-installs the scripts in Unix/Linux to ~/bin/ and ~/.local/bin/ so they will work both inside and outside the dev containers

Testing

Ran the scripts manually from the host machine. Example:

PS C:\Users\dchui\Documents\ipm\ipm2> docker exec -it ipm2-iris-1 ipm list

Node: 122fb10eb60e, Instance: IRIS

USER>
IPM (zpm)                     0.10.7-SNAPSHOT
vscode-per-namespace-settings 1.0.0
testcoverage                  4.1.2

Checklist

  • This branch has the latest changes from the main branch rebased or merged.
  • Changelog entry added.
  • Unit (zpm test -only) and integration tests (zpm verify -only) pass.
  • Style matches the style guide in the contributing guide.
  • Documentation has been/will be updated
    • Source controlled docs, e.g. README.md, should be included in this PR and Wiki changes should be made after this PR is merged (add an extra issue for this if needed)
  • Pull request correctly renders in the "Preview" tab.

Copy link
Copy Markdown
Collaborator

@isc-kiyer isc-kiyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@isc-dchui This is great! Just the one note. Seems like for now this is just for general dev use but is this something we would consider distributing with IPM so that if it is loaded into an instance, the scripts are available? We could say it only works for unix based systems to make life a bit easier for us to maintain

Comment thread ipm Outdated
Copy link
Copy Markdown
Contributor

@isc-tleavitt isc-tleavitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed with @isc-kiyer. We should install to ~/.local/bin and need to make sure mode is correct (+x)

@isc-dchui
Copy link
Copy Markdown
Collaborator Author

isc-dchui commented Apr 20, 2026

Added the installing to both ~/bin/ and ~/.local/bin/ in the install script and removed the lines from Dockerfile to keep installation in one place. This should work both inside and outside the container. ~/.local/bin/ is Unix/Linux standard and ~/bin/ is used by containers.

Copy link
Copy Markdown
Collaborator

@isc-kiyer isc-kiyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@isc-dchui few more small notes

Comment thread preload/cls/IPM/Installer.cls Outdated
Comment thread iriscli
Comment thread iriscli Outdated
@isc-dchui isc-dchui requested a review from isc-kiyer April 21, 2026 14:09
@isc-dchui isc-dchui merged commit fb59a26 into main Apr 22, 2026
15 checks passed
@isc-dchui isc-dchui deleted the container-scripts branch April 22, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OS/Container utility scripts

3 participants