Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update VW and Data Viz api parameters #167

Open
wants to merge 3 commits into
base: devel
Choose a base branch
from

Conversation

jimright
Copy link
Contributor

@jimright jimright commented Jan 7, 2025

Following the release of version 0.9.128 of the CDP CLI and API some CDW related parameters have been renamed:

  • In Virtual Warehouse creation template has been renamed to tShirtSize. API Docs Link
  • In Data Visualisation creation templateName has been renamed to resourceTemplate. API Docs Link

This PR handles the parameter renaming in the cloudera.cloud Ansible collection. Requires cloudera-labs/cdpy#106

Signed-off-by: Jim Enright <jenright@cloudera.com>
@jimright jimright added the enhancement New feature or request label Jan 7, 2025
@jimright jimright requested a review from a team January 7, 2025 16:10
@github-actions github-actions bot added the validated The pull request is passed the prechecks label Jan 7, 2025
Signed-off-by: Jim Enright <jenright@cloudera.com>
@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 Jan 7, 2025
@@ -356,7 +358,7 @@ def main():
id=dict(type="str"),
name=dict(type="str"),
config=dict(type="dict"),
template_name=dict(type="str"),
template_name=dict(type="str", aliases=["resource_template"]),
Copy link
Member

Choose a reason for hiding this comment

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

Let's swap these -- the old name should be the alias

@@ -71,13 +71,17 @@
- Required if C(state=present)
type: str
template:
description: The name of deployment template for the Virtual Warehouse
description:
- The name of deployment T-shirt size to use.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe reference the "deployment template" as an aside ("i.e. the deployment template")

@@ -600,7 +604,11 @@ def main():
catalog_id=dict(type="str", aliases=["dbc_id"]),
type=dict(type="str"),
name=dict(type="str"),
template=dict(type="str", choices=["xsmall", "small", "medium", "large"]),
template=dict(
Copy link
Member

Choose a reason for hiding this comment

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

Also swap the two

Signed-off-by: Jim Enright <jenright@cloudera.com>
@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 Jan 9, 2025
# for free to join this conversation on GitHub. Already have an account? # 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