-
Notifications
You must be signed in to change notification settings - Fork 100
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
Added ManageIQ Status with conditions, versions and ingress endpoint info #973
Conversation
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.
Apparently my comments from last week were not submitted... trying again
@Fryguy Can you also take a look at this? |
Overall LGTM, though I admit I'm not a go person. I agree with @bdunne's comments otherwise. |
@@ -166,6 +247,90 @@ func (r *ManageIQReconciler) SetupWithManager(mgr ctrl.Manager) error { | |||
|
|||
var logger = log.Log.WithName("controller_manageiq") | |||
|
|||
func FindDeployment(cr *miqv1alpha1.ManageIQ, client client.Client, name string) *appsv1.Deployment { |
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.
In a followup, do you think we should lower case these functions? I think they can be private.
Checked commits sudhir-kelkar/manageiq-pods@2ba13d9~...a9fa7ae with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint **
|
With this status implementation we are adding status conditions, ingress endpoint info and operator/operand version to the CR status.
Fixes
#https://github.ibm.com/katamari/dev-issue-tracking/issues/23259