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
[release-4.14] OCPBUGS-26548: Adds CloudConfigTransformer for Azure #321
[release-4.14] OCPBUGS-26548: Adds CloudConfigTransformer for Azure #321
Conversation
@theobarberbany: This pull request references Jira Issue OCPBUGS-26548, which is valid. The bug has been moved to the POST state. 6 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. In response to this: 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 openshift-eng/jira-lifecycle-plugin repository. |
c0e665f
to
88dfecb
Compare
4511952
to
42bad40
Compare
This change adds a boolean return value to the GetCloudConfigTransformer function to indicate when a the cloud config ConfigMap should be looked up from the CCO namespace. It is being added so that we can return a config transformer and also be able to lookup the default values from the CCO. This is primarily to address OCPBUGS-20213 which requires an updated config on Azure Stack Hub.
This change creates a CloudConfigTransformer for Azure Stack cloud provider. The transformer will unmarshal json from the cloud config into a `Config` struct from the azure cloud provider and then update the the `VMType` field if it is unset. This is being added because Azure has updated the default vm type to value that cannot be added to the basic load balancers which are associated with Azure Stack. The change also adds a test suite to accompany the transformer.
This adds a CloudConfigTransformer for Azure that will set the VMType to 'standard' if unset. See openshift#291, OCPBUGS-25483 and OCPBUGS-20213 for more information. This change also adds the functionality from the cluster-config-operator azure CloudConfigTransformer. The functionality we bring over is: 1. Ensure that the Cloud is set in the cloud.conf i. If it is set, verify that it is valid and does not conflict with the infrastructure config. If it conflicts, we want to error ii. If it is not set, default to public cloud (configv1.AzurePublicCloud) 2. Verify the cloud name set in the infra config is valid, if it is not bail with an informative error See openshift/cluster-config-operator#133 for the PR that initially added this functionality.
42bad40
to
ebb9993
Compare
/retest |
1 similar comment
/retest |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed 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 |
/label cherry-pick-approved |
/override ci/prow/e2e-alibaba-ovn |
@sdodson: Overrode contexts on behalf of sdodson: ci/prow/e2e-alibaba-ovn In response to this:
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. |
@theobarberbany: The following tests failed, say
Full PR test history. Your PR dashboard. 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. |
33a706e
into
openshift:release-4.14
@theobarberbany: Jira Issue OCPBUGS-26548: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-26548 has been moved to the MODIFIED state. In response to this: 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 openshift-eng/jira-lifecycle-plugin repository. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-cluster-cloud-controller-manager-operator-container-v4.14.0-202401122251.p0.g33a706e.assembly.stream for distgit ose-cluster-cloud-controller-manager-operator. |
Fix included in accepted release 4.14.0-0.nightly-2024-01-13-053216 |
No description provided.