Skip to content

Conversation

@Chaffelson
Copy link
Contributor

Move list of required GCP APIs into vars
Fetch listing of enabled services and check against required, only request new services to enable if necessary to avoid error where user lacks authorization to call the enable command

Signed-off-by: Daniel Chaffelson [email protected]

Fetch listing of enabled services and check against required, only request new services to enable if necessary to avoid error where user lacks authorization to call the enable command

Signed-off-by: Daniel Chaffelson <[email protected]>
@Chaffelson Chaffelson requested a review from a team October 3, 2022 12:46
Copy link
Contributor

@jimright jimright left a comment

Choose a reason for hiding this comment

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

Changes look good. Tested by deploying GCP environment which checked the list of services and skipped enable task because all were enabled. Approved.

@jimright jimright merged commit 5a1f9eb into cloudera-labs:devel Oct 3, 2022
@wmudge wmudge added the enhancement MINOR - New feature or enhancement in the CHANGELOG label Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement MINOR - New feature or enhancement in the CHANGELOG

Development

Successfully merging this pull request may close these issues.

3 participants