-
Notifications
You must be signed in to change notification settings - Fork 345
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
Automatically configure global build strategy #723
Conversation
e71e1f3
to
10f212c
Compare
I've skipped warming phase when the operator is global and I'm not setting builder affinity with the operator. Builder and Publisher, instead are linked by affinity, because they need to share their RWO volume. Publisher can bind to operator or builder pod depending on the config (not sure if we still can use Tested in global configuration in openshift and minikube. |
I feel more and more that we must spend some time improving the integration tests, because we need to make sure that all deployment configuration work... |
Awesome work! I agree the combinatory of use cases may become unsustainable without integration tests! |
Should we update the OLM manifest now in preparation of the next release? I just fear we forget to add the cluster-wide permissions for that new installation mode. |
I'll do some tests with that. I was thinking we can add a SNAPSHOT CSV that we keep aligned with latest changes and change it to stable during release. |
I agree, that'd be the best approach. |
7f092e2
to
aa25cd7
Compare
No need to set cluster level permissions, OLM creates a clusterrole if you install it for all namespaces. Tested on OCP4 in both local and global mode ;) |
Will open another issue to refactor the different roles... |
Somehow, I hoped it would for a few seconds when I reviewed the PR, otherwise, that would have meant each operator had to define the majorant of all the required roles for all the install modes. So that is a good news. |
May be similar to #516. |
If this gets into master, I'll try to cut 0.3.4 today. |
Fix #703
This works on OpenShift. I'm checking Kubernetes.