Skip to content

Conversation

@HarikaPonna
Copy link
Contributor

@HarikaPonna HarikaPonna commented Oct 14, 2025

Description

Added next steps url for the DA

Release required?

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

Added next steps url for the DA

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

HarikaPonna commented Oct 15, 2025

QuickStart - Financial Services edition
a

Standard - Financial Services edition
Screenshot 2025-10-15 at 12 31 59 PM (1)

Existing VPC
Screenshot 2025-10-23 at 11 02 34 AM

Copy link
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.

The landing zone VSI DA supports creating multiple VSIs - I dont think we can support linking to the individual VSIs because there may be many of them. Instead lets just link to the general VSI overview page - best we can do for this DA I think

@HarikaPonna
Copy link
Contributor Author

HarikaPonna commented Oct 16, 2025

@ocofaigh

  • Yes we are doing the same for the "Standard - Financial Services edition" variation, as multiple instances are created, we’re redirecting users to the overview page.

  • However, for the "QuickStart - Financial Services edition" variation, only two instances are created, and as suggested by @maheshwarishikha, instead of redirecting to the overview page, we plan to redirect users to the respective instance pages.

Please let us know if you have any suggestions on this approach.

ocofaigh

This comment was marked as outdated.

@ocofaigh
Copy link
Contributor

@HarikaPonna consumers of the "QuickStart - Financial Services edition" variation can change the number of VSIs they want to create by updating the value of the override_json_string input. You should use same approach for all variations

@HarikaPonna
Copy link
Contributor Author

HarikaPonna commented Oct 22, 2025

@HarikaPonna consumers of the "QuickStart - Financial Services edition" variation can change the number of VSIs they want to create by updating the value of the override_json_string input. You should use same approach for all variations

got it @ocofaigh thanks for clarifying ,updated the next step urls.

@zikra-iqbal
Copy link
Contributor

Tested the next-step URLs for the VPC landing zone.
Screenshot 2025-10-23 at 12 17 35 PM

@Aashiq-J
Copy link
Member

/run pipeline

@maheshwarishikha maheshwarishikha merged commit 587ffbd into main Oct 28, 2025
2 checks passed
@maheshwarishikha maheshwarishikha deleted the add_next_steps_url branch October 28, 2025 06:05
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 8.9.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.

7 participants