New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 786155 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Change misleading option "short_lived"

Project Member Reported by jkop@chromium.org, Nov 16 2017

Issue description


SetupTsMonGlobalState uses an option 'short-lived'. Defined here: cs/chromeos_public/chromite/lib/ts_mon_config.py?l=45

Ostensibly, this means, as it says in the docstring:
>    short_lived: Whether this process is short-lived and should use the autogen
>                 hostname prefix.

But in practice this is actually appropriate specifically for processes that run in parallel, and, for example, a simple single-threaded cron job which takes ~10 minutes is not an appropriate use-case for the option.

So its name should be changed. Probably to 'is-parallel' or 'allow-parallel'.
 
Cc: akes...@chromium.org pho...@chromium.org
Components: Infra>Client>ChromeOS
Labels: Hotlist-Fixit
Owner: ----
Status: Available (was: Untriaged)

Comment 2 by pho...@chromium.org, Nov 17 2017

Right, it's mostly used for concurrent processes. It's also necessary for processes which manually emit metrics and take less than 1 minute to run - but I don't think actually we have any processes which behave that way yet.
Hi, this bug has not been updated recently. Please acknowledge the bug and provide status within two weeks (6/22/2018), or the bug will be archived. Thank you.

Comment 4 by jkop@chromium.org, Jun 8 2018

Cc: -akes...@chromium.org -pho...@chromium.org jkop@chromium.org
Owner: akes...@chromium.org
Status: Untriaged (was: Available)
To akeshet@ to triage
Components: -Infra>Client>ChromeOS Infra>Client>ChromeOS>Test
Owner: ----
Status: Available (was: Untriaged)
fixit still seems appropriate. I will talk to cindy about the bug retention messages. I think it's ok to keep open some P3 fixits for a while.

Sign in to add a comment