Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RLQS: Add new throttling and disable flaky test that depends on pending stats implementation #33921

Merged
merged 2 commits into from
May 2, 2024
Merged

RLQS: Add new throttling and disable flaky test that depends on pending stats implementation #33921

merged 2 commits into from
May 2, 2024

Conversation

tyxia
Copy link
Member

@tyxia tyxia commented May 2, 2024 •
edited
Loading

Fix #33911

@tyxia
RLQS: Disable test
e040ca4 
Signed-off-by: Tianyu Xia <tyxia@google.com>
@repokitteh-read-only RepoKitteh - Read Only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #33921 was opened by tyxia.

see: more, trace.

@tyxia
add test
2c878ac 
Signed-off-by: Tianyu Xia <tyxia@google.com>
@tyxia tyxia changed the title RLQS: Disable test RLQS: Add new throttling and disable flaky test that depends on pending stats implementation May 2, 2024
@tyxia tyxia marked this pull request as ready for review May 2, 2024 14:28
@tyxia tyxia requested a review from yanavlasov as a code owner May 2, 2024 14:28
@tyxia
Copy link
Member Author

tyxia commented May 2, 2024

/assign @yanavlasov

@tyxia tyxia mentioned this pull request May 2, 2024
Closed
@yanavlasov yanavlasov merged commit f568d62 into envoyproxy:main May 2, 2024
51 checks passed
@phlax phlax mentioned this pull request May 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yanavlasov yanavlasov yanavlasov approved these changes

Assignees

@yanavlasov yanavlasov

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@tyxia @yanavlasov