Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make run-all-tests fails (sending with wrong VIP and verifying drop FAILS) #205

Closed
mhanif opened this issue Aug 30, 2022 · 17 comments · Fixed by #211
Closed

make run-all-tests fails (sending with wrong VIP and verifying drop FAILS) #205

mhanif opened this issue Aug 30, 2022 · 17 comments · Fixed by #211
Assignees

Comments

@mhanif
Copy link
Collaborator

mhanif commented Aug 30, 2022

Just cloned a fresh copy of DASH at the tip of master branch. Built and tested (unmodified) to get the baseline and saw that it is failing:

$ make run-saithrift-server
docker run --rm -it --net=host --name dash-saithrift-server-mhanif -v /home/mhanif/sw/dash-ipv6/DASH/dash-pipeline/bmv2/dash_pipeline.bmv2/dash_pipeline.json:/etc/dash/dash_pipeline.json -v /home/mhanif/sw/dash-ipv6/DASH/dash-pipeline/bmv2/dash_pipeline.bmv2/dash_pipeline_p4rt.txt:/etc/dash/dash_pipeline_p4rt.txt -v /home/mhanif/sw/dash-ipv6/DASH/dash-pipeline/SAI:/SAI -v /home/mhanif/sw/dash-ipv6/DASH/dash-pipeline/SAI/SAI/meta:/meta -w /SAI/rpc/usr/sbin -e LD_LIBRARY_PATH=/SAI/lib:/usr/local/lib chrissommers/dash-saithrift-bldr:220719
./saiserver
GRPC call SetForwardingPipelineConfig 0.0.0.0:9559 => /etc/dash/dash_pipeline.json, /etc/dash/dash_pipeline_p4rt.txt
Starting SAI RPC server on port 9092
GRPC call Write::INSERT OKtable_id: 45621839 match { field_id: 1 exact { value: "\254\020\001d" } } action { action { action_id: 26041632 } }
GRPC call Write::INSERT OKtable_id: 41103078 match { field_id: 1 exact { value: "\000\000d" } } action { action { action_id: 21912829 } }
GRPC call Write::INSERT OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\001" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::INSERT OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\002" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::INSERT OKtable_id: 34579306 match { field_id: 1 exact { value: "\000\003" } } action { action { action_id: 17187022 params { param_id: 1 value: "\000\000d" } } }
GRPC call Write::INSERT OKtable_id: 37987821 match { field_id: 1 exact { value: "\000\004" } } action { action { action_id: 27167550 params { param_id: 1 value: "\000\000'\020" } params { param_id: 2 value: "\000\001\206\240" } params { param_id: 3 value: "\000\001\206\240" } params { param_id: 4 value: "\001" } params { param_id: 5 value: "\254\020\001\001" } params { param_id: 6 value: "\000\000\t" } params { param_id: 7 value: "\000\003" } params { param_id: 8 value: "\000\000" } params { param_id: 9 value: "\000\000" } params { param_id: 10 value: "\000\000" } params { param_id: 11 value: "\000\000" } params { param_id: 12 value: "\000\000" } params { param_id: 13 value: "\000\000" } params { param_id: 14 value: "\000\000" } params { param_id: 15 value: "\000\000" } params { param_id: 16 value: "\000\000" } params { param_id: 17 value: "\000\000" } params { param_id: 18 value: "\000\000" } params { param_id: 19 value: "\000\000" } params { param_id: 20 value: "\000\000" } params { param_id: 21 value: "\000\000" } params { param_id: 22 value: "\000\000" } params { param_id: 23 value: "\000\000" } params { param_id: 24 value: "\000\000" } params { param_id: 25 value: "\000\000" } params { param_id: 26 value: "\000\000" } params { param_id: 27 value: "\000\000" } } }
GRPC call Write::INSERT OKtable_id: 37820776 match { field_id: 1 exact { value: "\000\314\314\314\314\314" } } action { action { action_id: 18228884 params { param_id: 1 value: "\000\004" } } }
GRPC call Write::INSERT OKtable_id: 37768533 match { field_id: 1 exact { value: "\000\004" } } match { field_id: 3 lpm { value: "\n\001\000\000" prefix_len: 112 } } match { field_id: 2 exact { value: "\000" } } action { action { action_id: 25364446 params { param_id: 1 value: "\000\003" } } }
GRPC call Write::INSERT OKtable_id: 34260339 match { field_id: 1 exact { value: "\000\003" } } match { field_id: 3 exact { value: "\n\001\0022" } } match { field_id: 2 exact { value: "\000" } } action { action { action_id: 25450412 params { param_id: 1 value: "\254\020\001\024" } params { param_id: 2 value: "\000\335\335\335\335\335" } params { param_id: 3 value: "\001" } } }
GRPC call Write::DELETE OKtable_id: 34260339 match { field_id: 1 exact { value: "\000\003" } } match { field_id: 3 exact { value: "\n\001\0022" } } match { field_id: 2 exact { value: "\000" } }
GRPC call Write::DELETE OKtable_id: 37768533 match { field_id: 1 exact { value: "\000\004" } } match { field_id: 3 lpm { value: "\n\001\000\000" prefix_len: 112 } } match { field_id: 2 exact { value: "\000" } }
GRPC call Write::DELETE OKtable_id: 37820776 match { field_id: 1 exact { value: "\000\314\314\314\314\314" } }
GRPC call Write::DELETE OKtable_id: 37987821 match { field_id: 1 exact { value: "\000\004" } } action { action { action_id: 27167550 params { param_id: 1 value: "\000\000'\020" } params { param_id: 2 value: "\000\001\206\240" } params { param_id: 3 value: "\000\001\206\240" } params { param_id: 4 value: "\001" } params { param_id: 5 value: "\254\020\001\001" } params { param_id: 6 value: "\000\000\t" } params { param_id: 7 value: "\000\003" } params { param_id: 8 value: "\000\000" } params { param_id: 9 value: "\000\000" } params { param_id: 10 value: "\000\000" } params { param_id: 11 value: "\000\000" } params { param_id: 12 value: "\000\000" } params { param_id: 13 value: "\000\000" } params { param_id: 14 value: "\000\000" } params { param_id: 15 value: "\000\000" } params { param_id: 16 value: "\000\000" } params { param_id: 17 value: "\000\000" } params { param_id: 18 value: "\000\000" } params { param_id: 19 value: "\000\000" } params { param_id: 20 value: "\000\000" } params { param_id: 21 value: "\000\000" } params { param_id: 22 value: "\000\000" } params { param_id: 23 value: "\000\000" } params { param_id: 24 value: "\000\000" } params { param_id: 25 value: "\000\000" } params { param_id: 26 value: "\000\000" } params { param_id: 27 value: "\000\000" } } }
GRPC call Write::DELETE OKtable_id: 34579306 match { field_id: 1 exact { value: "\000\003" } } action { action { action_id: 17187022 params { param_id: 1 value: "\000\000d" } } }
GRPC call Write::DELETE OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\002" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::DELETE OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\001" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::DELETE OKtable_id: 41103078 match { field_id: 1 exact { value: "\000\000d" } }
GRPC call Write::DELETE OKtable_id: 45621839 match { field_id: 1 exact { value: "\254\020\001d" } }


$ make run-all-tests

Ensure P4Runtime server is listening

t=5;
while [ ${t} -ge 1 ]; do
if sudo lsof -i:9559 | grep LISTEN >/dev/null; then
break;
else
sleep 1;
t=expr $t - 1;
fi;
done;
docker exec -w /tests/libsai/vnet_out simple_switch-mhanif ./vnet_out
[sudo] password for mhanif:
GRPC call SetForwardingPipelineConfig 0.0.0.0:9559 => /etc/dash/dash_pipeline.json, /etc/dash/dash_pipeline_p4rt.txt
GRPC call Write::INSERT OKtable_id: 41103078 match { field_id: 1 exact { value: "\000\000<" } } action { action { action_id: 21912829 } }
GRPC call Write::INSERT OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\001" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::INSERT OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\002" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::INSERT OKtable_id: 34579306 match { field_id: 1 exact { value: "\000\003" } } action { action { action_id: 17187022 params { param_id: 1 value: "\000\000\t" } } }
GRPC call Write::INSERT OKtable_id: 37987821 match { field_id: 1 exact { value: "\000\004" } } action { action { action_id: 27167550 params { param_id: 1 value: "\000\000'\020" } params { param_id: 2 value: "\000\001\206\240" } params { param_id: 3 value: "\000\001\206\240" } params { param_id: 4 value: "\001" } params { param_id: 5 value: "\254\020\003\001" } params { param_id: 6 value: "\000\000\t" } params { param_id: 7 value: "\000\003" } params { param_id: 27 value: "\000\000" } params { param_id: 26 value: "\000\000" } params { param_id: 25 value: "\000\000" } params { param_id: 24 value: "\000\000" } params { param_id: 23 value: "\000\000" } params { param_id: 17 value: "\000\000" } params { param_id: 16 value: "\000\000" } params { param_id: 15 value: "\000\000" } params { param_id: 14 value: "\000\000" } params { param_id: 13 value: "\000\000" } params { param_id: 22 value: "\000\002" } params { param_id: 21 value: "\000\002" } params { param_id: 20 value: "\000\002" } params { param_id: 19 value: "\000\002" } params { param_id: 18 value: "\000\002" } params { param_id: 12 value: "\000\001" } params { param_id: 11 value: "\000\001" } params { param_id: 10 value: "\000\001" } params { param_id: 9 value: "\000\001" } params { param_id: 8 value: "\000\001" } } }
GRPC call Write::INSERT OKtable_id: 37820776 match { field_id: 1 exact { value: "\252\314\314\314\314\314" } } action { action { action_id: 18228884 params { param_id: 1 value: "\000\004" } } }
GRPC call Write::DELETE OKtable_id: 37820776 match { field_id: 1 exact { value: "\252\314\314\314\314\314" } }
GRPC call Write::DELETE OKtable_id: 37987821 match { field_id: 1 exact { value: "\000\004" } } action { action { action_id: 27167550 params { param_id: 1 value: "\000\000'\020" } params { param_id: 2 value: "\000\001\206\240" } params { param_id: 3 value: "\000\001\206\240" } params { param_id: 4 value: "\001" } params { param_id: 5 value: "\254\020\003\001" } params { param_id: 6 value: "\000\000\t" } params { param_id: 7 value: "\000\003" } params { param_id: 27 value: "\000\000" } params { param_id: 26 value: "\000\000" } params { param_id: 25 value: "\000\000" } params { param_id: 24 value: "\000\000" } params { param_id: 23 value: "\000\000" } params { param_id: 17 value: "\000\000" } params { param_id: 16 value: "\000\000" } params { param_id: 15 value: "\000\000" } params { param_id: 14 value: "\000\000" } params { param_id: 13 value: "\000\000" } params { param_id: 22 value: "\000\002" } params { param_id: 21 value: "\000\002" } params { param_id: 20 value: "\000\002" } params { param_id: 19 value: "\000\002" } params { param_id: 18 value: "\000\002" } params { param_id: 12 value: "\000\001" } params { param_id: 11 value: "\000\001" } params { param_id: 10 value: "\000\001" } params { param_id: 9 value: "\000\001" } params { param_id: 8 value: "\000\001" } } }
GRPC call Write::DELETE OKtable_id: 34579306 match { field_id: 1 exact { value: "\000\003" } } action { action { action_id: 17187022 params { param_id: 1 value: "\000\000\t" } } }
Done.
GRPC call Write::DELETE OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\002" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::DELETE OKtable_id: 45323240 match { field_id: 1 exact { value: "\000\001" } } action { action { action_id: 25655048 params { param_id: 1 value: "\000\000\000\000" } } }
GRPC call Write::DELETE OKtable_id: 41103078 match { field_id: 1 exact { value: "\000\000<" } }
cd ../test/third-party/traffic_gen && ./deploy_ixiac.sh
.
Creating ixia-c-traffic-engine1-mhanif ... done
Creating ixia-c-traffic-engine2-mhanif ... done
Creating ixia-c-controller-mhanif ... done
docker run -u root --network=host --rm -w /tests/ -it
-w /tests/saithrift/ptf
local/dash-saithrift-client:latest
./run-saithrift-ptftests.sh
/usr/local/lib/python3.8/dist-packages/ptf-0.9.3-py3.8.egg/EGG-INFO/scripts/ptf:19: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import imp
Using packet manipulation module: ptf.packet_scapy
test_thrift_session.TestSaiThriftSession ... TestSaiThriftSession OK
ok


