Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
Signed-off-by: w3aman <[email protected]>
  • Loading branch information
w3aman committed Nov 7, 2024
1 parent 7c35eea commit d723b66
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
2 changes: 0 additions & 2 deletions common/k8stest/util_node.go
Original file line number Diff line number Diff line change
Expand Up @@ -478,8 +478,6 @@ func GetCsiNodeDriverTopologyKeys(nodeName string, driverName string) ([]string,

func CheckCsiNodeTopologyKeysPresent(nodeName string, driverName string, key []string) (bool, error) {
topologyKey, err := GetCsiNodeDriverTopologyKeys(nodeName, driverName)
fmt.Println("##############################")
fmt.Println(topologyKey)
if err != nil {
return false, err
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -246,18 +246,13 @@ func customTopologyWfcTest(decor string, engine common.OpenEbsEngine, volType co
productConfig.LvmEngineComponentDsPodLabelValue)

updatedEnvVarValue := fmt.Sprintf("kubernetes.io/hostname,%s", key)
fmt.Println(updatedEnvVarValue)

// we have to set env in the daemonset yaml for lvm-local like this
// kubectl set env daemonset/openebs-lvm-localpv-node -n openebs ALLOWED_TOPOLOGIES=kubernetes.io/hostname,key
// this updation will automatically restart daemonset pods
_, err = k8stest.UpdateDemonsetContainerEnv(productConfig.LvmEngineDaemonSetName, productConfig.LvmEnginePluginContainerName, common.NSOpenEBS(), allowedTopologyEnvVarName, updatedEnvVarValue)
Expect(err).To(BeNil(), "failed to update env values")

ds, err := k8stest.GetDaemonSet(productConfig.LvmEngineDaemonSetName, common.NSOpenEBS())
Expect(err).To(BeNil())
fmt.Println(ds)

// verify lvm daemonset to be ready
Eventually(func() bool {
return k8stest.DaemonSetReady(productConfig.LvmEngineDaemonSetName, common.NSOpenEBS())
Expand Down

0 comments on commit d723b66

Please sign in to comment.