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 #2083/9c53b6c1 backport][stable-8] backup_plan_info: bugfix to enable getting info of all backup plans #2122

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 4, 2024

This is a backport of PR #2083 as merged into main (9c53b6c).

SUMMARY

Fix being unable to fetch info of all backup plans.
With backup_plan_names being a required parameter, the functionality to get all plans info was not working

# Gather information about all backup plans
- name: Get info of all backup plans 
  amazon.aws.backup_plan_info:
  register: plan_info_result

gave
**********
fatal: [localhost]: FAILED! => {"changed": false, "msg": "missing required arguments: backup_plan_names"}

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

backup_plan_info

ADDITIONAL INFORMATION

…2083)

backup_plan_info: bugfix to enable getting info of all backup plans

SUMMARY

Fix being unable to fetch info of all backup plans.
With backup_plan_names being a required parameter, the functionality to get all plans info was not working
# Gather information about all backup plans
- name: Get info of all backup plans
  amazon.aws.backup_plan_info:
  register: plan_info_result

gave
**********
fatal: [localhost]: FAILED! => {"changed": false, "msg": "missing required arguments: backup_plan_names"}

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

backup_plan_info
ADDITIONAL INFORMATION

Reviewed-by: GomathiselviS
Reviewed-by: Alina Buzachis
Reviewed-by: Mandar Kulkarni <[email protected]>
Reviewed-by: Mark Chappell
(cherry picked from commit 9c53b6c)
Copy link

github-actions bot commented Jun 4, 2024

Docs Build 📝

Thank you for contribution!✨

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

Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 48s
✔️ build-ansible-collection SUCCESS in 17m 20s
✔️ ansible-test-splitter SUCCESS in 7m 35s
✔️ integration-amazon.aws-1 SUCCESS in 8m 53s
✔️ integration-community.aws-1 SUCCESS in 43m 33s
Skipped 42 jobs

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Jun 5, 2024
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 41s
✔️ build-ansible-collection SUCCESS in 15m 46s
✔️ ansible-test-splitter SUCCESS in 6m 59s
✔️ integration-amazon.aws-1 SUCCESS in 7m 58s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 6b7459e into stable-8 Jun 5, 2024
40 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-8/9c53b6c12e9c3e043f695b9791bfd499a770062a/pr-2083 branch June 5, 2024 08:35
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