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

Add link_ids read-only attribute to apstra_datacenter_connectivity_template_assignments and apstra_datacenter_connectivity_templates_assignment resources #701

Merged
merged 7 commits into from
Jul 8, 2024

Conversation

chrismarget-j
Copy link
Collaborator

@chrismarget-j chrismarget-j commented Jul 8, 2024

This PR adds two new attributes to these resources:

  • apstra_datacenter_connectivity_template_assignments
  • apstra_datacenter_connectivity_templates_assignment

The attributes are:

  • fetch_ip_link_ids (Optional, boolean)
  • ip_links_ids (Computed, 2-dimensional map)

fetch_ip_link_ids controls whether we perform the necessary queries to populate ip_link_ids. It is expected that ip_link_ids will be required infrequently. By not fetching this data when it's not needed, apply time can be reduced.

the ip_link_ids is a two-dimensional map for both resources: ID -> VLAN (string) -> Link ID

  • ID: this is the ID of the application point (in the case of apstra_datacenter_connectivity_template_assignments) or connectivity template (in the case of apstra_datacenter_connectivity_templates_assignment)
  • VLAN: This is a number between 0 and 4094 inclusive. It's represented as a string because that is a requirement for Terraform map keys. VLAN 0 represents "untagged" logical links.
  • Link ID: this is the ID of the logical_link type link nodes created by assigning a CT containing IP Link CT primitives.

@chrismarget-j chrismarget-j merged commit bcbe1cb into main Jul 8, 2024
1 check passed
@chrismarget-j chrismarget-j deleted the ct-application-link-ids branch July 8, 2024 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants