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

ec2_eni idempotence bug #255

Closed
sassoo opened this issue Feb 3, 2021 · 3 comments · Fixed by #337
Closed

ec2_eni idempotence bug #255

sassoo opened this issue Feb 3, 2021 · 3 comments · Fixed by #337
Labels
bug This issue/PR relates to a bug has_pr module module plugins plugin (any type)

Comments

@sassoo
Copy link

sassoo commented Feb 3, 2021

SUMMARY

The encoding of groups causes a bytes type does not equal a string type so idempotence fails. This is the line at fault I belive: https://github.com/ansible-collections/amazon.aws/blame/ac6b2cd478773befdde43bfadc0de40969ad4d0b/plugins/modules/ec2_eni.py#L772

ISSUE TYPE
  • Bug Report
COMPONENT NAME

ec2_eni

ANSIBLE VERSION
ansible 2.10.5
python version = 3.9.1 (default, Jan  8 2021, 17:17:17) [Clang 12.0.0 (clang-1200.0.32.28)]
STEPS TO REPRODUCE

Pass security group names to the ec2_eni module & it will always report a change.

EXPECTED RESULTS

It should not report a change.

@ansibullbot
Copy link

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug module module needs_triage plugins plugin (any type) labels Feb 3, 2021
@goneri
Copy link
Member

goneri commented Feb 10, 2021

Hi, thank you for the bug report. Do you have an example of a problematical playbook? And would you like to submit a Pull Request?

@abikouo abikouo added the has_pr label Jul 1, 2021
ansible-zuul bot pushed a commit that referenced this issue Jul 8, 2021
ec2_eni: Fix idempotency when security_groups is specified

SUMMARY

If security_groups attribute is specified - module always returns Changed = True.
Fixes #255.

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ec2_eni
ADDITIONAL INFORMATION



Example:
- amazon.aws.ec2_eni:
    name: my-eni
    security_groups:
      - my-sg
    subnet_id: subnet-123456

Will always return Changed = True.

Reviewed-by: Jill R <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Sep 18, 2023
…n of creation (ansible-collections#1822)

cloudfront_distribution: no longer crashes when waiting for completion of creation

SUMMARY
Fixes ansible-collections#255
Here we were referring to the ["Id"] member of a queried distribution, but there is level of embedding missing: ["Distribution"]["Id"]
(Just how it's used here )
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
cloudfront_distribution

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Sep 18, 2023
…n of creation (ansible-collections#1822)

cloudfront_distribution: no longer crashes when waiting for completion of creation

SUMMARY
Fixes ansible-collections#255
Here we were referring to the ["Id"] member of a queried distribution, but there is level of embedding missing: ["Distribution"]["Id"]
(Just how it's used here )
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
cloudfront_distribution

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Oct 24, 2023
…n of creation (ansible-collections#1822)

cloudfront_distribution: no longer crashes when waiting for completion of creation

SUMMARY
Fixes ansible-collections#255
Here we were referring to the ["Id"] member of a queried distribution, but there is level of embedding missing: ["Distribution"]["Id"]
(Just how it's used here )
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
cloudfront_distribution

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis
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 has_pr module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants