Skip to content

Comments

test: fixing a test flake#18899

Merged
yanavlasov merged 5 commits intoenvoyproxy:mainfrom
alyssawilk:flake
Nov 9, 2021
Merged

test: fixing a test flake#18899
yanavlasov merged 5 commits intoenvoyproxy:mainfrom
alyssawilk:flake

Conversation

@alyssawilk
Copy link
Contributor

Before: failed 11/8000, after passes all of 8000

Commit Message: n/a
Additional Description: n/a
Risk Level: n/a
Testing: lots
fixes #18881

Signed-off-by: Alyssa Wilk <[email protected]>
asraa
asraa previously approved these changes Nov 4, 2021
Copy link
Contributor

@asraa asraa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so so much!

Honestly a little confused why git isn't showing the diff removing the lines disabling coverage on this test...

Signed-off-by: Alyssa Wilk <[email protected]>
@alyssawilk
Copy link
Contributor Author

Ahh, i hadn't pulled in the disable because I hadn't seen we'd done that. Fixed!

asraa
asraa previously approved these changes Nov 4, 2021
@alyssawilk
Copy link
Contributor Author

/wait well that's weird this fixes the flake locally. will dig more

@alyssawilk alyssawilk marked this pull request as draft November 4, 2021 20:02
@alyssawilk
Copy link
Contributor Author

oh joy, it failed without actually logging what went wrong.
I can't repro this failure locally. trying again on CI and converting this to draft so we don't merge accidentally.

@alyssawilk alyssawilk marked this pull request as ready for review November 9, 2021 14:05
@yanavlasov yanavlasov merged commit 1f084f9 into envoyproxy:main Nov 9, 2021
mpuncel added a commit to mpuncel/envoy that referenced this pull request Nov 10, 2021
* main: (71 commits)
  bazel: fix macOS build (envoyproxy#18920)
  http: switching from 100 to 1xx (envoyproxy#18904)
  grpc: implement BufferedAsyncClient for bidirectional gRPC stream (envoyproxy#18129)
  bazel: add repository arg to benchmark_test (envoyproxy#18795)
  rocketmq_proxy: Improvement for map find (envoyproxy#18909)
  tls: unit test: spiffe signed by intermediate cert (envoyproxy#18914)
  Test for FilterConfigPerRoute dtor called on worker thread. (envoyproxy#18927)
  deps: Bump `com_google_protobuf` -> 3.19.1 (envoyproxy#18930)
  deps: Bump `com_googlesource_code_re2` -> 2021-11-01 (envoyproxy#18933)
  cvescan: Move cvescan data to yaml (envoyproxy#18947)
  remove unnecessary file level not unimplemented hide annotation (envoyproxy#18924)
  test: moving echo test (envoyproxy#18938)
  test: fixing a test flake (envoyproxy#18899)
  deps: Revert pyparsing bump (envoyproxy#18946)
  deps: Bump `build_bazel_rules_apple` -> 0.32.0 (envoyproxy#18932)
  deps: Bump `com_github_bazelbuild_buildtools` -> 4.2.3 (envoyproxy#18931)
  build(deps): bump pycparser from 2.20 to 2.21 in /tools/dependency (envoyproxy#18936)
  quic: supporting connections with zero initial available streams (envoyproxy#18775)
  test: moving proxy proto (envoyproxy#18939)
  build(deps): bump pyparsing from 3.0.4 to 3.0.5 in /tools/dependency (envoyproxy#18937)
  ...

Signed-off-by: Michael Puncel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Coverage flake //test/integration:multiplexed_integration_test

3 participants