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

[manual backport stable-5] rds - add the engine_mode parameter (#941) #1493

Conversation

alinabuzachis
Copy link
Collaborator

rds - add the engine_mode parameter

Depends-On: #1185
Add a new engine_mode parameter to rds_cluster and also adjust the test-suite follow-up to some recent API changes:

With the aurora engine, engine_mode default now on serverless. We cannot adjust the endpoint_port of a serverless cluster. aurora clusters are now encrypted by default (storage_encrypted)

The new engine_mode parameter also allows us to re-enable the Aurora test of the rds_instance integration tests.

Reviewed-by: Gonéri Le Bouder [email protected]
Reviewed-by: Alina Buzachis

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@github-actions
Copy link

github-actions bot commented Apr 27, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/e70b6e8c2b6f424eb6e65ab9cdf8e6a6

✔️ ansible-galaxy-importer SUCCESS in 3m 42s
✔️ build-ansible-collection SUCCESS in 12m 57s
✔️ ansible-test-splitter SUCCESS in 5m 09s
✔️ integration-amazon.aws-1 SUCCESS in 15m 06s
integration-amazon.aws-2 FAILURE in 45m 08s
✔️ integration-amazon.aws-3 SUCCESS in 20m 14s
Skipped 41 jobs

@tremble tremble added this to the 5.5.0 milestone Apr 27, 2023
@alinabuzachis
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/a07b5faf028e43f9b4857e777632e642

✔️ ansible-galaxy-importer SUCCESS in 4m 06s
✔️ build-ansible-collection SUCCESS in 13m 11s
✔️ ansible-test-splitter SUCCESS in 5m 02s
✔️ integration-amazon.aws-1 SUCCESS in 16m 28s
integration-amazon.aws-2 FAILURE in 39m 22s
integration-amazon.aws-3 RETRY_LIMIT in 1m 32s
Skipped 41 jobs

@alinabuzachis
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/34a6ef8eddbc4fcc93513eca6ea87511

✔️ ansible-galaxy-importer SUCCESS in 3m 55s
✔️ build-ansible-collection SUCCESS in 13m 15s
✔️ ansible-test-splitter SUCCESS in 5m 02s
✔️ integration-amazon.aws-1 SUCCESS in 13m 39s
integration-amazon.aws-2 FAILURE in 13m 16s
integration-amazon.aws-3 FAILURE in 11m 29s
Skipped 41 jobs

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/6aa3ee3624b34aa5a3494b9c5ad0c5c7

✔️ ansible-galaxy-importer SUCCESS in 4m 02s
✔️ build-ansible-collection SUCCESS in 13m 16s
✔️ ansible-test-splitter SUCCESS in 4m 48s
✔️ integration-amazon.aws-1 SUCCESS in 18m 53s
integration-amazon.aws-2 FAILURE in 15m 19s
integration-amazon.aws-3 FAILURE in 13m 19s
Skipped 41 jobs

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/75d231125ca84d4690d115bb374133dc

✔️ ansible-galaxy-importer SUCCESS in 5m 51s
✔️ build-ansible-collection SUCCESS in 13m 04s
✔️ ansible-test-splitter SUCCESS in 4m 45s
✔️ integration-amazon.aws-1 SUCCESS in 14m 13s
✔️ integration-amazon.aws-2 SUCCESS in 38m 52s
✔️ integration-amazon.aws-3 SUCCESS in 24m 04s
Skipped 41 jobs

rds - add the engine_mode parameter

Depends-On: ansible-collections#1185
Add a new engine_mode parameter to rds_cluster and also adjust the test-suite follow-up to some recent API changes:

With the aurora engine, engine_mode default now on serverless.
We cannot adjust the endpoint_port of a serverless cluster.
aurora clusters are now encrypted by default (storage_encrypted)

The new engine_mode parameter also allows us to re-enable the  Aurora test of the rds_instance integration tests.

Reviewed-by: Gonéri Le Bouder <[email protected]>
Reviewed-by: Alina Buzachis <None>
@tremble
Copy link
Contributor

tremble commented May 3, 2023

PR looks a little weird now, as far as I can tell, some of it got pulled in with #1491

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/ef8d339fe240483d9d104762a3aac84c

✔️ ansible-galaxy-importer SUCCESS in 3m 39s
✔️ build-ansible-collection SUCCESS in 13m 06s
✔️ ansible-test-splitter SUCCESS in 5m 02s
integration-amazon.aws-1 RETRY_LIMIT in 1m 35s
✔️ integration-amazon.aws-2 SUCCESS in 25m 50s
integration-amazon.aws-3 RETRY_LIMIT in 1m 36s
Skipped 41 jobs

@alinabuzachis
Copy link
Collaborator Author

recheck

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label May 3, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/d54d281555e64570b19be28c9c9d4cd1

✔️ ansible-galaxy-importer SUCCESS in 3m 59s
✔️ build-ansible-collection SUCCESS in 13m 14s
✔️ ansible-test-splitter SUCCESS in 5m 07s
✔️ integration-amazon.aws-1 SUCCESS in 15m 00s
✔️ integration-amazon.aws-2 SUCCESS in 23m 23s
✔️ integration-amazon.aws-3 SUCCESS in 22m 20s
Skipped 41 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 50845b0 into ansible-collections:stable-5 May 3, 2023
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jul 31, 2023
Fix rds_cluster tests on stable-5 branch

SUMMARY

This fixes two problems with the rds_cluster test target. Ansible 2.16 is removing ansible.builtin.include (and has removed it in the current milestone branch), so this has been replaced with ansible.builtin.include_tasks. In addition, this updates the engine as Aurora 1 has been retired.
I'm a little confused by the history of this test target. #1493 and #1504 seem to have been merged on the same day, with one overwriting some of the changes of the other. The tests for rds_cluster on the stable-5 branch are currently broken, though, as can be seen in the failure on #1664. If there's a better way to fix this, feel free to suggest something different.

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Jill R
Reviewed-by: Alina Buzachis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants