Skip to content

TPT-4399: Make nat_1_1 attribute as non-computed#2341

Open
zliang-akamai wants to merge 3 commits intolinode:devfrom
zliang-akamai:zhiwei/fix-instance-nat-1-1
Open

TPT-4399: Make nat_1_1 attribute as non-computed#2341
zliang-akamai wants to merge 3 commits intolinode:devfrom
zliang-akamai:zhiwei/fix-instance-nat-1-1

Conversation

@zliang-akamai
Copy link
Copy Markdown
Member

@zliang-akamai zliang-akamai commented May 2, 2026

📝 Description

Making nat_1_1 attribute as non-computed to trigger diff when the attribute is removed from the .tf config files.

✔️ How to Test

make test-int PKG_NAME=instanceconfig
make test-int PKG_NAME=instance

@zliang-akamai zliang-akamai changed the title Make nat_1_1 attribute as non-computed TPT-0000: Make nat_1_1 attribute as non-computed May 2, 2026
@zliang-akamai zliang-akamai changed the title TPT-0000: Make nat_1_1 attribute as non-computed TPT-4399: Make nat_1_1 attribute as non-computed May 2, 2026
@zliang-akamai zliang-akamai force-pushed the zhiwei/fix-instance-nat-1-1 branch from 837f313 to b7dcdd3 Compare May 4, 2026 18:31
@zliang-akamai zliang-akamai marked this pull request as ready for review May 4, 2026 18:32
@zliang-akamai zliang-akamai requested review from a team as code owners May 4, 2026 18:32
@zliang-akamai zliang-akamai requested review from a team, ckulinsk, jriddle-linode and mgwoj and removed request for a team May 4, 2026 18:32
@mgwoj
Copy link
Copy Markdown
Contributor

mgwoj commented May 7, 2026

Should we add test exposing this scenario?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Linode Instance network interface schema so nat_1_1 is no longer treated as a computed attribute, allowing Terraform to detect and plan changes when users remove nat_1_1 from their .tf configuration.

Changes:

  • Remove Computed: true from the nat_1_1 attribute within the VPC interface IPv4 schema.
  • Preserve existing behavior for nat_1_1 = "any" via the existing DiffSuppressFunc.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants