Skip to content

Commit

Permalink
Stop dumping k8s resources with kubetest2-kops
Browse files Browse the repository at this point in the history
Signed-off-by: Peter Rifel <[email protected]>
  • Loading branch information
rifelpet committed Dec 31, 2023
1 parent 697fd14 commit ce3066a
Showing 1 changed file with 12 additions and 3 deletions.
15 changes: 12 additions & 3 deletions tests/e2e/kubetest2-kops/deployer/dumplogs.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ import (

"k8s.io/klog/v2"
"k8s.io/kops/pkg/resources"
"k8s.io/kops/tests/e2e/pkg/kops"
"sigs.k8s.io/kubetest2/pkg/exec"
"sigs.k8s.io/yaml"
)
Expand All @@ -45,7 +46,7 @@ func (d *deployer) DumpClusterLogs() error {
}
klog.Info(strings.Join(args, " "))
cmd := exec.Command(args[0], args[1:]...)
cmd.SetEnv(d.env()...)
cmd.SetEnv(append(d.env(), "KOPS_TOOLBOX_DUMP_K8S_RESOURCES=1")...)
cmd.SetStdout(yamlFile)
cmd.SetStderr(os.Stderr)

Expand All @@ -60,11 +61,19 @@ func (d *deployer) DumpClusterLogs() error {
dumpErr = errors.Join(dumpErr, err)
}

if err := d.dumpClusterInfo(); err != nil {
klog.Warningf("cluster info dump failed: %v", err)
kopsVersion, err := kops.GetVersion(d.KopsBinaryPath)
if err != nil {
klog.Warningf("kops version failed: %v", err)
dumpErr = errors.Join(dumpErr, err)
}

if kopsVersion == "" || kopsVersion < "1.29" {
// TODO: remove when kubetest2-kops stops testing against kops 1.28 and older
if err := d.dumpClusterInfo(); err != nil {
klog.Warningf("cluster info dump failed: %v", err)
dumpErr = errors.Join(dumpErr, err)
}
}
return dumpErr
}

Expand Down

0 comments on commit ce3066a

Please sign in to comment.