User story #2222: PT distributePolicy: Check status of webapp and endpoint, and restart jetty if several continuous tests fail - Rudder - Issue Tracker

Actions

Copy link

User story #2222

closed

PT distributePolicy: Check status of webapp and endpoint, and restart jetty if several continuous tests fail

Added by Jonathan CLARKE almost 13 years ago. Updated almost 10 years ago.

Status:

Released

Priority:

Assignee:

Matthieu CERDA

Category:

Techniques

Target version:

2.4.0~alpha5

Pull Request:

UX impact:

Suggestion strength:

User visibility:

Effort required:

Name check:

Fix check:

Regression:

Description

Initially we can just run a curl check on a simple URL, and ensure we get a HTTP 200 OK response code.

Later, we can extend this to use the built-in REST API and call something like /api/status which should return a text string "OK".

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by Matthieu CERDA almost 13 years ago

Status changed from 2 to Pending technical review
% Done changed from 0 to 100

Applied in changeset commit:8ec83355feab16b7850fb5636e58016d28e1dbc9.

Actions

Copy link

Updated by Matthieu CERDA almost 13 years ago

Status changed from Pending technical review to Discussion

This commit implements the check only for the webapp. Indeed, the inventory endpoint only responds to POST methods and a GET returns a 405 error (Method not supported). I do not think that trying to post something at every run is a good idea, so it is for me a better idea to wait for the enhanced version that will use the REST API to check the status. Also, there were some discussions about merging the webapp and the endpoint !

Actions

Copy link

Updated by Jonathan CLARKE almost 13 years ago

Agreed.

Just one question: I don't see any "result_repaired" reports for when we restart jetty. Do they come from the bundle that is called, or have they been forgotten?

Actions

Copy link