-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat(opensearch): fetch logs from opensearch (#602) #602
feat(opensearch): fetch logs from opensearch (#602) #602
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #602 +/- ##
==========================================
+ Coverage 75.59% 76.75% +1.16%
==========================================
Files 15 16 +1
Lines 1565 1626 +61
==========================================
+ Hits 1183 1248 +65
+ Misses 382 378 -4
|
06e91d4
to
8423d28
Compare
8423d28
to
4471a2d
Compare
4471a2d
to
a2167cb
Compare
204bf90
to
ce6c14a
Compare
ce6c14a
to
00fbf00
Compare
00fbf00
to
f5514de
Compare
f5514de
to
79a288b
Compare
79a288b
to
15ec775
Compare
15ec775
to
4415b61
Compare
4415b61
to
802bd49
Compare
Read the Docs build fails with:
|
802bd49
to
13c3228
Compare
13c3228
to
98f7427
Compare
98f7427
to
20d3148
Compare
20d3148
to
7856218
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works nicely 👍
I've generated new docs/openapi.json
, fixed the unit tests due to the presence of new live_logs_enabled
property, and slightly amended the commit message.
7856218
to
46ae9cd
Compare
Optionally fetch live workflow and job logs from OpenSearch.
OpenSearch client is configurable via env variables, passed by Helm chart. Defaults are defined in
config.py
file.Read the Docs CI builds started to fail, added fix from https://about.readthedocs.com/blog/2024/07/addons-by-default/ in a separate commit.
Related to reanahub/reana#827.