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

tags/annotations cause error or don't get created #152

Closed
tigelane opened this issue Nov 2, 2020 · 3 comments
Closed

tags/annotations cause error or don't get created #152

tigelane opened this issue Nov 2, 2020 · 3 comments

Comments

@tigelane
Copy link

tigelane commented Nov 2, 2020

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version

Terraform v0.13.2
+ provider registry.terraform.io/ciscodevnet/aci v0.5.0

APIC version and APIC Platform

Affected Resource(s)

  • Many - Tenant, AP, VRF/CTX, Bridge Domain
  • Seems like anything that uses Annotations

Terraform Configuration Files

resource "aci_tenant" "tenant" {
  name        = "tagged_tenant"
  annotation  = "atag"
}
resource "aci_vrf" "vrf1" {
  tenant_dn   = aci_tenant.tenant.id
  name        = "single_tagged_vrf"
  annotation  = "atag"
}
resource "aci_application_profile" "ap" {
  tenant_dn   = aci_tenant.tenant.id
  name        = "single_tagged_ap"
  annotation  = "atag"
}
resource "aci_bridge_domain" "bd" {
  tenant_dn   = aci_tenant.tenant.id
  name        = "double_tagged_bd"
  annotation  = "atag,btag"
}

Expected Behavior

  • tenant, vrf1, ap to have a single annotation atag on it once created.
  • bd to have two annotations on it once created atag and btag.

Actual Behavior

  • tenant, vrf1, ap are created, but no annotations are applied.
  • bd is not created and errors (additional object with two tags error)
aci_tenant.tenant: Creating...
aci_tenant.tenant: Creation complete after 9s [id=uni/tn-tagged_tenant]
aci_application_profile.ap: Creating...
aci_vrf.vrf1: Creating...
aci_bridge_domain.bd: Creating...
aci_application_profile.ap: Creation complete after 1s [id=uni/tn-tagged_tenant/ap-single_tagged_ap]
aci_vrf.vrf1: Creation complete after 6s [id=uni/tn-tagged_tenant/ctx-single_tagged_vrf]

Error: property annotation of BD-double_tagged_bd failed validation for value 'atag,btag'

  on main.tf line 23, in resource "aci_bridge_domain" "bd":
  23: resource "aci_bridge_domain" "bd" {

Steps to Reproduce

  1. terraform apply
  2. Go look at created objects with single annotation or see error if you tried to add two annotations.

Important Factoids

Not that I can think of.

@tigelane tigelane changed the title tags/annotations fail or don't get created tags/annotations cause error or don't get created Nov 2, 2020
@nkatarmal-crest
Copy link
Contributor

@tigelane APIC API doesn't allow to pass multiple annotations that way. Also with new versions of APIC tags in the UI are pointing to tagInst object rather than annotation parameter. So you are not able to view it in the UI.

CC: @lhercot @vfiftyfive

@tigelane
Copy link
Author

tigelane commented Nov 2, 2020

Ah thank you @nkatarmal-crest It seems that tagInst isn't implemented in the provider yet. Maybe this should be a feature request vs bug?

@nkatarmal-crest
Copy link
Contributor

@tigelane It should be a feature request for a new resource. I am closing this one out as it's not causing any issues in the code.

RutvikS-crest added a commit that referenced this issue Aug 5, 2022
* Updated Read function in access_switch_policy_group

* User security role update (#151)

* Updated fabricNodeControl and userSecurityDomainRole

* Updated user security domain

* Updated Read Function for UI Deletion Test case (#152)

* added set for annotation in ldap_group_map

* Updated resources (#155)

* aaep_to_domain resource file updated, dn changed and description removed (#156)

* updated documentation

* updated documentation for match_rule

* resource_tagannotation updated (#159)

Co-authored-by: Parth-CDS <parth.patel@crestdatasys.com>

* tag_resource schema updated

* Updated Index file of Interface Blacklist

* Updated Mgmt Zone (#162)

* updated docs
RutvikS-crest added a commit that referenced this issue Aug 5, 2022
* Updated Read function in access_switch_policy_group

* User security role update (#151)

* Updated fabricNodeControl and userSecurityDomainRole

* Updated user security domain

* Updated Read Function for UI Deletion Test case (#152)

* added set for annotation in ldap_group_map

* Updated resources (#155)

* aaep_to_domain resource file updated, dn changed and description removed (#156)

* updated documentation

* updated documentation for match_rule

* resource_tagannotation updated (#159)

* tag_resource schema updated

* Updated Index file of Interface Blacklist

* Updated Mgmt Zone (#162)

* updated docs
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

No branches or pull requests

2 participants