Skip to content
This repository was archived by the owner on Jun 14, 2019. It is now read-only.

*: add azure4 as a known cluster profile #352

Merged
merged 1 commit into from
May 23, 2019

Conversation

abhinavdahiya
Copy link
Contributor

Adds azure4 as a known cluster profile. This will be the profile used for OpenShift 4 azure testing.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 23, 2019
@abhinavdahiya abhinavdahiya changed the title *: add as a known cluster profile *: add azure4 as a known cluster profile May 23, 2019
Copy link
Contributor

@stevekuznetsov stevekuznetsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 23, 2019
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhinavdahiya, stevekuznetsov

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 23, 2019
@openshift-merge-robot openshift-merge-robot merged commit fc24891 into openshift:master May 23, 2019
abhinavdahiya added a commit to abhinavdahiya/release that referenced this pull request May 30, 2019
Azure uses a separate base-domain because OpenshShift 4 Azure clusters need a DNS Zone in the same subscription as the cluster.
Therefore, the base domain logic is moved into the setup container where base domain is chosen per platform

The CLUSTER_TYPE for Azure tests is `azure4` to differentiate from pre-existing profile for other testing. PRs for ci-operator [1] and ci-operator-prowgen [2] should solidify the choice.

Also updates the test container to run Azure e2e and conformance tests.

[1]: openshift/ci-operator#352
[2]: openshift/ci-operator-prowgen#169
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants