Skip to content

Commit

Permalink
remove test tags
Browse files Browse the repository at this point in the history
  • Loading branch information
kobihk committed Nov 26, 2024
1 parent 3f86a41 commit 3547f04
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 40 deletions.
1 change: 0 additions & 1 deletion ods_ci/tasks/Resources/RHODS_OLM/install/oc_install.robot
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,6 @@ Wait For Pods Numbers
${status} Set Variable False
FOR ${counter} IN RANGE ${timeout}
${return_code} ${output} Run And Return Rc And Output oc get pod -n ${namespace} -l ${label_selector} | tail -n +2 | wc -l
Log To Console Output: ${output} RC: ${return_code} counter: ${counter} timeout: ${timeout} NS: ${namespace} label: ${label_selector}
IF ${output} == ${count}
${status} Set Variable True
Log To Console pods ${label_selector} created
Expand Down
2 changes: 1 addition & 1 deletion ods_ci/tests/Resources/Common.robot
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ Wait Until Csv Is Ready
Sleep ${check_interval}
${csv_created}= Run Process oc get csv --no-headers -n ${operators_namespace} | awk '/${display_name}/ {print \$1}' shell=yes
Log To Console The Result of csv_created Output is: ${csv_created.stdout} RC: ${csv_created.rc}
IF '${csv_created.stdout}' == '${EMPTY}' CONTINUE
IF '$csv_created.stdout' == '${EMPTY}' CONTINUE
# In case of upgrade there are 2 operators, we need to wait until only one will be available
${lines}= Split String ${csv_created.stdout} \n
${line_count}= Get Length ${lines}
Expand Down
4 changes: 3 additions & 1 deletion ods_ci/tests/Resources/OCP.resource
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,9 @@ Get Resource Attribute
Wait For Installplan And Approve It

Check warning

Code scanning / Robocop

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

Line is too long (126/120)
[Documentation] Wait for the operator's installplan to appear then approve it in case of Manual installplan approval
[Arguments] ${namespace} ${operator_name} ${operator_version}=${EMPTY}

Check warning

Code scanning / Robocop

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

Line is too long (124/120)
Wait Until Keyword Succeeds 10m 5s Check If Resource Exists By Command Output oc get installplans -n ${namespace} -o yaml | grep "${operator_name}"
${csv} = Evaluate '${operator_name}.${operator_version}' if '${operator_version}' else '${operator_name}'

Check warning

Code scanning / Robocop

The assignment sign is not consistent within the file. Expected '{{ expected_sign }}' but got '{{ actual_sign }}' instead Warning test

The assignment sign is not consistent within the file. Expected '=' but got ' =' instead

Check notice

Code scanning / Robocop

'{{ block_name }}' condition can be simplified Note test

'IF' condition can be simplified
Log To Console The csv is: ${csv}
Wait Until Keyword Succeeds 10m 5s Check If Resource Exists By Command Output oc get installplans -n ${namespace} -o yaml | grep "${csv}"

Check warning

Code scanning / Robocop

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

Line is too long (147/120)

Check warning

Code scanning / Robocop

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

Line is too long (153/120)
${installplan_name}= Run oc get subscription ${operator_name} -n ${namespace} -o json | jq '.status.installplan.name'
IF "${operator_version}" != "${EMPTY}"
${installplan_version_exist}= Run And Return Rc oc get installplan -n ${namespace} | grep "${operator_version}"
Expand Down
14 changes: 1 addition & 13 deletions ods_ci/tests/Resources/Page/Operators/ISVs.resource
Original file line number Diff line number Diff line change
Expand Up @@ -38,19 +38,7 @@ Install ISV Operator From OperatorHub Via CLI # robocop: disable
Oc Apply kind=Subscription src=${operator_sub_filepath}
Wait Until Keyword Succeeds 1 min 0 sec
... Is Resource Present Subscription ${subscription_name} ${namespace} ${IS_PRESENT}
# Wait For Installplan And Approve It ${namespace} ${operator_name}
WHILE "${installplan_name}" == "${EMPTY}" limit=5m
${installplan_name}= Get Resource Attribute ${namespace}
... Subscription ${subscription_name} .status.installPlanRef.name
Sleep 20s
END
${installplan_approval}= Get Resource Attribute ${namespace}
... InstallPlan ${installplan_name} .spec.approval
IF "${installplan_approval}" == "Manual"
${return_code} ${out}= Run And Return Rc And Output oc patch installplan ${installplan_name} -n ${namespace} --type='json' -p '[{"op": "replace", "path": "/spec/approved", "value": true}]' #robocop:disable
Should Be Equal As Integers ${return_code} 0 msg=Error while approving installplan
END
# until here
Wait For Installplan And Approve It ${namespace} ${operator_name}

