stats: revert "add a retry mechanism in a symlink test" 20/32920/3
authorBeno�t Ganne <bganne@cisco.com>
Fri, 25 Jun 2021 15:45:05 +0000 (15:45 +0000)
committerBenoît Ganne <bganne@cisco.com>
Fri, 25 Jun 2021 15:59:18 +0000 (17:59 +0200)
This reverts commit d70777aa60a87001f323f3b226d7e5cee2cd23d8.

Reason for revert: wrong fix

Type: fix

Change-Id: Ibff1b977eae8d2cc85888165ebf83c42eb661698
Signed-off-by: Benoît Ganne <bganne@cisco.com>
test/test_stats_client.py

index e5b6aa7..7e17e2a 100644 (file)
@@ -2,8 +2,6 @@
 
 import unittest
 import psutil
-import sys
-import six
 from vpp_papi.vpp_stats import VPPStats
 
 from framework import tag_fixme_vpp_workers
@@ -110,19 +108,7 @@ class StatsClientTestCase(VppTestCase):
         # We wait for nodes symlinks to update (interfaces created/deleted).
         # ... and packets to be sent
         self.sleep(0.1)
-        for _ in range(5):
-            try:
-                vectors = self.statistics.get_counter('/nodes/pg1-tx/vectors')
-                if vectors[0] == 0:
-                    raise ValueError("Nodes counters are not up to date")
-                break
-            except:
-                t, v, tb = sys.exc_info()
-
-                self.sleep(0.1)
-                continue
-        else:
-            six.reraise(t, v, tb)
+        vectors = self.statistics.get_counter('/nodes/pg1-tx/vectors')
 
         self.assertEqual(tx[0]['bytes'] - tx_before_sending[0]['bytes'],
                          bytes_to_send)