Conversation
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #89 +/- ##
==========================================
- Coverage 98.90% 98.89% -0.01%
==========================================
Files 15 15
Lines 1457 1452 -5
Branches 155 154 -1
==========================================
- Hits 1441 1436 -5
Misses 9 9
Partials 7 7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
After testing: #90 needs to be fixed |
…g ones (Log Volumes)
…ill-state-not-updating Remove LogCollector Fill Level panels from dashboard
…er-guide Add developer guide section to readthedocs for better structure
maldwg
left a comment
There was a problem hiding this comment.
Comments look good & code changes as well.
Only 1 thing: in the readme we say to start the docker compose we need to run
HOST_IP=127.0.0.1 docker-compose...
Thiscan be misleading as normally we should be using our correct internal IP 192.168.x.x if I am not mistaken ? Also in the config.yaml we should reflect this by using the Kafka external ports as default not the internal ones. What do you think?
lamr02n
left a comment
There was a problem hiding this comment.
There seem to be some outdated descriptions, specifically for the Inspection stage in pipeline.rst. We should update them according to our latest documentation format.
Sure, we should update that according to the actual functionality. @stefanDeveloper, can you clarify if these changes are correct? |
Yes. These are correct. |
Prepare and document the current state of this project for a RC1 to run in production.