Commit graph

45021 commits

Author SHA1 Message Date
RobW3LGA
bb50fc3889 construct_deep_url() (#53475)
* Updates aci.py with the ability to add ACI objects to any depth

Changes start at line 411 (construct_deep_url() and supporting functions). One minor change to line 633 (the original construct_url()) to provide for testability:  ...join(sorted(self.child_classes)) vs ...join(self.child_classes)

I am also attaching two test files. One characterizing the existing construct_url() and the matching test set for construct_deep_url() to support my efforts and proof of parity

* Two PyTest files to support construct_deep_url

These two files provide testing parity, one characterizing the original construct_url() function and the other proofing construct_deep_url(). The ...deep_url.py test file goes five layers deep to provide better validation for the function

* Correcting previous upload to incorrect folder

These two files provide testing parity, one characterizing the original construct_url() function and the other proofing construct_deep_url(). The ...deep_url.py test file goes five layers deep to provide better validation for the function

* Deleting for file name change per Matt Clay

* Deleting for file name change per Matt Clay

* Correcting file names per Matt Clay

@mattclay Thanks again for your continued guidance and patience. Please cancel the previous (incorrect) request

* Wrong location for test file

* Wrong location for test file

* First attempt to comply with suggestions

lib/ansible/module_utils/network/aci/aci.py:517:0: SyntaxWarning: "is not" with a literal. Did you mean "!="?
lib/ansible/module_utils/network/aci/aci.py:534:0: SyntaxWarning: "is not" with a literal. Did you mean "!="?
lib/ansible/module_utils/network/aci/aci.py:558:161: E501 line too long (210 > 160 characters)

* First attempt to comply with suggestions

test/units/module_utils/network/aci/test_aci_construct_url.py:1:14: SyntaxError: import pytest
test/units/module_utils/network/aci/test_aci_deep_url.py:1:14: SyntaxError: import pytest
test/units/module_utils/network/aci/test_aci_construct_url.py:0:0: use "\n" for line endings instead of "\r\n"
test/units/module_utils/network/aci/test_aci_deep_url.py:0:0: use "\n" for line endings instead of "\r\n"
Shortened test function names (less descriptive)

* Second attempt to comply with suggestions

* Second attempt to comply with suggestions

* Third attempt to comply with suggestions

* Third attempt to comply with suggestions

* Pro Tip: Convert from 'CRLF' to 'LF' in VSCode

It's on the status bar to the right

* Added setup() support for tests

* Continued corrections to support testing

* Added two mocks to support testing

I could not find where to place fakes/mocks, so please let me know if the current location is incorrect

* Adding tmpdir property to mock_basic.py

* Added last blank line to mock_basic.py

To pass sanity test

* Attempt to correct setup() issues

* Attempt to correct setup() issues

* Attempt to correct setup() issues

* Attempt to correct setup() issues

* Withdrawing pending injectability tweak to aci.py

* Withdrawing pending injectability tweak to aci.py

* Withdrawing pending injectability tweak to aci.py

* Withdrawing pending injectability tweak to aci.py
2019-05-23 00:46:43 +02:00
Felix Fontein
391a1042c2 openssl_pkcs12: fix crash due to wrong path used for loading key in check mode (#56808)
* Used wrong path for private key.

* Add changelog.
2019-05-22 23:34:42 +02:00
Brian Coca
eb9009c4f4 added missing version_added 2019-05-22 17:04:36 -04:00
Paddy Newman
57e42842b0 Add a new "log_folder" option to the log_plays callback plugin. (#56717)
* Add a new "log_folder" option to the log_plays callback plugin.

A a new option for the log_plays callback plugin which allows the user to
control where the callback where creates log files.

The option can be set via the ANSIBLE_LOG_FOLDER environment variable or in the
Ansible configuration file, e.g.:

[callback_log_plays]
log_folder = /path/to/my/log/folder
2019-05-22 17:00:00 -04:00
Matt Martz
8bb3274711
Improvement on setting VALID_INCLUDE_KEYWORDS on HandlerTaskInclude (#56801) 2019-05-22 15:24:25 -05:00
Fred-sun
584a32e028 modify azure_rm_devtestlabpolicy relate document (#56770) 2019-05-22 14:57:10 -05:00
Fred-sun
4680ed4525 modify azure_rm_devtestlabenvironment relate document (#56769) 2019-05-22 14:56:13 -05:00
Fred-sun
22d2c6282a Update azure_rm_cosmosdbaccount related module document (#56624)
* update azure_rm_cosmosdbaccount_facts document
2019-05-22 14:55:10 -05:00
Fred-sun
16fb3ffbda modify azure_rm_devtestlabcustomimage's document (#56763) 2019-05-22 14:53:21 -05:00
Fred-sun
71ec184688 modify document (#56762) 2019-05-22 14:52:37 -05:00
Kevyn Pérez
96d2dd3241 Adding example for add host to groups (#56756)
updates the documentation for the virtualbox plugin
2019-05-22 14:50:14 -05:00
Yunge Zhu
831d3c7b62 add azure_rm_keyvault_info (#55497) 2019-05-23 03:39:37 +08:00
Sloane Hertel
cea7151a5b The InventoryData class does not have a get_vars method and InventoryManager's get_vars never appears to have been used (#56804) 2019-05-22 13:56:47 -04:00
Brian Coca
8fe9618c84
preserve underlying block building error (#56465)
fixes #56462
2019-05-22 11:28:06 -04:00
Brian Coca
69b2d7e0f3 fix collection adjacent loading with --playbook-dir (#56194)
* added inventory adjacent test
2019-05-22 11:06:54 -04:00
Dag Wieers
4f89c1d508
Revert "ACI: Implement encoded query_string" (#56783)
This reverts commit 1690af04fe.
2019-05-22 15:44:22 +02:00
Nathan Swartz
e4239d9424 Fix workload profileId indexing in netapp_e_volume module (#56747) 2019-05-22 09:43:21 -04:00
Andrey Klychkov
f9108e23e7 postgresql_db: bugfix 56703 (#56734) 2019-05-22 05:59:21 -04:00
Fred-sun
f824a13952 remove azure_rm_cdnprofile quote (#56695) 2019-05-22 16:26:58 +08:00
Abhijeet Kasurde
459776eb82
VMware: Add folder option for vmware_dvswitch (#56707)
* Added folder param
* Mark datacenter as optional parameter
* Updated docs
* Updated tests

Fixes: #54986

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
2019-05-22 11:40:09 +05:30
Will Thames
4065692e13 k8s minor bug fixes (#56168)
Ensure `wait_condition`s with `Status: Unknown` actually
complete

Return k8s object after wait rather than k8s object before
wait when object is patched.
2019-05-22 08:08:22 +02:00
Jake Scaltreto
3e8ca0285f lookup: consul_kv: Add datacenter parameter (#56647) 2019-05-22 08:02:39 +02:00
Kevin Subileau
d8c1f67862 win_file_version: fix typos (#56751)
##### SUMMARY
Minor fixes in error messages


##### ISSUE TYPE
- Bugfix Pull Request

##### COMPONENT NAME
win_file_version

##### ADDITIONAL INFORMATION
2019-05-22 15:44:54 +10:00
CsilLAB
6900378766 [WIP] Tip to see the content of a single encrypted var (#37887)
* Tip to see the content of a single var encrypted with Ansible Vault
2019-05-21 15:37:25 -05:00
jimwitte
cb9be4ddaf Update get_certificate.py (#56594)
* Update get_certificate.py with an example to calculate number of days until cert expires from get_certificate result.

* Update lib/ansible/modules/crypto/get_certificate.py

Co-Authored-By: Felix Fontein <felix@fontein.de>
2019-05-21 15:31:34 -05:00
Felix Fontein
53ec9c8019 Clarify that GPLv3+ module_utils need core team approval (#56740)
* Clarify that GPLv3+ module_utils need core team approval.

* Update docs/docsite/rst/dev_guide/shared_snippets/licensing.txt

Co-Authored-By: Sam Doran <sdoran@redhat.com>

* More 'module_util' -> 'file in module_utils'.
2019-05-21 15:15:55 -05:00
Alexey Nikitin
b275dede0c Adds one more example for rollbar_deployment module (#56535)
- Also `name` added to existing example in order to comply documentation styleguide
2019-05-21 15:12:54 -05:00
Fred-sun
d7dc9586d5 Update azure_rm_deployment related document (#56627)
* add azure_rm_deployment documentation
2019-05-21 15:07:10 -05:00
Fred-sun
41cc198b61 Update azure_rm_devtestlab related module document (#56681)
* update azure_rm_devtestlab documention
2019-05-21 14:43:47 -05:00
Fred-sun
8495e3d9e4 update more azure documentation (#56700) 2019-05-21 14:41:11 -05:00
Sandra McCann
ac22cd31b6 fixed doc build errors (#56735) 2019-05-21 14:37:43 -05:00
Fred-sun
e64083e4b6 remove quote (#56690) 2019-05-21 14:35:26 -05:00
vicmunoz
a608ca0e86 na_ontap_volume - added qos policy module options (#56026)
* added qos policy module options

* reverting reformatting not needed

* removing unneeded variable

* updating docs and examples

* adding example with adaptive qos policy
2019-05-21 14:33:54 -04:00
Sandra McCann
388f5d0b2b Add links to quickstart guides (#56722)
* shell of quickstart

* link to existing quickstart guides
2019-05-21 10:48:50 -05:00
Martin Krizek
a68ac729ea yum: fix false error msg about autoremove support (#56459)
* yum: fix false error msg about autoremove support
2019-05-21 11:37:04 -04:00
Martin Krizek
77f997474a yum: use Custom Group in integration tests (#56439) 2019-05-21 11:32:36 -04:00
Jon Pugh
7f5b1e998f Out put the version of the dependency so we can better understand the problem. (#56379) 2019-05-21 11:28:16 -04:00
Marcin
afc678e2c7 Raise error in case of empty hosts list in playbook (#56354)
Fixes cases of playbook contaiing:
- hosts:
  -
ERROR! Unexpected Exception, this is probably a bug: sequence item 0: expected string, NoneType found
2019-05-21 11:11:15 -04:00
Daniel Skates
e13566140a Move update_cache to after self.yum_baseurl definition (#56613) 2019-05-21 11:43:44 +02:00
Ondra Machacek
a4144e15c0 kubevirt_vm: Improve create VM from template (#55927) 2019-05-21 09:31:58 +02:00
Jakob Ackermann
3b4b2e5021 [docker] support the lookup of images by digest (#56649)
* [docker] images: add support for lookup by sha256 digest

Signed-off-by: Jakob Ackermann <das7pad@outlook.com>

* [tests] docker image by digest: work on a minimal test case

Signed-off-by: Jakob Ackermann <das7pad@outlook.com>

* [docker] group branch conditions per lookup

Co-Authored-By: Felix Fontein <felix@fontein.de>

* [misc] add a news fragment for the added digest lookup for docker images

Signed-off-by: Jakob Ackermann <das7pad@outlook.com>
2019-05-21 02:54:05 -04:00
Will Thames
8e1dd58c2b Allow python_requirements_facts to cope with packages with dashes (#56166)
* Allow python_requirements_facts to cope with packages with dashes

```
python_requirements_facts:
  dependencies:
    - kubernetes-validate
```

should work as expected

* Ensure tests run for python_requirements_facts
2019-05-20 23:38:50 -04:00
Matthew Horoschun
d8a5efa00c iam_role.py remove_policies should remove all of the requested policies (not just the first) (#56331)
The remove_policies function in iam_role.py enumerates a list of policies to remove. However, due to an indentation issue on the return True line, only the first such policy would be removed.

This change outdents the return True so that all of the the requested policies are removed.
2019-05-20 17:09:26 -07:00
Matt Martz
493cf817a9 Don't rely on netloc for determining hostname and port, just use hostname and port (#56270)
* Add changelog fragment
* Fix IPv6 address parsing for py2.6, and add tests
* make sure hostname isn't None
2019-05-20 16:50:54 -04:00
Seth Hochberg
66bfa27685 Replace deprecated ansible-container references (#56427)
Link to ansible-bender instead of the deprecated ansible-container.

Co-Authored-By: Felix Fontein <felix@fontein.de>
2019-05-20 15:38:38 -05:00
mirumu
6496084697 Update unarchive.py (#56285)
Clarify that the unarchive module does not unpack a compressed file, only archive files.
2019-05-20 15:20:13 -05:00
Jon "The Nice Guy" Spriggs
77f3e8451c Update uri.py (#56395)
1. Note that uri doesn't honor the no_proxy environment variable (due to https://github.com/ansible/ansible/issues/52705), and suggest a work around.
2. Added an example showing a test waiting for a URL to become available (using the `until:`, `retries:` and `delay:` settings) - based on https://gist.github.com/mikeifomin/67e233cd461331de16707ef59a07e372#gistcomment-2718587

Co-Authored-By: Felix Fontein <felix@fontein.de>
2019-05-20 15:18:47 -05:00
Fred-sun
d56c975ce8 update azure_rm_availabilityset related module document (#56432)
* Update azure_rm_availabilityset.py
2019-05-20 15:09:11 -05:00
René Moser
ded7949879 vultr: streamline docs (#56563) 2019-05-20 14:52:37 -05:00
Arnaud
6b240433b6 Update vagrant.py (#56568)
`inventory_file` has been renamed to `inventory_path`.
2019-05-20 14:30:57 -05:00