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

Speed-up execution of keyword "Is RHODS Self-Managed" in local #1366

Conversation

jgarciao
Copy link
Contributor

@jgarciao jgarciao commented Apr 9, 2024

The keyword RHOSi Setup calls keyword Is RHODS Self-Managed, which is using the Robot Framework OpenShiftLibrary to perform Oc Get ...

The problem is that the OpenShiftLibrary library takes more than 10s to load for the first time, slowing down development in local.

This PR modifies the Is RHODS Self-Managed keyword not to use the OpenShift Library in order to speed up execution.

Tested successfully in local and rhods-ci-pr-test/2683

Original:
speed-if-rhods-is-self-managed-original-managed

With this PR:
speed-if-rhods-is-self-managed-updated-managed

RHOSi Setup is calling the keyword "Is RHODS Self-Managed", which is
currently using the Robot Framework OpenShiftLibrary. The problem is
that this library takes more than 10s to load, slowing down development
in local. This PR modifies the keyword not to use that library in this
particular keyword

Signed-off-by: Jorge Garcia Oncins <[email protected]>
@jgarciao jgarciao added the enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) label Apr 9, 2024
ods_ci/tests/Resources/OCP.resource Dismissed Show dismissed Hide dismissed
ods_ci/tests/Resources/OCP.resource Fixed Show fixed Hide fixed
@jgarciao jgarciao requested a review from asanzgom April 9, 2024 15:45
Copy link
Contributor

github-actions bot commented Apr 9, 2024

Robot Results

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

Signed-off-by: Jorge Garcia Oncins <[email protected]>
jiridanek
jiridanek previously approved these changes Apr 9, 2024
@jgarciao jgarciao requested a review from diegolovison April 9, 2024 16:00
@jgarciao jgarciao added the verified This PR has been tested with Jenkins label Apr 9, 2024
FedeAlonso
FedeAlonso previously approved these changes Apr 9, 2024
@jgarciao jgarciao dismissed stale reviews from FedeAlonso and jiridanek via 1d4e8cb April 9, 2024 16:22
Copy link

sonarqubecloud bot commented Apr 9, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.6% Duplication on New Code

See analysis details on SonarCloud

@jgarciao jgarciao merged commit 13fb4aa into red-hat-data-services:master Apr 9, 2024
8 checks passed
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) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants