Repository navigation
Bump versions for release - #102
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
✅ Files skipped from review due to trivial changes (2)
🚧 Files skipped from review as they are similar to previous changes (3)
📝 WalkthroughWalkthroughPinned multiple Theia Helm image tags to ChangesHelm values: image pins & landing page app
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
deployments/theia.artemis.cit.tum.de/values.yaml (1)
60-69:⚠️ Potential issue | 🟠 MajorStaging deployment values not updated in control-plane release.
The documented release process (docs/deployment-workflows.md:91–106) requires synchronized updates across staging and production for each image family. This PR updated production control-plane tags (
operator:latest-375ef32,service:latest-375ef32) but staging still uses floating:latesttags. The conversion-webhook is already synced (both atlatest-e431a13), butdeployments/theia-staging.artemis.cit.tum.de/values.yamlmust be updated with the same control-plane SHAs before merging.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@deployments/theia.artemis.cit.tum.de/values.yaml` around lines 60 - 69, Staging control-plane image tags are still floating (:latest); update the theia-staging deployment values.yaml to use the same control-plane SHAs used here by replacing image: ghcr.io/eduide/eduide-cloud/operator:latest and image: ghcr.io/eduide/eduide-cloud/service:latest with the pinned tags image: ghcr.io/eduide/eduide-cloud/operator:latest-375ef32 and image: ghcr.io/eduide/eduide-cloud/service:latest-375ef32 (the conversion-webhook is already synced), then commit the changed values file so staging and production use the same control-plane SHAs.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@deployments/theia.artemis.cit.tum.de/values.yaml`:
- Around line 60-69: Staging control-plane image tags are still floating
(:latest); update the theia-staging deployment values.yaml to use the same
control-plane SHAs used here by replacing image:
ghcr.io/eduide/eduide-cloud/operator:latest and image:
ghcr.io/eduide/eduide-cloud/service:latest with the pinned tags image:
ghcr.io/eduide/eduide-cloud/operator:latest-375ef32 and image:
ghcr.io/eduide/eduide-cloud/service:latest-375ef32 (the conversion-webhook is
already synced), then commit the changed values file so staging and production
use the same control-plane SHAs.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 802e0b60-fff1-4ada-9472-b5660e2f6054
📒 Files selected for processing (1)
deployments/theia.artemis.cit.tum.de/values.yaml
There was a problem hiding this comment.
Pull request overview
Updates the production Helm values for theia.artemis.cit.tum.de to point at newer container image tags as part of a release bump.
Changes:
- Bump
theia-cloud.operator.imageandtheia-cloud.service.imagetags. - Update preloading image tags (landing page + language images) and align
landingPage.image. - Bump
theia-appdefinitions.defaultImageTag.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Mtze
left a comment
Mtze
left a comment
There was a problem hiding this comment.
As discussed in the meeting - please use a "nice" release tag for the prod release
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ffb6f6e to
7170dbb
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ke it selectable through artemis flows
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Done in commit 64d0d5d — all prod image tags have been updated from |
64d0d5d to
8c01959
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary by CodeRabbit
New Features
Chores