User story (meta) #2222

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

Added by Jonathan CLARKE over 1 year ago. Updated over 1 year ago.

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

100%
Category:Policy Templates
Target version:Rudder - 2.4.0~alpha5 Estimated time:2.00 hours

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 Techniques - User story (meta) #2233: PT distributePolicy: Check status of webapp and endpoint,... Released 2012-02-02
blocked by Rudder - User story (meta) #856: Make promises in order to assure that Rudder services are... Released 2011-02-16

Associated revisions

Revision 8ec83355
Added by Matthieu CERDA over 1 year 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 over 1 year ago

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

#2 Updated by Matthieu CERDA over 1 year 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 over 1 year 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 over 1 year ago

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

#5 Updated by François ARMAND over 1 year ago

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

#6 Updated by Matthieu CERDA over 1 year ago

  • Status changed from Discussion to Pending technical review

Every step completed in the commit b6a0156e0523187a3fd8e60b6f975bffd5991860

#7 Updated by Jonathan CLARKE over 1 year ago

  • Status changed from Pending technical review to Pending functional review

Looks good to me!

#8 Updated by Jonathan CLARKE over 1 year ago

  • Status changed from Pending functional review to Released

Also available in: Atom PDF