Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[2.16.0] update the install type to cli instead of CLi #2145

Merged

Conversation

kobihk
Copy link
Contributor

@kobihk kobihk commented Dec 25, 2024

To make it easier to use the Install/Uninstall type which currently is CLi in part of the code and Cli in another part of the code.
changing it from CLi to Cli everywhere.

related to Jira ticket: RHOAIENG-2720

cherry-picked #2144

Signed-off-by: Kobi Hakimi <khakimi@redhat.com>
@kobihk kobihk self-assigned this Dec 25, 2024
@@ -63,12 +63,12 @@
FAIL Provided test environment and install type is not supported
END
ELSE IF "${cluster_type}" == "managed"
IF "${TEST_ENV}" in "${SUPPORTED_TEST_ENV}" and "${INSTALL_TYPE}" == "CLi" and "${UPDATE_CHANNEL}" == "odh-nightlies"
IF "${TEST_ENV}" in "${SUPPORTED_TEST_ENV}" and "${INSTALL_TYPE}" == "Cli" and "${UPDATE_CHANNEL}" == "odh-nightlies"

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning

Line is too long (124/120)
@@ -18,7 +18,7 @@
${SERVICE_MESH_OPERATOR_DEPLOYMENT_NAME} istio-operator
${SERVICE_MESH_CR_NS} istio-system
${SERVICE_MESH_CR_NAME} data-science-smcp
${INSTALL_TYPE} CLi
${INSTALL_TYPE} Cli

Check notice

Code scanning / Robocop

Variable '{{ name }}' is assigned but not used Note test

Variable '${INSTALL_TYPE}' is assigned but not used
@@ -19,7 +19,7 @@
${SERVICE_MESH_CR_NS} istio-system
${SERVICE_MESH_CR_NAME} data-science-smcp
${OLM_DIR} olm
${INSTALL_TYPE} CLi
${INSTALL_TYPE} Cli

Check notice

Code scanning / Robocop

Variable '{{ name }}' is assigned but not used Note test

Variable '${INSTALL_TYPE}' is assigned but not used
@kobihk kobihk changed the title update the install type to cli instead of CLi [2.16.0] update the install type to cli instead of CLi Dec 25, 2024
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
586 0 0 586 100

@kobihk kobihk merged commit 2939582 into red-hat-data-services:releases/2.16.0 Dec 26, 2024
11 checks passed
@kobihk kobihk deleted the update_cli_case_2.16.0 branch December 26, 2024 13:32
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