spotify / styx

"The path to execution", Styx is a service that schedules batch data processing jobs in Docker containers on Kubernetes.
Apache License 2.0
267 stars 50 forks source link

Add endpoint with enabled state #1056

Closed sonjaer closed 1 year ago

sonjaer commented 1 year ago

Hey, I just made a Pull Request!

Description

There is no endpoint for quickly getting all workflows enabled state. This PR adds it.

Motivation and Context

Currently you need to make one request per workflow which takes a long time. Since we are making some changes internally it would be good to store this information to be able to recover it.

Have you tested this? If so, how?

I have included unit tests

Checklist for PR author(s)

Checklist for PR reviewer(s)

codecov[bot] commented 1 year ago

Codecov Report

Merging #1056 (196f406) into master (35c827c) will decrease coverage by 0.05%. The diff coverage is 76.19%.

@@             Coverage Diff              @@
##             master    #1056      +/-   ##
============================================
- Coverage     92.40%   92.36%   -0.05%     
- Complexity     2145     2150       +5     
============================================
  Files           200      200              
  Lines          8293     8314      +21     
  Branches        502      502              
============================================
+ Hits           7663     7679      +16     
- Misses          517      521       +4     
- Partials        113      114       +1