Revert "Add support for IP interface loopback action (#2307)"

[prabhataravind]

This reverts commit 5043701.

Revert "[VS Test] Skip failing subport tests (#2370)"

This reverts commit 7175245.

What I did

Why I did it

How I verified it

Details if related

[prabhataravind]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[prsunny]

@liorghub , please take a look

[prsunny]

@liorghub , We just re-added the subport test in this PR - #2389 and it failed. So somehow it is linked to the changes. Can you please investigate this with priority?

[liorghub]

@liorghub , We just re-added the subport test in this PR - #2389 and it failed. So somehow it is linked to the changes. Can you please investigate this with priority?

@prsunny @dprital I tried to reproduce the issue that occured in vstest of PR #2389 but unfortunately I couldn't. In my run there were other failures.
vstest in PR #2389 failed since there were 2 keys in table ASIC_STATE:SAI_OBJECT_TYPE_VIRTUAL_ROUTER. I searched the code to understand what is the flow that creates a new key and found that it happens once on switch init and each time we create VRF or EVPN tunnel. My tests are not creating VRF or tunnel, therefore they could not cause the failure. If you still think otherwise, appreciate if we can work together to resolve the issue before reverting my commit (5043701)

[prsunny]

@liorghub , We just re-added the subport test in this PR - #2389 and it failed. So somehow it is linked to the changes. Can you please investigate this with priority?

@prsunny @dprital I tried to reproduce the issue that occured in vstest of PR #2389 but unfortunately I couldn't. In my run there were other failures. vstest in PR #2389 failed since there were 2 keys in table ASIC_STATE:SAI_OBJECT_TYPE_VIRTUAL_ROUTER. I searched the code to understand what is the flow that creates a new key and found that it happens once on switch init and each time we create VRF or EVPN tunnel. My tests are not creating VRF or tunnel, therefore they could not cause the failure. If you still think otherwise, appreciate if we can work together to resolve the issue before reverting my commit (5043701)

We are also not clear on the rootcause, but reverting your PR makes it pass, but with the changes, its failing. If you cannot repro it manually, can you try by reverting the changes one-by-one. Like remove intfmgrd but keep intforch and raise a PR. Lets try to narrow down.

[prsunny]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).