#7638 PR-CI: Make "Not enough resources configured" an error
Closed: fixed 5 years ago Opened 5 years ago by cheimes.

As a developer, I want to have all misconfigured test cases to fail with a clear and visible test failure. The pytest_multihost plugin turns lack of configured resources into silently skipped test instead of a failed test. For example the test case test_caless.py::TestClientInstall::()::test_client_install::setup is currently skipped with message:

"Skipped: Not enough resources configured: Domain 0 not configured: {'type': 'IPA', 'hosts': {'master': 1, 'replica': 0, 'client': 1}}"

but PR-CI lists the test as green. There are multiple more test runs that are silently skipped, because the job doesn't have enough resources configured.

I propose to turn any misconfiguration into an error, so we are able to spot these issues easily. Otherwise we'd have to verify each and every test job manually.


master:

  • 59ef537 Turn multihost config problems into errors

Metadata Update from @cheimes:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

5 years ago

ipa-4-7:

  • 52fa23c Add convenient template for temp commits
  • e44af22 Fix topology configuration of nightly runs
  • 43dde14 Making nigthly test definition editable by FreeIPA's contributors

Login to comment on this ticket.

Metadata