openaev icon indicating copy to clipboard operation
openaev copied to clipboard

If Caldera executor is enabled and Caldera is down, the platform should fail to start

Open RomuDeuxfois opened this issue 1 year ago • 1 comments

Related issues

  • https://github.com/OpenBAS-Platform/openbas/issues/1034

RomuDeuxfois avatar Jul 08 '24 08:07 RomuDeuxfois

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).

Files Patch % Lines
...cutors/caldera/service/CalderaExecutorService.java 6.89% 27 Missing :warning:
...va/io/openbas/service/PlatformSettingsService.java 0.00% 21 Missing :warning:
...enbas/rest/settings/response/PlatformSettings.java 0.00% 4 Missing :warning:
...njectors/caldera/client/CalderaInjectorClient.java 50.00% 1 Missing :warning:
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.

codecov[bot] avatar Jul 08 '24 08:07 codecov[bot]

@RomuDeuxfois do you think this PR is a good occasion to implement also: https://github.com/OpenBAS-Platform/openbas/issues/1206?

SamuelHassine avatar Jul 10 '24 06:07 SamuelHassine

@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 avatar Jul 10 '24 08:07 RomuDeuxfois

@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar : image It might be worth talking with @jborozco or @Jipegien and see if that's a deal breaker or not.

Dimfacion avatar Jul 16 '24 08:07 Dimfacion

@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar : image 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.

Jipegien avatar Jul 16 '24 08:07 Jipegien

@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar : image 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 image

RomuDeuxfois avatar Jul 16 '24 13:07 RomuDeuxfois

@RomuDeuxfois I hate to be that guy but the banner is displayed next to and not above the scrollbar : image 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 image

RomuDeuxfois avatar Jul 16 '24 13:07 RomuDeuxfois

@Dimfacion ready for a second round

RomuDeuxfois avatar Jul 16 '24 13:07 RomuDeuxfois