Skip to content

Conversation

@lucymcnatt
Copy link
Collaborator

@lucymcnatt lucymcnatt commented Nov 19, 2025

Jira ticket: https://broadworkbench.atlassian.net/browse/CTM-142

Summary of changes

What

Why

Testing these changes

What to test

Who tested and where

  • This change is covered by automated tests
    • NB: Rerun automation tests on this PR by commenting jenkins retest or jenkins multi-test.
  • I validated this change
  • Primary reviewer validated this change
  • I validated this change in the dev environment

@lucymcnatt lucymcnatt changed the title [AN-551] New terra base image [CTM-142] New terra base image Dec 1, 2025
@lucymcnatt lucymcnatt requested a review from LizBaldo December 1, 2025 16:29
Copy link
Collaborator

@LizBaldo LizBaldo left a comment

Choose a reason for hiding this comment

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

In your testing plan, could you make sure to:

  • Test the new base image with GPUs?
  • Test that the existing images (COS and dataproc) still work with these new startup scripts?

errorDetails: RuntimeErrorDetails,
mainInstance: Option[DataprocInstance],
deleteRuntime: Boolean = true
deleteRuntime: Boolean = false
Copy link
Collaborator

Choose a reason for hiding this comment

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

You know this already, but just a ping to remove before merging :)

JUPYTER_HOME=/etc/jupyter
JUPYTER_SCRIPTS=$JUPYTER_HOME/scripts
JUPYTER_USER_HOME=$(jupyterHomeDirectory)
JUPYTER_EXTENSIONS=$JUPYTER_HOME/extensions
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you leave a comment around what is in scripts vs. extensions?

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.

3 participants