Skip to content

Conversation

@imprateeksh
Copy link
Member

@imprateeksh imprateeksh commented Nov 26, 2025

Description

fixes: #257
This PR addresses the below changes -

a) Failure seen in renovate PR

Package Change Age Confidence
github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper v1.60.27 -> v1.60.31 age confidence

b) Changes w.r.t diagram updates this includes -

  • Added the missing Activity Tracker Event Routing in therag-stack-without-application.
  • Fixed typo in the existing rag-pattern diagram.
  • Updated rag-stack.svg to use SCC workload protection and use latest icons.
  • Existing diagram was a image, now changed to a .svg and can be edited in the respective tool.

c) Changed the index of variation to now include quickstart variation as 1st variation.
This is done to accommodate Quick start variation as the first variation. For details, please refer this comment.

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.

@imprateeksh
Copy link
Member Author

Please note - The indexing of variations has been updated as part of this PR

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

Ran test to ensure the latest go related changes are working fine. Found Test Failures as -

Failed to install provider
image

Secrets Manager failure was seen
image

This failure is unrelated to the changes in the PR and when checked the logs, secrets manager failed due to provider installation failure which is later seen as passed as well :

image

@imprateeksh
Copy link
Member Author

/run pipeline

@ocofaigh ocofaigh merged commit 7a35e3f into main Nov 27, 2025
1 of 2 checks passed
@ocofaigh ocofaigh deleted the fix-renovate branch November 27, 2025 14:58
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.

Review RAG stack diagrams

3 participants