Moving speed setting below interface type setting

[dgsudharsan]

What I did
Moved speed setting below interface type setting in the processing order. Sometimes the interface type settings are required to set proper speed and to avoid ambiguity (e.g. 100G can be set on 2 lanes in PAM4 case or 4 lanes in case of NRZ case which is decided by interface type).

When user configures speed and interface type commands next to next, it may be combined as a single update due to optimization in orchagent and with the current order it may result in speed getting processed before interface type.

User commands

2024 Aug 29 00:06:16.683596 mtvr-hippo-01 NOTICE sudo:    admin : TTY=pts/1 ; PWD=/home/admin ; USER=root ; COMMAND=/usr/local/bin/config interface type Ethernet336 none
2024 Aug 29 00:06:18.295811 mtvr-hippo-01 NOTICE sudo:    admin : TTY=pts/1 ; PWD=/home/admin ; USER=root ; COMMAND=/usr/local/bin/config interface speed Ethernet336 25000

SWSS record

2024-08-28.21:06:19.117532|PORT_TABLE:Ethernet336|SET|alias:etp43|index:43|lanes:336,337,338,339,340,341,342,343|speed:25000|autoneg:off|interface_type:none|adv_speeds:400000|adv_interface_types:CR8|fec:rs|mtu:9100|admin_status:down

Why I did it
Moved processing of speed setting after interface type in order to avoid any error in programming SAI.

How I verified it
Existing tests should verify the current functionality. Ordering cannot be tested with the current vs test infrastructure.

Details if related