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
Operator deployment #3
Operator deployment #3
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.
Should the directory be called manifests or install? It's install in MAO and CAO 🤔
labels: | ||
k8s-app: cloud-manager-operator | ||
spec: | ||
priorityClassName: system-node-critical |
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.
What are the options here? Does this mean this component is critical if we want nodes to work correctly?
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 feel like manifests is a better variant from those two. It is used mostly anywhere: https://github.com/openshift/cluster-kube-controller-manager-operator/tree/master/manifests |
So long as the manifests can be picked up in the release payload I'm happy |
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Fedosin 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 |
/test vendor |
/refresh |
/retest |
Manifest for image referencing and operator deployment, which will be later populated by CI.
To test image in production here is a temporary operator image location
quay.io/dgrigore/cluster-cloud-controller-manager-operator:latest
.