Skip to content

Conversation

@ChristopherJHart
Copy link
Contributor

Summary

Updates the BFD multi-hop template module to correctly map the schema attribute name to the updated provider attribute name for meticulous SHA-1 authentication.

Problem

The provider fixed an xpath bug and standardized the attribute name:

  • Schema: authentication_meticulous_sha1_keychain
  • Provider (old, broken): authentication_meticulous_sha_1keychain
  • Provider (new, fixed): authentication_meticulous_sha_1_keychain

The module needed to be updated to translate between the schema name and the new provider name.

Change

Updated iosxe_bfd.tf to map from the schema attribute to the provider attribute:

# Before (broken - referenced non-existent provider attribute)
authentication_meticulous_sha_1_keychain = try(template.authentication_meticulous_sha_1_keychain, ...)

# After (fixed - maps schema name to provider name)
authentication_meticulous_sha_1_keychain = try(template.authentication_meticulous_sha1_keychain, ...)

Dependencies

Requires provider fix: CiscoDevNet/terraform-provider-iosxe#433

Testing

Tested with the following data model:

bfd:
  multi_hop_templates:
    - name: issue-1097-test
      interval_milliseconds_both: 250
      interval_milliseconds_multiplier: 4
      authentication_meticulous_sha1_keychain: test-sha1-keychain

Verified on:

Platform Protocol Result
Cat8kv (17.15) RESTCONF ✅ Pass
Cat8kv (17.12) NETCONF ✅ Pass
Cat9kv (17.15) NETCONF ✅ Pass

Related: netascode/nac-iosxe#1097


🤖 Generated with Claude Code

Update module to correctly map the schema attribute name
(authentication_meticulous_sha1_keychain) to the provider attribute
name (authentication_meticulous_sha_1_keychain).

The schema uses a different naming convention than the provider,
so the module must translate between them.

Related: netascode/nac-iosxe#1097

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@aitestino aitestino added the bug Something isn't working label Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants