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

Disable S3 tests related to removing bucket encryption #1395

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Mar 1, 2023

SUMMARY

Amazon now enables S3-SSE bucket encryption by default and it's not possible to disable it. Disable the relevant tests but leave a minimal framework in place.

ISSUE TYPE
  • Tests Pull Request
COMPONENT NAME

s3_bucket

ADDITIONAL INFORMATION

@tremble tremble added backport-4 PR should be backported to the stable-4 branch backport-5 PR should be backported to the stable-5 branch labels Mar 1, 2023
@tremble tremble marked this pull request as ready for review March 1, 2023 12:20
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 54s
✔️ build-ansible-collection SUCCESS in 12m 53s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 14m 01s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 11m 16s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 11m 29s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 11m 35s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 11m 01s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 10m 03s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 10m 20s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 12m 07s
✔️ cloud-tox-py3 SUCCESS in 7m 23s
✔️ ansible-test-changelog SUCCESS in 4m 23s
✔️ ansible-test-splitter SUCCESS in 4m 53s
✔️ integration-amazon.aws-1 SUCCESS in 7m 01s
Skipped 43 jobs

Copy link
Collaborator

@alinabuzachis alinabuzachis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tremble Shouldn't we also add a note in module's documentation?

@github-actions
Copy link

github-actions bot commented Mar 1, 2023

Docs Build 📝

Thank you for contribution!✨

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

@tremble
Copy link
Contributor Author

tremble commented Mar 1, 2023

@tremble Shouldn't we also add a note in module's documentation?

Added a note. Since we support other clouds I'm not going to remove "none" as an option.

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Mar 1, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 30s
✔️ build-ansible-collection SUCCESS in 12m 28s
✔️ ansible-test-splitter SUCCESS in 4m 49s
✔️ integration-amazon.aws-1 SUCCESS in 10m 08s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 15d92a7 into ansible-collections:main Mar 1, 2023
@patchback
Copy link

patchback bot commented Mar 1, 2023

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/15d92a74d777ca23596e3cbcbaa59e97f20b9541/pr-1395

Backported as #1397

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 1, 2023
Disable S3 tests related to removing bucket encryption

SUMMARY
Amazon now enables S3-SSE bucket encryption by default and it's not possible to disable it.  Disable the relevant tests but leave a minimal framework in place.
ISSUE TYPE

Tests Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 15d92a7)
@patchback
Copy link

patchback bot commented Mar 1, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/15d92a74d777ca23596e3cbcbaa59e97f20b9541/pr-1395

Backported as #1398

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 1, 2023
Disable S3 tests related to removing bucket encryption

SUMMARY
Amazon now enables S3-SSE bucket encryption by default and it's not possible to disable it.  Disable the relevant tests but leave a minimal framework in place.
ISSUE TYPE

Tests Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 15d92a7)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 3, 2023
[PR #1395/15d92a74 backport][stable-4] Disable S3 tests related to removing bucket encryption

This is a backport of PR #1395 as merged into main (15d92a7).
SUMMARY
Amazon now enables S3-SSE bucket encryption by default and it's not possible to disable it.  Disable the relevant tests but leave a minimal framework in place.
ISSUE TYPE

Tests Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 3, 2023
s3_bucket: object lock enabled

Depends-On: #1395
SUMMARY
Fixes #1347
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 3, 2023
[PR #1395/15d92a74 backport][stable-5] Disable S3 tests related to removing bucket encryption

This is a backport of PR #1395 as merged into main (15d92a7).
SUMMARY
Amazon now enables S3-SSE bucket encryption by default and it's not possible to disable it.  Disable the relevant tests but leave a minimal framework in place.
ISSUE TYPE

Tests Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
tremble pushed a commit to tremble/amazon.aws that referenced this pull request Mar 3, 2023
s3_bucket: object lock enabled

Depends-On: ansible-collections#1395
SUMMARY
Fixes ansible-collections#1347
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
reparepo pushed a commit to reparepo/amazon.aws that referenced this pull request Mar 17, 2023
cloudwatchlogs: Condense cloudwatch tests

SUMMARY

Merging tests for cloudwatchlogs and cloudwatchlogs_log_group into combined suite cloudwatchlogs.
There are 2 test suites in community.aws with substantially overlapping functionality - cloudwatchlogs and cloudwatch_log_group.
The cloudwatchlogs runs for changes to the cloudwatchlogs_log_group and cloudwatchlogs_log_group_metric_filter modules, and the cloudwatch_log_group only runs for cloudwatchlogs_log_group_info.
Merging the 2 test suites to create a combined suite for testing the overlapping functionalities could be a good idea.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

cloudwatchlogs
cloudwatchlogs_log_group

Reviewed-by: Mike Graves <[email protected]>
Reviewed-by: Jill R <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@9660de0
@tremble tremble deleted the s3_bucket/encryption branch September 27, 2023 06:59
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
cloudwatchlogs: Condense cloudwatch tests

SUMMARY

Merging tests for cloudwatchlogs and cloudwatchlogs_log_group into combined suite cloudwatchlogs.
There are 2 test suites in community.aws with substantially overlapping functionality - cloudwatchlogs and cloudwatch_log_group.
The cloudwatchlogs runs for changes to the cloudwatchlogs_log_group and cloudwatchlogs_log_group_metric_filter modules, and the cloudwatch_log_group only runs for cloudwatchlogs_log_group_info.
Merging the 2 test suites to create a combined suite for testing the overlapping functionalities could be a good idea.

ISSUE TYPE


Feature Pull Request

COMPONENT NAME


cloudwatchlogs
cloudwatchlogs_log_group

Reviewed-by: Mike Graves <[email protected]>
Reviewed-by: Jill R <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4 PR should be backported to the stable-4 branch backport-5 PR should be backported to the stable-5 branch mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants