Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

add avg to singlestate panel #2006

Merged
merged 1 commit into from
Jan 15, 2019
Merged

add avg to singlestate panel #2006

merged 1 commit into from
Jan 15, 2019

Conversation

xudifsd
Copy link
Member

@xudifsd xudifsd commented Jan 15, 2019

clusterview dashboard uses many singlestate panel, two of them are yarn related and did not use aggregation expression. So whenever yarn service get restarted, there will be multiple time series returned by prometheus(one is from older yarn, another is from newly started yarn). This will make home page of Pai displaying error in these two panel.

Use avg aggregation expression to address the issue, although the code reader may be confused.

@xudifsd xudifsd requested a review from mzmssg January 15, 2019 06:41
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 52.647% when pulling addcfc2 on dixu/avg-on-single into 9c4792d on master.

@xudifsd xudifsd merged commit 4372019 into master Jan 15, 2019
@xudifsd xudifsd deleted the dixu/avg-on-single branch January 15, 2019 07:31
ydye added a commit that referenced this pull request Jan 16, 2019
…t passed in (#2010)

* add avg to singlestate panel (#2006)

* Add necessary rdma enviroment in azure to restserver's yarn container startup script.

* Issue Fix

* [Doc] update job tutorial doc about minFailedTaskCount and minSucceededTaskCount (#2009)

* update job tutorial doc

* fix comment

* fix comment

* fix min succeed task count

* Issue Fix

* fix_log_path (#2012)

* Issue Fix

* Issue Fix

* Issue Fix

* Issue Fix

* issue fix

* add more node related alerts (#2008)

* update virtual cluster doc (#1991)

* update virtual cluster doc

* change vc's definition

* add description of vc capacity and availability

* fix typo

* issue fix

* issue fix

* issue fix

* issue fix

* issue fix

* issue fix
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants