-
Notifications
You must be signed in to change notification settings - Fork 8
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
Remove Trustyai installation in downstream #196
Remove Trustyai installation in downstream #196
Conversation
(cherry picked from commit 0be94f0)
b731328
to
d7ccc88
Compare
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.
@VaishnaviHire I'm assuming removing the component, but keeping the alerting rules won't be an issue?
it is set to disabled, so the alerts wont be added into SRE prometheus regardless what value users set in DSC |
controllers/status/status.go
Outdated
if component == trustyai.ComponentName { | ||
condType = component + "Deprecated" | ||
} else { |
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.
think we should not set this,
eventually this status will be removed so not leave a TrustyAIDeprecated in the condition.
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.
Okay I will update this
86284b9
to
d02edc0
Compare
99116b6
into
red-hat-data-services:rhoai-2.7
Jira Issue: https://issues.redhat.com/browse/RHOAIENG-3391
Test Catalogssource: quay.io/vhire/opendatahub-operator-catalog:v2.7.0