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

Fix deprecation warning scope #719

Conversation

mandar242
Copy link
Contributor

SUMMARY

Fix deprecation warning scope to be not displayed while terminating instances.

Fixes #689

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_instance

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review module module needs_triage plugins plugin (any type) small_patch Hopefully easy to review labels Mar 18, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

Copy link
Contributor

@jatorcasso jatorcasso left a comment

Choose a reason for hiding this comment

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

LGTM after adding changelog!

@alinabuzachis alinabuzachis added backport-2 PR should be backported to the stable-2 branch backport-3 PR should be backported to the stable-3 branch labels Mar 21, 2022
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.

Even if the change is very small, you can add a changelog in either way.

@mandar242
Copy link
Contributor Author

Even if the change is very small, you can add a changelog in either way.

Added.

@ansibullbot ansibullbot removed the small_patch Hopefully easy to review label Mar 21, 2022
@mandar242 mandar242 force-pushed the ec2_instance_deprecation_fix branch from c54459a to 38172ca Compare March 21, 2022 15:41
@ansibullbot ansibullbot added the small_patch Hopefully easy to review label Mar 21, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@mandar242
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Mar 21, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit d6b1186 into ansible-collections:main Mar 21, 2022
@patchback
Copy link

patchback bot commented Mar 21, 2022

Backport to stable-2: 💔 cherry-picking failed — conflicts found

❌ Failed to cleanly apply d6b1186 on top of patchback/backports/stable-2/d6b118644c17ed900de83810d02f1a8535d40903/pr-719

Backporting merged PR #719 into main

  1. Ensure you have a local repo clone of your fork. Unless you cloned it
    from the upstream, this would be your origin remote.
  2. Make sure you have an upstream repo added as a remote too. In these
    instructions you'll refer to it by the name upstream. If you don't
    have it, here's how you can add it:
    $ git remote add upstream https://github.com/ansible-collections/amazon.aws.git
  3. Ensure you have the latest copy of upstream and prepare a branch
    that will hold the backported code:
    $ git fetch upstream
    $ git checkout -b patchback/backports/stable-2/d6b118644c17ed900de83810d02f1a8535d40903/pr-719 upstream/stable-2
  4. Now, cherry-pick PR Fix deprecation warning scope #719 contents into that branch:
    $ git cherry-pick -x d6b118644c17ed900de83810d02f1a8535d40903
    If it'll yell at you with something like fatal: Commit d6b118644c17ed900de83810d02f1a8535d40903 is a merge but no -m option was given., add -m 1 as follows intead:
    $ git cherry-pick -m1 -x d6b118644c17ed900de83810d02f1a8535d40903
  5. At this point, you'll probably encounter some merge conflicts. You must
    resolve them in to preserve the patch from PR Fix deprecation warning scope #719 as close to the
    original as possible.
  6. Push this branch to your fork on GitHub:
    $ git push origin patchback/backports/stable-2/d6b118644c17ed900de83810d02f1a8535d40903/pr-719
  7. Create a PR, ensure that the CI is green. If it's not — update it so that
    the tests and any other checks pass. This is it!
    Now relax and wait for the maintainers to process your pull request
    when they have some cycles to do reviews. Don't worry — they'll tell you if
    any improvements are necessary when the time comes!

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

@patchback
Copy link

patchback bot commented Mar 21, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/d6b118644c17ed900de83810d02f1a8535d40903/pr-719

Backported as #724

🤖 @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 21, 2022
Fix deprecation warning scope

SUMMARY

Fix deprecation warning scope to be not displayed while terminating instances.

Fixes #689
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ec2_instance

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
(cherry picked from commit d6b1186)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 23, 2022
[PR #719/d6b11864 backport][stable-3] Fix deprecation warning scope

This is a backport of PR #719 as merged into main (d6b1186).
SUMMARY

Fix deprecation warning scope to be not displayed while terminating instances.

Fixes #689
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ec2_instance
@mandar242 mandar242 deleted the ec2_instance_deprecation_fix branch March 23, 2022 20:26
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…t_subnet_group

Add integration tests for redshift_subnet_group

SUMMARY
Add integration tests for redshift_subnet_group

 CI / IAM Permissions

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
redshift_subnet_group
ADDITIONAL INFORMATION

Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…t_subnet_group

Add integration tests for redshift_subnet_group

SUMMARY
Add integration tests for redshift_subnet_group

 CI / IAM Permissions

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
redshift_subnet_group
ADDITIONAL INFORMATION

Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…t_subnet_group

Migrate redshift_subnet_group to boto3

SUMMARY
Migrate redshift_subnet_group to boto3
note: while there additional features (tagging springs to mind) that could be added, I'm trying to avoid scope creep and mostly just want to knock out migrations of the remaining old-boto modules.  That said, I am converting tags using the boto3_tag_list_to_ansible_dict helper, in the return values this is just to avoid needing to change formats in future
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
redshift_subnet_group
ADDITIONAL INFORMATION
Depends-On: ansible-collections#719

Reviewed-by: Markus Bergholz <git@osuv.de>
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
backport-2 PR should be backported to the stable-2 branch backport-3 PR should be backported to the stable-3 branch bug This issue/PR relates to a bug community_review mergeit Merge the PR (SoftwareFactory) module module needs_triage plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

amazon.aws.ec2_instance throws deprecation warning for teardown
4 participants