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

Create aws_eks_fargate_profile module #941

Closed
wants to merge 21 commits into from

Conversation

tjarra
Copy link
Contributor

@tjarra tjarra commented Feb 17, 2022

SUMMARY

New Module - aws_eks_fargate_profile
This create a new module to manage fargate profiles in EKS clusters.

Requires: ansible-collections/amazon.aws#651

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

aws_eks_fargate_profile

ADDITIONAL INFORMATION

Example:

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

@ansibullbot
Copy link

@tjarra this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

@ansibullbot ansibullbot added merge_commit This PR contains at least one merge commit. Please resolve! module module needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_triage new_contributor Help guide this first time contributor new_module New module new_plugin New plugin labels Feb 17, 2022
@ansibullbot
Copy link

@tjarra this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

…tjarra/community.aws into dev-eks_fargate_profile"

This reverts commit d87ecde, reversing
changes made to de60583.
@ansibullbot
Copy link

@tjarra This PR was evaluated as a potentially problematic PR for the following reasons:

  • More than 50 changed files.

Such PR can only be merged by human. Contact a Core team member to review this PR on IRC: #ansible-devel on irc.freenode.net

click here for bot help

@ansibullbot ansibullbot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Feb 18, 2022
@tjarra
Copy link
Contributor Author

tjarra commented Feb 18, 2022

I'm having problems with this PR, I'm going to close it and fix it locally.

@tjarra tjarra closed this Feb 18, 2022
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 31, 2022
New module aws_eks_fargate_profile

SUMMARY
(this is a copy of #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/amazon.aws#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 <[email protected]>
Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: None <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Jill R <None>
patchback bot pushed a commit that referenced this pull request Mar 31, 2022
New module aws_eks_fargate_profile

SUMMARY
(this is a copy of #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/amazon.aws#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 <[email protected]>
Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: None <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Jill R <None>
(cherry picked from commit 719483f)
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
)

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 ansible-collections#941 and adjust
the test-suite.
This commit temporarily disable the aurora integration tests for now.

Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
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>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
rds_cluster: split up the test target

Depends-On: ansible-collections#941
Split up the test target to avoid the parallel execution.

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Gonéri Le Bouder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge_commit This PR contains at least one merge commit. Please resolve! module module needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_triage new_contributor Help guide this first time contributor new_module New module new_plugin New plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants