openaev
openaev copied to clipboard
If Caldera executor is enabled and Caldera is down, the platform should fail to start
Related issues
- https://github.com/OpenBAS-Platform/openbas/issues/1034
Codecov Report
Attention: Patch coverage is 7.01754% with 53 lines in your changes missing coverage. Please review.
Project coverage is 24.71%. Comparing base (
8019106) to head (2e9c8b5).
Additional details and impacted files
@@ Coverage Diff @@
## master #1201 +/- ##
============================================
- Coverage 24.84% 24.71% -0.14%
- Complexity 1057 1062 +5
============================================
Files 487 489 +2
Lines 12121 12246 +125
Branches 661 667 +6
============================================
+ Hits 3012 3027 +15
- Misses 9018 9127 +109
- Partials 91 92 +1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@RomuDeuxfois do you think this PR is a good occasion to implement also: https://github.com/OpenBAS-Platform/openbas/issues/1206?
@SamuelHassine we can indeed, the subjects are close. But to be clear, for caldera the platform does not crash but displays an error message to the user as mentioned on the ticket by JB https://github.com/OpenBAS-Platform/openbas/issues/1034#issuecomment-2183013530.
In the case of mandatory dependencies, we align with OpenCTI, display logs and failed to start.
Is it all good?
@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar :
It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.
@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar :
It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.
something to align with what is currently implemented at OpenCTI side.
@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar :
It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.
You can still scroll without worries
@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar :
It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.
something to align with what is currently implemented at OpenCTI side.
It's align
@Dimfacion ready for a second round
It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.
It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.