UPI e2e: remove compute section in generated install-config

[vrutkovs]

This ensures masters are not labelled as workers on UPI AWS runs

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vrutkovs
To complete the pull request process, please assign tomassedovic
You can assign the PR to them by writing /assign @tomassedovic in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• ci-operator/templates/openshift/installer/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[vrutkovs]

/retest

[wking]

An alternative to this would be setting mastersSchedulable false, like this.

[vrutkovs]

/retest

[openshift-ci-robot]

@vrutkovs: The following tests failed, say /retest to rerun all failed tests:

Test name	Commit	Details	Rerun command
ci/rehearse/openshift/installer/fcos/e2e-vsphere	a02a578	link	/test pj-rehearse
ci/rehearse/openshift/installer/master/e2e-gcp-upi	a02a578	link	/test pj-rehearse
ci/rehearse/openshift/installer/master/e2e-aws-proxy	a02a578	link	/test pj-rehearse
ci/prow/pj-rehearse	a02a578	link	/test pj-rehearse

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[vrutkovs]

Okay, that worked - masters are no longer labelled as workers. However invoker test still fails (not sure if its related).

An alternative to this would be setting mastersSchedulable false, like this.

I think that would be a bit more confusing - masters would still be labelled as workers with this, but won't be schedulable?

[vrutkovs]

/cc @wking @smarterclayton

[wking]

#6741 talks about mastersSchedulable, and should have some rehearsals you can look at.

[jstuever]

Doesn't removing this from install-config.yaml cause it to just revert to default (with replicas: 3)?