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

SSM connection plugin - Add encryption tests #1657

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 13, 2023

Depends-On: ansible/ansible-zuul-jobs#1746

SUMMARY

Expand on the aws_ssm connection integration tests

  • Follow up on suggestion to use ssm_parameter to locate AMIs
  • Add tests for connection using encrypted buckets
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_ssm

ADDITIONAL INFORMATION

@tremble
Copy link
Contributor Author

tremble commented Jan 13, 2023

@markuman added your ssm_parameter suggestion

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration needs_triage tests tests labels Jan 13, 2023
@softwarefactory-project-zuul

This comment was marked as resolved.

@markuman
Copy link
Member

hmm

"msg": "Failed to access SSM parameter /aws/service/ami-amazon-linux-latest/amzn-ami-hvm-x86_64-gp2 (AccessDenied)"

The parameter itself is correct

ok: [localhost] => {
    "lookup('aws_ssm', '/aws/service/ami-amazon-linux-latest/amzn-ami-hvm-x86_64-gp2')": "ami-01ef0a53b405a1b2b"
}

@tremble
Copy link
Contributor Author

tremble commented Jan 15, 2023

@markuman Yeah, I've tested it locally. mattclay/aws-terminator#255 is the missing permission.

@gravesm
Copy link
Member

gravesm commented Jan 16, 2023

recheck

@softwarefactory-project-zuul

This comment was marked as resolved.

@tremble
Copy link
Contributor Author

tremble commented Jan 16, 2023

recheck

@softwarefactory-project-zuul

This comment was marked as resolved.

@goneri
Copy link
Member

goneri commented Jan 16, 2023

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 3m 57s
✔️ build-ansible-collection SUCCESS in 6m 01s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 58s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 02s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 21s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 10s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 22s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 09s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 15s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 13s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 7m 42s
✔️ ansible-test-changelog SUCCESS in 2m 45s
✔️ ansible-test-splitter SUCCESS in 2m 29s
integration-community.aws-1 FAILURE in 33m 05s
✔️ integration-community.aws-2 SUCCESS in 12m 36s
✔️ integration-community.aws-3 SUCCESS in 6m 52s
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@tremble
Copy link
Contributor Author

tremble commented Jan 17, 2023

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 3m 44s
✔️ build-ansible-collection SUCCESS in 5m 37s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 02s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 39s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 11s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 25m 34s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 38s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 38s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 58s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 49s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 7m 00s
✔️ ansible-test-changelog SUCCESS in 2m 38s
✔️ ansible-test-splitter SUCCESS in 2m 49s
integration-community.aws-1 FAILURE in 32m 57s
✔️ integration-community.aws-2 SUCCESS in 12m 06s
✔️ integration-community.aws-3 SUCCESS in 5m 42s
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 45s
✔️ build-ansible-collection SUCCESS in 5m 53s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 00s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 36s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 54s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 36s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 41s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 59s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 02s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 30s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 35s
✔️ ansible-test-changelog SUCCESS in 2m 55s
✔️ ansible-test-splitter SUCCESS in 2m 52s
✔️ integration-community.aws-1 SUCCESS in 11m 20s
✔️ integration-community.aws-2 SUCCESS in 10m 38s
✔️ integration-community.aws-3 SUCCESS in 6m 21s
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

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

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 6m 09s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 55s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 53s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 41s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 12m 01s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 11m 47s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 40s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 35s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 35s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 7m 25s
✔️ ansible-test-changelog SUCCESS in 2m 20s
✔️ ansible-test-splitter SUCCESS in 2m 49s
✔️ integration-community.aws-1 SUCCESS in 12m 28s
✔️ integration-community.aws-2 SUCCESS in 14m 43s
✔️ integration-community.aws-3 SUCCESS in 10m 10s
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 34e073d into ansible-collections:main Jan 17, 2023
@tremble tremble added the backport-5 PR should be backported to the stable-5 branch label Jan 18, 2023
@patchback
Copy link

patchback bot commented Jan 18, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/34e073d5473f8e2ec911553e381a19b1548ab953/pr-1657

Backported as #1658

🤖 @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 Jan 18, 2023
SSM connection plugin - Add encryption tests

Depends-On: ansible/ansible-zuul-jobs#1746
SUMMARY
Expand on the aws_ssm connection integration tests

Follow up on suggestion to use ssm_parameter to locate AMIs
Add tests for connection using encrypted buckets

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

(cherry picked from commit 34e073d)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 18, 2023
[PR #1657/34e073d5 backport][stable-5] SSM connection plugin - Add encryption tests

This is a backport of PR #1657 as merged into main (34e073d).
Depends-On: ansible/ansible-zuul-jobs#1746
SUMMARY
Expand on the aws_ssm connection integration tests

Follow up on suggestion to use ssm_parameter to locate AMIs
Add tests for connection using encrypted buckets

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
@tremble tremble deleted the tests/connection_aws_ssm-encryption branch February 15, 2023 09:24
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…EngineMode to the list of parameters that can be passed when creating or modifying a Multi-AZ RDS cluster. (ansible-collections#1657)

rds_cluster - enable creation or modification of AllocatedStorage, DBClusterInstanceClass, StorageType, Iops, and EngineMode

SUMMARY

Add AllocatedStorage, DBClusterInstanceClass, StorageType, Iops and EngineMode to the list of parameters that can be passed when creating or modifying a Multi-AZ RDS cluster.

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

rds_cluster

Reviewed-by: Taeho Park
Reviewed-by: Mike Graves <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-5 PR should be backported to the stable-5 branch community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) needs_triage tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants