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

Remove obsolete tests from test_caless.py #2779

Closed
wants to merge 1 commit into from

Conversation

wladich
Copy link

@wladich wladich commented Jan 22, 2019

Related issue #4270 is closed as "won't fix" after 4 years.
The tests are obsolete now.

See: https://pagure.io/freeipa/issue/4271
See: https://pagure.io/freeipa/issue/4270

Signed-off-by: Sergey Orlov sorlov@redhat.com

Related issue freeipa#4270 is closed as "won't fix" after 4 years.
The tests are obsolete now.

See: https://pagure.io/freeipa/issue/4271
See: https://pagure.io/freeipa/issue/4270

Signed-off-by: Sergey Orlov <sorlov@redhat.com>
@wladich wladich added the re-run Trigger a new run of PR-CI label Jan 23, 2019
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Jan 23, 2019
@wladich wladich added the needs review Pull Request is waiting for a review label Jan 23, 2019
@flo-renaud flo-renaud self-assigned this Jan 29, 2019
@flo-renaud
Copy link
Contributor

Hi @wladich
Thanks for the PR, LGTM.

@flo-renaud flo-renaud added ack Pull Request approved, can be merged and removed needs review Pull Request is waiting for a review labels Jan 29, 2019
@rcritten
Copy link
Contributor

master:

  • 13f2085 Remove obsolete tests from test_caless.py

@rcritten rcritten added the pushed Pull Request has already been pushed label Jan 29, 2019
@rcritten rcritten closed this Jan 29, 2019
@rcritten
Copy link
Contributor

Manual backport to ipa-4-7 branch is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants