Skip to content

ci: collect bazel profile and store in artifacts#5857

Merged
htuch merged 1 commit intoenvoyproxy:masterfrom
lizan:ci_profile
Feb 6, 2019
Merged

ci: collect bazel profile and store in artifacts#5857
htuch merged 1 commit intoenvoyproxy:masterfrom
lizan:ci_profile

Conversation

@lizan
Copy link
Member

@lizan lizan commented Feb 6, 2019

Signed-off-by: Lizan Zhou lizan@tetrate.io

Description:
Collect bazel JSON profile in CI

Risk Level: Low
Testing: CI
Docs Changes:
Release Notes:

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan lizan requested a review from htuch February 6, 2019 11:14
Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Nice

Copy link
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.

Chrome trace format is great, I find this a very useful performance visualization tool.

@htuch htuch merged commit 13a3bbc into envoyproxy:master Feb 6, 2019
@lizan lizan deleted the ci_profile branch February 6, 2019 21:14
fredlas pushed a commit to fredlas/envoy that referenced this pull request Mar 5, 2019
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Fred Douglas <fredlas@google.com>
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.

3 participants