-
Notifications
You must be signed in to change notification settings - Fork 91
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
Mark e2e testsuite obsolete and disable it before removal. #659
Mark e2e testsuite obsolete and disable it before removal. #659
Conversation
22a17c4
to
6f7f43a
Compare
/retest |
/lgtm |
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.
As CI shows, e2e-log-parser.sh
file should be also made no-op.
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pedjak 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 |
Motivation
This is the first step of removing the
e2e
test suite in favour ofacceptance
tests (https://issues.redhat.com/browse/APPSVC-648)Changes
This PR disables the
e2e
tests by default unlessENABLE_OBSOLETE_E2E
is set totrue
.A warning like the following is shown instead:
Testing
make test-e2e