Ran 1 test in 0.002s

OK
test_thrift_session.TestSaiThriftSaiHelper ... TestSaiThriftSaiHelper OK
ok


Ran 1 test in 0.001s

OK
test_saithrift_vnet.TestSaiThrift_outbound_udp_pkt ...

Sending packet with wrong vip...

<Ether dst=00:00:02:03:04:05 src=00:00:05:06:06:06 type=IPv4 |<IP ihl=None tos=0x0 id=1 flags= frag=0 ttl=64 proto=udp src=172.16.1.1 dst=172.16.100.100 |<UDP sport=11638 dport=4789 chksum=0x0 |<VXLAN flags=Instance reserved1=0 vni=0x64 reserved2=0x0 |<Ether dst=02:02:02:02:02:02 src=00:cc:cc:cc:cc:cc type=IPv4 |<IP ihl=None tos=0x0 id=1 flags= frag=0 ttl=64 proto=udp src=10.1.1.10 dst=10.1.2.50 |<UDP sport=1234 dport=80 |<Raw load='\x00\x01\x02\x03\x04\x05\x06\x07\x08\t\n\x0b\x0c\r\x0e\x0f\x10\x11\x12\x13\x14\x15\x16\x17\x18\x19\x1a\x1b\x1c\x1d\x1e\x1f !"#$%&'()*+,-./0123456789' |>>>>>>>>

Verifying drop...
FAIL

======================================================================
FAIL: test_saithrift_vnet.TestSaiThrift_outbound_udp_pkt