Create Operator Group
[Documentation] Creates the Operator Group object which might be needed by an operator.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ Resource ../../../Resources/Page/DistributedWorkloads/DistributedWorkl
Resource ../../../Resources/Page/DistributedWorkloads/WorkloadMetricsUI.resource
Suite Setup Dashboard Suite Setup
Suite Teardown RHOSi Teardown
Test Tags PreUpgradeKobi
Test Tags PreUpgrade


*** Variables ***
Expand All @@ -39,11 +39,6 @@ ${MODEL_CREATED}= ${FALSE}
${RUNTIME_NAME}= Model Serving Test

*** Test Cases ***
Test PreUpgrade
[Documentation] Test for pre upgrade
[Tags] PreUpgrade
Log To Console This is test of PreUpgrade

Set PVC Size Via UI
[Documentation] Sets a Pod toleration via the admin UI
[Tags] Upgrade
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Resource ../../../Resources/Page/ODH/ODHDashboard/ODHDashboardSetting
Resource ../../../Resources/Page/ODH/JupyterHub/ODHJupyterhub.resource
Library DebugLibrary
Library JupyterLibrary
Test Tags DuringUpgradeKobi
Test Tags DuringUpgrade


*** Variables ***
Expand All @@ -18,11 +18,6 @@ ${UPGRADE_TO_IIB}= ${EMPTY}


*** Test Cases ***
Test DuringUpgrade
[Documentation] Test for during upgrade
[Tags] DuringUpgrade
Log To Console This is test of DuringUpgrade

Long Running Jupyter Notebook
[Documentation] Launch a long running notebook before the upgrade
[Tags] Upgrade
Expand All @@ -37,7 +32,6 @@ Upgrade RHODS
[Documentation] Approve the install plan for the upgrade and make sure that upgrade has completed
[Tags] ODS-1766
... Upgrade
... DuringUpgrade
${initial_version} = Get RHODS Version
${initial_creation_date} = Get Operator Pod Creation Date
Set Suite Variable ${UPDATE_CHANNEL} ${UPGRADE_TO_UPDATE_CHANNEL}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ Resource ../../../Resources/Page/OCPDashboard/Pods/Pods.robot
Resource ../../../Resources/Page/OCPDashboard/Builds/Builds.robot
Resource ../../../Resources/Page/HybridCloudConsole/OCM.robot
Resource ../../../Resources/Page/DistributedWorkloads/DistributedWorkloads.resource
Test Tags PostUpgradeKobi
Test Tags PostUpgrade


*** Variables ***
Expand All @@ -35,11 +35,6 @@ ${RUNTIME_NAME}= Model Serving Test


*** Test Cases ***
Test PostUpgrade
[Documentation] Test for post upgrade
[Tags] PostUpgrade
Log To Console This is test of PostUpgrade

Verify PVC Size
[Documentation] Verify PVC Size after the upgrade
[Tags] Upgrade
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,6 @@ Test Tags ExcludeOnODH


*** Test Cases ***
Test Smoke
[Documentation] Test for smoke
[Tags] SmokeKobi
Log To Console This is test of SmokeKobi

Test Existence of Prometheus Alerting Rules
[Documentation] Verifies the prometheus alerting rules
[Tags] Smoke
Expand Down

0 comments on commit 3547f04

Please sign in to comment.