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
New module: manage Amazon CloudFront origin access identities (cloud/amazon/cloudfront_origin_access_identity) #35540
Conversation
@wilvk this PR contains the following merge commits: Please rebase your branch to remove these commits. |
96bc44e
to
009b493
Compare
- boto3 >= 1.0.0 | ||
- python >= 2.6 | ||
|
||
version_added: "2.4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version_added: "2.5"
options: | ||
state: | ||
description: | ||
- The state of the resource. Valid states are |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
description:: If the named resource should exist.
choices:
- present
- absent
default: present
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
|
||
# create an origin access identity | ||
|
||
- cloudfront_origin_access_identity: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use - name:
and not comments, e.g.
- name: Create an origin access identity
cloudfront_origin_access_identity:
...
e_tag = None | ||
changed = False | ||
|
||
module = AnsibleAWSModule(argument_spec=argument_spec, supports_check_mode=False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If checkmode isn't supported please add this in the
notes:
section in DOCUMENTATION
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
have added to documentation under notes: section. let me know if this is incorrect
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, appologies, I think this needs to be a list
@wilvk this PR contains the following merge commits: Please rebase your branch to remove these commits. |
16e2387
to
eeeea14
Compare
The test
The test
The test
The test
The test
|
The test
|
@Constantin007 @Constantin07 @Deepakkothandan @Etherdaemon @Java1Guy @Lujeni @Madhura-CSI @MichaelBaydoun @Sodki @adq @akazakov @alachaum @amir343 @anryko @bekelchik @bpennypacker @brandond @carsongee @defunctio @dkhenry @fiunchinho @fivethreeo @garethr @gunzy83 @gurumaia @hsingh @hyperized @iiibrad @infectsoldier @j-carl @jarv @Java1Guy @jimbydamonk @jmenga @joelthompson @jonhadfield @jonmer85 @joshsouza @jsdalton @jsmartin @kaczynskid @leedm777 @linuxdynasty @loia @lwade @MichaelBaydoun @michaeljs1990 @minichate @mjschultz @mmochan @nadirollo @nand0p @naslanidis @nickball @orthanc @piontas @pjodouin @prasadkatti @psykotox @pwnall @raags @rickmendes @roadmapper @ryansydnor @scicoin-project @scottanderson42 @shepdelacreme @silviud @steynovich @tastychutney @tedder @tgerla @timmahoney @tombamford @tsiganenok @viper233 @whiter @wimnat @zacblazic @zbal @zeekin @zimbatm As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
Thanks @wilvk for all the time you've spent on these modules! |
…amazon/cloudfront_origin_access_identity) (ansible#35540)
SUMMARY
As advised by @ryansb in PR #24292 this comprises the origin access identity component of the cloudfront module additions.
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/cloud/amazon/cloudfront_origin_access_identity.py
ANSIBLE VERSION
ADDITIONAL INFORMATION