Skip to content

Commit 02cec02

Browse files
Join vnet as a part of NC publish (Azure#451)
Join vnet before publishing NC. This is required as nmagent may be restarted which causes the NC publishing to fail.
1 parent 6ca873c commit 02cec02

File tree

1 file changed

+11
-18
lines changed

1 file changed

+11
-18
lines changed

cns/restserver/restserver.go

Lines changed: 11 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ func NewHTTPRestService(config *common.ServiceConfig) (HTTPService, error) {
113113

114114
serviceState := &httpRestServiceState{}
115115
serviceState.Networks = make(map[string]*networkInfo)
116+
serviceState.joinedNetworks = make(map[string]struct{})
116117

117118
return &HTTPRestService{
118119
Service: service,
@@ -124,7 +125,6 @@ func NewHTTPRestService(config *common.ServiceConfig) (HTTPService, error) {
124125
routingTable: routingTable,
125126
state: serviceState,
126127
}, nil
127-
128128
}
129129

130130
// Start starts the CNS listener.
@@ -980,10 +980,10 @@ func (service *HTTPRestService) setOrchestratorType(w http.ResponseWriter, r *ht
980980
log.Printf("[Azure CNS] setOrchestratorType")
981981

982982
var (
983-
req cns.SetOrchestratorTypeRequest
983+
req cns.SetOrchestratorTypeRequest
984984
returnMessage string
985-
returnCode int
986-
nodeID string
985+
returnCode int
986+
nodeID string
987987
)
988988

989989
err := service.Listener.Decode(w, r, &req)
@@ -1757,10 +1757,6 @@ func (service *HTTPRestService) isNetworkJoined(networkID string) bool {
17571757
namedLock.LockAcquire(stateJoinedNetworks)
17581758
defer namedLock.LockRelease(stateJoinedNetworks)
17591759

1760-
if service.state.joinedNetworks == nil {
1761-
service.state.joinedNetworks = make(map[string]struct{})
1762-
}
1763-
17641760
_, exists := service.state.joinedNetworks[networkID]
17651761

17661762
return exists
@@ -1819,16 +1815,13 @@ func (service *HTTPRestService) publishNetworkContainer(w http.ResponseWriter, r
18191815

18201816
switch r.Method {
18211817
case "POST":
1822-
// Join Network if not joined already
1823-
isNetworkJoined = service.isNetworkJoined(req.NetworkID)
1824-
if !isNetworkJoined {
1825-
publishResponse, publishError, err = service.joinNetwork(req.NetworkID, req.JoinNetworkURL)
1826-
if err == nil {
1827-
isNetworkJoined = true
1828-
} else {
1829-
returnMessage = err.Error()
1830-
returnCode = NetworkJoinFailed
1831-
}
1818+
// Join the network
1819+
publishResponse, publishError, err = service.joinNetwork(req.NetworkID, req.JoinNetworkURL)
1820+
if err == nil {
1821+
isNetworkJoined = true
1822+
} else {
1823+
returnMessage = err.Error()
1824+
returnCode = NetworkJoinFailed
18321825
}
18331826

18341827
if isNetworkJoined {

0 commit comments

Comments
 (0)