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

Issue 739580 link

Starred by 4 users

Issue metadata

Status: Archived
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

[CFM Test Automation] Sanity test for AutoZoom

Project Member Reported by harpreet@chromium.org, Jul 6 2017

Issue description

Status: Started (was: Untriaged)
Status: Fixed (was: Started)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 27 2017

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

commit 577ec0040e8520aecf9d4c60a1b1370f27cd4169
Author: harpreet <harpreet@google.com>
Date: Thu Jul 27 22:44:01 2017

Add autozoom sanity test.

Adding autozoom sanity test that enrolls the device into CFM and runs a
meetings session for a pre-defined length. After the session ends, it
examines the video sent resolution in JMI logs and outputs it into a
json file to be consumed by Chrome Perf Dashboard.

TODO's for this test include adding logic to examine CFM sent resolution
and taking appropriate action if resolution is not in the expected
range. Another TODO is to implement check_verify_rtanalytics_logs
method.

BUG= chromium:739580 
TEST=Tested on local setup.

Change-Id: I08d1d1c06ac6cd6ce5521920fd98f8a568831c50
Reviewed-on: https://chromium-review.googlesource.com/560627
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Dirk Kaeding <dkaeding@google.com>

[add] https://crrev.com/577ec0040e8520aecf9d4c60a1b1370f27cd4169/server/site_tests/enterprise_CFM_AutoZoomSanity/enterprise_CFM_AutoZoomSanity.py
[add] https://crrev.com/577ec0040e8520aecf9d4c60a1b1370f27cd4169/server/site_tests/enterprise_CFM_AutoZoomSanity/control

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/6e5002d6121e86570c4ffdadfb703f1cdce01e93

commit 6e5002d6121e86570c4ffdadfb703f1cdce01e93
Author: harpreet <harpreet@google.com>
Date: Tue Aug 01 20:01:40 2017

autotest-server-tests: Add autozoom and reboot stress tests

BUG= chromium:739580 
TEST=None

Change-Id: Icd19cedb7b9644693aa1b426166945d2b7d2924b
Reviewed-on: https://chromium-review.googlesource.com/592135
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Jon Deokule <jdeokule@chromium.org>

[modify] https://crrev.com/6e5002d6121e86570c4ffdadfb703f1cdce01e93/chromeos-base/autotest-server-tests/autotest-server-tests-9999.ebuild

Labels: M-61 Merge-Request-61
Project Member

Comment 6 by sheriffbot@chromium.org, Aug 25 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: We are only 10 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-61 Merge-Approved-61
Approving merge for M61.
Project Member

Comment 8 by bugdroid1@chromium.org, Aug 29 2017

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

commit 88ba4697a630c7540c8916d75b7f192ebf1b98fb
Author: harpreet <harpreet@google.com>
Date: Tue Aug 29 01:07:52 2017

Add autozoom sanity test.

Adding autozoom sanity test that enrolls the device into CFM and runs a
meetings session for a pre-defined length. After the session ends, it
examines the video sent resolution in JMI logs and outputs it into a
json file to be consumed by Chrome Perf Dashboard.

TODO's for this test include adding logic to examine CFM sent resolution
and taking appropriate action if resolution is not in the expected
range. Another TODO is to implement check_verify_rtanalytics_logs
method.

BUG= chromium:739580 
TEST=Tested on local setup.

Change-Id: I08d1d1c06ac6cd6ce5521920fd98f8a568831c50
Reviewed-on: https://chromium-review.googlesource.com/560627
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Dirk Kaeding <dkaeding@google.com>
(cherry picked from commit 577ec0040e8520aecf9d4c60a1b1370f27cd4169)
Reviewed-on: https://chromium-review.googlesource.com/636145
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>
Commit-Queue: Harpreet Grewal <harpreet@chromium.org>
Trybot-Ready: Harpreet Grewal <harpreet@chromium.org>

[add] https://crrev.com/88ba4697a630c7540c8916d75b7f192ebf1b98fb/server/site_tests/enterprise_CFM_AutoZoomSanity/enterprise_CFM_AutoZoomSanity.py
[add] https://crrev.com/88ba4697a630c7540c8916d75b7f192ebf1b98fb/server/site_tests/enterprise_CFM_AutoZoomSanity/control

Project Member

Comment 9 by bugdroid1@chromium.org, Aug 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/904e1254c2e345cadde9ea3147eabee4f299f6cc

commit 904e1254c2e345cadde9ea3147eabee4f299f6cc
Author: harpreet <harpreet@google.com>
Date: Tue Aug 29 01:10:23 2017

autotest-server-tests: Add autozoom and reboot stress tests

BUG= chromium:739580 
TEST=None

Change-Id: Icd19cedb7b9644693aa1b426166945d2b7d2924b
Reviewed-on: https://chromium-review.googlesource.com/592135
Commit-Ready: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Jon Deokule <jdeokule@chromium.org>
(cherry picked from commit 6e5002d6121e86570c4ffdadfb703f1cdce01e93)
Reviewed-on: https://chromium-review.googlesource.com/636523
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>
Commit-Queue: Harpreet Grewal <harpreet@chromium.org>
Trybot-Ready: Harpreet Grewal <harpreet@chromium.org>

[modify] https://crrev.com/904e1254c2e345cadde9ea3147eabee4f299f6cc/chromeos-base/autotest-server-tests/autotest-server-tests-9999.ebuild

Labels: -Merge-Approved-61
@Harpreet - this test has as far as I can tell never run - the control file list a 'autozoom' dependency but we don't have such a label. Only 'huddly' exists. I'll update the file
Project Member

Comment 12 by bugdroid1@chromium.org, Nov 13 2017

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

commit 8382bc51d101c915bb915da5c50716485bd37c96
Author: Denis Tosic <dtosic@chromium.org>
Date: Mon Nov 13 10:50:46 2017

[autotest][cfm] Fix AutoZoomSanity dependencies

The test never ran because the dependency 'autozoom' was never provided.
Instead, use 'huddly' and 'mimo'. Also adds the the test to the
bluestreak-release cq.

TEST=local setup
BUG= chromium:739580 
pre-cq-configs: bluestreak-pre-cq

Change-Id: I803273527ffef041c251cf1384f03ab45ca57630
Reviewed-on: https://chromium-review.googlesource.com/765470
Tested-by: Denis Tosic <dtosic@chromium.org>
Trybot-Ready: Denis Tosic <dtosic@chromium.org>
Reviewed-by: Denis Tosic <dtosic@chromium.org>

[modify] https://crrev.com/8382bc51d101c915bb915da5c50716485bd37c96/server/site_tests/enterprise_CFM_AutoZoomSanity/control

Comment 13 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment