Skip to content

add metrics service config example#17464

Merged
lizan merged 12 commits intoenvoyproxy:mainfrom
JaredTan95:config_example_for_metrics_service
Sep 3, 2021
Merged

add metrics service config example#17464
lizan merged 12 commits intoenvoyproxy:mainfrom
JaredTan95:config_example_for_metrics_service

Conversation

@JaredTan95
Copy link
Copy Markdown
Contributor

@JaredTan95 JaredTan95 commented Jul 23, 2021

Signed-off-by: JaredTan95 jian.tan@daocloud.io

Commit Message: add metrics service config example
Additional Description:
Risk Level:
Testing:
Docs Changes: add metrics service config example

Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to api/envoy/.
envoyproxy/api-shepherds assignee is @lizan
CC @envoyproxy/api-watchers: FYI only for changes made to api/envoy/.

🐱

Caused by: #17464 was opened by JaredTan95.

see: more, trace.

Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
@JaredTan95
Copy link
Copy Markdown
Contributor Author

JaredTan95 commented Aug 2, 2021

@phlax Hi, I am not very familiar with the format, could you give me a comment about how to fix CI fails? thanks

@alyssawilk
Copy link
Copy Markdown
Contributor

if you click through the failure you should be able to find logs. alternately running the "fix_format" scripts via ci/do_ci.sh may help

@yanavlasov
Copy link
Copy Markdown
Contributor

/wait

@yanavlasov
Copy link
Copy Markdown
Contributor

This is actually a proto format error. Did you run ./tools/proto_format/proto_format.sh fix

@JaredTan95
Copy link
Copy Markdown
Contributor Author

This is actually a proto format error. Did you run ./tools/proto_format/proto_format.sh fix

Thanks, I ran ./tools/code_format/check_format.py fix only, I will try it.

Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
@JaredTan95
Copy link
Copy Markdown
Contributor Author

api/envoy/

@phlax phlax requested a review from dmitri-d August 24, 2021 13:39
…metrics_service

Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
Signed-off-by: JaredTan95 <jian.tan@daocloud.io>
@JaredTan95
Copy link
Copy Markdown
Contributor Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17464 (comment) was created by @JaredTan95.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot removed the api label Sep 3, 2021
@lizan lizan merged commit ea9a806 into envoyproxy:main Sep 3, 2021
@JaredTan95 JaredTan95 deleted the config_example_for_metrics_service branch September 6, 2021 09:55
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.

7 participants