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

Chore: Added sleep and timeout for while loop #2029

Conversation

asanzgom
Copy link
Contributor

No description provided.

@asanzgom asanzgom added the enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) label Nov 14, 2024
@asanzgom asanzgom self-assigned this Nov 14, 2024
@asanzgom asanzgom enabled auto-merge (squash) November 14, 2024 16:05
@@ -311,7 +311,11 @@
${rc} ${value}= Run And Return Rc And Output
... oc get ${resource_type}/${resource_name} -n ${namespace} -o 'jsonpath={${attribute_path}}'
Should Be Equal "${rc}" "0" msg=${value}
Log To Console ${resource_type} ${resource_name} in the namespace ${namespace} attribute ${attribute_path} value is ${value}
IF "${value}" != "${EMPTY}"
Log To Console ${resource_type} ${resource_name} in the namespace ${namespace} attribute ${attribute_path} value is ${value}

Check warning

Code scanning / Robocop

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

Line is too long (135/120)
Log To Console ${resource_type} ${resource_name} in the namespace ${namespace} attribute ${attribute_path} value is ${value}
IF "${value}" != "${EMPTY}"
Log To Console ${resource_type} ${resource_name} in the namespace ${namespace} attribute ${attribute_path} value is ${value}
ELSE

Check warning

Code scanning / Robocop

Trailing whitespace at the end of line Warning test

Trailing whitespace at the end of line
IF "${value}" != "${EMPTY}"
Log To Console ${resource_type} ${resource_name} in the namespace ${namespace} attribute ${attribute_path} value is ${value}
ELSE
Log To Console ${resource_type} ${resource_name} in the namespace ${namespace} attribute ${attribute_path} value is EMPTY

Check warning

Code scanning / Robocop

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

Line is too long (132/120)
Copy link
Contributor

Robot Results

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

Copy link
Contributor

@MarianMacik MarianMacik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@asanzgom asanzgom merged commit 00e2391 into red-hat-data-services:master Nov 14, 2024
8 checks passed
@asanzgom asanzgom deleted the hotfix/Install_ISV_Operator_From_OperatorHub_Via_CLI branch November 14, 2024 16:21
CFSNM pushed a commit to CFSNM/ods-ci that referenced this pull request Nov 15, 2024
…2029)

* Added sleep and timeout for while loop

* Added If else structure for Gettting Resource Attribute Kw for when value comes empty
CFSNM pushed a commit to CFSNM/ods-ci that referenced this pull request Nov 15, 2024
…2029)

* Added sleep and timeout for while loop

* Added If else structure for Gettting Resource Attribute Kw for when value comes empty
CFSNM added a commit that referenced this pull request Nov 18, 2024
* Added sleep and timeout for while loop

* Added If else structure for Gettting Resource Attribute Kw for when
value comes empty
CFSNM added a commit that referenced this pull request Nov 18, 2024
* Added sleep and timeout for while loop

* Added If else structure for Gettting Resource Attribute Kw for when
value comes empty
lenahorsley pushed a commit to lenahorsley/ods-ci that referenced this pull request Nov 18, 2024
…2029)

* Added sleep and timeout for while loop

* Added If else structure for Gettting Resource Attribute Kw for when value comes empty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants