Skip to content

fix: add deprecation notice for the vpn_gateways input#1041

Merged
ocofaigh merged 30 commits intomainfrom
deprecate_refs
Jan 20, 2026
Merged

fix: add deprecation notice for the vpn_gateways input#1041
ocofaigh merged 30 commits intomainfrom
deprecate_refs

Conversation

@imprateeksh
Copy link
Copy Markdown
Member

@imprateeksh imprateeksh commented Aug 18, 2025

Description

This PR is to include the documentation about the deprecation of in-module VPN gateway provisioning in the Landing Zone VPC module mentioning about planned removal in upcoming release. The intention is to remove the direct usage of resource in root module and consume site to site VPN module.

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content
  • Resource address changes:
    Root‑level ibm_is_vpn_gateway resource will be refactored into the vpn_gateways module. Once this is done, users must migrate state using terraform state mv command to avoid recreation of ibm_is_vpn_gateway resource.

    Example:

      terraform state mv \
        'module.slz_vpc.ibm_is_vpn_gateway.vpn_gateway["vpg1"]' \
        'module.slz_vpc.module.vpn_gateways["vpg1"].ibm_is_vpn_gateway.vpn_gateway[0]'

    For more details about the migration of vpn gateway resources, please refer the migration guide.

Run the pipeline

If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.

Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:

/run pipeline

Checklist for reviewers

  • If relevant, a test for the change is included or updated with this PR.
  • If relevant, documentation for the change is included or updated with this PR.

For mergers

  • Use a conventional commit message to set the release level. Follow the guidelines.
  • Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
  • Use the Squash and merge option.

vburckhardt
vburckhardt previously approved these changes Sep 3, 2025
@imprateeksh
Copy link
Copy Markdown
Member Author

/run pipeline

Comment thread README.md Outdated
Comment thread README.md Outdated
@imprateeksh imprateeksh requested a review from ocofaigh December 4, 2025 12:36
@imprateeksh
Copy link
Copy Markdown
Member Author

Marked this as major release as per the comment given in the internal issue i.e. getting the new major version released with a migration guide.

@imprateeksh
Copy link
Copy Markdown
Member Author

Before migrating state - Recreation of resource can be seen

image image image

After Migration : Resource recreation is not happening
image

image

Copy link
Copy Markdown
Contributor

@ocofaigh ocofaigh left a comment

Choose a reason for hiding this comment

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

left some comments

Comment thread .docs/migration_guide.md Outdated
Comment thread README.md Outdated
Comment thread variables.tf Outdated
Comment thread README.md Outdated
Comment thread main.tf Outdated
@imprateeksh
Copy link
Copy Markdown
Member Author

/run pipeline

Comment thread main.tf
ocofaigh
ocofaigh previously approved these changes Jan 16, 2026
@imprateeksh
Copy link
Copy Markdown
Member Author

/run pipeline

@imprateeksh
Copy link
Copy Markdown
Member Author

/run pipeline

@ocofaigh ocofaigh changed the title docs: deprecate in-module VPN gateway and introduce site-to-site-vpn module fix: add deprecation notice for the vpn_gateways input Jan 16, 2026
Copy link
Copy Markdown
Contributor

@ocofaigh ocofaigh left a comment

Choose a reason for hiding this comment

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

@imprateeksh You didn't add the ignore to all tests. The TestFullyConfigurableWithFlowLogs test failed with:

        	Messages:   	Resource(s) identified to be destroyed 
        	            	Name: deprecation_warning
        	            	Address: module.vpc.terraform_data.deprecation_warning[0]
        	            	Actions: [delete create]
        	            	DIFF:
        	            	  Before: 
        	            		{"id":"574c7dfe-7846-a7a8-0a22-39e4fa5e2f08","triggers_replace":{"always_run":"2026-01-16T21:31:24Z"}}
        	            	  After: 
        	            		{"triggers_replace":{}}

@imprateeksh
Copy link
Copy Markdown
Member Author

/run pipeline

@imprateeksh
Copy link
Copy Markdown
Member Author

PR failures are now seen with different errors:

@ocofaigh
Copy link
Copy Markdown
Contributor

@imprateeksh There is a bug (here) in the code - the VPC region function should not be failing the tests. @Khuzaima05 is going to fix it in #1082

@maheshwarishikha
Copy link
Copy Markdown
Member

maheshwarishikha commented Jan 19, 2026

I am seeing one deprecate notice related failure in other repo - terraform-ibm-modules/terraform-ibm-observability-da#577 (comment)

Is something missing here?

@ocofaigh
Copy link
Copy Markdown
Contributor

@imprateeksh
Copy link
Copy Markdown
Member Author

/run pipeline

@ocofaigh
Copy link
Copy Markdown
Contributor

Failed with:

         2026/01/20 11:46:10 Terraform destroy | Error: ---
         2026/01/20 11:46:10 Terraform destroy | id: terraform-c907de16
         2026/01/20 11:46:10 Terraform destroy | summary: 'isWaitForSubnetDeleted failed: timeout while waiting for state to become
         2026/01/20 11:46:10 Terraform destroy |   ''done, '' (last state: ''deleting'', timeout: 10m0s)'
         2026/01/20 11:46:10 Terraform destroy | severity: error
         2026/01/20 11:46:10 Terraform destroy | resource: ibm_is_subnet
         2026/01/20 11:46:10 Terraform destroy | operation: delete
         2026/01/20 11:46:10 Terraform destroy | component:
         2026/01/20 11:46:10 Terraform destroy |   name: github.com/IBM-Cloud/terraform-provider-ibm
         2026/01/20 11:46:10 Terraform destroy |   version: 1.87.1
         2026/01/20 11:46:10 Terraform destroy | ---

Unrelated to changes in this Pr, so retrying..

@ocofaigh
Copy link
Copy Markdown
Contributor

/run pipeline

@ocofaigh ocofaigh merged commit 7f953b8 into main Jan 20, 2026
2 checks passed
@ocofaigh ocofaigh deleted the deprecate_refs branch January 20, 2026 12:50
@terraform-ibm-modules-ops
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 8.11.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ocofaigh
Copy link
Copy Markdown
Contributor

ocofaigh commented Jan 21, 2026

@imprateeksh It seems you missed adding this to the IgnoreDestroy:

        	Test:       	TestRunUpgradeFullyConfigurable
        	Messages:   	Resource(s) identified to be destroyed 
        	            	Name: deprecation_warning
        	            	Address: module.vpc.terraform_data.deprecation_warning[0]
        	            	Actions: [delete create]
        	            	DIFF:
        	            	  Before: 
        	            		{"id":"23c3f4a1-5028-f353-8e0a-ee93fec98fe2","triggers_replace":{"always_run":"2026-01-21T17:33:33Z"}}
        	            	  After: 
        	            		{"triggers_replace":{}}

fixed in #1113

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants