autotest: hostap_config: Allow setting max STA count |
||
Issue descriptionHostapd provides the max_num_sta option for configuring how many STAs can connect to an AP. The option is useful for inducing connection failures due to too many STAs being connected to an AP, and so should be added to the HostapConfig class in the server.cros.network.hostap_config package for use in various autotests. Based on a quick test, it seems like this option also works for a max_num_sta value of 0, so we can use this to induce connection failures even with just a single STA.
,
Jun 30 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/0d9e8aff8d902a883b18380f8c506dba7a2de6e3 commit 0d9e8aff8d902a883b18380f8c506dba7a2de6e3 Author: Alex Khouderchah <akhouderchah@chromium.org> Date: Sat Jun 30 09:14:23 2018 autotest: hostap_config: Allow configuring max STA count Hostapd provides a configuration option to determine how many STAs an AP will allow to be connected to it. This change adds the 'max_stas' option to HostapConfig in order to expose this configuration option to autotests. BUG= chromium:857580 TEST=Modified network_WiFi_Perf tests to use max_stas=0 and ensured that the STA failed to connect (provided reason was 'Association timed out'). Also ran the original network_WiFi_Perf test to verify that leaving max_stas unspecified works as expected. Change-Id: I03f06d7a76e98806e69c34f2da1b6ade5947f31f Reviewed-on: https://chromium-review.googlesource.com/1119080 Commit-Ready: Kirtika Ruchandani <kirtika@chromium.org> Tested-by: Kirtika Ruchandani <kirtika@chromium.org> Tested-by: Alex Khouderchah <akhouderchah@chromium.org> Reviewed-by: Kirtika Ruchandani <kirtika@chromium.org> Reviewed-by: Brian Norris <briannorris@chromium.org> [modify] https://crrev.com/0d9e8aff8d902a883b18380f8c506dba7a2de6e3/server/cros/network/hostap_config.py
,
Jul 2
|
||
►
Sign in to add a comment |
||
Comment 1 by briannorris@chromium.org
, Jun 28 2018Owner: akhouderchah@chromium.org
Status: Started (was: Untriaged)