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

REDUCER_DISABLE option for targets #826

Merged
merged 2 commits into from
Sep 13, 2023
Merged

REDUCER_DISABLE option for targets #826

merged 2 commits into from
Sep 13, 2023

Conversation

jmthomas
Copy link
Member

closes #801

@jmthomas jmthomas requested a review from ryanmelt September 13, 2023 15:24
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Patch coverage: 75.00% and project coverage change: -0.01% ⚠️

Comparison is base (1eda95c) 72.75% compared to head (a01a562) 72.75%.
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #826      +/-   ##
==========================================
- Coverage   72.75%   72.75%   -0.01%     
==========================================
  Files         472      472              
  Lines       30772    30782      +10     
  Branches      877      877              
==========================================
+ Hits        22389    22396       +7     
- Misses       8288     8292       +4     
+ Partials       95       94       -1     
Flag Coverage Δ
frontend 60.45% <ø> (-0.04%) ⬇️
ruby-api 50.67% <ø> (ø)
ruby-backend 80.54% <75.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
openc3/lib/openc3/models/target_model.rb 72.71% <75.00%> (+0.01%) ⬆️

... and 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# Disable data reduction (min, hour, day) on this target data
# No 'ruby reducer_microservice.rb DEFAULT__REDUCER__EXAMPLE'
# will appear in the the process list
DISABLE_REDUCER
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to REDUCER_DISABLE to be consistent of starting with REDUCER_

<% end %>

<% if include_inst2 %>
TARGET INST <%= inst2_target_name %>
LOG_RETAIN_TIME <%= log_retain_time %>
REDUCED_LOG_RETAIN_TIME <%= reduced_log_retain_time %>
TLM_LOG_CYCLE_TIME 600
# Allow the reducer microservice to take 10% of the cpu (default 30%)
REDUCER_MAX_CPU_UTILIZATION 10
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really low for the heaviest target

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll change it ... but speaking of that. Can the reducer know it's falling behind and raise a notification?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the minute reducer takes longer than a minute, then there is probably a problem

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are cases where that would be fine though. Like if you dropped in a whole bunch of files to process at once and it just takes a long time to work through

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably worth a notification.

@jmthomas jmthomas changed the title DISABLE_REDUCER option for targets REDUCER_DISABLE option for targets Sep 13, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jmthomas jmthomas merged commit b320bfd into main Sep 13, 2023
@jmthomas jmthomas deleted the disable_reducer branch September 13, 2023 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ability to Disable Data Reduction
2 participants