Skip to content

ECO-11034 | Develop <- 3.3.1 fixes | merge to develop #103

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

Merged

Conversation

Uddipaan-Hazarika
Copy link
Contributor

@Uddipaan-Hazarika Uddipaan-Hazarika commented Nov 28, 2024

https://delphix.atlassian.net/browse/ECO-11034
This has the fix to database_name issue and vdb_groups issue. Also we aim at fixing the acceptance tests.
Below are the acceptance tests results:

uddipaan.hazarika@Uddipans-MacBook-Pro terraform-provider-delphix % make testacc
TF_ACC=1 go test $(go list ./... | grep -v 'vendor') -v -timeout 120m
? terraform-provider-delphix [no test files]
=== RUN TestProvider
--- PASS: TestProvider (0.00s)
=== RUN TestProvider_impl
--- PASS: TestProvider_impl (0.00s)
=== RUN TestDsource_create_positive
--- PASS: TestDsource_create_positive (93.91s)
=== RUN Test_source_create_positive
--- PASS: Test_source_create_positive (67.33s)
=== RUN TestAccEnvironment_positive
--- PASS: TestAccEnvironment_positive (84.74s)
=== RUN TestOracleDsource_create_positive
--- PASS: TestOracleDsource_create_positive (254.09s)
=== RUN TestAccVdb_create_vdb_group_positive
--- PASS: TestAccVdb_create_vdb_group_positive (187.13s)
=== RUN TestAccVdb_provision_positive
--- PASS: TestAccVdb_provision_positive (213.90s)
=== RUN TestAccVdb_bookmark_provision
--- PASS: TestAccVdb_bookmark_provision (465.91s)
=== RUN TestAccVdb_appdata_provision
--- PASS: TestAccVdb_appdata_provision (52.78s)
PASS
ok terraform-provider-delphix/internal/provider 1421.055s
uddipaan.hazarika@Uddipans-MacBook-Pro terraform-provider-delphix % date
Tue Nov 26 19:03:05 IST 2024

@elima-sugunan elima-sugunan merged commit eea289a into delphix-integrations:develop Nov 29, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants