Skip to content

Commit

Permalink
fix alerts
Browse files Browse the repository at this point in the history
  • Loading branch information
bdattoma committed Feb 27, 2024
1 parent 37c30bd commit 5c141ee
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 6 deletions.
9 changes: 6 additions & 3 deletions ods_ci/tests/Resources/CLI/Minio.resource
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ Documentation Collcetion of keywords to deploy and remove MinIO storage
Library OperatingSystem
Resource ../Common.robot


*** Variables ***
${MINIO_RESOURCES_DIRPATH}= ods_ci/tests/Resources/Files/minio
${MINIO_POD_FILEPATH}= ${MINIO_RESOURCES_DIRPATH}/minio.yaml
Expand Down Expand Up @@ -30,11 +31,13 @@ Deploy MinIO
Set Test Variable ${route_name}

Check notice

Code scanning / Robocop

{{ set_variable_keyword }} can be replaced with VAR Note test

Set Test Variable can be replaced with VAR

Check warning

Code scanning / Robocop

Test, suite and global variables should be uppercase Warning test

Test, suite and global variables should be uppercase
Create File From Template ${MINIO_POD_FILEPATH} ${MINIO_RESOURCES_DIRPATH}/minio_filled.yaml
${rc}= Run And Return Rc oc new-project ${namespace}

Check warning

Code scanning / Robocop

Local variable '{{ name }}' is overwritten before usage Warning test

Local variable '${rc}' is overwritten before usage
${rc} ${out}= Run And Return Rc And Output oc apply -f ${MINIO_RESOURCES_DIRPATH}/minio_filled.yaml -n ${namespace}
${rc} ${out}= Run And Return Rc And Output
... oc apply -f ${MINIO_RESOURCES_DIRPATH}/minio_filled.yaml -n ${namespace}
Wait For Pods To Be Ready label_selector=app=minio
... namespace=${namespace}
END
${rc} ${route}= Run And Return Rc And Output oc get route ${route_name} -n ${namespace} --template={{.spec.host}}
${rc} ${route}= Run And Return Rc And Output
... oc get route ${route_name} -n ${namespace} --template={{.spec.host}}
Should Be Equal As Integers ${rc} ${0}
RETURN ${route}

Expand All @@ -58,4 +61,4 @@ Get Minio Credentials
Remove Minio
[Documentation] Delete the MinIO objects from the cluster
Log message=To Be Implemented
# for the time being the test using Minio is deleting the NS.
# for the time being the test using Minio is deleting the NS.
3 changes: 1 addition & 2 deletions ods_ci/tests/Resources/CLI/ModelServing/llm.resource
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ Set Project And Runtime
[Documentation] Creates the DS Project (if not exists), creates the data connection for the models,
... creates caikit runtime. This can be used as test setup
[Arguments] ${namespace} ${enable_metrics}=${FALSE} ${runtime}=caikit-tgis-runtime ${protocol}=grpc
... ${access_key_id}=${S3.AWS_ACCESS_KEY_ID} ${access_key}=${S3.AWS_SECRET_ACCESS_KEY}
... ${access_key_id}=${S3.AWS_ACCESS_KEY_ID} ${access_key}=${S3.AWS_SECRET_ACCESS_KEY}

Check notice

Code scanning / Robocop

There is too many arguments per continuation line ({{ arguments_count }} / {{ max_arguments_count }}) Note test

There is too many arguments per continuation line (2 / 1)
... ${endpoint}=${MODELS_BUCKET.ENDPOINT} ${verify_ssl}=${TRUE}

Check notice

Code scanning / Robocop

There is too many arguments per continuation line ({{ arguments_count }} / {{ max_arguments_count }}) Note test

There is too many arguments per continuation line (2 / 1)
Set Up Test OpenShift Project test_ns=${namespace}
Create Secret For S3-Like Buckets endpoint=${endpoint}
Expand Down Expand Up @@ -112,7 +112,6 @@ Create Secret For S3-Like Buckets
... oc annotate secret ${name} -n ${namespace} serving.kserve.io/s3-verifyssl='false' --overwrite
Should Be Equal As Integers ${rc} ${0}
END


Compile Inference Service YAML
[Documentation] Prepare the Inference Service YAML file in order to deploy a model
Expand Down
10 changes: 9 additions & 1 deletion ods_ci/tests/Resources/Files/minio/minio.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ metadata:
app: minio
name: ${podname}
spec:
automountServiceAccountToken: false
containers:
- args:
- server
Expand All @@ -45,4 +46,11 @@ spec:
value: ${pw}
image: ${minio_image}
imagePullPolicy: Always
name: minio
name: minio
resources:
limits:
cpu: 250m
memory: 10Gi
requests:
cpu: 20m
memory: 400Mi

0 comments on commit 5c141ee

Please sign in to comment.