-
Notifications
You must be signed in to change notification settings - Fork 714
/
test_buffer.py
3308 lines (2855 loc) · 154 KB
/
test_buffer.py
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
import logging
import time
import re
import json
import math
from natsort import natsorted
import pytest
from tests.common import config_reload
from tests.common.utilities import wait_until
from tests.common.helpers.assertions import pytest_assert, pytest_require
from tests.common.fixtures.conn_graph_facts import conn_graph_facts # noqa F401
from tests.common.mellanox_data import is_mellanox_device
from tests.common.innovium_data import is_innovium_device
from tests.common.plugins.loganalyzer.loganalyzer import LogAnalyzer
from tests.common.utilities import check_qos_db_fv_reference_with_table
from tests.common.utilities import skip_release
from tests.common.dualtor.dual_tor_utils import is_tunnel_qos_remap_enabled, dualtor_ports # noqa F401
from tests.qos.buffer_helpers import DutDbInfo
pytestmark = [
pytest.mark.topology('any')
]
profile_format = 'pg_lossless_{}_{}_profile'
LOSSLESS_PROFILE_PATTERN = 'pg_lossless_([1-9][0-9]*000)_([1-9][0-9]*m)_profile'
DEFAULT_CABLE_LENGTH_LIST = None
DEFAULT_LOSSLESS_HEADROOM_DATA = None
DEFAULT_INGRESS_POOL_NUMBER = 0
DEFAULT_SHARED_HEADROOM_POOL_ENABLED = False
DEFAULT_OVER_SUBSCRIBE_RATIO = None
DEFAULT_SHARED_HEADROOM_POOL_SIZE = None
DEFAULT_MTU = None
PORT_TO_TEST = None
NUMBER_OF_LANES = None
PORTS_WITH_8LANES = None
ASIC_TYPE = None
TESTPARAM_HEADROOM_OVERRIDE = None
TESTPARAM_LOSSLESS_PG = None
TESTPARAM_SHARED_HEADROOM_POOL = None
TESTPARAM_EXTRA_OVERHEAD = None
TESTPARAM_ADMIN_DOWN = None
BUFFER_MODEL_DYNAMIC = True
ASIC_TABLE_KEYS_LOADED = False
CELL_SIZE = None
PIPELINE_LATENCY = None
MAC_PHY_DELAY = None
LOSSLESS_TRAFFIC_PATTERN_KEYS_LOADED = False
LOSSLESS_MTU = None
SMALL_PACKET_PERCENTAGE = None
KEY_2_LOSSLESS_QUEUE = "2_lossless_queues"
KEY_4_LOSSLESS_QUEUE = "4_lossless_queues"
def detect_buffer_model(duthost):
"""Detect the current buffer model (dynamic or traditional) and store it for further use.
Called only once when the module is initialized
Args:
duthost: The DUT host object
"""
global BUFFER_MODEL_DYNAMIC
buffer_model = duthost.shell(
'redis-cli -n 4 hget "DEVICE_METADATA|localhost" buffer_model')['stdout']
BUFFER_MODEL_DYNAMIC = (buffer_model == 'dynamic')
def detect_ingress_pool_number(duthost):
"""Detect the number of ingress buffer pools and store it for further use.
Called only once when the module is initialized
Args:
duthost: The DUT host object
"""
global DEFAULT_INGRESS_POOL_NUMBER
pools = duthost.shell(
'redis-cli -n 4 keys "BUFFER_POOL|ingress*"')['stdout']
DEFAULT_INGRESS_POOL_NUMBER = len(pools.split())
def detect_shared_headroom_pool_mode(duthost):
"""Detect whether shared headroom pool is enabled
Args:
duthost: The DUT host object
"""
global DEFAULT_SHARED_HEADROOM_POOL_ENABLED
global DEFAULT_SHARED_HEADROOM_POOL_SIZE
global DEFAULT_OVER_SUBSCRIBE_RATIO
over_subscribe_ratio = duthost.shell(
'redis-cli -n 4 hget "DEFAULT_LOSSLESS_BUFFER_PARAMETER|AZURE" over_subscribe_ratio')['stdout']
if over_subscribe_ratio and over_subscribe_ratio != '0':
DEFAULT_SHARED_HEADROOM_POOL_ENABLED = True
DEFAULT_OVER_SUBSCRIBE_RATIO = int(over_subscribe_ratio)
shared_headroom_pool_size = duthost.shell(
'redis-cli -n 4 hget "BUFFER_POOL|ingress_lossless_pool" xoff')['stdout']
if shared_headroom_pool_size and shared_headroom_pool_size != '0':
DEFAULT_SHARED_HEADROOM_POOL_ENABLED = True
DEFAULT_SHARED_HEADROOM_POOL_SIZE = int(shared_headroom_pool_size)
def detect_default_mtu(duthost, port_to_test):
"""Detect the mtu and store it for further use. Called only once when the module is initialized
Args:
duthost: The DUT host object
"""
global DEFAULT_MTU
if not DEFAULT_MTU:
DEFAULT_MTU = duthost.shell(
'redis-cli -n 4 hget "PORT|{}" mtu'.format(port_to_test))['stdout']
logging.info("Default MTU {}".format(DEFAULT_MTU))
def get_asic_table_data_from_db(duthost):
"""
Load CELL_SIZE, PIPELINE_LATENCY and MAC_PHY_DELAY from ASIC_TABLE
"""
# Get cell size from state DB
# Command: redis-cli -n 6 hget "ASIC_TABLE|MELLANOX-SPECTRUM-2" 'cell_size'
asic_keys = duthost.shell('redis-cli -n 6 keys *ASIC_TABLE*')['stdout']
cell_size = float(duthost.shell(
'redis-cli -n 6 hget "{}" "cell_size"'.format(asic_keys))['stdout'])
# Get PIPELINE_LATENCY from state DB
# Command: redis-cli -n 6 hget "ASIC_TABLE|MELLANOX-SPECTRUM-2" 'pipeline_latency'
pipeline_latency = float(
duthost.shell('redis-cli -n 6 hget "{}" "pipeline_latency"'.format(asic_keys))['stdout']) * 1024
# Get MAC_PHY_DELAY from state DB
# Command: redis-cli -n 6 hget "ASIC_TABLE|MELLANOX-SPECTRUM-2" 'mac_phy_delay'
mac_phy_delay = float(duthost.shell(
'redis-cli -n 6 hget "{}" "mac_phy_delay"'.format(asic_keys))['stdout']) * 1024
return cell_size, pipeline_latency, mac_phy_delay
def detect_asic_table_keys(duthost):
"""
Get CELL_SIZE, PIPELINE_LATENCY and MAC_PHY_DELAY by function get_asic_table_data_from_db
"""
global CELL_SIZE
global PIPELINE_LATENCY
global MAC_PHY_DELAY
global ASIC_TABLE_KEYS_LOADED
CELL_SIZE, PIPELINE_LATENCY, MAC_PHY_DELAY = get_asic_table_data_from_db(
duthost)
ASIC_TABLE_KEYS_LOADED = True
def get_lossless_traffic_pattern_data_from_db(duthost):
"""
Load LOSSLESS_MTU, SMALL_PACKET_PERCENTAGE from LOSSLESS_TRAFFIC_PATTERN table
"""
# Get LOSSLESS_MTU from config DB
# Command: redis-cli -n 4 hget 'LOSSLESS_TRAFFIC_PATTERN|AZURE' 'mtu'
lossless_traffic_keys = duthost.shell(
'redis-cli -n 4 keys LOSSLESS_TRAFFIC_PATTERN*')['stdout']
lossless_mtu = float(duthost.shell(
'redis-cli -n 4 hget "{}" "mtu"'.format(lossless_traffic_keys))['stdout'])
# Get SMALL_PACKET_PERCENTAGE from config DB
# Command: redis-cli -n 4 hget 'LOSSLESS_TRAFFIC_PATTERN|AZURE' 'small_packet_percentage'
small_packet_percentage = float(
duthost.shell('redis-cli -n 4 hget "{}" "small_packet_percentage"'.format(lossless_traffic_keys))['stdout'])
return lossless_mtu, small_packet_percentage
def detect_lossless_traffic_pattern_keys(duthost):
"""
Get LOSSLESS_MTU, SMALL_PACKET_PERCENTAGE by calling function get_lossless_traffic_pattern_data_from_db
"""
global LOSSLESS_MTU
global SMALL_PACKET_PERCENTAGE
global LOSSLESS_TRAFFIC_PATTERN_KEYS_LOADED
LOSSLESS_MTU, SMALL_PACKET_PERCENTAGE = get_lossless_traffic_pattern_data_from_db(
duthost)
LOSSLESS_TRAFFIC_PATTERN_KEYS_LOADED = True
def load_lossless_headroom_data(duthost):
"""Load test parameters from the json file. Called only once when the module is initialized
Args:
duthost: the DUT host object
"""
global DEFAULT_LOSSLESS_HEADROOM_DATA
if not DEFAULT_LOSSLESS_HEADROOM_DATA:
dut_hwsku = duthost.facts["hwsku"]
dut_platform = duthost.facts["platform"]
skudir = "/usr/share/sonic/device/{}/{}/".format(
dut_platform, dut_hwsku)
lines = duthost.shell(
'cat {}/pg_profile_lookup.ini'.format(skudir))["stdout"]
DEFAULT_LOSSLESS_HEADROOM_DATA = {}
for line in lines.split('\n'):
if line[0] == '#':
continue
tokens = line.split()
speed = tokens[0]
cable_length = tokens[1]
size = tokens[2]
xon = tokens[3]
xoff = tokens[4]
if not DEFAULT_LOSSLESS_HEADROOM_DATA.get(speed):
DEFAULT_LOSSLESS_HEADROOM_DATA[speed] = {}
DEFAULT_LOSSLESS_HEADROOM_DATA[speed][cable_length] = {
'size': size, 'xon': xon, 'xoff': xoff}
DEFAULT_LOSSLESS_HEADROOM_DATA = DEFAULT_LOSSLESS_HEADROOM_DATA
def load_test_parameters(duthost):
"""Load test parameters from the json file. Called only once when the module is initialized
Args:
duthost: The DUT host object
"""
global DEFAULT_CABLE_LENGTH_LIST
global TESTPARAM_HEADROOM_OVERRIDE
global TESTPARAM_LOSSLESS_PG
global TESTPARAM_SHARED_HEADROOM_POOL
global TESTPARAM_EXTRA_OVERHEAD
global TESTPARAM_ADMIN_DOWN
global ASIC_TYPE
global MAX_SPEED_8LANE_PORT
param_file_name = "qos/files/dynamic_buffer_param.json"
with open(param_file_name) as file:
params = json.load(file)
logging.info("Loaded test parameters {} from {}".format(
params, param_file_name))
ASIC_TYPE = duthost.facts['asic_type']
vendor_specific_param = params[ASIC_TYPE]
DEFAULT_CABLE_LENGTH_LIST = vendor_specific_param['default_cable_length']
TESTPARAM_HEADROOM_OVERRIDE = vendor_specific_param['headroom-override']
TESTPARAM_LOSSLESS_PG = vendor_specific_param['lossless_pg']
TESTPARAM_SHARED_HEADROOM_POOL = vendor_specific_param['shared-headroom-pool']
TESTPARAM_EXTRA_OVERHEAD = vendor_specific_param['extra_overhead']
TESTPARAM_ADMIN_DOWN = vendor_specific_param['admin-down']
MAX_SPEED_8LANE_PORT = vendor_specific_param['max_speed_8lane_platform'].get(
duthost.facts['platform'])
# For ingress profile list, we need to check whether the ingress lossy profile exists
ingress_lossy_pool = duthost.shell(
'redis-cli -n 4 keys "BUFFER_POOL|ingress_lossy_pool"')['stdout']
if ingress_lossy_pool:
ingress_profile_list = TESTPARAM_ADMIN_DOWN.get(
'BUFFER_PORT_INGRESS_PROFILE_LIST_TABLE')
ingress_profile_list.append(
'[BUFFER_PROFILE_TABLE:ingress_lossy_zero_profile]')
# 'admin-down' section contains references to buffer profiles
# We need to convert the format of the references according to whether table name should be in the reference
if not check_qos_db_fv_reference_with_table(duthost):
expected_pgs = TESTPARAM_ADMIN_DOWN.get('BUFFER_PG_TABLE')
if expected_pgs:
new_pgs = {}
for pg, profile in list(expected_pgs.items()):
new_pgs[pg] = profile.replace(
'[BUFFER_PROFILE_TABLE:', '').replace(']', '')
TESTPARAM_ADMIN_DOWN['BUFFER_PG_TABLE'] = new_pgs
expected_queues = TESTPARAM_ADMIN_DOWN.get('BUFFER_QUEUE_TABLE')
if expected_queues:
new_queues = {}
for queue, profile in list(expected_queues.items()):
new_queues[queue] = profile.replace(
'[BUFFER_PROFILE_TABLE:', '').replace(']', '')
TESTPARAM_ADMIN_DOWN['BUFFER_QUEUE_TABLE'] = new_queues
expected_ingress_profile_list = TESTPARAM_ADMIN_DOWN.get(
'BUFFER_PORT_INGRESS_PROFILE_LIST_TABLE')
if expected_ingress_profile_list:
new_list = []
for profile in expected_ingress_profile_list:
new_list.append(profile.replace(
'[BUFFER_PROFILE_TABLE:', '').replace(']', ''))
TESTPARAM_ADMIN_DOWN['BUFFER_PORT_INGRESS_PROFILE_LIST_TABLE'] = new_list
expected_egress_profile_list = TESTPARAM_ADMIN_DOWN.get(
'BUFFER_PORT_EGRESS_PROFILE_LIST_TABLE')
if expected_egress_profile_list:
new_list = []
for profile in expected_egress_profile_list:
new_list.append(profile.replace(
'[BUFFER_PROFILE_TABLE:', '').replace(']', ''))
TESTPARAM_ADMIN_DOWN['BUFFER_PORT_EGRESS_PROFILE_LIST_TABLE'] = new_list
def configure_shared_headroom_pool(duthost, enable):
"""Enable or disable the shared headroom pool according to the argument
Args:
duthost: The DUT host object
enable: True to enable and false to disable the shared headroom pool
"""
if enable:
duthost.shell(
"config buffer shared-headroom-pool over-subscribe-ratio 2")
else:
duthost.shell(
"config buffer shared-headroom-pool over-subscribe-ratio 0")
time.sleep(20)
@pytest.fixture(scope="module", autouse=True)
def setup_module(duthosts, rand_one_dut_hostname, request):
"""Set up module. Called only once when the module is initialized
Args:
duthost: The DUT host object
"""
global DEFAULT_SHARED_HEADROOM_POOL_ENABLED
global DEFAULT_OVER_SUBSCRIBE_RATIO
duthost = duthosts[rand_one_dut_hostname]
detect_buffer_model(duthost)
if not is_mellanox_device(duthost) and not is_innovium_device(duthost):
load_lossless_headroom_data(duthost)
yield
return
# Disable BGP neighbors
# There are a lot of routing entries learnt with BGP neighbors enabled.
# There are a lot of speed changing operations during the buffer test,
# which causes port operational down and routing entries withdrawn.
# Since orchagent works in a single thread model, this can causes buffer related notifications
# pended in the queue and can not be drained until routing entries handled,
# which in turn significantly slows down the process in orchagent and makes many checks timeout.
# As the buffer test has already taken ~30 minutes, we don't want to extend the wait time.
# So disabling BGP neighbors is a reasonal way to tolerance this situation.
bgp_neighbors = duthost.shell(
'redis-cli -n 4 keys BGP_NEIGHBOR*')['stdout']
if bgp_neighbors:
duthost.shell('config bgp shutdown all')
logging.info(
"Shutting down BGP neighbors and waiting for all routing entries withdrawn")
time.sleep(60)
enable_shared_headroom_pool = request.config.getoption(
"--enable_shared_headroom_pool")
need_to_disable_shared_headroom_pool_after_test = False
if BUFFER_MODEL_DYNAMIC:
detect_ingress_pool_number(duthost)
detect_shared_headroom_pool_mode(duthost)
detect_asic_table_keys(duthost)
detect_lossless_traffic_pattern_keys(duthost)
load_lossless_headroom_data(duthost)
load_test_parameters(duthost)
logging.info("Cable length: default {}".format(
DEFAULT_CABLE_LENGTH_LIST))
logging.info("Ingress pool number {}".format(
DEFAULT_INGRESS_POOL_NUMBER))
logging.info("Lossless headroom data {}".format(
DEFAULT_LOSSLESS_HEADROOM_DATA))
if enable_shared_headroom_pool and not DEFAULT_SHARED_HEADROOM_POOL_ENABLED:
configure_shared_headroom_pool(duthost, True)
DEFAULT_SHARED_HEADROOM_POOL_ENABLED = True
DEFAULT_OVER_SUBSCRIBE_RATIO = 2
logging.info(
"Shared headroom pool enabled according to test option")
need_to_disable_shared_headroom_pool_after_test = True
else:
load_lossless_headroom_data(duthost)
logging.info("Lossless headroom data {}".format(
DEFAULT_LOSSLESS_HEADROOM_DATA))
yield
if need_to_disable_shared_headroom_pool_after_test:
configure_shared_headroom_pool(duthost, False)
if bgp_neighbors:
duthost.shell("config bgp startup all")
time.sleep(60)
def skip_traditional_model():
if not BUFFER_MODEL_DYNAMIC:
pytest.skip("Skip test in traditional model")
def init_log_analyzer(duthost, marker, expected, ignored=None):
loganalyzer = LogAnalyzer(ansible_host=duthost, marker_prefix=marker)
marker = loganalyzer.init()
loganalyzer.load_common_config()
loganalyzer.expect_regex = []
loganalyzer.expect_regex.extend(expected)
if ignored:
loganalyzer.ignore_regex.extend(ignored)
return loganalyzer, marker
def check_log_analyzer(loganalyzer, marker):
loganalyzer.analyze(marker)
return loganalyzer
def check_pool_size(duthost, ingress_lossless_pool_oid, **kwargs):
"""Check whether the pool size has been updated correctly
The expected pool size will be calculated based on the input arguments on a per-vendor basis
After that, it will check the expected value against the buffer pool size in BUFFER_POOL_TABLE
and in the ASIC_DB
Args:
ingress_lossless_pool_oid: The SAI OID of the ingress lossless pool in ASIC_DB
kwargs: The parameters based on which the expected pool size is calculated.
They are represented in form of kwargs because different vendor can require different parameters
For Mellanox, it includes:
- Old / new pg size
- Old / new pg xoff (required only over subscribe ratio is defined)
- Old / new pg numbers
- Old_ratio / new_ratio / conn_graph_facts (required only over subscribe ratio is defined)
- Current pool size
- Current shared headroom pool size (required only over subscribe ratio is defined)
- The expected pool size is calculated as following:
- Shared headroom pool disabled:
current_pool_size + old_pg_num * old_pg_size - new_pg_num * new_pg_size
- Shared headroom pool enabled by over subscribe ratio:
current_pool_size + old_pg_num * old_pg_size - new_pg_num * new_pg_size
+ (old_pg_num * old_pg_xoff - new_pg_num * new_pg_xoff) * over_subscribe_ratio
"""
def _fetch_size_difference_for_8lane_ports(duthost, conn_graph_facts): # noqa F811
"""Calculate the difference in buffer pool size caused by 8-lane ports on Mellanox platform
Args:
duthost: The duthost object
conn_graph_facts: The connection graph facts object
"""
global PORTS_WITH_8LANES
hostname = list(conn_graph_facts['device_conn'].keys())[0]
ports_info = conn_graph_facts['device_conn'][hostname]
if not ports_info:
ports = [port.split('|')[1] for port in duthost.shell(
'redis-cli -n 4 keys "PORT|*"')['stdout'].split('\n')]
else:
ports = list(ports_info.keys())
if PORTS_WITH_8LANES is None:
PORTS_WITH_8LANES = []
for port in ports:
lanes = duthost.shell(
'redis-cli -n 4 hget "PORT|{}" lanes'.format(port))['stdout']
if len(lanes.split(',')) == 8:
PORTS_WITH_8LANES.append(port)
lossless_pgs = duthost.shell(
'redis-cli keys "BUFFER_PG_TABLE:Ethernet*:3-4"')['stdout'].split()
lossless_pgs_8lane = [
pg for pg in lossless_pgs if pg[16:-4] in PORTS_WITH_8LANES]
return len(lossless_pgs_8lane) * 2 * 9216
logging.debug("Kwargs {}".format(kwargs))
if duthost.facts['asic_type'] == 'mellanox':
if kwargs.get("old_ratio") and kwargs.get("new_ratio"):
curr_pool_size = int(kwargs["pool_size"])
curr_shp_size = int(kwargs["shp_size"])
old_ratio = int(kwargs.get("old_ratio"))
new_ratio = int(kwargs.get("new_ratio"))
conn_graph_facts_kwargs = kwargs.get("conn_graph_facts")
original_memory = curr_pool_size * DEFAULT_INGRESS_POOL_NUMBER + curr_shp_size
if new_ratio == 0:
expected_shp_size = 0
expected_pool_size = (
original_memory - curr_shp_size * old_ratio)
if old_ratio != 0:
expected_pool_size = expected_pool_size - \
_fetch_size_difference_for_8lane_ports(
duthost, conn_graph_facts_kwargs)
expected_pool_size = expected_pool_size / DEFAULT_INGRESS_POOL_NUMBER
else:
expected_shp_size = curr_shp_size * old_ratio / new_ratio
expected_pool_size = (
original_memory - expected_shp_size) / DEFAULT_INGRESS_POOL_NUMBER
elif kwargs.get("config_shp_size"):
expected_shp_size = int(kwargs.get("config_shp_size"))
expected_pool_size = None
else:
curr_pool_size = int(kwargs["pool_size"])
if "old_pg_number" in kwargs:
old_pg_number = int(kwargs["old_pg_number"])
else:
old_pg_number = 2
if old_pg_number:
old_size = int(kwargs["old_size"])
else:
old_size = 0
if "new_pg_number" in kwargs:
new_pg_number = int(kwargs["new_pg_number"])
else:
new_pg_number = old_pg_number
if new_pg_number:
if "new_size" in kwargs:
new_size = int(kwargs["new_size"])
else:
new_size = old_size
new_reserved = new_pg_number * new_size
else:
new_reserved = 0
if "adjust_extra_overhead" in kwargs:
adjust_extra_overhead = int(kwargs["adjust_extra_overhead"])
else:
adjust_extra_overhead = 0
original_memory = curr_pool_size * DEFAULT_INGRESS_POOL_NUMBER + \
old_size * old_pg_number + adjust_extra_overhead
if DEFAULT_OVER_SUBSCRIBE_RATIO:
private_headroom_str = TESTPARAM_SHARED_HEADROOM_POOL.get(
"private_pg_headroom")
if private_headroom_str:
private_headroom_number = int(private_headroom_str)
else:
private_headroom_number = 0
curr_shp_size = int(kwargs["shp_size"])
if old_pg_number:
old_xoff = int(kwargs["old_xoff"])
else:
old_xoff = 0
if new_pg_number and "new_xoff" in kwargs:
new_xoff = int(kwargs["new_xoff"])
else:
new_xoff = old_xoff
original_memory += curr_shp_size
shp_size_diff = new_xoff * new_pg_number - old_xoff * old_pg_number
if old_pg_number != 0:
original_memory += private_headroom_number
shp_size_diff += private_headroom_number
if new_pg_number != 0:
new_reserved += private_headroom_number
shp_size_diff -= private_headroom_number
expected_shp_size = curr_shp_size + shp_size_diff / DEFAULT_OVER_SUBSCRIBE_RATIO
new_reserved += expected_shp_size
else:
expected_shp_size = None
curr_shp_size = None
expected_pool_size = (
original_memory - new_reserved) / DEFAULT_INGRESS_POOL_NUMBER
logging.debug("Expected pool {}, expec shp {}, curr_shp {} default ovs {}".format(
expected_pool_size, expected_shp_size, curr_shp_size, DEFAULT_OVER_SUBSCRIBE_RATIO))
pytest_assert(
ensure_pool_size(duthost, 20, expected_pool_size, expected_shp_size, ingress_lossless_pool_oid),
"Pool size isn't correct in database: expected pool {} shp {}, "
"size in APPL_DB pool {} shp {}, size in ASIC_DB {}".format(
expected_pool_size,
expected_shp_size,
duthost.shell(
'redis-cli hget "BUFFER_POOL_TABLE:ingress_lossless_pool" size')['stdout'],
duthost.shell(
'redis-cli hget "BUFFER_POOL_TABLE:ingress_lossless_pool" xoff')['stdout'],
get_pool_size_from_asic_db(duthost, ingress_lossless_pool_oid))
if DEFAULT_OVER_SUBSCRIBE_RATIO else
"Pool size isn't correct in database: expected {}, size in APPL_DB pool {}, size in ASIC_DB {}".format(
expected_pool_size,
duthost.shell(
'redis-cli hget "BUFFER_POOL_TABLE:ingress_lossless_pool" size')['stdout'],
get_pool_size_from_asic_db(duthost, ingress_lossless_pool_oid))
)
def get_pool_size_from_asic_db(duthost, ingress_lossless_pool_oid):
pool_sai = _compose_dict_from_cli(duthost.shell(
'redis-cli -n 1 hgetall ASIC_STATE:SAI_OBJECT_TYPE_BUFFER_POOL:{}'
.format(ingress_lossless_pool_oid))['stdout'].split('\n'))
return pool_sai.get('SAI_BUFFER_POOL_ATTR_SIZE'), pool_sai.get('SAI_BUFFER_POOL_ATTR_XOFF_SIZE')
def ensure_pool_size(duthost, timeout, expected_pool_size, expected_shp_size, ingress_lossless_pool_oid=None):
"""Make sure the size of the buffer pool and shared headroom pool is as expected
within a given time in APPL_DB and ASIC_DB (optional)
Args:
timeout: The time out value
expected_pool_size: The expected shared buffer pool size
expected_shp_size: The expected shared headroom pool size
ingress_lossless_pool_oid: The SAI OID of ingress lossless buffer pool.
If it is omitted, the ASIC DB won't be checked
"""
def _ensure_pool_size(duthost, expected_pool_size, expected_shp_size, ingress_lossless_pool_oid=None):
pool_app = _compose_dict_from_cli(duthost.shell(
'redis-cli hgetall "BUFFER_POOL_TABLE:ingress_lossless_pool"')['stdout'].split('\n'))
if expected_pool_size and int(pool_app['size']) != expected_pool_size:
return False
if DEFAULT_OVER_SUBSCRIBE_RATIO and int(pool_app['xoff']) != expected_shp_size:
return False
if ingress_lossless_pool_oid:
pool_size, shp_size = get_pool_size_from_asic_db(
duthost, ingress_lossless_pool_oid)
if expected_pool_size and int(pool_size) != expected_pool_size:
return False
if expected_shp_size and expected_shp_size != int(shp_size):
return False
return True
if timeout >= 5:
delay = timeout / 5
else:
delay = 1
return wait_until(timeout, delay, 0, _ensure_pool_size, duthost,
expected_pool_size, expected_shp_size, ingress_lossless_pool_oid)
def check_pg_profile(duthost, pg, expected_profile, fail_test=True):
"""Check whether the profile in BUFFER_PG match the expected value
in a wait_until loop with maximum timeout as 10 seconds
Args:
pg: The key of buffer pg in BUFFER_PG table. Format: BUFFER_PG|<port>|<pg>
expected_profile: The name of the expected profile
fail_test: Fail the test by pytest_assert in case expected_profile not found within given time
Returns:
Whether the expected profile has been found within given time
"""
def _check_pg_profile(duthost, pg, expected_profile):
if check_qos_db_fv_reference_with_table(duthost) is True:
profile = duthost.shell(
'redis-cli hget {} profile'.format(pg))['stdout'][1:-1]
return (profile == 'BUFFER_PROFILE_TABLE:' + expected_profile)
else:
profile = duthost.shell(
'redis-cli hget {} profile'.format(pg))['stdout']
return (profile == expected_profile)
if wait_until(10, 2, 0, _check_pg_profile, duthost, pg, expected_profile):
return True
else:
if fail_test:
pytest_assert(False, "Profile in PG {} isn't {}".format(
pg, expected_profile))
else:
return False
def check_pfc_enable(duthost, port, expected_pfc_enable_map):
"""Check whether the pfc_enable map in port table is correct
in a wait_until loop with maximum timeout as 10 seconds
Args:
port: The port to be checked
expected_pfc_enable_map: The expected pfc_enable map
"""
def _check_pfc_enable(duthost, port, expected_pfc_enable_map):
pfc_enable = duthost.shell(
'redis-cli -n 4 hget "PORT_QOS_MAP|{}" pfc_enable'.format(port))['stdout']
return (expected_pfc_enable_map == pfc_enable)
pytest_assert(wait_until(10, 2, 0, _check_pfc_enable, duthost, port, expected_pfc_enable_map),
"Port {} pfc enable check failed expected: {} got: {}".format(
port,
expected_pfc_enable_map,
duthost.shell('redis-cli -n 4 hget "PORT_QOS_MAP|{}" pfc_enable'.format(port))['stdout']))
def check_lossless_profile_removed(duthost, profile, sai_oid=None):
"""Check whether the lossless profile has been removed from APPL_DB, STATE_DB and ASIC_DB (if sai_oid provided)
Args:
profile: The name of the buffer profile to be checked
sai_oid: The SAI OID in ASIC_DB of the buffer profile
If it is None the ASIC_DB won't be checked
"""
profile_info = duthost.shell(
'redis-cli -n 6 hgetall "BUFFER_PROFILE_TABLE|{}"'.format(profile))['stdout']
pytest_assert(not profile_info,
"Profile {} isn't removed from STATE_DB".format(profile))
profile_info = duthost.shell(
'redis-cli hgetall "BUFFER_PROFILE_TABLE:{}"'.format(profile))['stdout']
pytest_assert(not profile_info,
"Profile {} isn't removed from APPL_DB".format(profile))
logging.debug(
'Profile {} has been removed from STATE_DB and APPL_DB'.format(profile))
if sai_oid:
profile_info = duthost.shell(
'redis-cli -n 1 hgetall {}'.format(sai_oid))['stdout']
pytest_assert(
not profile_info, "Profile {} hasn't been removed from ASIC_DB".format(sai_oid))
def fetch_initial_asic_db(duthost):
profiles_in_asicdb = duthost.shell(
'redis-cli -n 1 keys "ASIC_STATE:SAI_OBJECT_TYPE_BUFFER_PROFILE*"')['stdout']
return set(profiles_in_asicdb.split('\n'))
def _compose_dict_from_cli(fields_list):
"""Convert the out put of hgetall command to a dict object containing the field,
key pairs of the database table content
Args:
fields_list: A list of lines, the output of redis-cli hgetall command
"""
return dict(list(zip(fields_list[0::2], fields_list[1::2])))
def check_buffer_profile_details(duthost, initial_profiles, profile_name, profile_oid, pool_oid, port_to_test):
"""Check buffer profile details.
The following items are tested:
- Whether the headroom information, like xoff, is correct.
For version 202106 and before, this is tested by comparing with standard profile in pg_profile_lookup table
For version after 202106, this is tested by comparing with the returned value
from function calculate_headroom_data
- Whether the profile information in APPL_DB matches that in ASIC_DB
Args:
initial_profiles: The keys of buffer profiles in ASIC_DB at the beginning of the test
profile_name: Name of the profile
profile_oid: SAI OID of the profile
pool_oid: SAI OID of ingress lossless pool
"""
profile_appldb = _compose_dict_from_cli(duthost.shell(
'redis-cli hgetall BUFFER_PROFILE_TABLE:{}'.format(profile_name))['stdout'].split('\n'))
logging.debug("APPL_DB buffer profile {}: {} ".format(
profile_name, profile_appldb))
# Check the profile against the standard value
m = re.search(LOSSLESS_PROFILE_PATTERN, profile_name)
if m:
# This means it's a dynamic profile
if check_qos_db_fv_reference_with_table(duthost) is True:
# SONiC version is 202106 and before, compare with standard profile in pg_profile_lookup table
speed = m.group(1)
cable_length = m.group(2)
std_profiles_for_speed = DEFAULT_LOSSLESS_HEADROOM_DATA.get(speed)
if std_profiles_for_speed:
std_profile = std_profiles_for_speed.get(cable_length)
if std_profile:
# This means it's a profile with std speed and cable length.
# We can check whether the headroom data is correct
pytest_assert(
profile_appldb['xon'] == std_profile['xon']
and profile_appldb['xoff'] == std_profile['xoff']
and (profile_appldb['size'] == std_profile['size']
or DEFAULT_SHARED_HEADROOM_POOL_ENABLED),
"Generated profile {} doesn't match the std profile {}".format(profile_appldb, std_profile))
else:
for std_cable_len, std_profile in list(std_profiles_for_speed.items()):
if int(std_cable_len[:-1]) > int(cable_length[:-1]):
pytest_assert(
int(std_profile['xoff']) >= int(profile_appldb['xoff']),
"XOFF of generated profile {} is greater than standard profile {} "
"while its cable length is less".format(profile_appldb, std_profile))
else:
pytest_assert(
int(std_profile['xoff']) <= int(profile_appldb['xoff']),
"XOFF of generated profile {} is less than standard profile {} "
"while its cable length is greater".format(profile_appldb, std_profile))
else:
logging.info(
"Skip headroom checking because headroom information is not provided for speed {}".format(speed))
else:
# SONiC version is after 202106, compare with the returned value from function calculate_headroom_data
ret, head_room_data = calculate_headroom_data(
duthost, port_to_test)
if ret:
# This means it's a profile with std speed and cable length.
# We can check whether the headroom data is correct
pytest_assert(
int(profile_appldb['xon']) == head_room_data['xon']
and int(profile_appldb['xoff']) == head_room_data['xoff']
and (int(profile_appldb['size']) == head_room_data['size']
or DEFAULT_SHARED_HEADROOM_POOL_ENABLED),
"Generated profile {} doesn't match the std profile {}".format(profile_appldb, head_room_data))
else:
logging.info(
"Skip headroom checking because headroom information is not able to be calculated for speed {}"
.format(speed))
profiles_in_asicdb = set(duthost.shell(
'redis-cli -n 1 keys "ASIC_STATE:SAI_OBJECT_TYPE_BUFFER_PROFILE*"')['stdout'].split('\n'))
diff = profiles_in_asicdb - initial_profiles
if len(diff) == 1:
profile_oid = diff.pop()
pytest_assert(profile_oid, "Unable to fetch SAI OID for profile {}, initial SAI OID set {} current set {}".format(
profile_name, initial_profiles, profiles_in_asicdb))
logging.debug("Initial profiles {} and current profiles {} have the following difference(s) {}".format(
initial_profiles, profiles_in_asicdb, diff))
profile_sai = _compose_dict_from_cli(duthost.shell(
'redis-cli -n 1 hgetall {}'.format(profile_oid))['stdout'].split('\n'))
logging.debug("SAI object for new profile {}: oid {} content {}".format(
profile_name, profile_oid, profile_sai))
if pool_oid is None:
pool_oid = profile_sai['SAI_BUFFER_PROFILE_ATTR_POOL_ID']
if profile_appldb.get('dynamic_th'):
sai_threshold_value = profile_appldb['dynamic_th']
sai_threshold_mode = 'SAI_BUFFER_PROFILE_THRESHOLD_MODE_DYNAMIC'
else:
sai_threshold_value = profile_appldb['static_th']
sai_threshold_mode = 'SAI_BUFFER_PROFILE_THRESHOLD_MODE_STATIC'
assert profile_sai == {'SAI_BUFFER_PROFILE_ATTR_XON_TH': profile_appldb['xon'],
'SAI_BUFFER_PROFILE_ATTR_XOFF_TH': profile_appldb['xoff'],
'SAI_BUFFER_PROFILE_ATTR_RESERVED_BUFFER_SIZE': profile_appldb['size'],
'SAI_BUFFER_PROFILE_ATTR_POOL_ID': pool_oid,
'SAI_BUFFER_PROFILE_ATTR_THRESHOLD_MODE': sai_threshold_mode,
'SAI_BUFFER_PROFILE_ATTR_SHARED_DYNAMIC_TH': sai_threshold_value}
return profile_oid, pool_oid
def make_expected_profile_name(speed, cable_length, **kwargs):
"""Make the name of an expected profile according to parameters
Args:
speed: The speed of the port (on which the profile is applied)
cable_length: The cable length of the port
other_factors: Other factors that affects profile name, like mtu, threshold, etc
It can be omitted.
Return:
The name of the profile
"""
expected_profile = 'pg_lossless_{}_{}_'.format(speed, cable_length)
other_factors = kwargs.get('other_factors')
if other_factors:
expected_profile += '_'.join(other_factors) + '_'
if ASIC_TYPE == 'mellanox':
number_of_lanes = kwargs.get('number_of_lanes')
if number_of_lanes is not None:
if number_of_lanes == 8 and speed != MAX_SPEED_8LANE_PORT:
expected_profile += '8lane_'
elif NUMBER_OF_LANES == 8 and speed != MAX_SPEED_8LANE_PORT:
expected_profile += '8lane_'
expected_profile += 'profile'
return expected_profile
@pytest.fixture(params=['50000', '10000'])
def speed_to_test(request):
"""Used to parametrized test cases for speeds
Args:
param request: The pytest request object
Return:
speed_to_test
"""
return request.param
@pytest.fixture(params=['15m', '40m'])
def cable_len_to_test(request):
"""Used to parametrized test cases for cable length
Args:
request: The pytest request object
Return:
cable_len_to_test
"""
return request.param
@pytest.fixture(params=['1500', '9100'])
def mtu_to_test(request):
"""Used to parametrized test cases for mtu
Args:
request: The pytest request object
Return:
cable_len_to_test
"""
return request.param
@pytest.fixture(scope="module", autouse=True)
def port_to_test(request, duthost):
"""Used to parametrized test cases for port
Args:
request: The pytest request object
Return:
port_to_test
"""
global PORT_TO_TEST
global NUMBER_OF_LANES
if PORT_TO_TEST:
return PORT_TO_TEST
dutLagInterfaces = []
mgFacts = duthost.minigraph_facts(host=duthost.hostname)['ansible_facts']
for _, lag in list(mgFacts["minigraph_portchannels"].items()):
dutLagInterfaces += lag["members"]
testPort = set(mgFacts["minigraph_ports"].keys())
lagMembers = set(dutLagInterfaces)
testPort -= lagMembers
pytest_require(len(testPort) > 0, "No port to run test")
PORT_TO_TEST = request.config.getoption("--port_to_test")
if PORT_TO_TEST in lagMembers:
logging.info(
"LAG member port {} can not be used for dynamic buffer test".format(PORT_TO_TEST))
PORT_TO_TEST = None
if not PORT_TO_TEST:
PORT_TO_TEST = list(testPort)[0]
lanes = duthost.shell(
'redis-cli -n 4 hget "PORT|{}" lanes'.format(PORT_TO_TEST))['stdout']
NUMBER_OF_LANES = len(lanes.split(','))
logging.info("Port to test {}, number of lanes {}".format(
PORT_TO_TEST, NUMBER_OF_LANES))
return PORT_TO_TEST
@pytest.fixture(params=['3-4', '6'])
def pg_to_test(request):
"""Used to parametrized test cases for PGs under test
Args:
request: The pytest request object
Return:
pg_to_test
"""
return request.param
def test_change_speed_cable(duthosts, rand_one_dut_hostname, conn_graph_facts, # noqa F811
port_to_test, speed_to_test, mtu_to_test, cable_len_to_test):
"""The testcase for changing the speed and cable length of a port
Change the variables of the port, including speed, mtu and cable length,
in different ways and observe whether the DUT behaves correctly
For any of the variable, if it matches the current port configuration, we will skip configuring it.
If all of the speed_to_test, mtu_to_test and cable_len_to_test match the current value, the test will be skipped
The flow of the test case:
1. Update the port configuration according to input parameters
2. Determine whether the profile removing behavior can be verified:
If neither mtu nor cable length is default value, they will be applied on the port_to_test only,
and the generated profile will be removed after the configuration change
because the profile is referenced by this port only.
For example:
The mtu_to_test 1500 only applied on the port_to_test,
thus the *_mtu1500_* profile is referenced by the port only
The *_mtu1500_* mtu will be removed after the mtu of the port is updated to default value.
In this case, we are able to verify whether the buffer profile is removed
after mtu reverted or all PGs are removed.
Other the other hand, if the mtu is 9100, the buffer profile can be referenced by many other ports
and it's less possible for us to verify the removing behavior.
We will remove and readd an extra PG 6 to verify the removing behavior as well.
3. Each time the port configuration updated, the following items will be checked as much as possible:
- Whether the new profile is generated in APPL_DB, STATE_DB and ASIC_DB.
- Whether the pool size is updated in APPL_DB and ASIC_DB.
4. Each time the PG on a port is added or removed, the following items will be checked:
- Whether the profile referenced by PGs is as expected according to the port configuration.
- Whether the profile is removed if all PGs are removed
and we are able to check removing behavior (result of step 2).
- Whether the pfc_enable filed of the port has been updated accordingly.
Args:
port_to_test: On which port will the test be performed
speed_to_test: To what speed will the port's be changed
mtu_to_test: To what mtu will the port's be changed
cable_len_to_test: To what cable length will the port's be changed
"""
skip_traditional_model()