Rely api healthcheck on status code rather than json decoding

[fabiolab]

Proposed changes:

• Change the way to check api status from ui: set it on status_code rather than json decoding

It is related to this issue: #1866

Status (please check what you already did):

• First draft (up for discussions & feedback)
• Final code
• Added tests
• Updated documentation

[ZanSara]

Amazing, thank you for the tests especially! Feel free to request a review to me or @tstadel when you think it's ready to be merged 🙂

[tstadel]

Yes, looks like it's ready to review :-).
@fabiolab just feel free, to request a review from me and/or @ZanSara.

[ZanSara]

I noticed your tests are failing on the CI. I'll add a commit to your PR to fix the CI, currently is not installing the UI dependencies because we haven't had UI tests before 😅

[fabiolab]

Yes, looks like it's ready to review :-). @fabiolab just feel free, to request a review from me and/or @ZanSara.

Yes, I don't see what to do else! It's ready from my point of view. But I can't find out how to assign the PR to one of you.

[ZanSara]

There should be a box on the sidebar on the right of your first comment, titled Reviewers. I can't assign myself as a reviewer, but if you can't find it I will assign @tstadel or ask him to assign me 🙂

[fabiolab]

Mmmh, I can see the "assignees" section on the right sidebar, but I can't edit it in anyway. I guess I don't have the appropriate level access, so feel free to assign it to one of you!

[ZanSara]

Thanks again for your contribution 🙂 As soon as the CI completes on master, the new docker images should be uploaded on the Hub and ready to use.