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

tickets/DM-44731: two pythons #176

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

tickets/DM-44731: two pythons #176

wants to merge 8 commits into from

Conversation

athornton
Copy link
Member

Base stack container on python3.12 and get the stack from lsstinstall. Run user terminals and the LSST kernel in the DM Stack context, but Jupyterlab in its own venv (probably overkill, we could put all this stuff in the system python, but, you know, why not do it cleanly (well, an answer would be, we can't, apparently, hide the venv python the way we can the system one...) ?)

Also: refactor a lot of the build and runtime process.

In general, Phalanx-but-not-DM-specific things are moving under /usr/local/share/jupyterlab rather than /opt/lsst/software. Then we have to add a compatibility layer so we can spawn old and new Lab containers from the same Nublado, but we could eventually use that when existing ones age out.

We almost certainly can pare down the Jupyterlab Python environment farther, and I may have overdone it with the moving to /usr/local/share/jupyterlab because some of these things, like the custom logging addenda in 20-logging.py are really RSP-specific.

There's actually quite a lot more to think about here in terms of organization. For example, the Hub menu interaction that goes through Gafaelfawr as well is generic to Phalanx (but not generic-to-world). But the "Rubin" firefly menu thing is obviously RSP-specific, as is displaying the image tag in the bottom frame. So we should refactor the RSP Jupyter extensions into Phalanx Jupyter extensions and RSP-specific extensions, and stuff like that.

Anyway, it's a start, and it generates containers that run.

@athornton athornton force-pushed the tickets/DM-44731 branch 5 times, most recently from 7b630db to a7336a9 Compare August 9, 2024 00:03
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.

1 participant