Skip to content

Commit 10c66fa

Browse files
authored
Revert "Switch coverage CI targets to EngFlow (#39269)" (#39299)
Commit Message: This reverts commit e0420ee. It unexpectedly resulted in lower coverage numbers - we should rollback while we are investigating what's going on to avoid disruptions. Additional Description: Some relevant discussions can be found in #39030 which prompted me to work on this in the first place. And I will use #39248 as a tracking bug for the coverage changes. Signed-off-by: Mikhail Krinkin <[email protected]>
1 parent e0420ee commit 10c66fa

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/_check_coverage.yml

+2-1
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
uses: ./.github/workflows/_run.yml
3939
name: ${{ matrix.name ||matrix.target }}
4040
with:
41-
bazel-extra: '--config=remote-envoy-engflow'
41+
# bazel-extra: '--config=remote-envoy-engflow'
4242
cache-build-image: ${{ fromJSON(inputs.request).request.build-image.default }}
4343
concurrency-suffix: -${{ matrix.target }}
4444
diskspace-hack: ${{ matrix.diskspace-hack && true || false }}
@@ -50,6 +50,7 @@ jobs:
5050
lower than limit
5151
gcs-cache-bucket: ${{ inputs.gcs-cache-bucket }}
5252
rbe: true
53+
rbe-google: true
5354
request: ${{ inputs.request }}
5455
runs-on: ${{ fromJSON(inputs.request).config.ci.agent-ubuntu }}
5556
steps-post: |

0 commit comments

Comments
 (0)