CZERTAINLY / CZERTAINLY-Core

CZERTAINLY - core of the platform managing certificate lifecycle management related tasks
https://www.czertainly.com
MIT License
4 stars 13 forks source link

Periodic discovery marked Failed even if it succeeded. #805

Closed semik closed 1 month ago

semik commented 1 month ago

Describe the bug Periodic discovery marked Failed even if it succeeded.

To Reproduce Steps to reproduce the behavior:

  1. Create periodic discovery for single hostname with cron 0 0/1 * * * ? *
  2. Let it execute for serveral times
  3. See scheduler that Last Execution Status is Failed and all past executions are also failed.
  4. Click on Details of status and see that it actually finished without any problem
  5. Click on discoveries and see that some are completed and some are stuck in Processing.

Expected behavior I think that list of executions of periodic discovery should show true result value.

Screenshots image image image

Additional context Tested on czertainly-core:2.12.0 and on develop version sha256:b4c4a35528beca47b2b2d4a414857fdb0a9d226e8536a1e53c6ab17885a76b3a

Initially I thought it is related to failing notification provider, that is not true. I was able to reproduce this on other instance where notification provider is not set up at all.