Traceback (most recent call last):
File "./vnet/test_saithrift_vnet.py", line 225, in runTest
raise ae
File "./vnet/test_saithrift_vnet.py", line 146, in runTest
verify_no_other_packets(self)
File "/usr/local/lib/python3.8/dist-packages/ptf-0.9.3-py3.8.egg/ptf/testutils.py", line 3291, in verify_no_other_packets
test.fail(
AssertionError: A packet was received on device 0, port 0, but we expected no packets.
========== RECEIVED ==========
0000 33 33 00 00 00 16 7A EF 5F 17 69 A9 86 DD 60 00 33....z._.i...`.
0010 00 00 00 4C 00 01 FE 80 00 00 00 00 00 00 78 EF ...L..........x.
0020 5F FF FE 17 69 A9 FF 02 00 00 00 00 00 00 00 00 _...i...........
0030 00 00 00 00 00 16 3A 00 05 02 00 00 01 00 8F 00 ......:.........
0040 C1 64 00 00 00 03 04 00 00 00 FF 02 00 00 00 00 .d..............
0050 00 00 00 00 00 01 FF 00 00 00 04 00 00 00 FF 02 ................
0060 00 00 00 00 00 00 00 00 00 01 FF 17 69 A9 04 00 ............i...
0070 00 00 FF 02 00 00 00 00 00 00 00 00 00 00 00 00 ................
0080 00 02 ..


Ran 1 test in 0.138s

FAILED (failures=1)


ATTENTION: SOME TESTS DID NOT PASS!!!

The following tests failed:
TestSaiThrift_outbound_udp_pkt


make: *** [Makefile:374: run-saithrift-ptftests] Error 1


$ docker ps
CONTAINER ID IMAGE COMMAND CREATED STATUS PORTS NAMES
35725a3a4693 ixiacom/ixia-c-traffic-engine:1.4.1.26 "./entrypoint.sh" 4 minutes ago Up 4 minutes ixia-c-traffic-engine2-mhanif
584cfe0175de ixiacom/ixia-c-controller:0.0.1-2934 "./bin/controller --…" 4 minutes ago Up 4 minutes ixia-c-controller-mhanif
478016d7cb2f ixiacom/ixia-c-traffic-engine:1.4.1.26 "./entrypoint.sh" 4 minutes ago Up 4 minutes ixia-c-traffic-engine1-mhanif
bbf3233c21f5 chrissommers/dash-saithrift-bldr:220719 "./saiserver" 8 minutes ago Up 8 minutes dash-saithrift-server-mhanif
509dc995d6ed chrissommers/dash-bmv2-bldr:220630 "env LD_LIBRARY_PATH…" About an hour ago Up About an hour simple_switch-mhanif

@chrispsommers
Copy link
Collaborator

@mhanif Thanks for reporting. I saw something similar while I was working the other day, but it somehow resolved and I can't say exactly how, but it might have to do with version of PTF and p4c. I was developing/testing #202. You might want to clone my dev fork https://github.com/chrispsommers/, checkout branch fix-docker-perms and give that a try.

The curious thing is this version appears to run CI tests fine, so it is possible the failure results from some previous state. Did you make docker-saithrift-client ?

@mhanif
Copy link
Collaborator Author

mhanif commented Aug 30, 2022

The curious thing is this version appears to run CI tests fine, so it is possible the failure results from some previous state. Did you make docker-saithrift-client ?

@chrispsommers I did "make all" which does do " make docker-saithrift-client". I will try with your "fix-docker-perms" branch and see if I have any luck :-) Thanks!

@mukeshmv
Copy link
Collaborator

mukeshmv commented Aug 31, 2022

This looks like a stray MLD report ICMPv6 packet. So we may need to guard against that - this is something Chris had commented earlier when the drop packet test was introduced.

Internet Protocol Version 6
    0110 .... = Version: 6
    .... 0000 0000 .... .... .... .... .... = Traffic Class: 0x00 (DSCP: CS0, ECN: Not-ECT)
    .... 0000 0000 0000 0000 0000 = Flow Label: 0x00000
    Payload Length: 76
    Next Header: IPv6 Hop-by-Hop Option (0)
    Hop Limit: 1
    Source Address: fe80::78ef:5fff:fe17:69a9
    Destination Address: ff02::16
    IPv6 Hop-by-Hop Option
Internet Control Message Protocol v6
    Type: Multicast Listener Report Message v2 (143)
    Code: 0
    Checksum: 0xc164 [correct]
    Checksum Status: Good
    Reserved: 0000
    Number of Multicast Address Records: 3
    Multicast Address Record Changed to exclude: ff02::1:ff00:0
    Multicast Address Record Changed to exclude: ff02::1:ff17:69a9
    Multicast Address Record Changed to exclude: ff02::2

@mukeshmv
Copy link
Collaborator

mukeshmv commented Aug 31, 2022

Our Makefile has these lines to disable IPv6

network:
  sudo sysctl net.ipv6.conf.veth0.disable_ipv6=1
  sudo sysctl net.ipv6.conf.veth1.disable_ipv6=1
  sudo sysctl net.ipv6.conf.veth2.disable_ipv6=1
  sudo sysctl net.ipv6.conf.veth3.disable_ipv6=1

And when I check in my system I see that ipv6 is disabled on these interfaces.

mukesh@mukesh$ sysctl net.ipv6.conf.veth0.disable_ipv6
net.ipv6.conf.veth0.disable_ipv6 = 1
mukesh@mukesh$ sysctl net.ipv6.conf.veth1.disable_ipv6
net.ipv6.conf.veth1.disable_ipv6 = 1
mukesh@mukesh$ sysctl net.ipv6.conf.veth2.disable_ipv6
net.ipv6.conf.veth2.disable_ipv6 = 1
mukesh@mukesh$ sysctl net.ipv6.conf.veth3.disable_ipv6
net.ipv6.conf.veth3.disable_ipv6 = 1

@mhanif can you check this on your side. With this I believe we should not be seeing IPv6 packets.

@chrispsommers
Copy link
Collaborator

@mukeshmv thanks for the insight, I missed that. It might explain the sporadic nature of this problem. Despite all my efforts to prevent IPv6 contamination of the test-cases (all those sysctl commands I added to the initial ones provided by @marian-pritsak), it still seems to creep in - you can see in Wireshark monitoring veth's. I've previously spent hours on this and didn't get a perfect solution. Maybe a few more magic sysclt commands would solve it. Perhaps the use of container and device namespaces would help. Any advice is welcome.

@mukeshmv
Copy link
Collaborator

mukeshmv commented Sep 1, 2022

@mhanif if the disable_ipv6 sysctl parameters are already set in your case and you are still seeing the issue can you try setting these MLD params to see if it helps

sudo sysctl -w net.ipv6.conf.veth0.mldv2_unsolicited_report_interval=0
sudo sysctl -w net.ipv6.conf.veth1.mldv2_unsolicited_report_interval=0
sudo sysctl -w net.ipv6.conf.veth2.mldv2_unsolicited_report_interval=0
sudo sysctl -w net.ipv6.conf.veth3.mldv2_unsolicited_report_interval=0

Or if 0 is not switching it off then alternatively you could try increasing the interval to a very high value (set in millisecs).

By the way, I don't see any IPv6 packet in my envrironment on these 4 interfaces even before making any changes. I tried capturing using tcpdump before, after and while I was running the tests.

@chrispsommers
Copy link
Collaborator

I have some more empirical evidence. In my environment, I see numerous IPv6 packets appear on the veths after executing the following sequence. You don't need to make run-switch or make run-saithrift-server.

  • make network # console 1 - creates veths
  • sudo tcpdump -i veth0 # console 2
  • make deploy-ixiac # console 1
    dash-pkt-capture.txt

You can see IPv6 packets appear on the veths over a period of a few seconds. See attachment for example. This seems to happen only on first invocation of make network and make deploy-ixiac and test failure seems to be environment-dependent.

I will investigate further why this happens. In the meantime, I think the remedy could be to delete the deploy-ixiac prerequisite from the following make target located here: https://github.com/Azure/DASH/blob/191bb85bf07c7746ef81113e590cdb04cb7e12e8/dash-pipeline/Makefile#L6

The apparent impact of this prerequisite is to produce these IPv6 packets during the execution of the subsequent run-saithrift-ptftests (one of the components of run-saithrift-client-tests) which breaks the test because "expected packet" ends up being a random IPv6 packet. Removing the prerequisite won't interfere with PTF tests because ixia-c isn't used there. (In fact there are no tests currently enabled using ixia-c.) In any case, deploy-ixia-c is also a prerequisite for run-saithrift-pytests so it's OK to remove from run-all-tests.

Sorry for the long explanation. In summary:

  • @mhanif could you please try removing deploy-ixiac from run-all-tests and retry the usual make run-switch, make run-saithrift-server, make run-all-tests? Please execute make network-clean undeploy-ixiac first to get to the initial conditions.
  • Assuming this works, I can do a PR to remedy this in the near-term
  • I will pursue the root cause of the IPv6 packets emitted during deploy-ixiac

@mukeshmv
Copy link
Collaborator

mukeshmv commented Sep 1, 2022

Good find @chrispsommers. I was only doing run-saithrift-ptftests after make all .. thats why I didn't see it.

And I also noticed that IPv6 is getting enabled back after make deploy-ixiac.

sysctl net.ipv6.conf.veth0.disable_ipv6
net.ipv6.conf.veth0.disable_ipv6 = 0

So I guess we don't need to dig further into sysctls. Just keeping the current one disabled should work.

@mhanif
Copy link
Collaborator Author

mhanif commented Sep 1, 2022

@chrispsommers @mukeshmv Thanks a lot for all your help and suggestions. After todays meeting, I re-ran the command "make run-all-tests" and it passed with the message - "3 passed, 1 skipped in 0.76s". Not sure if it is supposed to skip one test? In any case, I didn't make any changes to my environment or issued any sysctl command. I will re-run this a few times to see if there are intermittent failures. In case of any failures, I will try your suggestions. Thanks again.

@chrispsommers
Copy link
Collaborator

@mhanif Thanks and you're welcome!

Indeed, one test was commented out awhile back so it is "skipped." It's here:
https://github.com/Azure/DASH/blob/66671be26f56c431208241046ce96d22d022e666/dash-pipeline/tests/saithrift/pytest/echo/test_echo_port.py#L6

@mukesh, can this be reenabled yet? I forget the rationale here.

@mhanif
Copy link
Collaborator Author

mhanif commented Sep 1, 2022

@chrispsommers After I re-ran the tests few more time and I occasionally (more than once) see that the test stops right away with the following message. Haven't debug to see why

$ make run-all-tests

Ensure P4Runtime server is listening

t=5;
while [ ${t} -ge 1 ]; do
if sudo lsof -i:9559 | grep LISTEN >/dev/null; then
break;
else
sleep 1;
t=expr $t - 1;
fi;
done;
docker exec -w /tests/libsai/vnet_out simple_switch-mhanif ./vnet_out
GRPC call SetForwardingPipelineConfig 0.0.0.0:9559 => /etc/dash/dash_pipeline.json, /etc/dash/dash_pipeline_p4rt.txt
GRPC ERROR[7]: Not primary, GRPC call Write::INSERT ERROR:
table_id: 41103078 match { field_id: 1 exact { value: "\000\000<" } } action { action { action_id: 21912829 } }Failed to create Direction Lookup Entry
make: *** [Makefile:253: run-libsai-test] Error 1

@chrispsommers
Copy link
Collaborator

Hi @mhanif It seems like the P4Runtime server inside bmv2 is getting a client connection request when another P4Runtime client already has "mastership." The clue is: Not primary. Check to see which containers are running via docker ps in case something is leftover.

@mukeshmv
Copy link
Collaborator

mukeshmv commented Sep 1, 2022

@mukesh, can this be reenabled yet? I forget the rationale here.

@chrispsommers no dash pipeline does not support echo of non-vxlan packet. If I remember right we concluded that you will go ahead and replicate the PTF outbound packet test as a Pytest. That skip reason given is outdated, I should have changed it.

@chrispsommers
Copy link
Collaborator

OK, thanks for clarifying. Thanks for reminding my to create a PyTest!

@chrispsommers chrispsommers self-assigned this Sep 1, 2022
@mhanif
Copy link
Collaborator Author

mhanif commented Sep 1, 2022

Hi @mhanif It seems like the P4Runtime server inside bmv2 is getting a client connection request when another P4Runtime client already has "mastership." The clue is: Not primary. Check to see which containers are running via docker ps in case something is leftover.

Hi @chrispsommers - it seems like that there must be a delay of 10+ seconds when I rerun the same test using "make run-all-tests". I was just doing the "up-arrow" and recalling the command on the Linux prompt. If I wait a little bit and rerun the test, this issue doesn't occur. Seems like the previous incarnation of the P4RT server takes a little bit of time before it cleans up and die. Thanks.

@chrispsommers
Copy link
Collaborator

@mhanif you could be right, thanks for the details.

@chrispsommers
Copy link
Collaborator

@mukeshmv @mhanif If you get a chance please try out #211, thanks.

KrisNey-MSFT pushed a commit that referenced this issue Sep 11, 2022
* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>
@mhanif mhanif removed their assignment Sep 12, 2022
chrispsommers added a commit that referenced this issue Sep 12, 2022
* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
chrispsommers added a commit that referenced this issue Sep 12, 2022
* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
mmiele added a commit to mmiele/DASH that referenced this issue Sep 18, 2022
commit 9700152
Merge: 32332b5 d003f4e
Author: Michael Miele <[email protected]>
Date:   Sat Sep 17 16:43:45 2022 -0700

    Merge branch 'Azure:main' into main

commit d003f4e
Author: Chris Sommers <[email protected]>
Date:   Fri Sep 16 22:43:03 2022 -0700

    Upgrade to ixia-c 0.0.1-3383 (sonic-net#226)

    * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch.
    Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.)

    * Update ixia-c traffic engine to match controller.

    * Update snappi version to match ixia-c-components.

    Co-authored-by: Chris Sommers <[email protected]>

commit 543df5c
Author: Ze Gan <[email protected]>
Date:   Fri Sep 16 01:36:01 2022 +0800

    [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223)

    According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17.

commit 94b0168
Author: Chris Sommers <[email protected]>
Date:   Wed Sep 14 10:27:22 2022 -0700

    Slide deck for today's SAI Challenger presentation

commit 10056d7
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:33:36 2022 -0700

    Update .wordlist.txt

commit f1c5245
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:31:16 2022 -0700

    Fix Spelling (sonic-net#221)

    * Update AMD-Pensando_HA_Proposal.md

    Explicitly add 'parallel' to state synchronization stages (per @lguohan )

    * Update AMD-Pensando_HA_Proposal.md (sonic-net#210)

    * SAI apigen support for tables with no action parameters and a single action (sonic-net#207)

    * Update .wordlist.txt

    * Update AMD-Pensando_HA_Proposal.md

    * Doc dash as submodule (sonic-net#203)

    * Document third-party workflows using DASH as a Git submodule.

    * Add URL to sample project.

    * Incorporate review feedback (typo; missing file).

    * Spellcheck fixes.

    * Spellcheck

    * Add .wordlist.txt to CI triggers.

    * Spellcheck wordlist.

    Co-authored-by: Chris Sommers <[email protected]>

    * Split SAI API (sonic-net#201)

    Make APIs compatible with SONiC bulk infra

    Signed-off-by: Marian Pritsak <[email protected]>

    * Add APP_DB to SAI mapping (sonic-net#102)

    * Add APP_DB to SAI mapping

    * Update .wordlist.txt

    Updating w/Chris

    Signed-off-by: Marian Pritsak <[email protected]>
    Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Marian Pritsak <[email protected]>

    Signed-off-by: Marian Pritsak <[email protected]>
    Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Marian Pritsak <[email protected]>

commit 8694184
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:25:46 2022 -0700

    Update program-scale-testing-requirements-draft.md

    Update Scale #'s per Keysight/MSFT call; document is a version which needs an update.

commit 2ce121e
Author: Chris Sommers <[email protected]>
Date:   Sun Sep 11 13:37:40 2022 -0700

    Fix ipv6 packet noise (sonic-net#211)

    * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205.

    * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

    * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

    Co-authored-by: Chris Sommers <[email protected]>

commit e31982c
Author: Chris Sommers <[email protected]>
Date:   Sun Sep 11 13:36:47 2022 -0700

    Fix docker & Makefile perms (sonic-net#202)

    * Incremental work on docker permissions fixes.

    * More progress on docker and make permissions issues.

    * Trial remove -u root from CI file.

    * Makefile and CI file perms

    * Fix CI docker flags

    * Final purge of obs sudo commands to fix perms.

    * Test for veth's before deleting.

    Co-authored-by: Chris Sommers <[email protected]>

commit 32332b5
Merge: 9f62724 6d1c50a
Author: Michael Miele <[email protected]>
Date:   Wed Sep 7 17:46:27 2022 -0700

    Merge branch 'Azure:main' into main

commit 9f62724
Merge: 9b51bb7 e5e461a
Author: Michael Miele <[email protected]>
Date:   Thu Aug 25 09:11:30 2022 -0700

    Merge branch 'Azure:main' into main

commit 9b51bb7
Merge: 54eca5a 3c15d95
Author: Michael Miele <[email protected]>
Date:   Mon Jul 18 18:40:31 2022 -0700

    Merge branch 'Azure:main' into main

commit 54eca5a
Merge: 30c65ab 7b5ce5e
Author: Michael Miele <[email protected]>
Date:   Tue Jun 28 12:10:45 2022 -0700

    Merge branch 'Azure:main' into main

commit 30c65ab
Merge: 48ae39c c9612ff
Author: Michael Miele <[email protected]>
Date:   Mon Jun 20 10:27:44 2022 -0700

    Merge branch 'Azure:main' into main

commit 48ae39c
Merge: e4b3b64 49d9f2c
Author: Michael Miele <[email protected]>
Date:   Fri Jun 17 10:53:03 2022 -0700

    Merge branch 'Azure:main' into main

commit e4b3b64
Merge: 4120634 231212b
Author: Michael Miele <[email protected]>
Date:   Tue Jun 14 13:39:40 2022 -0700

    Merge branch 'Azure:main' into main

commit 4120634
Merge: b5c1700 a9ec96a
Author: Michael Miele <[email protected]>
Date:   Mon Jun 13 09:41:12 2022 -0700

    Merge branch 'Azure:main' into main

commit b5c1700
Merge: f541d44 2e28aef
Author: Michael Miele <[email protected]>
Date:   Mon Jun 13 09:31:01 2022 -0700

    Merge branch 'Azure:main' into main

commit f541d44
Merge: 1927911 8433203
Author: Michael Miele <[email protected]>
Date:   Wed Jun 8 13:40:39 2022 -0700

    Merge branch 'Azure:main' into main

commit 1927911
Merge: 569c32a d0807c8
Author: Michael Miele <[email protected]>
Date:   Tue May 31 13:49:13 2022 -0700

    Merge branch 'Azure:main' into main

commit 569c32a
Merge: d956391 e420737
Author: Michael Miele <[email protected]>
Date:   Fri May 20 16:25:33 2022 -0700

    Merge branch 'Azure:main' into main

commit d956391
Author: KrisNey-MSFT <[email protected]>
Date:   Tue May 10 16:34:33 2022 -0700

    Create sdn-packet-transforms-vm-to-vm.svg

    Edited Fast Path and Slow Path images

commit 18f88a4
Merge: b2c18a7 221c401
Author: Michael Miele <[email protected]>
Date:   Mon May 9 08:48:29 2022 -0700

    Merge branch 'Azure:main' into main

commit b2c18a7
Author: Michael Miele <[email protected]>
Date:   Mon May 9 08:47:53 2022 -0700

    sync-with-azure-dash (#8)

    * dash-holistic-view (sonic-net#104)

    * draft document

    * Updated content and images

    * added image and updated content

    * fixed image

    * Update dash-architecture.md

    * Fixed image and caption

    * Update dash-architecture.md

    * added/reorganized images

    * Update dash-architecture.md

    * fixed image and content

    * Update dash-architecture.md

    * Update dash-hld-ecosystem.svg

    * Renamed file.

    * Update dash-hld-ecosystem.svg

    * Update dash-hld-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-hld-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * reorganized content

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Fixed images and content based on feedback

    * move images, added captions, fixed content

    * Update dash-hld-project-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md".

    * Update dash-architecture-view.md

    * fixed card opacity

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Fixed image

    * fixed image

    * fixed image and related content

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md (#3)

    Merged in the PR: #2.

    * Renamed article and added content.

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * renamed file

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * fixed formating

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Fixed images

    * Fixed TOC

    * Update documentation/general/design/dash-holistic-view.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update documentation/general/design/dash-holistic-view.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Apply suggestions from code review

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Apply suggestions from code review

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Updated content and added note

    * Fixed content

    * fixed metadata

    * Created .gitignore and removed bkp files

    * Fixed content based on feedback.

    * Fixed content based on feedback. Added links and special notes.  Fixed image.

    * Fixed diagrams: using gNMI container instead of DASH container.

    * Fixed title and typos

    * Added reference links

    * Update documentation/general/design/dash-holistic-view.md

    Change committed.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-high-level-smart-switch.svg

    Changed "DASH container" name to "gNMI container".

    * Fixed image and moved `.gitignore` to the main folder.

    * Apply suggestions from code review

    Committed Kristina's suggestions.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-holistic-view.md

    Renamed the section API to "SAI headers and behavioral model". Added links to the related README files.

    * Removed old dash-high-level-design.md

    deleted dash-high-level-design.md
    renamed dash-holistic-view.md to dash-high-level-design.md

    * Added the list of words to generate the "word cloud".

    * Update README.md

    Reversed the link order and fixed descriptions.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-high-level-design.md

    * Update dash-high-level-design.md

    * Update dash-hld-project-ecosystem.svg

    * Update dash-high-level-design.md

    Co-authored-by: KrisNey-MSFT <[email protected]>
KrisNey-MSFT added a commit that referenced this issue Sep 20, 2022
* sync-with-azure-dash (#8)

* dash-holistic-view (#104)

* draft document

* Updated content and images

* added image and updated content

* fixed image

* Update dash-architecture.md

* Fixed image and caption

* Update dash-architecture.md

* added/reorganized images

* Update dash-architecture.md

* fixed image and content

* Update dash-architecture.md

* Update dash-hld-ecosystem.svg

* Renamed file.

* Update dash-hld-ecosystem.svg

* Update dash-hld-ecosystem.svg

* Update dash-architecture-view.md

* Update dash-hld-ecosystem.svg

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* reorganized content

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Fixed images and content based on feedback

* move images, added captions, fixed content

* Update dash-hld-project-ecosystem.svg

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md".

* Update dash-architecture-view.md

* fixed card opacity

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Fixed image

* fixed image

* fixed image and related content

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md (#3)

Merged in the PR: mmiele#2.

* Renamed article and added content.

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* renamed file

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* fixed formating

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Fixed images

* Fixed TOC

* Update documentation/general/design/dash-holistic-view.md

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update documentation/general/design/dash-holistic-view.md

Co-authored-by: KrisNey-MSFT <[email protected]>

* Apply suggestions from code review

Co-authored-by: KrisNey-MSFT <[email protected]>

* Apply suggestions from code review

Co-authored-by: KrisNey-MSFT <[email protected]>

* Updated content and added note

* Fixed content

* fixed metadata

* Created .gitignore and removed bkp files

* Fixed content based on feedback.

* Fixed content based on feedback. Added links and special notes.  Fixed image.

* Fixed diagrams: using gNMI container instead of DASH container.

* Fixed title and typos

* Added reference links

* Update documentation/general/design/dash-holistic-view.md

Change committed.

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update dash-high-level-smart-switch.svg

Changed "DASH container" name to "gNMI container".

* Fixed image and moved `.gitignore` to the main folder.

* Apply suggestions from code review

Committed Kristina's suggestions.

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update dash-holistic-view.md

Renamed the section API to "SAI headers and behavioral model". Added links to the related README files.

* Removed old dash-high-level-design.md

deleted dash-high-level-design.md
renamed dash-holistic-view.md to dash-high-level-design.md

* Added the list of words to generate the "word cloud".

* Update README.md

Reversed the link order and fixed descriptions.

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update dash-high-level-design.md

* Update dash-high-level-design.md

* Update dash-hld-project-ecosystem.svg

* Update dash-high-level-design.md

Co-authored-by: KrisNey-MSFT <[email protected]>

* Create sdn-packet-transforms-vm-to-vm.svg

Edited Fast Path and Slow Path images

* Reorganized dataplane folder.

* Fixed links

* Reorganized gnmi folder.

* Reorganized encrypt-gw-service and express-route-service folders

* Reorganized general folder

* Fixed link

* Reorganized high-avail folder.

* Reorganized load-bal-service folder

* Reorganized sai folder.

* Reorganized stpl-service folder.

* Reorganized vnet-peering-service folder.

* Reorganized vnet-to-vnet-service folder.

* Fixed reference links.

* Fixed links to point to the reorganized folders.

* Update vnet-to-vnet-service.md

Updated metadata

* Squashed commit of the following:

commit 9700152
Merge: 32332b5 d003f4e
Author: Michael Miele <[email protected]>
Date:   Sat Sep 17 16:43:45 2022 -0700

    Merge branch 'Azure:main' into main

commit d003f4e
Author: Chris Sommers <[email protected]>
Date:   Fri Sep 16 22:43:03 2022 -0700

    Upgrade to ixia-c 0.0.1-3383 (#226)

    * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch.
    Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.)

    * Update ixia-c traffic engine to match controller.

    * Update snappi version to match ixia-c-components.

    Co-authored-by: Chris Sommers <[email protected]>

commit 543df5c
Author: Ze Gan <[email protected]>
Date:   Fri Sep 16 01:36:01 2022 +0800

    [doc]: Fix the typo of TCP UDP protocol number (#223)

    According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17.

commit 94b0168
Author: Chris Sommers <[email protected]>
Date:   Wed Sep 14 10:27:22 2022 -0700

    Slide deck for today's SAI Challenger presentation

commit 10056d7
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:33:36 2022 -0700

    Update .wordlist.txt

commit f1c5245
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:31:16 2022 -0700

    Fix Spelling (#221)

    * Update AMD-Pensando_HA_Proposal.md

    Explicitly add 'parallel' to state synchronization stages (per @lguohan )

    * Update AMD-Pensando_HA_Proposal.md (#210)

    * SAI apigen support for tables with no action parameters and a single action (#207)

    * Update .wordlist.txt

    * Update AMD-Pensando_HA_Proposal.md

    * Doc dash as submodule (#203)

    * Document third-party workflows using DASH as a Git submodule.

    * Add URL to sample project.

    * Incorporate review feedback (typo; missing file).

    * Spellcheck fixes.

    * Spellcheck

    * Add .wordlist.txt to CI triggers.

    * Spellcheck wordlist.

    Co-authored-by: Chris Sommers <[email protected]>

    * Split SAI API (#201)

    Make APIs compatible with SONiC bulk infra

    Signed-off-by: Marian Pritsak <[email protected]>

    * Add APP_DB to SAI mapping (#102)

    * Add APP_DB to SAI mapping

    * Update .wordlist.txt

    Updating w/Chris

    Signed-off-by: Marian Pritsak <[email protected]>
    Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Marian Pritsak <[email protected]>

    Signed-off-by: Marian Pritsak <[email protected]>
    Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Marian Pritsak <[email protected]>

commit 8694184
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:25:46 2022 -0700

    Update program-scale-testing-requirements-draft.md

    Update Scale #'s per Keysight/MSFT call; document is a version which needs an update.

commit 2ce121e
Author: Chris Sommers <[email protected]>
Date:   Sun Sep 11 13:37:40 2022 -0700

    Fix ipv6 packet noise (#211)

    * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

    * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

    * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

    Co-authored-by: Chris Sommers <[email protected]>

commit e31982c
Author: Chris Sommers <[email protected]>
Date:   Sun Sep 11 13:36:47 2022 -0700

    Fix docker & Makefile perms (#202)

    * Incremental work on docker permissions fixes.

    * More progress on docker and make permissions issues.

    * Trial remove -u root from CI file.

    * Makefile and CI file perms

    * Fix CI docker flags

    * Final purge of obs sudo commands to fix perms.

    * Test for veth's before deleting.

    Co-authored-by: Chris Sommers <[email protected]>

commit 32332b5
Merge: 9f62724 6d1c50a
Author: Michael Miele <[email protected]>
Date:   Wed Sep 7 17:46:27 2022 -0700

    Merge branch 'Azure:main' into main

commit 9f62724
Merge: 9b51bb7 e5e461a
Author: Michael Miele <[email protected]>
Date:   Thu Aug 25 09:11:30 2022 -0700

    Merge branch 'Azure:main' into main

commit 9b51bb7
Merge: 54eca5a 3c15d95
Author: Michael Miele <[email protected]>
Date:   Mon Jul 18 18:40:31 2022 -0700

    Merge branch 'Azure:main' into main

commit 54eca5a
Merge: 30c65ab 7b5ce5e
Author: Michael Miele <[email protected]>
Date:   Tue Jun 28 12:10:45 2022 -0700

    Merge branch 'Azure:main' into main

commit 30c65ab
Merge: 48ae39c c9612ff
Author: Michael Miele <[email protected]>
Date:   Mon Jun 20 10:27:44 2022 -0700

    Merge branch 'Azure:main' into main

commit 48ae39c
Merge: e4b3b64 49d9f2c
Author: Michael Miele <[email protected]>
Date:   Fri Jun 17 10:53:03 2022 -0700

    Merge branch 'Azure:main' into main

commit e4b3b64
Merge: 4120634 231212b
Author: Michael Miele <[email protected]>
Date:   Tue Jun 14 13:39:40 2022 -0700

    Merge branch 'Azure:main' into main

commit 4120634
Merge: b5c1700 a9ec96a
Author: Michael Miele <[email protected]>
Date:   Mon Jun 13 09:41:12 2022 -0700

    Merge branch 'Azure:main' into main

commit b5c1700
Merge: f541d44 2e28aef
Author: Michael Miele <[email protected]>
Date:   Mon Jun 13 09:31:01 2022 -0700

    Merge branch 'Azure:main' into main

commit f541d44
Merge: 1927911 8433203
Author: Michael Miele <[email protected]>
Date:   Wed Jun 8 13:40:39 2022 -0700

    Merge branch 'Azure:main' into main

commit 1927911
Merge: 569c32a d0807c8
Author: Michael Miele <[email protected]>
Date:   Tue May 31 13:49:13 2022 -0700

    Merge branch 'Azure:main' into main

commit 569c32a
Merge: d956391 e420737
Author: Michael Miele <[email protected]>
Date:   Fri May 20 16:25:33 2022 -0700

    Merge branch 'Azure:main' into main

commit d956391
Author: KrisNey-MSFT <[email protected]>
Date:   Tue May 10 16:34:33 2022 -0700

    Create sdn-packet-transforms-vm-to-vm.svg

    Edited Fast Path and Slow Path images

commit 18f88a4
Merge: b2c18a7 221c401
Author: Michael Miele <[email protected]>
Date:   Mon May 9 08:48:29 2022 -0700

    Merge branch 'Azure:main' into main

commit b2c18a7
Author: Michael Miele <[email protected]>
Date:   Mon May 9 08:47:53 2022 -0700

    sync-with-azure-dash (#8)

    * dash-holistic-view (#104)

    * draft document

    * Updated content and images

    * added image and updated content

    * fixed image

    * Update dash-architecture.md

    * Fixed image and caption

    * Update dash-architecture.md

    * added/reorganized images

    * Update dash-architecture.md

    * fixed image and content

    * Update dash-architecture.md

    * Update dash-hld-ecosystem.svg

    * Renamed file.

    * Update dash-hld-ecosystem.svg

    * Update dash-hld-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-hld-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * reorganized content

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Fixed images and content based on feedback

    * move images, added captions, fixed content

    * Update dash-hld-project-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md".

    * Update dash-architecture-view.md

    * fixed card opacity

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Fixed image

    * fixed image

    * fixed image and related content

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md (#3)

    Merged in the PR: mmiele#2.

    * Renamed article and added content.

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * renamed file

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * fixed formating

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Fixed images

    * Fixed TOC

    * Update documentation/general/design/dash-holistic-view.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update documentation/general/design/dash-holistic-view.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Apply suggestions from code review

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Apply suggestions from code review

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Updated content and added note

    * Fixed content

    * fixed metadata

    * Created .gitignore and removed bkp files

    * Fixed content based on feedback.

    * Fixed content based on feedback. Added links and special notes.  Fixed image.

    * Fixed diagrams: using gNMI container instead of DASH container.

    * Fixed title and typos

    * Added reference links

    * Update documentation/general/design/dash-holistic-view.md

    Change committed.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-high-level-smart-switch.svg

    Changed "DASH container" name to "gNMI container".

    * Fixed image and moved `.gitignore` to the main folder.

    * Apply suggestions from code review

    Committed Kristina's suggestions.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-holistic-view.md

    Renamed the section API to "SAI headers and behavioral model". Added links to the related README files.

    * Removed old dash-high-level-design.md

    deleted dash-high-level-design.md
    renamed dash-holistic-view.md to dash-high-level-design.md

    * Added the list of words to generate the "word cloud".

    * Update README.md

    Reversed the link order and fixed descriptions.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-high-level-design.md

    * Update dash-high-level-design.md

    * Update dash-hld-project-ecosystem.svg

    * Update dash-high-level-design.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

* Update vnet-to-vnet-service.md

Updated metadata.

* Update vnet-to-vnet-service.md

Update date

* Update vnet-to-vnet-service.md

Changed date

* Update vnet-to-vnet-service.md

Fixed date

* Update vnet-to-vnet-service.md

Updated

* Update vnet-to-vnet-service.md

Fixed date

* Create dash-sonic-hld.md

Sync with main

* Create program-scale-testing-requirements-draft.md

Sync with main

* Create AMD-Pensando_HA_Proposal.md

Sync with main

* Update vnet-to-vnet-service.md

Update

* Delete dash-sonic-hld.md

Reorganized general folder.

* Delete program-scale-testing-requirements-draft.md

Reorganized general folder.

* Update README.md

Removed the section "Organization of Design & Requirements Documents" because it is obsolete.

* Delete AMD-Pensando_HA_Proposal.md

Removed design folder.

* Fixed broken links

* Fixed links and removed obsolete one.

* Removed obsolete links.

* Update README.md

Removed obsolete link.

* Update README.md

Removed obsolete links.

* Removed obsolete links

Co-authored-by: KrisNey-MSFT <[email protected]>
chrispsommers added a commit that referenced this issue Oct 8, 2022
* Create dash-docker-p4c-bmv2-acr.yml

* Update Makefile

* Experimenting with ACR publish workflows

Revert p4c-bmv2 tag

* Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml

* Experimental ACR publish; change regname

Use sonicdash ACR for p4c-bmv2. No publish yet.

* Update dash-p4c-bmv2-docker-acr.yml

Publish to ACR using secrets

* Fix macro errors

* Update dash-p4c-bmv2-docker-acr.yml

Force trigger.

* Uncomment docker push command

* Build dash-grpc:1.43.2 docker and publish to ACR

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Publish docker images to ACR and use those in build process. (#208)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

Co-authored-by: Chris Sommers <[email protected]>

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Incremental progress - publish to ACRs (#212)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

Co-authored-by: Chris Sommers <[email protected]>

* Call docker publish make targets from all CI scripts which publish.

* Incremental progress publishing dockers to ACR (#213)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

Co-authored-by: Chris Sommers <[email protected]>

* Spellcheck - add to wordslist

* Sync to main via intermediate dev branch. (#216)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Incremental progress publishing dockers to ACR (#219)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

* Incremental progress publishing dockers to ACR. (#220)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Trial of env var to define docker img

* Add docker .env variable to CI triggers.

* Fix stray conflict markers in Makefile.

* Fix CI triggers.

* Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers.

* add missing path

* Fix path.

* Trivial change to force a CI build.

* Change Makefile to force CI rebuild.

* Trivial change to docker image .env file to test CI triggers.

* Change .env file to force CI.

* Change grpc docker .env file to trigger CI rebuild.

* Change saithrift-bldr docker .env to force CI rebuild.

* Change saithrift-client .env file to force CI rebuild/

* Trial docker build/conditional publish.

* Get env variable into context.

* Correct env filename

* Debugging env vars & docker save

* Fix CI syntax

* Add Makefile targets to upload/download docker image tarball artifact.

* Revert to simple conditional job for build+publish.

* Conditional CI for non-publish docker job.

* Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo.

* Documenting Docker workflows.

* Spellcheck fixes.

* Spellcheck fixes.

* Add screen cap of skipped job.

* Spellcheck

* Update PR from dev branch (sync to main) (#224)

* Update program-scale-testing-requirements-draft.md

Update Scale #'s per Keysight/MSFT call; document is a version which needs an update.

* Fix Spelling (#221)

* Update AMD-Pensando_HA_Proposal.md

Explicitly add 'parallel' to state synchronization stages (per @lguohan )

* Update AMD-Pensando_HA_Proposal.md (#210)

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Update AMD-Pensando_HA_Proposal.md

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Update .wordlist.txt

Updating w/Chris

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Update .wordlist.txt

* Slide deck for today's SAI Challenger presentation

* [doc]: Fix the typo of TCP UDP protocol number (#223)

According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17.

* Proofreading, image clarity.

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
Co-authored-by: Ze Gan <[email protected]>

* Use env variable for DASH_ACR_LOGIN_SERVER instead of secret.

* Use literal expression instead of env variable.

* Use literal expression for ACR repository instead of secret.

* Clarify docker publishing.

* Spellcheck.

* Document another dockerfile workflow.

* spellcheck

* CI triggers on PR to any branch, not just main.

* Fixes per review feedback #225 (comment):
 - fix image; (docxkerhub->ACR in one case
 - replace "local" with "host"
Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review).
Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix.

* Spellcheck.

* Spellcheck.

* Spellcheck

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
Co-authored-by: Ze Gan <[email protected]>
chrispsommers added a commit that referenced this issue Nov 1, 2022
…ion branches (#272)

* Create dash-docker-p4c-bmv2-acr.yml

* Update Makefile

* Experimenting with ACR publish workflows

Revert p4c-bmv2 tag

* Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml

* Experimental ACR publish; change regname

Use sonicdash ACR for p4c-bmv2. No publish yet.

* Update dash-p4c-bmv2-docker-acr.yml

Publish to ACR using secrets

* Fix macro errors

* Update dash-p4c-bmv2-docker-acr.yml

Force trigger.

* Uncomment docker push command

* Build dash-grpc:1.43.2 docker and publish to ACR

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Publish docker images to ACR and use those in build process. (#208)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

Co-authored-by: Chris Sommers <[email protected]>

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Incremental progress - publish to ACRs (#212)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

Co-authored-by: Chris Sommers <[email protected]>

* Call docker publish make targets from all CI scripts which publish.

* Incremental progress publishing dockers to ACR (#213)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

Co-authored-by: Chris Sommers <[email protected]>

* Added SAI-Challenger to CI setup.

    1. Added SAI-Challenger submodule.
    2. Added SAI-Challenger basic test.
    3. Changed Makefile to build/start/stop SAI-Challenger.

Signed-off-by: Maksym Hedeon <[email protected]>

* Added saithrift to SAI-Challenger client docker image

- Added saithrift to SAI-Challenger client docker image
- Changed SAI-C submodule branch

Signed-off-by: Maksym Hedeon <[email protected]>

* Update example vnet test.

    1. Added SAI-Challenger submodule.
    2. Added SAI-Challenger basic test.
    3. Changed Makefile to build/start/stop SAI-Challenger.

Signed-off-by: Kostiantyn Goloveshko <[email protected]>
Co-authored-by: Maksym Hedeon <[email protected]>

* Add vnet outbound test based on SAI description.

Signed-off-by: Maksym Prytoliuk <[email protected]>

* Fixup double network host usage for sai-challenger-client

Signed-off-by: Konstantin Goloveshko <[email protected]>

* Kdt 17: Update docker environment (#8)

* Added cgyang submodule
* Updated docker env for thrift tests
* Makefile refactor

Signed-off-by: Maksym Hedeon <[email protected]>

* Fixup VNET inbound cleanup removals in test config.

Signed-off-by: Konstantin Goloveshko <[email protected]>

* Update CA_TO_PA entry in outbound test

Signed-off-by: Maksym Prytoliuk <[email protected]>

* Add vnet scenarios in DASH config format.

* Added dash-style VNET inbound routing test
* Added dash-style VNET outbound routing test
* Read config from json (temp workaround)
* Added JSON setup configs for testbed

Signed-off-by: Anton Putria <[email protected]>
Co-authored-by: Maksym Prytoliuk <[email protected]>

* Fixed saigen links.

Signed-off-by: Anton Putria <[email protected]>

* Fixes in test scanario.

Signed-off-by: Anton Putria <[email protected]>

* Fixed path to saigen. Fixed inbound test.

Signed-off-by: Anton Putria <[email protected]>

* Update submodule

Signed-off-by: Anton Putria <[email protected]>

* Final test fixes to align with submodue version.

Signed-off-by: Anton Putria <[email protected]>

* Spellcheck - add to wordslist

* Sync to main via intermediate dev branch. (#216)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Incremental progress publishing dockers to ACR (#219)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

* Incremental progress publishing dockers to ACR. (#220)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix #205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Trial of env var to define docker img

* Add docker .env variable to CI triggers.

* Fix stray conflict markers in Makefile.

* Fix CI triggers.

* Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers.

* add missing path

* Fix path.

* Trivial change to force a CI build.

* Change Makefile to force CI rebuild.

* Trivial change to docker image .env file to test CI triggers.

* Change .env file to force CI.

* Change grpc docker .env file to trigger CI rebuild.

* Change saithrift-bldr docker .env to force CI rebuild.

* Change saithrift-client .env file to force CI rebuild/

* Fixed VNET outbound test scenario.

Signed-off-by: Anton Putria <[email protected]>

* Trial docker build/conditional publish.

* Get env variable into context.

* Correct env filename

* Debugging env vars & docker save

* Fix CI syntax

* Add Makefile targets to upload/download docker image tarball artifact.

* Revert to simple conditional job for build+publish.

* Conditional CI for non-publish docker job.

* Outbound test passes with traffic

Fixed configuration based on changes in main.

Signed-off-by: Maksym Prytoliuk <[email protected]>

* Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo.

* Documenting Docker workflows.

* Spellcheck fixes.

* Spellcheck fixes.

* Add screen cap of skipped job.

* Spellcheck

* Update PR from dev branch (sync to main) (#224)

* Update program-scale-testing-requirements-draft.md

Update Scale #'s per Keysight/MSFT call; document is a version which needs an update.

* Fix Spelling (#221)

* Update AMD-Pensando_HA_Proposal.md

Explicitly add 'parallel' to state synchronization stages (per @lguohan )

* Update AMD-Pensando_HA_Proposal.md (#210)

* SAI apigen support for tables with no action parameters and a single action (#207)

* Update .wordlist.txt

* Update AMD-Pensando_HA_Proposal.md

* Doc dash as submodule (#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (#102)

* Add APP_DB to SAI mapping

* Update .wordlist.txt

Updating w/Chris

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Update .wordlist.txt

* Slide deck for today's SAI Challenger presentation

* [doc]: Fix the typo of TCP UDP protocol number (#223)

According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17.

* Proofreading, image clarity.

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
Co-authored-by: Ze Gan <[email protected]>

* Use env variable for DASH_ACR_LOGIN_SERVER instead of secret.

* Use literal expression instead of env variable.

* Use literal expression for ACR repository instead of secret.

* Clarify docker publishing.

* Spellcheck.

* Document another dockerfile workflow.

* spellcheck

* CI triggers on PR to any branch, not just main.

* Update documentation

- Added dash-test-sai-challenger.md doc file under test/docs
- Updated Makefile make run-saic-test-thrift target with passing parameters.

Signed-off-by: Maksym Hedeon <[email protected]>

* Updated SAI-Challenger submodule.

Signed-off-by: Anton Putria <[email protected]>

* Improvements to SAI-Challenger docker flows.

- Use docker hub to pull images
- Fixed make run-saic-tests to do not show errors
- Updated manuals

Signed-off-by: Anton Putria <[email protected]>

* Fixes per review feedback #225 (comment):
 - fix image; (docxkerhub->ACR in one case
 - replace "local" with "host"
Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review).
Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix.

* Spellcheck.

* Spellcheck.

* Update dash-test-sai-challenger.md

Signed-off-by: Mircea Dan Gheorghe <[email protected]>

* Add bmv2 SAI port attributes (num_active, port_list), dflt vlan, dflt vrf)

* Added SAI-Challenger docker build verification to GitHub actions

- Added SAI-Challenger docker verification action.
- Fixed spellchecker and docker build issues.

Signed-off-by: Anton Putria <[email protected]>

* Add default .1Q bridge to bmv2

Signed-off-by: Maksym Prytoliuk <[email protected]>

* Fixed SAI-Challenger user guide and file namings.

Signed-off-by: Anton Putria <[email protected]>

* Spellcheck

* Scaled VNET outbound test using snappi and saigen. (1st edition)

- Split scale and simple tests / jsons
- Updated ixia controller version
- Added dash_helper for traffic profile scaling.

Signed-off-by: Maksym Hedeon <[email protected]>

* spellcheck

* Move saichallenger client docker to ACR.

* Fix .yml formatting.

* Fix dependency in action file.

* Add missing .env file

* remove space in action script, add .env dependencies

* fix include path

* Add saichalleneger tests to run-all-tests and CI

* Fix docker image name.

* Correct mixup in image names.

* Remove stray hyphen

* Run saichallenger workflows on any branch PR.

* Added few more VNET scenario examples.

- Extended VNET traffic scenarios.
- Added docstrings for helper methods.
- Removed unused code.

Signed-off-by: Anton Putria <[email protected]>
Co-authored-by: Maksym Hedeon <[email protected]>

* Slides from DASH WG Meeting (SAI Challenger Episode 2) (#254)

* Add outbound diagram

Signed-off-by: Maksym Prytoliuk <[email protected]>

* Added inbound to xfail test results. (#44)

- Marked VNET Inbound Routing with xfail
- Added pytest.ini with marks definitions.
- Decreased duration of tests.

Signed-off-by: Anton Putria <[email protected]>

* Scaling test configuration documentation

- Added scaling approach documentation.
- Vnet2vnet traffic scaling scenario description.
- Added docstring to the test cases.
- Added more test examples.

Signed-off-by: Anton Putria <[email protected]>
Co-authored-by: Maksym Hedeon <[email protected]>
Co-authored-by: Maksym Prytoliuk <[email protected]>

* Submodule update and rename

* Rename and update the SAI-Challenger submodule
* Enabled additional tests based on the fixes in the SAI-Challenger.
* Renamed test folder.

Signed-off-by: Anton Putria <[email protected]>

* PTF: add basic test infra and more test cases (#231)

* ptf: add vnet sample test cases for dash

- Common VNet API for all tests
- Test cases:
  - Inbound VNI/ENI
  - Inbound Routing PA validate
  - Inbound Routing VNI match
  - Inbound Routing PA SRC IP
  - Outbound Routing Vnet direct
  - Outbound Routing direct
  - CT (just placeholder)
  - Route (basic)

Signed-off-by: Volodymyr Mytnyk <[email protected]>
Signed-off-by: Yuriy Harhas <[email protected]>

* ptf: run only bmv2 ready test case on CI

Signed-off-by: Volodymyr Mytnyk <[email protected]>

* ptf: test_saithrift_vnet: removed unused libs

Signed-off-by: Volodymyr Mytnyk <[email protected]>

* test-cases folder restructure.

- created two subfolders in root: functional and scale
- removed outdated bmv2_model folder
- moved all existing content of test-cases to test-cases/scale

Signed-off-by: Anton Putria <[email protected]>

* Moved saidashvnet.py to a proper location.

Signed-off-by: Anton Putria <[email protected]>

* Fixed ENI creation in Outbound tests

Signed-off-by: Yuriy Harhas <[email protected]>

* Spellchecker related fixes.

Signed-off-by: Anton Putria <[email protected]>

* Updated VNET test cases based on the PR #238.

Signed-off-by: Yuriy Harhas <[email protected]>

Signed-off-by: Volodymyr Mytnyk <[email protected]>
Signed-off-by: Yuriy Harhas <[email protected]>
Signed-off-by: Anton Putria <[email protected]>
Co-authored-by: Anton Putria <[email protected]>
Co-authored-by: Yuriy Harhas <[email protected]>

* Adding updated license files (#266)

* Fixbug: SAI Server crash on DASH ACL API (#264)

1. Fix duplicate SAI ID
2. Wrong search action
3. Double delete entry if SAI write error
4. Wrong format of error message

Signed-off-by: Ze Gan <[email protected]>

* Add VNET v6 PTF test case; Fix v4 PTF test case to address test hang issue (#263)

This PR adds VNET v6 test case and also fixes the PTF thrift client issue (the v6 test case just hangs) #237. This is a joint work with @chrispsommers and @aputriax.

* Fixed links in the README after renaming folders. (#262)

Signed-off-by: Anton Putria <[email protected]>

Signed-off-by: Anton Putria <[email protected]>

* Correct CI file, was publishing wrong image. Some prior merge must have caused a mixup.

Signed-off-by: Maksym Hedeon <[email protected]>
Signed-off-by: Kostiantyn Goloveshko <[email protected]>
Signed-off-by: Maksym Prytoliuk <[email protected]>
Signed-off-by: Konstantin Goloveshko <[email protected]>
Signed-off-by: Anton Putria <[email protected]>
Signed-off-by: Marian Pritsak <[email protected]>
Signed-off-by: Mircea Dan Gheorghe <[email protected]>
Signed-off-by: Volodymyr Mytnyk <[email protected]>
Signed-off-by: Yuriy Harhas <[email protected]>
Signed-off-by: Anton Putria <[email protected]>
Signed-off-by: Ze Gan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Maksym Hedeon <[email protected]>
Co-authored-by: Kostiantyn Goloveshko <[email protected]>
Co-authored-by: Maksym Hedeon <[email protected]>
Co-authored-by: Maksym Prytoliuk <[email protected]>
Co-authored-by: Anton Putria <[email protected]>
Co-authored-by: Maksym Prytoliuk <[email protected]>
Co-authored-by: Anton Putria <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
Co-authored-by: Ze Gan <[email protected]>
Co-authored-by: MirceaDan <[email protected]>
Co-authored-by: Volodymyr Mytnyk <[email protected]>
Co-authored-by: Anton Putria <[email protected]>
Co-authored-by: Yuriy Harhas <[email protected]>
Co-authored-by: Mohammad Hanif <[email protected]>
vijasrin pushed a commit to vijasrin/DASH that referenced this issue Dec 8, 2022
* sync-with-azure-dash (sonic-net#8)

* dash-holistic-view (sonic-net#104)

* draft document

* Updated content and images

* added image and updated content

* fixed image

* Update dash-architecture.md

* Fixed image and caption

* Update dash-architecture.md

* added/reorganized images

* Update dash-architecture.md

* fixed image and content

* Update dash-architecture.md

* Update dash-hld-ecosystem.svg

* Renamed file.

* Update dash-hld-ecosystem.svg

* Update dash-hld-ecosystem.svg

* Update dash-architecture-view.md

* Update dash-hld-ecosystem.svg

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* reorganized content

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Fixed images and content based on feedback

* move images, added captions, fixed content

* Update dash-hld-project-ecosystem.svg

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md".

* Update dash-architecture-view.md

* fixed card opacity

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Fixed image

* fixed image

* fixed image and related content

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md (#3)

Merged in the PR: mmiele#2.

* Renamed article and added content.

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* Update dash-architecture-view.md

* renamed file

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* fixed formating

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Update dash-holistic-view.md

* Fixed images

* Fixed TOC

* Update documentation/general/design/dash-holistic-view.md

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update documentation/general/design/dash-holistic-view.md

Co-authored-by: KrisNey-MSFT <[email protected]>

* Apply suggestions from code review

Co-authored-by: KrisNey-MSFT <[email protected]>

* Apply suggestions from code review

Co-authored-by: KrisNey-MSFT <[email protected]>

* Updated content and added note

* Fixed content

* fixed metadata

* Created .gitignore and removed bkp files

* Fixed content based on feedback.

* Fixed content based on feedback. Added links and special notes.  Fixed image.

* Fixed diagrams: using gNMI container instead of DASH container.

* Fixed title and typos

* Added reference links

* Update documentation/general/design/dash-holistic-view.md

Change committed.

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update dash-high-level-smart-switch.svg

Changed "DASH container" name to "gNMI container".

* Fixed image and moved `.gitignore` to the main folder.

* Apply suggestions from code review

Committed Kristina's suggestions.

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update dash-holistic-view.md

Renamed the section API to "SAI headers and behavioral model". Added links to the related README files.

* Removed old dash-high-level-design.md

deleted dash-high-level-design.md
renamed dash-holistic-view.md to dash-high-level-design.md

* Added the list of words to generate the "word cloud".

* Update README.md

Reversed the link order and fixed descriptions.

Co-authored-by: KrisNey-MSFT <[email protected]>

* Update dash-high-level-design.md

* Update dash-high-level-design.md

* Update dash-hld-project-ecosystem.svg

* Update dash-high-level-design.md

Co-authored-by: KrisNey-MSFT <[email protected]>

* Create sdn-packet-transforms-vm-to-vm.svg

Edited Fast Path and Slow Path images

* Reorganized dataplane folder.

* Fixed links

* Reorganized gnmi folder.

* Reorganized encrypt-gw-service and express-route-service folders

* Reorganized general folder

* Fixed link

* Reorganized high-avail folder.

* Reorganized load-bal-service folder

* Reorganized sai folder.

* Reorganized stpl-service folder.

* Reorganized vnet-peering-service folder.

* Reorganized vnet-to-vnet-service folder.

* Fixed reference links.

* Fixed links to point to the reorganized folders.

* Update vnet-to-vnet-service.md

Updated metadata

* Squashed commit of the following:

commit 9700152
Merge: 32332b5 d003f4e
Author: Michael Miele <[email protected]>
Date:   Sat Sep 17 16:43:45 2022 -0700

    Merge branch 'Azure:main' into main

commit d003f4e
Author: Chris Sommers <[email protected]>
Date:   Fri Sep 16 22:43:03 2022 -0700

    Upgrade to ixia-c 0.0.1-3383 (sonic-net#226)

    * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch.
    Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.)

    * Update ixia-c traffic engine to match controller.

    * Update snappi version to match ixia-c-components.

    Co-authored-by: Chris Sommers <[email protected]>

commit 543df5c
Author: Ze Gan <[email protected]>
Date:   Fri Sep 16 01:36:01 2022 +0800

    [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223)

    According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17.

commit 94b0168
Author: Chris Sommers <[email protected]>
Date:   Wed Sep 14 10:27:22 2022 -0700

    Slide deck for today's SAI Challenger presentation

commit 10056d7
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:33:36 2022 -0700

    Update .wordlist.txt

commit f1c5245
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:31:16 2022 -0700

    Fix Spelling (sonic-net#221)

    * Update AMD-Pensando_HA_Proposal.md

    Explicitly add 'parallel' to state synchronization stages (per @lguohan )

    * Update AMD-Pensando_HA_Proposal.md (sonic-net#210)

    * SAI apigen support for tables with no action parameters and a single action (sonic-net#207)

    * Update .wordlist.txt

    * Update AMD-Pensando_HA_Proposal.md

    * Doc dash as submodule (sonic-net#203)

    * Document third-party workflows using DASH as a Git submodule.

    * Add URL to sample project.

    * Incorporate review feedback (typo; missing file).

    * Spellcheck fixes.

    * Spellcheck

    * Add .wordlist.txt to CI triggers.

    * Spellcheck wordlist.

    Co-authored-by: Chris Sommers <[email protected]>

    * Split SAI API (sonic-net#201)

    Make APIs compatible with SONiC bulk infra

    Signed-off-by: Marian Pritsak <[email protected]>

    * Add APP_DB to SAI mapping (sonic-net#102)

    * Add APP_DB to SAI mapping

    * Update .wordlist.txt

    Updating w/Chris

    Signed-off-by: Marian Pritsak <[email protected]>
    Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Marian Pritsak <[email protected]>

    Signed-off-by: Marian Pritsak <[email protected]>
    Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Chris Sommers <[email protected]>
    Co-authored-by: Marian Pritsak <[email protected]>

commit 8694184
Author: KrisNey-MSFT <[email protected]>
Date:   Mon Sep 12 16:25:46 2022 -0700

    Update program-scale-testing-requirements-draft.md

    Update Scale #'s per Keysight/MSFT call; document is a version which needs an update.

commit 2ce121e
Author: Chris Sommers <[email protected]>
Date:   Sun Sep 11 13:37:40 2022 -0700

    Fix ipv6 packet noise (sonic-net#211)

    * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205.

    * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

    * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

    Co-authored-by: Chris Sommers <[email protected]>

commit e31982c
Author: Chris Sommers <[email protected]>
Date:   Sun Sep 11 13:36:47 2022 -0700

    Fix docker & Makefile perms (sonic-net#202)

    * Incremental work on docker permissions fixes.

    * More progress on docker and make permissions issues.

    * Trial remove -u root from CI file.

    * Makefile and CI file perms

    * Fix CI docker flags

    * Final purge of obs sudo commands to fix perms.

    * Test for veth's before deleting.

    Co-authored-by: Chris Sommers <[email protected]>

commit 32332b5
Merge: 9f62724 6d1c50a
Author: Michael Miele <[email protected]>
Date:   Wed Sep 7 17:46:27 2022 -0700

    Merge branch 'Azure:main' into main

commit 9f62724
Merge: 9b51bb7 e5e461a
Author: Michael Miele <[email protected]>
Date:   Thu Aug 25 09:11:30 2022 -0700

    Merge branch 'Azure:main' into main

commit 9b51bb7
Merge: 54eca5a 3c15d95
Author: Michael Miele <[email protected]>
Date:   Mon Jul 18 18:40:31 2022 -0700

    Merge branch 'Azure:main' into main

commit 54eca5a
Merge: 30c65ab 7b5ce5e
Author: Michael Miele <[email protected]>
Date:   Tue Jun 28 12:10:45 2022 -0700

    Merge branch 'Azure:main' into main

commit 30c65ab
Merge: 48ae39c c9612ff
Author: Michael Miele <[email protected]>
Date:   Mon Jun 20 10:27:44 2022 -0700

    Merge branch 'Azure:main' into main

commit 48ae39c
Merge: e4b3b64 49d9f2c
Author: Michael Miele <[email protected]>
Date:   Fri Jun 17 10:53:03 2022 -0700

    Merge branch 'Azure:main' into main

commit e4b3b64
Merge: 4120634 231212b
Author: Michael Miele <[email protected]>
Date:   Tue Jun 14 13:39:40 2022 -0700

    Merge branch 'Azure:main' into main

commit 4120634
Merge: b5c1700 a9ec96a
Author: Michael Miele <[email protected]>
Date:   Mon Jun 13 09:41:12 2022 -0700

    Merge branch 'Azure:main' into main

commit b5c1700
Merge: f541d44 2e28aef
Author: Michael Miele <[email protected]>
Date:   Mon Jun 13 09:31:01 2022 -0700

    Merge branch 'Azure:main' into main

commit f541d44
Merge: 1927911 8433203
Author: Michael Miele <[email protected]>
Date:   Wed Jun 8 13:40:39 2022 -0700

    Merge branch 'Azure:main' into main

commit 1927911
Merge: 569c32a d0807c8
Author: Michael Miele <[email protected]>
Date:   Tue May 31 13:49:13 2022 -0700

    Merge branch 'Azure:main' into main

commit 569c32a
Merge: d956391 e420737
Author: Michael Miele <[email protected]>
Date:   Fri May 20 16:25:33 2022 -0700

    Merge branch 'Azure:main' into main

commit d956391
Author: KrisNey-MSFT <[email protected]>
Date:   Tue May 10 16:34:33 2022 -0700

    Create sdn-packet-transforms-vm-to-vm.svg

    Edited Fast Path and Slow Path images

commit 18f88a4
Merge: b2c18a7 221c401
Author: Michael Miele <[email protected]>
Date:   Mon May 9 08:48:29 2022 -0700

    Merge branch 'Azure:main' into main

commit b2c18a7
Author: Michael Miele <[email protected]>
Date:   Mon May 9 08:47:53 2022 -0700

    sync-with-azure-dash (sonic-net#8)

    * dash-holistic-view (sonic-net#104)

    * draft document

    * Updated content and images

    * added image and updated content

    * fixed image

    * Update dash-architecture.md

    * Fixed image and caption

    * Update dash-architecture.md

    * added/reorganized images

    * Update dash-architecture.md

    * fixed image and content

    * Update dash-architecture.md

    * Update dash-hld-ecosystem.svg

    * Renamed file.

    * Update dash-hld-ecosystem.svg

    * Update dash-hld-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-hld-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * reorganized content

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Fixed images and content based on feedback

    * move images, added captions, fixed content

    * Update dash-hld-project-ecosystem.svg

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md".

    * Update dash-architecture-view.md

    * fixed card opacity

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Fixed image

    * fixed image

    * fixed image and related content

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md (#3)

    Merged in the PR: mmiele#2.

    * Renamed article and added content.

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * Update dash-architecture-view.md

    * renamed file

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * fixed formating

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Update dash-holistic-view.md

    * Fixed images

    * Fixed TOC

    * Update documentation/general/design/dash-holistic-view.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update documentation/general/design/dash-holistic-view.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Apply suggestions from code review

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Apply suggestions from code review

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Updated content and added note

    * Fixed content

    * fixed metadata

    * Created .gitignore and removed bkp files

    * Fixed content based on feedback.

    * Fixed content based on feedback. Added links and special notes.  Fixed image.

    * Fixed diagrams: using gNMI container instead of DASH container.

    * Fixed title and typos

    * Added reference links

    * Update documentation/general/design/dash-holistic-view.md

    Change committed.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-high-level-smart-switch.svg

    Changed "DASH container" name to "gNMI container".

    * Fixed image and moved `.gitignore` to the main folder.

    * Apply suggestions from code review

    Committed Kristina's suggestions.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-holistic-view.md

    Renamed the section API to "SAI headers and behavioral model". Added links to the related README files.

    * Removed old dash-high-level-design.md

    deleted dash-high-level-design.md
    renamed dash-holistic-view.md to dash-high-level-design.md

    * Added the list of words to generate the "word cloud".

    * Update README.md

    Reversed the link order and fixed descriptions.

    Co-authored-by: KrisNey-MSFT <[email protected]>

    * Update dash-high-level-design.md

    * Update dash-high-level-design.md

    * Update dash-hld-project-ecosystem.svg

    * Update dash-high-level-design.md

    Co-authored-by: KrisNey-MSFT <[email protected]>

* Update vnet-to-vnet-service.md

Updated metadata.

* Update vnet-to-vnet-service.md

Update date

* Update vnet-to-vnet-service.md

Changed date

* Update vnet-to-vnet-service.md

Fixed date

* Update vnet-to-vnet-service.md

Updated

* Update vnet-to-vnet-service.md

Fixed date

* Create dash-sonic-hld.md

Sync with main

* Create program-scale-testing-requirements-draft.md

Sync with main

* Create AMD-Pensando_HA_Proposal.md

Sync with main

* Update vnet-to-vnet-service.md

Update

* Delete dash-sonic-hld.md

Reorganized general folder.

* Delete program-scale-testing-requirements-draft.md

Reorganized general folder.

* Update README.md

Removed the section "Organization of Design & Requirements Documents" because it is obsolete.

* Delete AMD-Pensando_HA_Proposal.md

Removed design folder.

* Fixed broken links

* Fixed links and removed obsolete one.

* Removed obsolete links.

* Update README.md

Removed obsolete link.

* Update README.md

Removed obsolete links.

* Removed obsolete links

Co-authored-by: KrisNey-MSFT <[email protected]>
vijasrin pushed a commit to vijasrin/DASH that referenced this issue Dec 8, 2022
* Create dash-docker-p4c-bmv2-acr.yml

* Update Makefile

* Experimenting with ACR publish workflows

Revert p4c-bmv2 tag

* Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml

* Experimental ACR publish; change regname

Use sonicdash ACR for p4c-bmv2. No publish yet.

* Update dash-p4c-bmv2-docker-acr.yml

Publish to ACR using secrets

* Fix macro errors

* Update dash-p4c-bmv2-docker-acr.yml

Force trigger.

* Uncomment docker push command

* Build dash-grpc:1.43.2 docker and publish to ACR

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Publish docker images to ACR and use those in build process. (sonic-net#208)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

Co-authored-by: Chris Sommers <[email protected]>

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Incremental progress - publish to ACRs (sonic-net#212)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

Co-authored-by: Chris Sommers <[email protected]>

* Call docker publish make targets from all CI scripts which publish.

* Incremental progress publishing dockers to ACR (sonic-net#213)

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

Co-authored-by: Chris Sommers <[email protected]>

* Spellcheck - add to wordslist

* Sync to main via intermediate dev branch. (sonic-net#216)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (sonic-net#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (sonic-net#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (sonic-net#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (sonic-net#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Incremental progress publishing dockers to ACR (sonic-net#219)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (sonic-net#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (sonic-net#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (sonic-net#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (sonic-net#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (sonic-net#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (sonic-net#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

* Incremental progress publishing dockers to ACR. (sonic-net#220)

* AMD-Pensando HA Proposal document

* Add CI script for saithrift-bldr-docker

* Remove superfluous ACR publish steps from regular docker build CI script.

* Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR).

* SAI apigen support for tables with no action parameters and a single action (sonic-net#207)

* Update .wordlist.txt

* Use make target to publish to ACR (just bmv2-bldr as first candidate).

* Call docker publish make targets from all CI scripts which publish.

* Doc dash as submodule (sonic-net#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (sonic-net#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (sonic-net#102)

* Add APP_DB to SAI mapping

* Spellcheck - add to wordslist

* Fix docker & Makefile perms (sonic-net#202)

* Incremental work on docker permissions fixes.

* More progress on docker and make permissions issues.

* Trial remove -u root from CI file.

* Makefile and CI file perms

* Fix CI docker flags

* Final purge of obs sudo commands to fix perms.

* Test for veth's before deleting.

Co-authored-by: Chris Sommers <[email protected]>

* Fix ipv6 packet noise (sonic-net#211)

* Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205.

* CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces.

* Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target.

Co-authored-by: Chris Sommers <[email protected]>

* Remove publish steps from non-ACR CI scripts (consequence of merge).

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Trial of env var to define docker img

* Add docker .env variable to CI triggers.

* Fix stray conflict markers in Makefile.

* Fix CI triggers.

* Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers.

* add missing path

* Fix path.

* Trivial change to force a CI build.

* Change Makefile to force CI rebuild.

* Trivial change to docker image .env file to test CI triggers.

* Change .env file to force CI.

* Change grpc docker .env file to trigger CI rebuild.

* Change saithrift-bldr docker .env to force CI rebuild.

* Change saithrift-client .env file to force CI rebuild/

* Trial docker build/conditional publish.

* Get env variable into context.

* Correct env filename

* Debugging env vars & docker save

* Fix CI syntax

* Add Makefile targets to upload/download docker image tarball artifact.

* Revert to simple conditional job for build+publish.

* Conditional CI for non-publish docker job.

* Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo.

* Documenting Docker workflows.

* Spellcheck fixes.

* Spellcheck fixes.

* Add screen cap of skipped job.

* Spellcheck

* Update PR from dev branch (sync to main) (sonic-net#224)

* Update program-scale-testing-requirements-draft.md

Update Scale #'s per Keysight/MSFT call; document is a version which needs an update.

* Fix Spelling (sonic-net#221)

* Update AMD-Pensando_HA_Proposal.md

Explicitly add 'parallel' to state synchronization stages (per @lguohan )

* Update AMD-Pensando_HA_Proposal.md (sonic-net#210)

* SAI apigen support for tables with no action parameters and a single action (sonic-net#207)

* Update .wordlist.txt

* Update AMD-Pensando_HA_Proposal.md

* Doc dash as submodule (sonic-net#203)

* Document third-party workflows using DASH as a Git submodule.

* Add URL to sample project.

* Incorporate review feedback (typo; missing file).

* Spellcheck fixes.

* Spellcheck

* Add .wordlist.txt to CI triggers.

* Spellcheck wordlist.

Co-authored-by: Chris Sommers <[email protected]>

* Split SAI API (sonic-net#201)

Make APIs compatible with SONiC bulk infra

Signed-off-by: Marian Pritsak <[email protected]>

* Add APP_DB to SAI mapping (sonic-net#102)

* Add APP_DB to SAI mapping

* Update .wordlist.txt

Updating w/Chris

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>

* Update .wordlist.txt

* Slide deck for today's SAI Challenger presentation

* [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223)

According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17.

* Proofreading, image clarity.

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
Co-authored-by: Ze Gan <[email protected]>

* Use env variable for DASH_ACR_LOGIN_SERVER instead of secret.

* Use literal expression instead of env variable.

* Use literal expression for ACR repository instead of secret.

* Clarify docker publishing.

* Spellcheck.

* Document another dockerfile workflow.

* spellcheck

* CI triggers on PR to any branch, not just main.

* Fixes per review feedback sonic-net#225 (comment):
 - fix image; (docxkerhub->ACR in one case
 - replace "local" with "host"
Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review).
Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix.

* Spellcheck.

* Spellcheck.

* Spellcheck

Signed-off-by: Marian Pritsak <[email protected]>
Co-authored-by: Chris Sommers <[email protected]>
Co-authored-by: Sanjay Thyamagundalu <[email protected]>
Co-authored-by: Mukesh Moopath Velayudhan <[email protected]>
Co-authored-by: KrisNey-MSFT <[email protected]>
Co-authored-by: Marian Pritsak <[email protected]>
Co-authored-by: Ze Gan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants