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

update the install type to Cli instead of CLi #2144

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

kobihk
Copy link
Contributor

@kobihk kobihk commented Dec 24, 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

Signed-off-by: Kobi Hakimi <khakimi@redhat.com>
@@ -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
Copy link
Contributor

Robot Results

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

@kobihk kobihk added the verified This PR has been tested with Jenkins label Dec 24, 2024
@kobihk kobihk changed the title update the install type to cli instead of CLi update the install type to Cli instead of CLi Dec 24, 2024
@kobihk kobihk merged commit 114f6f2 into red-hat-data-services:master Dec 25, 2024
11 checks passed
@kobihk kobihk deleted the update_cli_case_master branch December 25, 2024 13:39
kobihk added a commit that referenced this pull request Dec 26, 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
kobihk added a commit that referenced this pull request Dec 26, 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
kobihk added a commit that referenced this pull request Dec 26, 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
kobihk added a commit that referenced this pull request Dec 26, 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
kobihk added a commit that referenced this pull request Dec 26, 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
kobihk added a commit that referenced this pull request Dec 26, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants