Skip to content

Commit

Permalink
Revert "Create workaround for NETCONF-1285"
Browse files Browse the repository at this point in the history
This reverts commit fb343f7.

Signed-off-by: tobias.pobocik <[email protected]>
  • Loading branch information
Tobianas committed Apr 30, 2024
1 parent 9196e54 commit 7c42f98
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ do \
assertHttpStatusCode $(curl -o /dev/null -s -w "%{http_code} GET %{url_effective}\n" --user admin:admin -H "Content-Type: application/json" --insecure http://$pod_controller_ip:8558/cluster/members) \
;done
sleep 1
: ' FIXME uncomment this after NETCONF-1285 is resolved

#Pods healthcheck (:8888/restconf/operations)

for pod_controller_ip in $POD_CONTROLLER_IPS; \
Expand All @@ -100,7 +100,7 @@ sleep 1

#Service healthcheck (:30888/restconf/operations)
assertHttpStatusCode $(curl -o /dev/null -s -w "%{http_code} GET %{url_effective}\n" --user admin:admin -H "Content-Type: application/json" --insecure http://$MINIKUBE_IP:$CONTROLLER_PORT/restconf/operations)
'

# add gNMI node into gNMI topology
assertHttpStatusCode $(curl -X PUT -o /dev/null -s -w "%{http_code} PUT %{url_effective}\n" \
http://"$MINIKUBE_IP":$CONTROLLER_PORT/restconf/data/network-topology:network-topology/topology=gnmi-topology/node=node-"${MINIKUBE_IP//.}" \
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -182,15 +182,15 @@ do \
"http://$pod_controller_ip:8558/cluster/members")" \
;done
validateTestStatus
: ' FIXME uncomment this after NETCONF-1285 is resolved

# Pods health check (:8888/restconf/operations)
for pod_controller_ip in $POD_CONTROLLER_IPS; \
do \
assertHttpStatusCode "$(curl -o /dev/null -s -w "%{http_code} GET %{url_effective}\n" \
-H "Content-Type: application/json" \
"http://$pod_controller_ip:$CONTROLLER_PORT/restconf/operations")" \
;done
'

# Add node into topology
assertHttpStatusCode "$(curl -X PUT -o /dev/null -s -w "%{http_code} PUT %{url_effective}\n" \
"http://$CTRL0_IP:$CONTROLLER_PORT/restconf/data/network-topology:network-topology/topology=topology-netconf/node=node-${SIMULATOR_IP//.}" \
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/lighty-rnc-app/tests-lighty-rnc-app.sh
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ do \
assertHttpStatusCode $(curl -o /dev/null -s -w "%{http_code} GET %{url_effective}\n" --user admin:admin -H "Content-Type: application/json" --insecure http://$pod_controller_ip:8558/cluster/members) \
;done
sleep 1
: ' FIXME uncomment this after NETCONF-1285 is resolved

# Pods healthcheck (:8888/restconf/operations)

for pod_controller_ip in $POD_CONTROLLER_IPS; \
Expand All @@ -92,7 +92,7 @@ sleep 1
# Service healthcheck (:30888/restconf/operations)
assertHttpStatusCode $(curl -o /dev/null -s -w "%{http_code} GET %{url_effective}\n" --user admin:admin -H "Content-Type: application/json" --insecure http://$MINIKUBE_IP:$CONTROLLER_PORT/restconf/operations)
sleep 1
'

# add node into topology
assertHttpStatusCode $(curl -X PUT -o /dev/null -s -w "%{http_code} PUT %{url_effective}\n" http://"$MINIKUBE_IP":$CONTROLLER_PORT/restconf/data/network-topology:network-topology/topology=topology-netconf/node=node-"${SIMULATOR_IP//.}" \
-H 'Content-Type: application/json' \
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,8 @@ public void simpleApplicationTest() throws Exception {
}
]
}""");
/* FIXME uncomment this after NETCONF-1285 is resolved
operations = restClient.GET("restconf/operations");
assertEquals(operations.statusCode(), 200);
*/
operations = restClient.GET("restconf/data/network-topology:network-topology?content=config");
assertEquals(operations.statusCode(), 200);
operations = restClient.GET("restconf/data/network-topology:network-topology?content=nonconfig");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,8 @@ public void simpleApplicationTest() throws IOException, InterruptedException {
]
}""");

/* FIXME uncomment this after NETCONF-1285 is resolved
operations = restClient.GET("rests/operations");
Assert.assertEquals(operations.statusCode(), 200);
*/
operations = restClient.GET("rests/data/network-topology:network-topology?content=config");
Assert.assertEquals(operations.statusCode(), 200);
operations = restClient.GET("rests/data/network-topology:network-topology?content=nonconfig");
Expand Down

0 comments on commit 7c42f98

Please sign in to comment.