Skip to content
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

ARO-5950 | feat: add initial ARO-HCP Azure attributes #927

Conversation

miguelsorianod
Copy link
Contributor

Add initial set of attributes for ARO-HCP clusters. These will be needed when creating ARO-HCP clusters.

The set of attributes in this PR is only the initial ones, that we know should be needed. More will be added over time and there is work in progress to identify and design them. Future PRs will be created for those.


// Microsoft Azure settings of a cluster.
struct Azure {
// [Required] The Azure Subscription ID associated with the cluster. It must belong to `tenant_id`.
Copy link
Contributor Author

@miguelsorianod miguelsorianod Apr 10, 2024

Choose a reason for hiding this comment

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

Unfortunately we can't configure a required OpenAPI attribute with the metamodel so we document it

@JameelB
Copy link

JameelB commented Apr 10, 2024

lgtm 👍

Copy link
Collaborator

@zgalor zgalor left a comment

Choose a reason for hiding this comment

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

lgtm

@miguelsorianod miguelsorianod changed the title [WIP] ARO-5950 | feat: add initial ARO-HCP Azure attributes ARO-5950 | feat: add initial ARO-HCP Azure attributes Apr 10, 2024
@miguelsorianod miguelsorianod merged commit 25ec4d4 into openshift-online:main Apr 10, 2024
2 checks passed
@miguelsorianod miguelsorianod deleted the add-initial-arohcp-azure-attrs branch April 10, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants