Skip to content

Commit

Permalink
mobile: Add the no-remote-exec tag to the Swift stats tests (envoyp…
Browse files Browse the repository at this point in the history
…roxy#32567)

For reasons we haven't figured out yet, RBE does not work well with the
Swift tests and causes frequent timeouts on CI.

See envoyproxy#32551 for details.

We tagged the Swift tests with `no-remote-exec` but forgot to do so for
the swift/stats tests, so we're doing that here. This test timed out
recently on CI because we didn't have the `no-remote-exec` tag.

Signed-off-by: Ali Beyad <[email protected]>
  • Loading branch information
abeyad authored Feb 26, 2024
1 parent cf1816a commit 9a47bc9
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions mobile/test/swift/stats/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ envoy_mobile_swift_test(
"TagsBuilderTests.swift",
],
flaky = True, # TODO(jpsim): Fix timeouts when running these tests on CI
tags = ["no-remote-exec"], # TODO(32551): Re-enable remote exec
visibility = ["//visibility:public"],
deps = [
"//library/objective-c:envoy_engine_objc_lib",
Expand Down

0 comments on commit 9a47bc9

Please sign in to comment.