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

Add extra Azure parameters to cloudera.cloud.dw_cluster #140

Merged
merged 3 commits into from
May 9, 2024

Conversation

jimright
Copy link
Contributor

@jimright jimright commented Mar 15, 2024

@jimright jimright added the enhancement New feature or request label Mar 15, 2024
@jimright jimright requested a review from a team March 15, 2024 15:21
@github-actions github-actions bot added the validated The pull request is passed the prechecks label Mar 15, 2024
@@ -337,6 +392,14 @@ def main():
private_load_balancer=dict(type='bool', default=False),
az_subnet=dict(type='str'),
az_enable_az=dict(type='bool'),
az_managed_identity=dict(type='str'),
Copy link
Member

Choose a reason for hiding this comment

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

With so many Azure-related options, might want to revisit nesting them within a key, like azure?

@jimright jimright force-pushed the feature/azure_dw_extra_params branch from 3876678 to db9137a Compare May 8, 2024 09:34
@github-actions github-actions bot added validated The pull request is passed the prechecks and removed validated The pull request is passed the prechecks labels May 8, 2024
Signed-off-by: Jim Enright <[email protected]>
@github-actions github-actions bot added validated The pull request is passed the prechecks and removed validated The pull request is passed the prechecks labels May 8, 2024
@github-actions github-actions bot added validated The pull request is passed the prechecks and removed validated The pull request is passed the prechecks labels May 9, 2024
@wmudge wmudge merged commit 46c3a99 into cloudera-labs:devel May 9, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request validated The pull request is passed the prechecks
Development

Successfully merging this pull request may close these issues.

2 participants