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

CDPCP-12181 Azure Data Lake acceptance tests #211

Merged
merged 1 commit into from
Feb 17, 2025
Merged

CDPCP-12181 Azure Data Lake acceptance tests #211

merged 1 commit into from
Feb 17, 2025

Conversation

tevesz
Copy link
Contributor

@tevesz tevesz commented Feb 13, 2025

Creates a cdp credential using the Azure subscription properties. Provisions an environment and data lake with the terraform testing framework.
The new variables are added to the e2e testing infrastructure as environment variables.

This change also includes fixes to:

  • fix flexible_server_subnet_ids issue where a resource update always reported this property as a changed property. The state should be used if the status is unknown.
  • fix encryption_user_managed_identity null value change to empty string value. Provisioning failed if the property was not set. The property is not computed, using the empty state resulted caused this issue.

@tevesz tevesz requested a review from a team as a code owner February 13, 2025 22:43
Copy link

github-actions bot commented Feb 13, 2025

Go test coverage: 35.5% for commit 2adc7b1
⚠️ 2 of 17 packages have zero coverage.
  • github.com/cloudera/terraform-provider-cdp/resources/de
  • github.com/cloudera/terraform-provider-cdp/resources/ml
View coverage for all packages
# Package Name                                                                    | Coverage
+ github.com/cloudera/terraform-provider-cdp/cdpacctest                           |    14.3%
+ github.com/cloudera/terraform-provider-cdp/provider                             |    78.7%
+ github.com/cloudera/terraform-provider-cdp/resources/datahub                    |    35.8%
+ github.com/cloudera/terraform-provider-cdp/resources/datalake                   |     9.2%
- github.com/cloudera/terraform-provider-cdp/resources/de                         |     0.0%
+ github.com/cloudera/terraform-provider-cdp/resources/dw/cluster/aws             |    71.9%
+ github.com/cloudera/terraform-provider-cdp/resources/dw/databasecatalog         |    65.8%
+ github.com/cloudera/terraform-provider-cdp/resources/dw/dataviz                 |    78.0%
+ github.com/cloudera/terraform-provider-cdp/resources/dw/virtualwarehouse/hive   |    77.7%
+ github.com/cloudera/terraform-provider-cdp/resources/dw/virtualwarehouse/impala |    70.2%
+ github.com/cloudera/terraform-provider-cdp/resources/environments               |    44.4%
+ github.com/cloudera/terraform-provider-cdp/resources/iam                        |     2.7%
- github.com/cloudera/terraform-provider-cdp/resources/ml                         |     0.0%
+ github.com/cloudera/terraform-provider-cdp/resources/opdb                       |    25.4%
+ github.com/cloudera/terraform-provider-cdp/resources/recipe                     |    12.0%
+ github.com/cloudera/terraform-provider-cdp/utils                                |    56.1%
+ github.com/cloudera/terraform-provider-cdp/utils/test                           |    75.0%

Creates a cdp credential using the Azure subscription properties.
Provisions an environment and data lake with the terraform testing
framework.
The new variables are added to the e2e testing infrastructure as
environment variables.

This change also includes fixes to:
 - fix flexible_server_subnet_ids issue where a resource update
   always reported this property as a changed property. The state should
   be used if the status is unknown.
 - fix encryption_user_managed_identity null value change to empty
   string value. Provisioning failed if the property was not set. The
   property is not computed, using the empty state resulted caused this
   issue.
@gregito gregito merged commit 9165320 into main Feb 17, 2025
14 checks passed
@gregito gregito deleted the CDPCP-12181 branch February 17, 2025 16:53
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.

2 participants