-
Notifications
You must be signed in to change notification settings - Fork 21
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
OLS-1058 - Adding OLS e2e tests #416
base: main
Are you sure you want to change the base?
Conversation
Adding OLS e2e tests
d519a73
to
dfb52a6
Compare
.tekton/integration-tests/pipelines/lightspeed-integration-test-pipeline.yaml
Show resolved
Hide resolved
.tekton/integration-tests/pipelines/lightspeed-integration-test-pipeline.yaml
Outdated
Show resolved
Hide resolved
dfb52a6
to
b8c14f3
Compare
Adding kubeconfig Adding kubeconfig Adding kubeconfig Adding kubeconfig
a49133a
to
03839f2
Compare
2991531
to
5c232e8
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: raptorsun The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/override "Red Hat Konflux / ols-enterprise-contract / test-bundle" |
@raptorsun: Overrode contexts on behalf of raptorsun: Red Hat Konflux / ols-enterprise-contract / bundle, Red Hat Konflux / ols-enterprise-contract / test-bundle In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
5c232e8
to
38c12a6
Compare
New changes are detected. LGTM label has been removed. |
38c12a6
to
5878ca0
Compare
make install-deps | ||
make install-deps-test | ||
pdm run pytest tests/e2e -s --durations=0 -o junit_suite_name="${SUITE_ID}" -m "${TEST_TAGS}" --junit-prefix="${SUITE_ID}" --junit-xml="${ARTIFACT_DIR}/junit_e2e_${SUITE_ID}.xml" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not make test-e2e
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make test-e2e
was causing a bunch of dependency issues:
python not installed
pytest not installed
ReportPortalClient not installed
Basically everything that was dependent on pdm was not working. So, due to the short timeframe we have for this, I decided to add the pdm run pytest ...
89b996c
to
936ff7e
Compare
8654506
to
1145ed8
Compare
Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Attempting to fix image reference issue Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding 4.17 cluster pool for ols Adding OLS installation via operator-sdk to work in konflux Adding OLS installation via operator-sdk to work in konflux Adding exception to operator installation for konflux Adding exception to operator installation for konflux Adding exception to operator installation for konflux Adding exception to operator installation for konflux Adding exception to operator installation for konflux Adding exception to operator installation for konflux Adding exception to operator installation for konflux Adding operator tests for 4.15 and 4.17 Adding operator tests for 4.15 and 4.17 Adding operator tests for 4.15 and 4.17 Adding operator tests for 4.15 and 4.17 Adding operator tests for 4.15 and 4.17 Removing the get
1145ed8
to
9052a52
Compare
@JoaoFula: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Description
Type of change
Related Tickets & Documents
Checklist before requesting a review
Testing