Skip to content

Conversation

@imprateeksh
Copy link
Member

@imprateeksh imprateeksh commented Oct 21, 2025

Description

This PR introduces the quick start variation of Landing zone for cloud-native AI applications aka RAG Stack.

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content
  • Quick start variation of Landing zone for cloud-native AI applications
  • Quick provisioning of watsonx.ai resources, watsonx Assistant, watson Discovery, Databases for Elasticsearch, Container Registry and Code Engine for rapid deployment of RAG based AI application along with Object Storage and Key Protect services.

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 imprateeksh self-assigned this Oct 21, 2025
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.

@imprateeksh I left some initial comments. I think we need to get the overall content reviewed because the DA is getting renamed to "Cloud-native AI" with less focus on the term "RAG".
I also think I'll create a new repo for the quickstart variation so its separate to the stack code. They can have different release cadence then since the code is not re-used across the stack and quickstart variations. We can continue with this PR until I get the new repo set up

@ocofaigh
Copy link
Contributor

ocofaigh commented Nov 3, 2025

@imprateeksh do you want to move the code over to https://github.com/terraform-ibm-modules/terraform-ibm-cloud-native-ai ? Please bare in mind that by doing that we will currently have to duplicate the offering level content in the ibm_catalog.json until catalog give us a way to not duplicate.

healthcheck = false
}

provider "restapi" {
Copy link
Contributor

Choose a reason for hiding this comment

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

what module requires restapi?

Copy link
Member Author

@imprateeksh imprateeksh Nov 6, 2025

Choose a reason for hiding this comment

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

watsonx.ai requires restapi, if not declared, like we usually do and let the referenced module take care of it, getting the below error, hence explicitly added the provider block.

image

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 tests should be using the schematics test function. All DA tests should be using schematics as thats how consumer will be deploying it. It also allows us to use private endpoints,

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh imprateeksh marked this pull request as ready for review November 7, 2025 10:08
@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

imprateeksh commented Nov 7, 2025

I am looking into the Test failure in QuickStart DA - seen due to code engine build module but other existing Tests have also failed.

@ocofaigh
Copy link
Contributor

ocofaigh commented Nov 7, 2025

@imprateeksh No point in running the tests in this repo - your going to trigger the stack tests too. Please move to the other repo

@imprateeksh
Copy link
Member Author

Closing this PR as the new repo will be used for quick start variation.

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.

3 participants