Skip to content

fix: safe default for SessionType if the CRD is not up-to-date#1120

Merged
leafty merged 1 commit into
mainfrom
leafty/fix-sessions-older-crd
May 8, 2026
Merged

fix: safe default for SessionType if the CRD is not up-to-date#1120
leafty merged 1 commit into
mainfrom
leafty/fix-sessions-older-crd

Conversation

@leafty
Copy link
Copy Markdown
Member

@leafty leafty commented May 8, 2026

This change makes amalthea work when the installed CRD is pre-job (no SessionType field).

/deploy

@leafty leafty marked this pull request as ready for review May 8, 2026 11:53
@leafty leafty requested review from a team and olevski as code owners May 8, 2026 11:53
@RenkuBot
Copy link
Copy Markdown
Contributor

RenkuBot commented May 8, 2026

You can access the deployment of this PR at https://renku-ci-am-1120.dev.renku.ch

Copy link
Copy Markdown
Member

@eikek eikek left a comment

Choose a reason for hiding this comment

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

Thank you

@leafty leafty merged commit 57ba8d9 into main May 8, 2026
48 of 54 checks passed
@leafty leafty deleted the leafty/fix-sessions-older-crd branch May 8, 2026 14:32
@RenkuBot
Copy link
Copy Markdown
Contributor

RenkuBot commented May 8, 2026

Tearing down the temporary RenkuLab deplyoment for this PR.

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