Merge pull request #4099 from hashicorp/feat/status-reports/auto-run-…

…check-on-deploy-release

internal/runner: Run a status report on Deployments and Releases

.changelog/4099.txt

@@ -0,0 +1,4 @@
+```release-note:improvement
+core: Auto run a status report after a deployment or release operation rather
+than only if `waypoint deploy` or `waypoint release` CLI is run.
+```

internal/cli/deployment_create.go

@@ -70,18 +70,6 @@ func (c *DeploymentCreateCommand) Run(args []string) int {
  hostname = hostnamesResp.Hostnames[0]
  }
 
- // Status Report
- app.UI.Output("")
- _, err = app.StatusReport(ctx, &pb.Job_StatusReportOp{
- Target: &pb.Job_StatusReportOp_Deployment{
- Deployment: deployment,
- },
- })
- if err != nil {
- app.UI.Output(clierrors.Humanize(err), terminal.WithErrorStyle())
- return ErrSentinel
- }
-
  // Release if we're releasing
  var releaseUrl string
  if c.flagRelease {

internal/cli/release_create.go

@@ -182,18 +182,6 @@ func (c *ReleaseCreateCommand) Run(args []string) int {
  tbl := fmtVariablesOutput(resp.VariableFinalValues)
  c.ui.Table(tbl)
 
- // Status Report
- app.UI.Output("")
- _, err = app.StatusReport(ctx, &pb.Job_StatusReportOp{
- Target: &pb.Job_StatusReportOp_Release{
- Release: result.Release,
- },
- })
- if err != nil {
- app.UI.Output(clierrors.Humanize(err), terminal.WithErrorStyle())
- return ErrSentinel
- }
-
  if result.Release.Url == "" {
  app.UI.Output("\n"+strings.TrimSpace(releaseNoUrl),
  deploy.Id,

internal/runner/operation_deploy.go

@@ -52,6 +52,12 @@ func (r *Runner) executeDeployOp(
  return nil, err
  }
 
+ // Run a status report on the recent deployment
+ _, err = app.DeploymentStatusReport(ctx, deployment)
+ if err != nil {
+ return nil, err
+ }
+
  return &pb.Job_Result{
  Deploy: &pb.Job_DeployResult{
  Deployment: deployment,

internal/runner/operation_release.go

@@ -211,5 +211,11 @@ func (r *Runner) executeReleaseOp(
  }
  }
 
+ // Run a status report operation on the recent release
+ _, err = app.ReleaseStatusReport(ctx, release)
+ if err != nil {
+ return nil, err
+ }
+
  return result, nil
 }