[Asterisk-code-review] tests/rest api/channels/originate: Reduce number of channels. (testsuite[master])

[Anonymous Coward]

Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: tests/rest_api/channels/originate: Reduce number of channels.
......................................................................


tests/rest_api/channels/originate: Reduce number of channels.

The testsuite can get overwhelmed when 25 channels are originated
at the same time causing its AMI capability to slow down and
disconnect. When this happens it proceeds to originate a second
channel to kick off the whole process again which causes more
problems.

This change merely reduces the number of originated channels
down to 15 so the slower Jenkins nodes can tolerate it.

Change-Id: I51ee075a097dfeef568f0069f8bd22975106c56e
---
M tests/rest_api/channels/originate/channel_originate.py
M tests/rest_api/channels/originate/test-config.yaml
2 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/tests/rest_api/channels/originate/channel_originate.py b/tests/rest_api/channels/originate/channel_originate.py
index af1586c..2382fdd 100644
--- a/tests/rest_api/channels/originate/channel_originate.py
+++ b/tests/rest_api/channels/originate/channel_originate.py
@@ -15,7 +15,7 @@
 
 def on_kickoff_start(ari, event, test_object):
     LOGGER.debug("on_kickoff_start(%r)" % event)
-    for x in xrange(25):
+    for x in xrange(15):
         ari.post('channels',
                  endpoint='Local/1000 at default',
                  app='testsuite',
@@ -33,7 +33,7 @@
     global CHANNELS
     LOGGER.debug("on_channel_destroyed: %s" % str(event.get('channel')))
     CHANNELS += 1
-    if CHANNELS == 50:
+    if CHANNELS == 30:
         LOGGER.info("All channels destroyed")
         test_object.set_passed(True)
         test_object.stop_reactor()
diff --git a/tests/rest_api/channels/originate/test-config.yaml b/tests/rest_api/channels/originate/test-config.yaml
index ed9abf5..9209520 100644
--- a/tests/rest_api/channels/originate/test-config.yaml
+++ b/tests/rest_api/channels/originate/test-config.yaml
@@ -33,7 +33,7 @@
                     type: StasisStart
                     application: testsuite
                     args: ['blast']
-            count: 25
+            count: 15
             callback:
                 module: channel_originate
                 method: on_blast_start
@@ -41,7 +41,7 @@
                 match:
                     type: ChannelDestroyed
                     application: testsuite
-            count: 50
+            count: 30
             callback:
                 module: channel_originate
                 method: on_channel_destroyed

-- 
To view, visit https://gerrit.asterisk.org/2353
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I51ee075a097dfeef568f0069f8bd22975106c56e
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>