Skip to content

Conversation

@akocbek
Copy link
Contributor

@akocbek akocbek commented Dec 19, 2024

Description

Issue: #297

  • existing_kms_key_crn is currently in the required list but should be moved to optiona
  • move use_existing_resource_group next to resource_group_name in ui (on required).
  • elser_model_type -> this is only applicable if plan is platinum (not for enterprise). At minumum indicate it in the description of the variable.
  • skip_es_sm_auth_policy -> use full service name when they are not obvious to typical end-user (skip_elastic_search_secret_manager_auth_policy or skip_elastic_search_to_secret_manager_auth_policy)
  • existing_db_instance_crn -> use elastic_search instead of db for consistency in naming.
  • skip_iam_authorization_policy and skip_es_sm_auth_policy are inconsistent in term of naming.
  • group related inputs together in ui - eg: all kms related input together.

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.

@akocbek
Copy link
Contributor Author

akocbek commented Dec 19, 2024

/run pipeline

2 similar comments
@akocbek
Copy link
Contributor Author

akocbek commented Jan 8, 2025

/run pipeline

@akocbek
Copy link
Contributor Author

akocbek commented Jan 15, 2025

/run pipeline

@akocbek
Copy link
Contributor Author

akocbek commented Feb 10, 2025

/run pipeline

@akocbek
Copy link
Contributor Author

akocbek commented Feb 10, 2025

/run pipeline

@akocbek
Copy link
Contributor Author

akocbek commented Feb 10, 2025

/run pipeline

@akocbek
Copy link
Contributor Author

akocbek commented Feb 10, 2025

/run pipeline

@akocbek
Copy link
Contributor Author

akocbek commented Feb 10, 2025

/run pipeline

@ocofaigh
Copy link
Contributor

/run pipeline

@ocofaigh ocofaigh merged commit 65f7a90 into main Feb 17, 2025
2 checks passed
@ocofaigh ocofaigh deleted the small_fixes branch February 17, 2025 16:10
@terraform-ibm-modules-ops
Copy link
Contributor

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

5 participants