Skip to content

bazel: fix for consuming builds#1868

Merged
mattklein123 merged 1 commit intomasterfrom
bazel_fixes
Oct 16, 2017
Merged

bazel: fix for consuming builds#1868
mattklein123 merged 1 commit intomasterfrom
bazel_fixes

Conversation

@mattklein123
Copy link
Copy Markdown
Member

Signed-off-by: Matt Klein mklein@lyft.com

Signed-off-by: Matt Klein <mklein@lyft.com>
@mattklein123
Copy link
Copy Markdown
Member Author

@htuch @jmillikin-stripe

Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

LGTM, still OS X issues persist, which look independent of this.

@mattklein123 mattklein123 merged commit 794217a into master Oct 16, 2017
@mattklein123 mattklein123 deleted the bazel_fixes branch October 16, 2017 20:55
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Picking up [Envoy changes](3941b63...5423325).

Specifically:

- #18418

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Picking up [Envoy changes](3941b63...5423325).

Specifically:

- #18418

Signed-off-by: JP Simard <jp@jpsim.com>
mathetake pushed a commit that referenced this pull request Mar 3, 2026
**Description**

mcpProxyBackendName is a function, producing the following output:
> Creating MCP proxy Backend {"namespace": "xxx", "nameError": "json:
unsupported type: func(*v1alpha1.MCPRoute) string"}

This small patch will make sure we log the rendered name.

Signed-off-by: Matthieu Blottiere Seroux <matthieu.blottiere@owkin.com>
Co-authored-by: Ignasi Barrera <ignasi@tetrate.io>
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.

2 participants