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

Migrate ec2_vpc_endpoint* modules and tests #435

Merged

Conversation

alinabuzachis
Copy link
Contributor

@alinabuzachis alinabuzachis commented Aug 3, 2021

Migrate ec2_vpc_endpoint* modules and tests

@ansibullbot
Copy link

@alinabuzachis: Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information.

Here are the items we could not find in your description:

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot
Copy link

@ansibullbot ansibullbot added integration tests/integration module module needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage new_module New module new_plugin New plugin plugins plugin (any type) tests tests labels Aug 3, 2021
@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_endpoint branch 4 times, most recently from a748800 to 5e9fd7d Compare August 6, 2021 08:33
@ansibullbot ansibullbot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Aug 10, 2021
@ansibullbot ansibullbot removed needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Aug 12, 2021
@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_endpoint branch 3 times, most recently from ce7e5a2 to 27207f5 Compare August 13, 2021 13:32
@tremble
Copy link
Contributor

tremble commented Aug 18, 2021

When this is merged please make sure to migrate the issues (there's a nice button for it)

https://github.com/ansible-collections/community.aws/issues?q=is%3Aissue+is%3Aopen+ec2_vpc_endpoint

@alinabuzachis
Copy link
Contributor Author

recheck

@ansibullbot ansibullbot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Aug 23, 2021
tremble and others added 21 commits August 26, 2021 15:23
* Ensure ec2_vpc_endpoint returns True when deleting an Endpoint
Return not changed when state=absent and endpoint has already been deleted

* Add minimal endpoint tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@a89ec90
* More aliases cleanup
* Mark ec2_classic_lb tests unstable
* Add more comments about why tests aren't enabled

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@cb55efa
* Added support for 'vpc_endpoint_type'.
* Integration test for the 'vpc_endpoint_type' feature.
* Added choices in documentation.
* Added changelog.

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@4da568a
* Add no_log=False to mark some more false-positives of the no_log check.

* More false-positives confirmed by tremble.

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@67b6d75
* New module - ec2_vpc_endpoint_service_info

* Deprecate querying services through ec2_vpc_endpoint_info

* Attempt to cope with some services which have who possible endpoints

Co-authored-by: Jill R <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@b48c1ee
* Stabilize and improve ec2_vpc_endpoint modules

- Add tagging support
- Make idempotent
- Better exception handling
- Better check_mode support
- Use module_utils for common functions
- Enable retries on common AWS failures

* Make endpoint deletion idempotent in check_mode

* Sanity fixes

* Address review feedback

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@f1d338d
This collection has been operating on Zuul CI for some weeks now

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@4e0d83c
ansible-zuul bot added a commit to ansible/project-config that referenced this pull request Aug 26, 2021
…-strategy_26775

amazon.aws: temp turn on the merge strategy

Disable the squash-merge merge strategy just the time we land

ansible-collections/amazon.aws#435
ansible-collections/amazon.aws#410

Reviewed-by: None <None>
@jillr jillr added the gate label Aug 26, 2021
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

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

LGTM!

goneri added a commit to goneri/project-config that referenced this pull request Aug 26, 2021
@ansible-zuul ansible-zuul bot merged commit 6f44e02 into ansible-collections:main Aug 26, 2021
ansible-zuul bot added a commit to ansible/project-config that referenced this pull request Aug 27, 2021
…e-merge-strategy_24403

Revert "amazon.aws: temp turn on the merge strategy"

Depends-On: ansible-collections/amazon.aws#435
Depends-On: ansible-collections/amazon.aws#410
This reverts commit 7dd6b71 since we
don't need it anymore.

Reviewed-by: None <None>
ansible-zuul bot added a commit to ansible-collections/community.aws that referenced this pull request Aug 27, 2021
Migrate ec2_vpc_endpoint* modules and tests

Depends-On: ansible-collections/amazon.aws#435
Remove ec2_vpc_endpoint* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration tests/integration module module needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly new_module New module new_plugin New plugin plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants