Skip to content

Commit

Permalink
Merge pull request #105831 from renatolabs/rc/backport-100403-101222-…
Browse files Browse the repository at this point in the history
…104343-104777-105514
  • Loading branch information
renatolabs authored Jul 4, 2023
2 parents dfa1630 + 3a12d96 commit a9d0bbe
Show file tree
Hide file tree
Showing 7 changed files with 240 additions and 141 deletions.
7 changes: 4 additions & 3 deletions pkg/cmd/roachprod/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -655,7 +655,8 @@ var runCmd = &cobra.Command{
`,
Args: cobra.MinimumNArgs(1),
Run: wrap(func(_ *cobra.Command, args []string) error {
return roachprod.Run(context.Background(), config.Logger, args[0], extraSSHOptions, tag, secure, os.Stdout, os.Stderr, args[1:])
return roachprod.Run(context.Background(), config.Logger, args[0], extraSSHOptions, tag,
secure, os.Stdout, os.Stderr, args[1:], install.WithWaitOnFail())
}),
}

Expand Down Expand Up @@ -802,7 +803,7 @@ multiple nodes the destination file name will be prefixed with the node number.
if len(args) == 3 {
dest = args[2]
}
return roachprod.Get(config.Logger, args[0], src, dest)
return roachprod.Get(context.Background(), config.Logger, args[0], src, dest)
}),
}

Expand Down Expand Up @@ -858,7 +859,7 @@ Examples:
if cmd.CalledAs() == "pprof-heap" {
pprofOpts.Heap = true
}
return roachprod.Pprof(config.Logger, args[0], pprofOpts)
return roachprod.Pprof(context.Background(), config.Logger, args[0], pprofOpts)
}),
}

Expand Down
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -1969,7 +1969,7 @@ func (c *clusterImpl) Get(
}
c.status(fmt.Sprintf("getting %v", src))
defer c.status("")
return errors.Wrap(roachprod.Get(l, c.MakeNodes(opts...), src, dest), "cluster.Get")
return errors.Wrap(roachprod.Get(ctx, l, c.MakeNodes(opts...), src, dest), "cluster.Get")
}

// Put a string into the specified file on the remote(s).
Expand Down
Loading

0 comments on commit a9d0bbe

Please sign in to comment.