Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

Commit

Permalink
test: enhance info message when switching deployments
Browse files Browse the repository at this point in the history
When a deployment was created in a different namespace than the one
that is expected now, the log output was a bit confusing:

Dec 18 09:31:55.805: INFO: want lvm-testing PMEM-CSI deployment = {HasDriver:true HasOperator:false HasOLM:false Mode:lvm Namespace:default Testing:true Version:}
Dec 18 09:31:55.805: INFO: have lvm-testing PMEM-CSI deployment, want lvm-testing -> delete existing deployment

Re-deploying in the intended namespace however worked fine.
  • Loading branch information
pohly committed Dec 18, 2020
1 parent cab939c commit 0492963
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion test/e2e/deploy/deploy.go
Original file line number Diff line number Diff line change
Expand Up @@ -811,7 +811,9 @@ func EnsureDeploymentNow(f *framework.Framework, deployment *Deployment) {
}
return
}
framework.Logf("have %s PMEM-CSI deployment, want %s -> delete existing deployment", running.Name(), deployment.Name())
framework.Logf("have %s PMEM-CSI deployment in namespace %s, want %s in namespace %s-> delete existing deployment",
running.Name(), running.Namespace,
deployment.Name(), deployment.Namespace)

if running.HasOLM {
cmd := exec.Command("test/stop-operator.sh", "-olm")
Expand Down

0 comments on commit 0492963

Please sign in to comment.