Skip to content

Conversation

@HarikaPonna
Copy link
Contributor

@HarikaPonna HarikaPonna commented Oct 8, 2025

Remove BNPP reference

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content

Removed BNPP reference from the codebase as part of cleanup.

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.

@HarikaPonna
Copy link
Contributor Author

/run pipeline

@HarikaPonna HarikaPonna changed the title chore : Remove BNPP reference refactor : Remove BNPP reference Oct 8, 2025
@HarikaPonna
Copy link
Contributor Author

/run pipeline

1 similar comment
@Ak-sky
Copy link
Member

Ak-sky commented Oct 10, 2025

/run pipeline

@Ak-sky
Copy link
Member

Ak-sky commented Oct 10, 2025

  • TestUpgradeFullyConfigurable Test is failing with below
    probably because of

This field is deprecated and will be removed starting with this 1.82.0 release. Please use iam_id field

instead.

         2025/10/10 07:18:23 Terraform apply | This field is deprecated and will be removed starting with this 1.82.0
         2025/10/10 07:18:23 Terraform apply | release. Please use iam_id field instead.
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply | (and one more similar warning elsewhere)
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply | Error: ---
         2025/10/10 07:18:23 Terraform apply | id: terraform-a963604a
         2025/10/10 07:18:23 Terraform apply | summary: 'ReplaceSettingsWithContext failed: Service Unavailable'
         2025/10/10 07:18:23 Terraform apply | severity: error
         2025/10/10 07:18:23 Terraform apply | resource: ibm_config_aggregator_settings
         2025/10/10 07:18:23 Terraform apply | operation: create
         2025/10/10 07:18:23 Terraform apply | component:
         2025/10/10 07:18:23 Terraform apply |   name: github.com/IBM-Cloud/terraform-provider-ibm
         2025/10/10 07:18:23 Terraform apply |   version: 1.82.1
         2025/10/10 07:18:23 Terraform apply | ---
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply |   with module.app_config.ibm_config_aggregator_settings.config_aggregator_settings[0],
         2025/10/10 07:18:23 Terraform apply |   on ../../main.tf line 179, in resource "ibm_config_aggregator_settings" "config_aggregator_settings":
         2025/10/10 07:18:23 Terraform apply |  179: resource "ibm_config_aggregator_settings" "config_aggregator_settings" {
  • @HarikaPonna, can you please check with AppConfig team over slack.
  • You can create another issue for this error - > This field is deprecated and will be removed starting with this 1.82.0 release. Please use iam_id field and work on this seperately.

@HarikaPonna
Copy link
Contributor Author

  • TestUpgradeFullyConfigurable Test is failing with below
    probably because of

This field is deprecated and will be removed starting with this 1.82.0 release. Please use iam_id field

instead.

         2025/10/10 07:18:23 Terraform apply | This field is deprecated and will be removed starting with this 1.82.0
         2025/10/10 07:18:23 Terraform apply | release. Please use iam_id field instead.
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply | (and one more similar warning elsewhere)
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply | Error: ---
         2025/10/10 07:18:23 Terraform apply | id: terraform-a963604a
         2025/10/10 07:18:23 Terraform apply | summary: 'ReplaceSettingsWithContext failed: Service Unavailable'
         2025/10/10 07:18:23 Terraform apply | severity: error
         2025/10/10 07:18:23 Terraform apply | resource: ibm_config_aggregator_settings
         2025/10/10 07:18:23 Terraform apply | operation: create
         2025/10/10 07:18:23 Terraform apply | component:
         2025/10/10 07:18:23 Terraform apply |   name: github.com/IBM-Cloud/terraform-provider-ibm
         2025/10/10 07:18:23 Terraform apply |   version: 1.82.1
         2025/10/10 07:18:23 Terraform apply | ---
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply | 
         2025/10/10 07:18:23 Terraform apply |   with module.app_config.ibm_config_aggregator_settings.config_aggregator_settings[0],
         2025/10/10 07:18:23 Terraform apply |   on ../../main.tf line 179, in resource "ibm_config_aggregator_settings" "config_aggregator_settings":
         2025/10/10 07:18:23 Terraform apply |  179: resource "ibm_config_aggregator_settings" "config_aggregator_settings" {
  • @HarikaPonna, can you please check with AppConfig team over slack.
  • You can create another issue for this error - > This field is deprecated and will be removed starting with this 1.82.0 release. Please use iam_id field and work on this seperately.

@Ak-sky thankyou , I will check on this

@HarikaPonna
Copy link
Contributor Author

https://ibm-cloudplatform.slack.com/archives/C018V8U2P1R/p1760098382341849
Will try to re-run and check if issue is still persisting

@HarikaPonna
Copy link
Contributor Author

/run pipeline

@HarikaPonna HarikaPonna requested a review from Aashiq-J October 10, 2025 15:35
@HarikaPonna
Copy link
Contributor Author

HarikaPonna commented Oct 13, 2025

resolved conflicts,will re-run pipeline

@HarikaPonna
Copy link
Contributor Author

/run pipeline

@HarikaPonna
Copy link
Contributor Author

/run pipeline

@maheshwarishikha maheshwarishikha merged commit 5376973 into main Oct 13, 2025
2 checks passed
@maheshwarishikha maheshwarishikha deleted the Remove_bnpp branch October 13, 2025 13:20
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 1.12.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

5 participants