Skip to content

test: 6x speedup of overload integration test.#45647

Open
paul-r-gall wants to merge 1 commit into
envoyproxy:mainfrom
paul-r-gall:speedup-overload-tests
Open

test: 6x speedup of overload integration test.#45647
paul-r-gall wants to merge 1 commit into
envoyproxy:mainfrom
paul-r-gall:speedup-overload-tests

Conversation

@paul-r-gall

Copy link
Copy Markdown
Contributor

Commit Message: test: 6x speedup of overload integration test.
Additional Description: Required using simulated time and some refactors/rewrites to make simulated time better. The code is slightly more complicated now due to the need for if (simulated) {..} else {..} blocks.

  • Switch overload_integration_test.cc to use simulated time.
  • Fix simulated time deadlocks by making pending updates tracking thread-aware.
  • Refactor test helpers (IntegrationCodecClient, IntegrationStreamDecoder, IntegrationTcpClient) to store time system reference.
  • Update helper methods to accept time system references.

Risk Level: low
Testing: test-only
AI Usage: This PR was generated by Gemini.

- Switch overload_integration_test.cc to use simulated time.
- Fix simulated time deadlocks by making pending updates tracking thread-aware.
- Refactor test helpers (IntegrationCodecClient, IntegrationStreamDecoder, IntegrationTcpClient) to store time system reference.
- Update helper methods to accept time system references.

Signed-off-by: Paul Ogilby <pgal@google.com>
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.

1 participant