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

[PR #1589/614b792b backport][stable-6] cloudwatchevent_rule should return false when there is no change done to the rule #1640

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 4, 2023

This is a backport of PR #1589 as merged into main (614b792).

SUMMARY

Fixes #1080

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugins/modules/cloudwatchevent_rule.py

ADDITIONAL INFORMATION

… to the rule (#1589)

cloudwatchevent_rule should return false when there is no change done to the rule

SUMMARY

Fixes #1080
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

plugins/modules/cloudwatchevent_rule.py
ADDITIONAL INFORMATION

Reviewed-by: GomathiselviS
Reviewed-by: Mark Chappell
Reviewed-by: Jill R
Reviewed-by: Alina Buzachis
(cherry picked from commit 614b792)
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 04s
✔️ build-ansible-collection SUCCESS in 13m 04s
✔️ ansible-test-splitter SUCCESS in 4m 48s
✔️ integration-amazon.aws-1 SUCCESS in 8m 00s
Skipped 43 jobs

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Jul 4, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 08s
✔️ build-ansible-collection SUCCESS in 14m 28s
✔️ ansible-test-splitter SUCCESS in 5m 11s
✔️ integration-amazon.aws-1 SUCCESS in 6m 01s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 4adb1b6 into stable-6 Jul 4, 2023
36 of 38 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-6/614b792be81a8a64c8c955093c853733ca1f57c3/pr-1589 branch July 4, 2023 11:43
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
ecs_cluster capacity provider strategy

SUMMARY
Fixes ansible-collections#770 - Add AWS ECS_Cluster Capacity Provider Strategy Support
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_cluster
ADDITIONAL INFORMATION
When creating or updating an ECS Cluster, configure the capacity providers and capacity provider strategy as provided by the user.
Given playbook task:
- name: Create an ECS Cluster with Capacity Providers
  ecs_cluster:
    name: default
    state: present
    capacity_providers:
      - FARGATE
      - FARGATE_SPOT
    capacity_provider_strategy:
      - capacity_provider: FARGATE
        base: 1
        weight: 1
      - capacity_provider: FARGATE_SPOT
        weight: 100

Previously would throw "Unsupported parameter" and no other parameter exists to expose these features.
Now you should see changed: [localhost] with the resultant created ECS Cluster having the same providers and provider_strategy fields as provided by the user.

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Justin McCormick <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
ecs_cluster: add default value to doc

SUMMARY
missing default value in the docs section of ansible-collections#1640
not released yet.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ecs_cluster

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
ecs_cluster capacity provider strategy

SUMMARY
Fixes ansible-collections#770 - Add AWS ECS_Cluster Capacity Provider Strategy Support
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_cluster
ADDITIONAL INFORMATION
When creating or updating an ECS Cluster, configure the capacity providers and capacity provider strategy as provided by the user.
Given playbook task:
- name: Create an ECS Cluster with Capacity Providers
  ecs_cluster:
    name: default
    state: present
    capacity_providers:
      - FARGATE
      - FARGATE_SPOT
    capacity_provider_strategy:
      - capacity_provider: FARGATE
        base: 1
        weight: 1
      - capacity_provider: FARGATE_SPOT
        weight: 100

Previously would throw "Unsupported parameter" and no other parameter exists to expose these features.
Now you should see changed: [localhost] with the resultant created ECS Cluster having the same providers and provider_strategy fields as provided by the user.

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Justin McCormick <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
ecs_cluster: add default value to doc

SUMMARY
missing default value in the docs section of ansible-collections#1640
not released yet.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ecs_cluster

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
ecs_cluster capacity provider strategy

SUMMARY
Fixes ansible-collections#770 - Add AWS ECS_Cluster Capacity Provider Strategy Support
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_cluster
ADDITIONAL INFORMATION
When creating or updating an ECS Cluster, configure the capacity providers and capacity provider strategy as provided by the user.
Given playbook task:
- name: Create an ECS Cluster with Capacity Providers
  ecs_cluster:
    name: default
    state: present
    capacity_providers:
      - FARGATE
      - FARGATE_SPOT
    capacity_provider_strategy:
      - capacity_provider: FARGATE
        base: 1
        weight: 1
      - capacity_provider: FARGATE_SPOT
        weight: 100

Previously would throw "Unsupported parameter" and no other parameter exists to expose these features.
Now you should see changed: [localhost] with the resultant created ECS Cluster having the same providers and provider_strategy fields as provided by the user.

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Justin McCormick <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
ecs_cluster: add default value to doc

SUMMARY
missing default value in the docs section of ansible-collections#1640
not released yet.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ecs_cluster

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
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