don't inherit OpenstackModule class from AnsibleModule class to
prevent occasional overriding Ansible methods or vars and failing
module.
Change-Id: Ic34fff0c938eb87cc0d2c5e98fbafed64bf349f6
One is for resource changing module, like server start or delete,
second one is for info collection about a specific resource.
Change-Id: I78b35075111731fff2fd50837fa4e6e0c61c55a0
The ignores can just list the directory and from them it will
ignore the directory too.
Add .gitreview and .gitignore and the importer_result.json.
Change-Id: Ibe5e56ccd3fc54695bf64dd696adb2ec2bb0f3c2
Run galaxy-importer on the built artifact and fail if galaxy is
going to fail.
Remove license_file entry in galaxy.yml
Change-Id: I6c2ac22ccb56f52f8efcefc5891b10875ae8c0b5
The collection routing implementation for 2.10:
https://github.com/ansible/ansible/pull/67684
just update the filename to be meta/runtime.yml.
Change-Id: I858f53e66bef2c40fb7ec2821d412bd966417106
We need 2.10 for building collection tarballs with build_ignore.
Also, build_ignore takes globs, not leading filepaths, so add
globs to the dirs in our list so that we properly don't include
these.
Change-Id: I809e100fa80bffafda4002487590a6dcd6aea0c1
Check only things we build into collection.
Ignore sanity files from full CI jobs.
Add queens to gate because it's voting.
Add files to ignoring when building collection.
Remove non-required lines from ignorign sanity files.
Change-Id: I0dde756efa133cf75b67d072af78a1f8307ce932
This is separate from the previous patch - it's just the results
of running the script so we can review the two a little independently.
We should probably squash them.
Change-Id: I838f15cf4a32455a5be20033c8ddc27db6ca15c0
The 2.10 transition has a routing.yml file that points each
individual module from ansible/ansible to a new location. That means
we can put:
os_server:
redirect: openstack.cloud.server
into lib/ansible/config/routing.yml in ansible/ansible and have
the result be the end user's playbooks still working with the
old names while providing new names that are less ugly.
This adds a routing file to our collection repo, as well as the
script used to generate the new mapping.
Change-Id: Ia5d18282b14ad0d86a347343be8bb477ae47130a
We can't run linters in an install because installs expand
symlinks to regular files so the ansible-test code that detects
symlinks can't work so it looks at docs for symlink aliases
as if they were real.
Change-Id: I0a5659a1f693261a21a5b29528f950b846205f2e
Prior to this change, the os_server_action module could not rebuild
servers, because it passed no arguments to the "rebuild" object.
The imageRef parameter is mandatory. If the client does not specify
imageRef, Nova takes no action to rebuild the server.
Change-Id: I0edbdb634feb90d4c54476a4089fcb014c0b83b2
As per the REST Networking API v2.0 (Subnet) docs, if subnets pools
are not specified, OpenStack networking automatically allocates pools
covering all IP addresses in the CIDR.
In custom vendor specific environments, subnets can be created without
allocation pools via UI due to which subnets will not have any existing
allocation pools. Under this scenario, module throws an
"IndexError: list index out of range" error.
Also, allow to add more allocation pools on top of existing allocation
pools.
Change-Id: Ib8becf5e958f1bc8e5c9fd76f1722536bf1c9f1a
Keystone supports setting custom properties for projects. This changes add
properties to os_project to enable setting of custom properties.
Depends-On: https://review.opendev.org/715255
Change-Id: I23e2834d6b9d40ce86db4b32a0ed8bf76b3d9879
In addition to running ansible-test from stable it's helpful to run
ansible-test from 'devel' where new sanity tests will land. These
sanity tests are generally applicable to existing modules, but aren't
back-ported.
Change-Id: I8df3dd43cbc4cba643d7860e77a1089c15f5f092
Depends-On: https://review.opendev.org/717118
