Skip to content

Commit

Permalink
move aws var in setup
Browse files Browse the repository at this point in the history
  • Loading branch information
bdattoma committed Mar 11, 2024
1 parent 9bcda52 commit 2a92882
Showing 1 changed file with 4 additions and 12 deletions.
16 changes: 4 additions & 12 deletions ods_ci/tests/Resources/CLI/ModelServing/llm.resource
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,7 @@ Set Project And Runtime
Set Up Test OpenShift Project test_ns=${namespace}
IF ${download_in_pvc}
Create PVC And Download Model From S3 model_name=${model_name} namespace=${namespace} bucket_name=${MODELS_BUCKET.NAME}
... endpoint=${MODELS_BUCKET.ENDPOINT} region=${MODELS_BUCKET.REGION} access_key_id=${S3.AWS_ACCESS_KEY_ID}
... access_key=${S3.AWS_SECRET_ACCESS_KEY} use_https=${USE_BUCKET_HTTPS} download_timeout=${download_timeout}
... use_https=${USE_BUCKET_HTTPS} download_timeout=${download_timeout}
... storage_size=${storage_size} model_path=${model_path}
ELSE
Create Secret For S3-Like Buckets endpoint=${endpoint}
Expand Down Expand Up @@ -671,17 +670,13 @@ Clean Up Test Project
END

Create PVC And Download Model From S3
[Arguments] ${model_name} ${bucket_name} ${endpoint}
... ${region} ${access_key_id} ${access_key}
[Arguments] ${model_name} ${bucket_name}

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)
... ${use_https} ${namespace} ${storage_size}
... ${model_path} ${download_timeout}=500s
Set Log Level NONE
Set Test Variable ${model_name}
Set Test Variable ${bucket_name}
Set Test Variable ${endpoint}
Set Test Variable ${region}
Set Test Variable ${access_key_id}
Set Test Variable ${access_key}
# AWS variables defined in the Setup Test Variables
Set Test Variable ${use_https}
Set Test Variable ${namespace}
Set Test Variable ${storage_size}
Expand Down Expand Up @@ -716,10 +711,7 @@ Download Prompts Weights In PVC
Set Test Variable ${prompts_path}

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
Set Test Variable ${model_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
Set Test Variable ${bucket_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
# Set Test Variable ${endpoint}
# Set Test Variable ${region}
# Set Test Variable ${access_key_id}
# Set Test Variable ${access_key}
# AWS variables defined in the Setup Test Variables
Set Test Variable ${use_https}

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
Set Test Variable ${namespace}

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
Set Test Variable ${storage_size}

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
Expand Down

0 comments on commit 2a92882

Please sign in to comment.