-
Notifications
You must be signed in to change notification settings - Fork 397
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
cloudfront_distribution.py - wait_until_processed appears broken #255
Labels
affects_2.10
bug
This issue/PR relates to a bug
module
module
needs_verified
plugins
plugin (any type)
python3
traceback
waiting_on_contributor
Needs help. Feel free to engage to get things unblocked
Comments
Files identified in the description: If these files are inaccurate, please update the |
ansibullbot
added
affects_2.10
bug
This issue/PR relates to a bug
module
module
needs_triage
plugins
plugin (any type)
python3
traceback
labels
Nov 16, 2020
goneri
added
needs_verified
waiting_on_contributor
Needs help. Feel free to engage to get things unblocked
and removed
needs_triage
labels
Feb 19, 2021
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this issue
May 25, 2022
…ollections#337) ec2_eni: Fix idempotency when security_groups is specified SUMMARY If security_groups attribute is specified - module always returns Changed = True. Fixes ansible-collections#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>
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jun 1, 2023
…n of creation (#1822) cloudfront_distribution: no longer crashes when waiting for completion of creation SUMMARY Fixes #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
patchback bot
pushed a commit
that referenced
this issue
Jun 1, 2023
…n of creation (#1822) cloudfront_distribution: no longer crashes when waiting for completion of creation SUMMARY Fixes #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 (cherry picked from commit dbf6362)
alinabuzachis
pushed a commit
to alinabuzachis/community.aws
that referenced
this issue
Jun 1, 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 (cherry picked from commit dbf6362)
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jun 1, 2023
…n of creation (#1822) (#1838) [manual backport stable-5] cloudfront_distribution: no longer crashes when waiting for completion of creation (#1822) cloudfront_distribution: no longer crashes when waiting for completion of creation SUMMARY Fixes #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 (cherry picked from commit dbf6362) SUMMARY ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Markus Bergholz <[email protected]>
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jun 8, 2023
…n of creation (#1822) (#1835) [PR #1822/dbf6362d backport][stable-6] cloudfront_distribution: no longer crashes when waiting for completion of creation This is a backport of PR #1822 as merged into main (dbf6362). SUMMARY Fixes #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: Alina Buzachis
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
affects_2.10
bug
This issue/PR relates to a bug
module
module
needs_verified
plugins
plugin (any type)
python3
traceback
waiting_on_contributor
Needs help. Feel free to engage to get things unblocked
SUMMARY
A cloudfront_distribution which performs its operation and the cloudfront distribution is created successfully. However, when
wait: yes
is enabled the error is detailed below.Digging around the source I've been unable to find anything on my own where the key error could be originating from.
This behavior only occurs on creation with
state: present
, not on deletion withstate: absent
ISSUE TYPE
COMPONENT NAME
cloudfront_distribution
ANSIBLE VERSION
CONFIGURATION
OS / ENVIRONMENT
STEPS TO REPRODUCE
I haven't been able to fully deconstruct it yet to a minimal case, I will try to update with one if I can. Right now I can reproduce by using
wait: yes
, and it goes away withwait: no
EXPECTED RESULTS
The ansible routine to wait until the distribution is made and reports healthy and then exit with my registered variable populated.
ACTUAL RESULTS
The text was updated successfully, but these errors were encountered: