Define instance type for logging

[jcantrill]

This PR specifies the instance type for worker nodes when running logging jobs.

Depends on #14350

cc @wking

/assign @periklis

[openshift-merge-robot]

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

Test name	Commit	Details	Rerun command
ci/rehearse/openshift/cluster-logging-operator/master/e2e-operator	2e23502	link	/test pj-rehearse

Full PR test history. Your PR dashboard.

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.

[alanconway]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alanconway, jcantrill

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• ci-operator/config/openshift/cluster-logging-operator/OWNERS [alanconway,jcantrill]
• ci-operator/config/openshift/elasticsearch-operator/OWNERS [alanconway,jcantrill]
• ci-operator/config/openshift/elasticsearch-proxy/OWNERS [alanconway,jcantrill]
• ci-operator/config/openshift/origin-aggregated-logging/OWNERS [alanconway,jcantrill]

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

[openshift-ci-robot]

@jcantrill: Updated the ci-operator-master-configs configmap in namespace ci at cluster app.ci using the following files:

• key openshift-cluster-logging-operator-master.yaml using file ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-master.yaml
• key openshift-elasticsearch-operator-master.yaml using file ci-operator/config/openshift/elasticsearch-operator/openshift-elasticsearch-operator-master.yaml
• key openshift-elasticsearch-proxy-master.yaml using file ci-operator/config/openshift/elasticsearch-proxy/openshift-elasticsearch-proxy-master.yaml
• key openshift-origin-aggregated-logging-master.yaml using file ci-operator/config/openshift/origin-aggregated-logging/openshift-origin-aggregated-logging-master.yaml

Details

In response to this:

This PR specifies the instance type for worker nodes when running logging jobs.

Depends on #14350

cc @wking

/assign @periklis

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.

[wking]

this PR merged before many of the rehearsals had completed, but I'd expect you'd need to also set some other environment variables to avoid clearing them. See #14184 for similar recovery. I have an internal ticket open with DPTP about rendering convenient docs for available environment variables here and similar. Anyhow, if your CI is doing what you want, you can ignore this. But if the jobs start behaving strangely, this may be why (and next time you may want to wait out the rehearsals and review their success before merging the release PR ;).

[jcantrill]

this PR merged before many of the rehearsals had completed, but I'd expect you'd need to also set some other environment variables to avoid clearing them.

Do we have any idea what might need to be set or is this simply trial and error?