turbonomic / kubeturbo

140 stars 75 forks source link

[TRB-45107] operator watch ns #910

Closed shayne-lin closed 1 year ago

shayne-lin commented 1 year ago

Intent

Resolves bug https://jsw.ibm.com/browse/TRB-45107

Background

Make changes to operator.yaml to make sure that the operator is only monitoring CRs in the same ns as the operator. Otherwise, this might cause conflicts when there are multiple operators watching over the same ns.

Testing

It's tested as part of the kubeturbo operator deployment test here where multiple operators are deployed simultaneously. The WATCH_NAMESPACE is already limited to the same ns as the operator.

Also tested manually by deploying kubeturbo through operator and made sure that it only monitored CRs created in the same ns and that the kubeturbo deployment is successful with these changes to the operator.

Checklist

These are the items that must be done by the developer and by reviewers before the change is ready to merge. Please strikeout any items that are not applicable, but don't delete them

Audience

(@ mention any review/... groups or people that should be aware of this merge request)