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.

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

chore(revert): revert "refactor(xds): route builder" #8011

Closed
wants to merge 1 commit into from

Conversation

slonka
Copy link
Contributor

@slonka slonka commented Oct 11, 2023

Reverts #7211

@mmorel-35 we're seeing failing tests on master branch and we suspect this is the issue.

@slonka slonka requested a review from a team as a code owner October 11, 2023 11:02
@slonka slonka requested review from Automaat and bartsmykla and removed request for a team October 11, 2023 11:02
@slonka slonka added the ci/run-full-matrix PR: Runs all possible e2e test combination (expensive use carefully) label Oct 11, 2023
@slonka slonka changed the title chore(revert): Revert "refactor(xds): route builder" chore(revert): revert "refactor(xds): route builder" Oct 11, 2023
@slonka slonka enabled auto-merge (squash) October 11, 2023 11:12
@lahabana
Copy link
Contributor

So the errors we suspected were caused by this are still here so I guess we can close that one!

@slonka slonka closed this Oct 11, 2023
auto-merge was automatically disabled October 11, 2023 12:09

Pull request was closed

@slonka slonka deleted the revert-7211-issue-2538/route branch October 12, 2023 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/run-full-matrix PR: Runs all possible e2e test combination (expensive use carefully)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants