New issue
Advanced search Search tips

Issue 769554 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Oct 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Remove graphics_WebGLAquarium/meminfo_MemUsed

Project Member Reported by gurcheta...@chromium.org, Sep 28 2017

Issue description

The test doesn't really find memory leaks and we usually ignore the results.  Let's remove since it spams cros-gfx-alerts.


Performance dashboard identified a 169.8% regression in graphics_WebGLAquarium/meminfo_MemUsed on cros-veyron_jaq at revision range 32060000991600001:32070000991700000. Graph: https://chromeperf.appspot.com/report?masters=ChromeOS_Graphics&bots=cros-veyron_jaq&tests=graphics_WebGLAquarium%2Fmeminfo_MemUsed&checked=meminfo_MemUsed%2Cmeminfo_MemUsed_ref%2Cref&rev=32070000991700000

 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/fbb11babe3b4eb702d245b7f3cebe1e5fb230ad1

commit fbb11babe3b4eb702d245b7f3cebe1e5fb230ad1
Author: Po-Hsien Wang <pwang@chromium.org>
Date: Tue Oct 03 03:25:46 2017

graphics_WebGLAquarium: Remove memory alert report

Remove meminfo alert from graphics_WebGLAquarium as it is spamming
cros-gfx-alerts.

BUG= chromium:769554 
TEST=test_that dut graphics_WebGLAquarium. and check results-chart.json

Change-Id: I23fbd9dde1cbb60843c34e897d7ff6cc136ab5d9
Reviewed-on: https://chromium-review.googlesource.com/690446
Commit-Ready: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Po-Hsien Wang <pwang@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

[modify] https://crrev.com/fbb11babe3b4eb702d245b7f3cebe1e5fb230ad1/client/site_tests/graphics_WebGLAquarium/graphics_WebGLAquarium.py

Project Member

Comment 2 by bugdroid1@chromium.org, Oct 3 2017

Labels: merge-merged-release-R61-9765.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/a3f025e3235a662ef7504cbce36964a50a85701b

commit a3f025e3235a662ef7504cbce36964a50a85701b
Author: Po-Hsien Wang <pwang@chromium.org>
Date: Tue Oct 03 22:02:30 2017

graphics_WebGLAquarium: Remove memory alert report

Remove meminfo alert from graphics_WebGLAquarium as it is spamming
cros-gfx-alerts.

BUG= chromium:769554 
TEST=test_that dut graphics_WebGLAquarium. and check results-chart.json

Change-Id: I23fbd9dde1cbb60843c34e897d7ff6cc136ab5d9
Reviewed-on: https://chromium-review.googlesource.com/690446
Commit-Ready: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Po-Hsien Wang <pwang@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
(cherry picked from commit fbb11babe3b4eb702d245b7f3cebe1e5fb230ad1)
Reviewed-on: https://chromium-review.googlesource.com/699331
Tested-by: Ilja H. Friedel <ihf@chromium.org>

[modify] https://crrev.com/a3f025e3235a662ef7504cbce36964a50a85701b/client/site_tests/graphics_WebGLAquarium/graphics_WebGLAquarium.py

Project Member

Comment 3 by bugdroid1@chromium.org, Oct 3 2017

Labels: merge-merged-release-R62-9901.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/b1a6ec6a80148cddeafcd01463d1c319335e52a2

commit b1a6ec6a80148cddeafcd01463d1c319335e52a2
Author: Po-Hsien Wang <pwang@chromium.org>
Date: Tue Oct 03 22:02:54 2017

graphics_WebGLAquarium: Remove memory alert report

Remove meminfo alert from graphics_WebGLAquarium as it is spamming
cros-gfx-alerts.

BUG= chromium:769554 
TEST=test_that dut graphics_WebGLAquarium. and check results-chart.json

Change-Id: I23fbd9dde1cbb60843c34e897d7ff6cc136ab5d9
Reviewed-on: https://chromium-review.googlesource.com/690446
Commit-Ready: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Po-Hsien Wang <pwang@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
(cherry picked from commit fbb11babe3b4eb702d245b7f3cebe1e5fb230ad1)
Reviewed-on: https://chromium-review.googlesource.com/699330
Tested-by: Ilja H. Friedel <ihf@chromium.org>

[modify] https://crrev.com/b1a6ec6a80148cddeafcd01463d1c319335e52a2/client/site_tests/graphics_WebGLAquarium/graphics_WebGLAquarium.py

Comment 4 by ihf@chromium.org, Oct 3 2017

Status: Verified (was: Available)

Sign in to add a comment