ts_mon: make /chrome/infra prefix configurable |
||||
Issue descriptionThis is for both Python and Go versions. Context: request by tanin@ so ts_mon can be used by other teams.
,
Jul 1 2016
,
Jul 1 2016
I suppose you meant to assign yourself as an owner (status Assigned without an owner doesn't make sense :-)
,
Jul 1 2016
I thought I did put myself as the owner... noogler's mistake :s
,
Jul 1 2016
For this, I think we are going to add --ts-mon-metrics-name-prefix as a command-line argument. I'm worried about defaulting its value to '/chrome/infra'. Because users might accidentally push metrics to /chrome/infra. We should require this argument. But that will break the current user (which is chrome-infra team). any thought?
,
Jul 1 2016
I'd say, let's do one change at a time. 1. Add the new argument and default it to '/chrome/infra/' - this will be backwards compatible with all of our current usage. This step alone should be the scope of this bug. 2. Upgrade all tools and services to require the new argument, or have it supplied through the config file. This step will take a while, and may need to involve many people's projects. 3. Make the new flag required. This is a breaking change, and will require an announcement and close monitoring + possibly last minute fixes. In the meantime, we should also start checking for the correct prefix at our proxy, to make sure the parameter is used correctly.
,
Jul 1 2016
> 1. Add the new argument and default it to '/chrome/infra/' - this will be backwards compatible with all of our current usage. This step alone should be the scope of this bug. I agree. I will go ahead and add it.
,
Jul 1 2016
,
Jul 19 2016
The fix landed 2 weeks ago (thanks!) Closing the bug. |
||||
►
Sign in to add a comment |
||||
Comment 1 by benhenry@chromium.org
, Jun 29 2016