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
#261 - compute overall state #262
Conversation
# Conflicts: # alignak_backend/timeseries.py
0e0196e
to
35c0839
Compare
Compute overall host/service state on insertion Fix broken tests
alignak_backend/app.py
Outdated
|
||
# Host overall was computed, update the host overall state | ||
lookup = {"_id": item['_id']} | ||
patch_internal('host', {"_overall_state_id": overall_state}, False, False, **lookup) |
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.
I think (but not sure) the patch_internal modify the _etag
, so need return the new _etag
. if we don't do this the _etag
returned by the post method will be wrong
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.
Îndeed, yes, it modifies the _etag. this is why I had to modify the tests ... but how do you modify the returned _etag? Please do the modification ...
I added code to have the right (new) etag in the response |
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.
Approved. Rebase / squash and we can merge
No description provided.