-
Notifications
You must be signed in to change notification settings - Fork 328
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
rds - add the engine_mode parameter #941
rds - add the engine_mode parameter #941
Conversation
recheck |
e9e7be5
to
1cdb9ab
Compare
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
Follow up to a recent behaviour change, the Aurora clusters are created with `engine_mode=serverless` by default. When we try to add a new instance with `rds_instance`, we get the following errror: ``` Unable to create DB instance: An error occurred (InvalidParameterValue) when calling the CreateDBInstance operation: Instances cannot be added to Aurora Serverless clusters. ``` e.g: https://ansible.softwarefactory-project.io/zuul/build/f1f7cb085f0c4038b905a216a118c229/logs Address the problem, we will need to add the `engine_mode` parameter to `rds_cluster` through ansible-collections#941 and adjust the test-suite. This commit temporarily disable the aurora integration tests for now.
Follow up to a recent behaviour change, the Aurora clusters are created with `engine_mode=serverless` by default. When we try to add a new instance with `rds_instance`, we get the following errror: ``` Unable to create DB instance: An error occurred (InvalidParameterValue) when calling the CreateDBInstance operation: Instances cannot be added to Aurora Serverless clusters. ``` e.g: https://ansible.softwarefactory-project.io/zuul/build/f1f7cb085f0c4038b905a216a118c229/logs Address the problem, we will need to add the `engine_mode` parameter to `rds_cluster` through ansible-collections#941 and adjust the test-suite. This commit temporarily disable the aurora integration tests for now.
rds_instance: disable aurora integration tests Follow up to a recent behaviour change, the Aurora clusters are created with engine_mode=serverless by default. When we try to add a new instance with rds_instance, we get the following errror: Unable to create DB instance: An error occurred (InvalidParameterValue) when calling the CreateDBInstance operation: Instances cannot be added to Aurora Serverless clusters. e.g: https://ansible.softwarefactory-project.io/zuul/build/f1f7cb085f0c4038b905a216a118c229/logs Address the problem, we will need to add the engine_mode parameter to rds_cluster through #941 and adjust the test-suite. This commit temporarily disable the aurora integration tests for now. Reviewed-by: Alina Buzachis <None>
plugins/module_utils/rds.py
Outdated
@@ -185,8 +185,17 @@ def handle_errors(module, exception, method_name, parameters): | |||
accepted_engines = [ | |||
'aurora', 'aurora-mysql', 'aurora-postgresql' | |||
] | |||
accepted_engine_modes = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this redundant since we already validate the user inputs with argument_spec
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't remember the exact reason behind it now. I guess we can remove these now https://github.com/ansible-collections/amazon.aws/pull/941/files#diff-93f87b21085d00540e096e681c0d162bcc8575fe1c5995fc4b11623df4c6622dR191-R194 as long as we validate the specification.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With engine
, engine_mode
and engine_version
There is a large range of potential combinations. The API returns a clear message and I would prefer to just rely on this. Also, in the future AWS will probably adjust some of these constraints, if we've got a strict check system we will have to release a new collection version.
1cdb9ab
to
20b3549
Compare
recheck |
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
…ds_cluster Signed-off-by: Alina Buzachis <abuzachis@redhat.com>
2e78c41
to
220e143
Compare
rds_cluster: split up the test target Depends-On: #941 Split up the test target to avoid the parallel execution. Reviewed-by: Alina Buzachis <None> Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
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 <goneri@lebouder.net> Reviewed-by: Alina Buzachis <None>
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 <goneri@lebouder.net> Reviewed-by: Alina Buzachis <None>
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 <goneri@lebouder.net> Reviewed-by: Alina Buzachis <None>
…#1493) [manual backport stable-5] rds - add the engine_mode parameter (#941) 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 goneri@lebouder.net Reviewed-by: Alina Buzachis SUMMARY ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Mark Chappell
New module aws_eks_fargate_profile SUMMARY (this is a copy of ansible-collections#941 because i messed up my original branch) New Module - aws_eks_fargate_profile This create a new module to manage fargate profiles in EKS clusters. Requires: ansible-collections#651 ISSUE TYPE New Module Pull Request COMPONENT NAME aws_eks_fargate_profile ADDITIONAL INFORMATION community.aws.aws_eks_fargate_profile - name: create Fargate Profile aws_eks_fargate_profile: name: 'my-profile' state: present cluster_name: 'my-eks-cluster' role_arn: 'arn:aws:iam::999999999999:role/eks-FargatePodExecutionRole' subnets: - subnet-aaaa1111 selectors: - namespace: 'test_nm' labels: label1: test wait: true tags: env: test foo: bar Reviewed-by: Markus Bergholz <git@osuv.de> Reviewed-by: Mark Woolley <mw@marknet15.com> Reviewed-by: None <None> Reviewed-by: Alina Buzachis <None> Reviewed-by: Jill R <None>
New module aws_eks_fargate_profile SUMMARY (this is a copy of ansible-collections#941 because i messed up my original branch) New Module - aws_eks_fargate_profile This create a new module to manage fargate profiles in EKS clusters. Requires: ansible-collections#651 ISSUE TYPE New Module Pull Request COMPONENT NAME aws_eks_fargate_profile ADDITIONAL INFORMATION community.aws.aws_eks_fargate_profile - name: create Fargate Profile aws_eks_fargate_profile: name: 'my-profile' state: present cluster_name: 'my-eks-cluster' role_arn: 'arn:aws:iam::999999999999:role/eks-FargatePodExecutionRole' subnets: - subnet-aaaa1111 selectors: - namespace: 'test_nm' labels: label1: test wait: true tags: env: test foo: bar Reviewed-by: Markus Bergholz <git@osuv.de> Reviewed-by: Mark Woolley <mw@marknet15.com> Reviewed-by: None <None> Reviewed-by: Alina Buzachis <None> Reviewed-by: Jill R <None>
New module aws_eks_fargate_profile SUMMARY (this is a copy of ansible-collections#941 because i messed up my original branch) New Module - aws_eks_fargate_profile This create a new module to manage fargate profiles in EKS clusters. Requires: ansible-collections#651 ISSUE TYPE New Module Pull Request COMPONENT NAME aws_eks_fargate_profile ADDITIONAL INFORMATION community.aws.aws_eks_fargate_profile - name: create Fargate Profile aws_eks_fargate_profile: name: 'my-profile' state: present cluster_name: 'my-eks-cluster' role_arn: 'arn:aws:iam::999999999999:role/eks-FargatePodExecutionRole' subnets: - subnet-aaaa1111 selectors: - namespace: 'test_nm' labels: label1: test wait: true tags: env: test foo: bar Reviewed-by: Markus Bergholz <git@osuv.de> Reviewed-by: Mark Woolley <mw@marknet15.com> Reviewed-by: None <None> Reviewed-by: Alina Buzachis <None> Reviewed-by: Jill R <None>
Depends-On: #1185
Add a new
engine_mode
parameter tords_cluster
and also adjust the test-suite follow-up to some recent API changes:engine_mode
default now onserverless
.endpoint_port
of aserverless
cluster.storage_encrypted
)The new
engine_mode
parameter also allows us to re-enable the Aurora test of therds_instance
integration tests.