Skip to content
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

metal_device_network_type #56

Open
t0mk opened this issue Mar 27, 2023 · 1 comment
Open

metal_device_network_type #56

t0mk opened this issue Mar 27, 2023 · 1 comment

Comments

@t0mk
Copy link
Contributor

t0mk commented Mar 27, 2023

Create module for metal_device_network_type.

Analogical to resource equinix_metal_device_network_type in terraform-provider-equinix:

Module should

  • be in plugins/modules/metal_device_network_type.py
  • follow development conventions
  • be idempotent
  • support check_mode if it's an _info module
  • have doc fields for (ansible-specdoc)[https://github.com/linode/ansible-specdoc] (see metal_project.py for an example
    • module_spec
    • specdoc_example
    • result_sample
    • SPECDOC_METAL
    • (DOCUMENATION, EXAMPLES and RETURN will be generated by make docs)
  • have an integration test in tests/integration/targets/metal_device_network_type/tasks/main.yml
  • (semi-optional) pass sanity test:
    $ cd .ansible/collections/ansible_collections/equinix/cloud
    $ ansible-test sanity -v --docker --python 3.10 metal_device_network_type.py

See the newest added module to get an idea how to do implement this.

@displague
Copy link
Member

This is a Terraform-ism that we potentially don't need if the ports are made to be configurable in the Metal Device.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants