feat(mannheim): pin thm-java-25 to pr-170 tag - #134
Open
Mtze wants to merge 1 commit into
Open
Conversation
Mannheim already offers the TH-Mannheim Java 25 image (thm-java-25) as its only app, but the image tag was passed at deploy time via ide_tag because the image is unmerged in EduIDE (#170) and no numbered release publishes it yet. Pin it in-repo with versions.ide: pr-170 so the environment renders and deploys without a manual ide_tag input. Mannheim offers only thm-java-25, so this moves that one image and nothing else. Drop the block once #170 merges and a release publishes a numbered tag. Verified: helm template renders ghcr.io/eduide/eduide/thm-java-25:pr-170, and test-deploy-logic.sh is green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012jym5phu2W2EPwmQ53ydNz
|
Warning Review limit reachedNext included review available in 26 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Pins the Mannheim environment's IDE image tag to
pr-170so the TH-Mannheim Java 25 image (thm-java-25) deploys from the current EduIDE PR build.Mannheim already registers
thm-java-25as its only app (all othersnull). Until now the tag was passed at deploy time via theide_tagdispatch input, because the image is unmerged in EduIDE#170 and no numbered release publishes it yet. This pins it in-repo instead.Why scoped this way
environments/mannheim.eduide.aet.cit.tum.de/values.yaml, so no other installation is touched.thm-java-25, soversions.idemoves that single image. The landing page, operator and REST service keep their own versions.Verification
helm templaterenders exactly one AppDefinition →ghcr.io/eduide/eduide/thm-java-25:pr-170../scripts/test-deploy-logic.sh→ ALL PASS.Follow-up
pr-170is a floating PR tag. Drop thisversions.ideblock once EduIDE#170 merges and a release publishes a numberedthm-java-25tag; the image then resolves from the chartappVersionlike every other image.🤖 Generated with Claude Code