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 #1701/19b89f68 backport][stable-6] Backup selection bugfix #1710

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Aug 23, 2023

This is a backport of PR #1701 as merged into main (19b89f6).

SUMMARY

Fix bug in backup_selection where an existing backup selection was not updated to add Conditions if that value was previously empty.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

backup_selection

ADDITIONAL INFORMATION

Split into two commits, the first commit adds comprehensive integration tests to cover the scenarios that were failing and the second commit fixes the code.

@patchback patchback bot mentioned this pull request Aug 23, 2023
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Aug 23, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 19s
✔️ build-ansible-collection SUCCESS in 13m 24s
✔️ ansible-test-splitter SUCCESS in 4m 58s
integration-amazon.aws-1 FAILURE in 8m 06s
Skipped 43 jobs

@hakbailey hakbailey force-pushed the patchback/backports/stable-6/19b89f682227f8755322250c929355812dc37a2c/pr-1701 branch from 919d0ff to 81e99a3 Compare August 30, 2023 13:42
@hakbailey
Copy link
Contributor

Rebased onto current stable-6.

@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 00s
✔️ build-ansible-collection SUCCESS in 12m 26s
✔️ ansible-test-splitter SUCCESS in 4m 40s
✔️ integration-amazon.aws-1 SUCCESS in 8m 35s
Skipped 43 jobs

Backup selection bugfix

SUMMARY
Fix bug in backup_selection where an existing backup selection was not updated to add Conditions if that value was previously empty.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
backup_selection
ADDITIONAL INFORMATION
Split into two commits, the first commit adds comprehensive integration tests to cover the scenarios that were failing and the second commit fixes the code.

Reviewed-by: Mike Graves <[email protected]>
Reviewed-by: Alina Buzachis
(cherry picked from commit 19b89f6)
@hakbailey hakbailey force-pushed the patchback/backports/stable-6/19b89f682227f8755322250c929355812dc37a2c/pr-1701 branch from 81e99a3 to b5a008a Compare September 1, 2023 15:25
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 05s
✔️ build-ansible-collection SUCCESS in 13m 05s
✔️ ansible-test-splitter SUCCESS in 5m 51s
✔️ integration-amazon.aws-1 SUCCESS in 7m 41s
Skipped 43 jobs

@hakbailey hakbailey merged commit 47b0125 into stable-6 Sep 1, 2023
35 checks passed
@hakbailey hakbailey deleted the patchback/backports/stable-6/19b89f682227f8755322250c929355812dc37a2c/pr-1701 branch September 1, 2023 16:18
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 27, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@88872b3
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 27, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@88872b3
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 27, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@88872b3
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 6, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@88872b3
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
iam_access_key - fix example docs

SUMMARY
Closes ansible-collections#1710
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
iam_access_key

Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants