CI: Tweak our code coverage profile behavior

.github/workflows/codecov.yml

@@ -9,7 +9,7 @@ permissions: read-all
 jobs:
   test:
     name: Code Coverage
-    runs-on: gh-hosted-runners-4cores-1
+    runs-on: gh-hosted-runners-16cores-1
 
     steps:
     - name: Check out code
@@ -106,8 +106,14 @@ jobs:
 
         eatmydata -- make unit_test_cover
 
+        # Restore the files we deleted as codecov tries to fix their paths.
+        git reset --hard HEAD
+
     - name: Upload coverage reports to codecov.io
       if: steps.changes.outputs.changed_files == 'true'
-      uses: codecov/codecov-action@v3
+      uses: codecov/codecov-action@v4-beta
+      with:
+        fail_ci_if_error: true
+        verbose: true
       env:
         CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

Makefile

@@ -216,7 +216,7 @@ e2e_test: build
 # Run the code coverage tools, compute aggregate.
 unit_test_cover: build dependency_check demo
 	source build.env
-	go test $(VT_GO_PARALLEL) -count=1 -coverprofile=coverage.out ./go/...
+	go test $(VT_GO_PARALLEL) -count=1 -failfast -covermode=atomic -coverpkg=./... -coverprofile=coverage.out ./go/...
 	# Handle go tool cover failures due to not handling `//line` directives, which
 	# the goyacc compiler adds to the generated parser in sql.go. See:
 	# https://github.com/golang/go/issues/41222

codecov.yml

@@ -0,0 +1,26 @@
+# https://docs.codecov.com/docs/
+# https://docs.codecov.com/docs/codecov-yaml
+
+codecov:
+  branch: main # Set the default branch
+
+parsers:
+  go:
+    partials_as_hits: true
+
+ignore:
+  # Ignore our end-to-end test code
+  - "go/test/endtoend"
+  - "go/*/endtoend"
+  - "go/vt/*/endtoend"
+  - "go/cmd/vttestserver" # This relies on end-to-end test packages
+
+comment: # https://docs.codecov.com/docs/pull-request-comments
+    hide_project_coverage: false
+
+coverage:
+  status: # https://docs.codecov.com/docs/commit-status
+    project:
+      default:
+        informational: true # Don't ever fail the codecov/project or codecov/patch tests
+

go/vt/wrangler/vexec.go

@@ -880,8 +880,9 @@ func (wr *Wrangler) ListAllWorkflows(ctx context.Context, keyspace string, activ
 }
 
 // ShowWorkflow will return all of the relevant replication related information for the given workflow.
-func (wr *Wrangler) ShowWorkflow(ctx context.Context, workflow, keyspace string, shards []string) (*ReplicationStatusResult, error) {
-	replStatus, err := wr.getStreams(ctx, workflow, keyspace, shards)
+// If shardSubset is nil, then all shards will be queried.
+func (wr *Wrangler) ShowWorkflow(ctx context.Context, workflow, keyspace string, shardSubset []string) (*ReplicationStatusResult, error) {
+	replStatus, err := wr.getStreams(ctx, workflow, keyspace, shardSubset)
 	if err != nil {
 		return nil, err
 	}