Add API /state endpoint
Add new endpoint for sentinel components state. Currently there are following components:
- FWLogs
- Minipot
- Turris Survey
- Sentinel proxy
Valid states are:
- running
- disabled
- sending
- failed
- unknown
- uninstalled
Closes: #10 (closed)
Blocked by: turris/foris-controller/foris-controller-sentinel-module#9 (closed)
Merge request reports
Activity
added Blocked label
- Resolved by Filip Hron
Looks pretty simple and straightforward. Please add test to newly added endpoint.
//edit
I just went through the tests and found out that there are only tests for update actions. It is either by mistake or it rather seems by design. so we face decision:
- we let the test be only for
POST
update requests - add test for this request and all the other
GET
also
It appears to me that that is more of a design, but you have more experience in that regard.
Edited by Filip Hron - we let the test be only for
- Resolved by Filip Hron
I will test on Router when I am able to make
forris-controller
backend changes work.
added 2 commits
marked this merge request as draft from 52e35930
added 2 commits
added 2 commits
assigned to @agumroian and unassigned @fhron
changed milestone to %Data Collection Plugin 0.3.0
- Resolved by Aleksandr Gumroian
We know this is not working due to required changes in backend. However having this merged does not generate any error unless is used explicitly. And it can happen that SYSteam would like to have it ASAP. I'd consider flask part
complete
orready
.