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_port_vlan_attachment #62

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

metal_port_vlan_attachment #62

t0mk opened this issue Mar 27, 2023 · 1 comment

Comments

@t0mk
Copy link
Contributor

t0mk commented Mar 27, 2023

Create module for metal_port_vlan_attachment.

Analogical to resource equinix_metal_port_vlan_attachment in terraform-provider-equinix:

Module should

  • be in plugins/modules/metal_port_vlan_attachment.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_port_vlan_attachment/tasks/main.yml
  • (semi-optional) pass sanity test:
    $ cd .ansible/collections/ansible_collections/equinix/cloud
    $ ansible-test sanity -v --docker --python 3.10 metal_port_vlan_attachment.py

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

@displague
Copy link
Member

displague commented Mar 29, 2023

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

@displague displague added this to the v0.3 milestone May 31, 2023
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