Skip to content

Conversation

@KSmigielski
Copy link
Contributor

No description provided.

@KSmigielski KSmigielski changed the title [WIP] remove xDS support remove xDS support Nov 14, 2022
@KSmigielski KSmigielski marked this pull request as ready for review November 14, 2022 07:54
Ferdudas97
Ferdudas97 previously approved these changes Nov 15, 2022
matb4r
matb4r previously approved these changes Nov 15, 2022
@KSmigielski KSmigielski dismissed stale reviews from matb4r and Ferdudas97 via 3417e63 November 16, 2022 17:24
Ferdudas97
Ferdudas97 previously approved these changes Nov 18, 2022
matb4r
matb4r previously approved these changes Nov 18, 2022
@KSmigielski KSmigielski temporarily deployed to ci November 30, 2022 08:38 Inactive
@KSmigielski KSmigielski temporarily deployed to ci December 27, 2022 07:16 — with GitHub Actions Inactive
# Conflicts:
#	CHANGELOG.md
#	envoy-control-tests/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/EndpointMetadataMergingTests.kt
@KSmigielski KSmigielski dismissed stale reviews from matb4r and Ferdudas97 via 7c3cd6f February 8, 2023 16:58
Ferdudas97
Ferdudas97 previously approved these changes Feb 13, 2023
matb4r
matb4r previously approved these changes Feb 15, 2023
@KSmigielski KSmigielski temporarily deployed to ci February 16, 2023 07:59 — with GitHub Actions Inactive
import pl.allegro.tech.servicemesh.envoycontrol.config.envoy.EnvoyExtension

interface ServiceTagsAndCanaryTestBase {
class ServiceTagsAndCanaryTestBase {
Copy link
Contributor

@MarcinFalkowski MarcinFalkowski Feb 16, 2023

Choose a reason for hiding this comment

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

I think this is not related to xDS, but to custom test implementation in AEC. It may be still needed

# Conflicts:
#	CHANGELOG.md
#	envoy-control-core/src/main/resources/lua/ingress_rbac_logging.lua
#	envoy-control-tests/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/routing/ServiceTagsAndCanaryTest.kt
@KSmigielski KSmigielski dismissed stale reviews from matb4r and Ferdudas97 via 7feaa29 March 22, 2023 08:14
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.

5 participants