Merge pull request #182 from kishen-v/move-to-artifacts

Move logs under artifacts

kubetest2-tf/deployer/deployer.go

@@ -21,6 +21,7 @@ import (
 
 	"k8s.io/client-go/tools/clientcmd"
 	"k8s.io/klog/v2"
+	"sigs.k8s.io/kubetest2/pkg/artifacts"
 	"sigs.k8s.io/kubetest2/pkg/types"
 
 	"github.com/ppc64le-cloud/kubetest2-plugins/kubetest2-tf/deployer/options"
@@ -121,7 +122,7 @@ var _ types.Deployer = &deployer{}
 func New(opts types.Options) (types.Deployer, *pflag.FlagSet) {
 	d := &deployer{
 		commonOptions: opts,
-		logsDir:       filepath.Join(opts.RunDir(), "logs"),
+		logsDir:       filepath.Join(artifacts.BaseDir(), "logs"),
 		BuildOptions: &options.BuildOptions{
 			CommonBuildOptions: &build.Options{
 				Builder:         &build.NoopBuilder{},

kubetest2-tf/deployer/dumplogs.go

@@ -63,6 +63,8 @@ func (d *deployer) DumpClusterLogs() error {
 	for _, machineIP := range d.machineIPs {
 		klog.Infof("Collecting node level information from PowerVS instance %s", machineIP)
 		for logFile, command := range commandFilename {
+			stdOut.Reset()
+			stdErr.Reset()
 			commandArgs := []string{
 				"ssh",
 				"-i",
@@ -85,6 +87,7 @@ func (d *deployer) DumpClusterLogs() error {
 				errors = append(errors, fmt.Errorf("Failed to create a log-file: %v", err))
 				continue
 			} else {
+				outfile.WriteString(string(stdOut.Bytes()))
 				outfile.Close()
 			}
 		}