[ansible][ACO] - Contribution questions

Jakob Meng jmeng at redhat.com
Tue Dec 27 10:40:00 UTC 2022


Hello Gaël,
thank you for giving us feedback on our Ansible modules and actually 
submitting a new module ☺️ We are currently lagging a bit in responses 
because we are trying to get release 2.0.0 of the Ansible OpenStack 
collection out of the door in January 2023.

As part of this effort we also refactored our CI integration tests, it 
is more consistent nowadays but still not complete. With 
compute_service_info you picked our worst case, it is tested in role 
nova_services 😬 A relict from the past.

Initially we planned to write one Ansible role per module, e.g. role 
project_info for openstack.cloud.project_info. But doing so produced a 
lot of redundant code. So during this year we changed our plan. Now we 
merge tests for *_info modules with their non-info equivalents. For 
example, tests for both modules federation_mapping and 
federation_mapping_info can be found in role federation_mapping.

Integration tests for volume_service_info would be located in Ansible 
role volume_service.

Instead of compute_service_info better take neutron_rbac_policies_info 
as an example of how to write and test *_info modules. Refactoring 
compute_service_info is still on my todo list 😅 Same goes for our docs 
on how to write modules etc. 🙈

Best,
Jakob

On 27.12.22 00:11, Gaël THEROND wrote:
> Hi sadi,
>
> Thanks for this feedback!
> I’ll wait for this patch to be merged then, no biggies as it’s 
> currently greetings season’s so no rush xD
>
> I’ll probably have few patches after that especially around unifying 
> options (filtering especially) on few modules.
>
> Thanks for the answer!
>
> Le lun. 26 déc. 2022 à 21:06, Sagi Shnaidman <sshnaidm at redhat.com> a 
> écrit :
>
>     Hi,  Gael,
>
>     Thanks for your contribution! Currently the tox-2.12 CI job always
>     fails, it's because of tox version 4 changes. I add a workaround
>     in the patch
>     https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868607
>
>     When it (or other solution) is merged, you're good to go with your
>     patch. Sorry for the inconvenience .
>
>     Thanks
>
>
>     On Sat, Dec 24, 2022 at 4:22 PM Gaël THEROND
>     <gael.therond at bitswalk.com> wrote:
>
>         Hi ansible collections openstack team!
>
>         I finally had time to list all my issues met with the project,
>         created few bug reports and even contributed to a patch today
>         (minor, mainly copy/paste) however I’ve few questions
>         regarding the CI process!
>
>         Overall, what’s the rule with the CI code testing?
>         I’ve read the contributing guide and had an eye on previous
>         patches to see how it’s used but I’m having a hard time to
>         find a real unified method. For instance, it seems that some
>         module miss CI tasks (such as compute_service_info) or did I
>         missed something?
>
>         Thanks a lot for all the good job!
>
>
>
>     -- 
>     Best regards
>     Sagi Shnaidman
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.openstack.org/pipermail/openstack-discuss/attachments/20221227/470ade8d/attachment-0001.htm>


More information about the openstack-discuss mailing list