Skip to content

Conversation

@vkuma17
Copy link
Contributor

@vkuma17 vkuma17 commented Aug 11, 2025

Description

test

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.

terraform-ibm-modules-ops and others added 30 commits June 5, 2024 09:33
* chore(deps): update ci dependencies

* Update catalogValidationValues.json.template

---------

Co-authored-by: Conall Ó Cofaigh <[email protected]>
…rotection` to consider use case when data lookup checks current default version (terraform-ibm-modules#455)
* fix(deps): update terraform-module

* fix the minimum provider version required to 1.66.0 (terraform-ibm-modules#453)

* update version in fscloud

---------

Co-authored-by: Aashiq-J <[email protected]>
Co-authored-by: [email protected] <[email protected]>
…IBM Cloud private endpoints for use cases where the runtime may not have access to the public network. This can be achieved by setting the `use_private_endpoint` variable to `true`.<br>- As part of this effort the `ibmcloud_api_key` input variable has been removed as an input for the module. (terraform-ibm-modules#442)
…odules#469)

* fix: fixed advanced example to handle ingress check SKIP UPGRADE TEST

* fix: fixed gateways
…e inside the `worker_pools` input variable list object. Validation has also been added to ensure only the following allowed values are passed: `REDHAT_8_64`, `RHCOS` (only allowed in OCP version 4.15 or later) (terraform-ibm-modules#466)
mukulpalit-ibm and others added 28 commits July 17, 2025 19:26
… a cluster with a public endpoint enabled by default. This can be toggled using the `disable_public_service_endpoint` input.<br>- If VPC DA is selected (it is by default), there is now an option to configure the network ACLs. By default, it will create ACLs to allow public (port 80 and 443) so the Openshift console can be reached.(terraform-ibm-modules#740)
…tance_crn`, `existing_cluster_kms_key_crn` and `existing_boot_volume_kms_key_crn` inputs (terraform-ibm-modules#751)
…`, which means it will default to the default resource group in your account. Previously it was defaulting to "Default" which may not exist in every account. (terraform-ibm-modules#766)
* feat: Quickstart OCP DA

* Updated catalog,variables.tf,main.tf,pr_test.go

* Added catalogValidationValues.json.template

* fix: Resolved pipeline error

* Removed log file

* fix: Defined Resourcegroup in pr_test

* Resolved precommit error

* updated code

* Removed logfile

* Addressed review comments

* Updated catalog

* SKIP UPGRADE TEST

* Updated permission

* Addressed review comments

* Adressed review comments

* Addressed review comments

* Addressed review comments

* Updated code

* Addressed comments

* Updated decsription

* Addressed review comments

* Updated diagram

* Addressed review comments

* Updated mark down file and variable.tf

* resolved review comments

* addressed review comments

* Update ibm_catalog.json

Co-authored-by: Shikha Maheshwari <[email protected]>

* Update ibm_catalog.json

Co-authored-by: Shikha Maheshwari <[email protected]>

* Update solutions/quickstart/variables.tf

Co-authored-by: Shikha Maheshwari <[email protected]>

* Addressed comments

* Updated catalog validation,resource_group an output.tf

* Resolved precommit error

---------

Co-authored-by: Arya Girish K <[email protected]>
Co-authored-by: Shikha Maheshwari <[email protected]>
@vkuma17
Copy link
Contributor Author

vkuma17 commented Aug 11, 2025

Closing this PR. Raised by mistake.

@vkuma17 vkuma17 closed this Aug 11, 2025
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.