User story #2222

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

Added by Jonathan CLARKE about 3 years ago. Updated about 1 month ago.

Status:ReleasedStart date:2012-01-31
Priority:3Due date:
Assignee:Matthieu CERDA% Done:

100%

Category:TechniquesEstimated time:2.00 hours
Target version:2.4.0~alpha5
Pull Request:

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

Related to Rudder - User story #2233: PT distributePolicy: Check status of webapp and endpoint,... Released 2012-02-02
Blocked by Rudder - User story #856: Make promises in order to assure that Rudder services are... Released 2011-02-16

Associated revisions

Revision 8ec83355
Added by Matthieu CERDA about 3 years ago

Fixes #2222: The distributePolicy now implements a check to see if the webapp returns a 200 http code

History

#1 Updated by Matthieu CERDA about 3 years ago

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

Applied in changeset commit:8ec83355feab16b7850fb5636e58016d28e1dbc9.

#2 Updated by Matthieu CERDA about 3 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 !

#3 Updated by Jonathan CLARKE about 3 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?

#4 Updated by Matthieu CERDA about 3 years ago

They come from the called bundle indeed. Nothing forgotten :)

#5 Updated by François ARMAND about 3 years ago

  • Target version changed from 2.4.0~alpha4 to 2.4.0~alpha5

#6 Updated by Matthieu CERDA about 3 years ago

  • Status changed from Discussion to Pending technical review

Every step completed in the commit commit:b6a0156e0523187a3fd8e60b6f975bffd5991860

#7 Updated by Jonathan CLARKE about 3 years ago

  • Status changed from Pending technical review to Pending functional review

Looks good to me!

#8 Updated by Jonathan CLARKE about 3 years ago

  • Status changed from Pending functional review to Released

#9 Updated by Benoît PECCATTE about 1 month ago

  • Project changed from Techniques to Rudder
  • Category changed from Policy Templates to Techniques

Also available in: Atom PDF