discovery server should allow new publisher in down state

[Deepinder Setia]

Some applications need to set SERVICE DOWN while publishing. This is generally because application may have many gating factors that all need to open before declaring service to be up. Currently discovery server immediately declares the service up when published and starts to assign to clients.

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12475
Submitter: Deepinder Setia (<email address hidden>)

[Foucault de Bonneval (foucaultdebonneval)]

Hi Deepinder,

About this comportement there is a very interesting behavior in Ceph about the OSD status.

As described in [0], OSD can have 2 parameter with two status each :
 - in / out
 - up / down

Adding in / out states in addition to up / down could be a way to make the difference between a up and not-ready component that gets ready after a sync period.

Enabling an administrative in / out state could also make operations easier to for "draining" of a component we would like to put in maintenance mode for example.

What do you think ?

Foucault

[0] : http://ceph.com/docs/master/rados/operations/monitoring-osd-pg/

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12601
Submitter: Deepinder Setia (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/12601
Committed: http://github.org/Juniper/contrail-controller/commit/66d50b02a29b328d237eff89caf4a4e0fc040baa
Submitter: Zuul
Branch: R2.20

commit 66d50b02a29b328d237eff89caf4a4e0fc040baa
Author: Deepinder Setia <email address hidden>
Date: Thu Jul 23 20:24:37 2015 -0700

Bug: 1475924 - Accept admin state in publish request

Change-Id: I7389a6abac64f3cbeb3887ceed4d69d82a965ee9

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12834
Submitter: Deepinder Setia (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/13011
Submitter: Nipa Kumar (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/12834
Committed: http://github.org/Juniper/contrail-controller/commit/dce4a585430d6c71c60d2a0937da76c752a1b2dd
Submitter: Zuul
Branch: master

commit dce4a585430d6c71c60d2a0937da76c752a1b2dd
Author: Deepinder Setia <email address hidden>
Date: Thu Jul 23 20:24:37 2015 -0700

Bug: 1475924 - Accept admin state in publish request

Change-Id: I7389a6abac64f3cbeb3887ceed4d69d82a965ee9

[Deepinder Setia (dsetia)]

foucault, sorry I had not read your comment about CEFT earlier and just saw it. Discovery now has administrative and operational states (admin-state and oped-state respectively). As the name implies, former is intended for administrator to bring the service in/out while latter is to be used by application to control its own status based on its run state. Hope this will provide enough control.

[Ankit Jain (ankitja)]

Test case added :
https://review.opencontrail.org/#/c/13902/

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/13927
Submitter: Vinay Vithal Mahuli (<email address hidden>)