-
Notifications
You must be signed in to change notification settings - Fork 32
Pre release fix script for RHDH 1.7 #1107
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
Pre release fix script for RHDH 1.7 #1107
Conversation
Signed-off-by: Josef Kopriva <[email protected]>
|
WalkthroughThe Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant pre-release.sh
participant SubscriptionConfig
participant YAMLFile
User->>pre-release.sh: Run pre-release script
pre-release.sh->>SubscriptionConfig: configure_rhdh("developerHub", "fast-1.7")
pre-release.sh->>SubscriptionConfig: configure_subscription("installer/charts/tssc-subscriptions/values.yaml")
SubscriptionConfig->>YAMLFile: Apply changes using updated values
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5-10 minutes Possibly related PRs
Suggested labels
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📚 Learning: for the rhtap-cli repository, chart version bumps in helm charts are handled at release time, not wh...
Applied to files:
📚 Learning: for the rhtap-cli project, chart versions in helm charts are updated during release time rather than...
Applied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
🔇 Additional comments (1)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jkopriva, Roming22 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
2742438
into
redhat-appstudio:main
Summary by CodeRabbit