Skip to content

Conversation

@ocofaigh
Copy link
Contributor

@ocofaigh ocofaigh commented Dec 12, 2025

Description

  • Doc cleanup
  • Removed duplicate inputs from ibm_catalog.json (this is now caught in updated pre-commit hook)
  • updated the standard variations to use Elasticsearch platinum plan, b3c.4x16.encrypted host flavor, 3 dedicated CPUs, and deployment of the elsr model. These all used to be in the the standard varitation but seems were incorrect removed in v4.
  • Bumped to latest RAG sample app DA which has several fixes in it

Release required?

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

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.

@ocofaigh
Copy link
Contributor Author

/run pipeline

@ocofaigh
Copy link
Contributor Author

/run pipeline

@ocofaigh
Copy link
Contributor Author

github issues:

 2025/12/13 12:26:32 Terraform init | Error: Failed to install provider
 2025/12/13 12:26:32 Terraform init | 
 2025/12/13 12:26:32 Terraform init | Error while installing ibm-cloud/ibm v1.84.3: could not query provider
 2025/12/13 12:26:32 Terraform init | registry for registry.terraform.io/ibm-cloud/ibm: failed to retrieve
 2025/12/13 12:26:32 Terraform init | authentication checksums for provider: the request failed after 2 attempts,
 2025/12/13 12:26:32 Terraform init | please try again later: 504 Gateway Timeout returned from github.com

retrying..

@ocofaigh
Copy link
Contributor Author

/run pipeline

@ocofaigh ocofaigh mentioned this pull request Dec 16, 2025
6 tasks
Copy link
Member

@imprateeksh imprateeksh left a comment

Choose a reason for hiding this comment

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

The changes look good. Thank you.

@ocofaigh
Copy link
Contributor Author

There are currently some issues with projects in production. Test failed with:
TestProjectsBasicExistingResourcesTest test failed with:

2025/12/16 10:13:25 Error creating config from JSON: Internal Server Error

However both the other tests passed so proceeding with merge here..

@ocofaigh ocofaigh merged commit aff7d0d into main Dec 16, 2025
1 of 2 checks passed
@ocofaigh ocofaigh deleted the index-doc branch December 16, 2025 13:31
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 5.2.0 🎉

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.

4 participants