-
Notifications
You must be signed in to change notification settings - Fork 53
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
[resolution pt.7] resolver integration #101
[resolution pt.7] resolver integration #101
Conversation
3c860df
to
36c8a56
Compare
36c8a56
to
3e231aa
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.
Again, great work @perdasilva!
for _, operator := range operatorList.Items { | ||
apimeta.SetStatusCondition(&operator.Status.Conditions, metav1.Condition{ | ||
Type: operatorsv1alpha1.TypeReady, | ||
Status: status, | ||
Reason: reason, | ||
Message: message, | ||
ObservedGeneration: op.GetGeneration(), | ||
}) | ||
if varID, ok := packageVariableIDMap[operator.Spec.PackageName]; ok { | ||
operator.Status.BundlePath = varID | ||
} | ||
if err := r.Client.Status().Update(ctx, &operator); err != nil { | ||
return ctrl.Result{}, err | ||
} | ||
} |
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.
We may want to consider if we should update each of the operator CR's status here or if we should just retrigger the reconciler for each of them.
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.
I'll follow this up with another PR where I update deppy and how we handle the Solution
struct changes
Signed-off-by: perdasilva <perdasilva@redhat.com>
Signed-off-by: perdasilva <perdasilva@redhat.com>
3e231aa
to
f75647a
Compare
This PR integrates the solver with the reconciler. It also update the Operator API to include a
BundlePath
on Operator.Status