Skip to content

Revert "Change pytest CLI arguments position for discovering subfolder conftest.py"#2169

Closed
daall wants to merge 1 commit intomasterfrom
revert-2146-run_tests_conftest
Closed

Revert "Change pytest CLI arguments position for discovering subfolder conftest.py"#2169
daall wants to merge 1 commit intomasterfrom
revert-2146-run_tests_conftest

Conversation

@daall
Copy link
Contributor

@daall daall commented Sep 1, 2020

Reverts #2146

This PR somehow changed the pretest step in the run_tests script. announce_routes is not running, which is causing the BGP tests to fail in the mgmt and build-image repos. Need to back this out while we investigate so those repos are not blocked.

@daall
Copy link
Contributor Author

daall commented Sep 1, 2020

retest vsimage please

@daall daall marked this pull request as ready for review September 2, 2020 02:08
@daall
Copy link
Contributor Author

daall commented Sep 2, 2020

retest vsimage please

@daall daall requested a review from a team September 2, 2020 02:08
@daall
Copy link
Contributor Author

daall commented Sep 2, 2020

Closing in favor of #2173

@daall daall closed this Sep 2, 2020
@wangxin wangxin deleted the revert-2146-run_tests_conftest branch February 18, 2022 05:51
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
5bb99c7 Validate LAG has members before mirror session create (sonic-net#2130)
ec6c8af [vxlan] Remove tunnel map objects on VNET tunnel removal (sonic-net#2150)
7e7db19 [BFD]Registering BFD state change callback during session creation (sonic-net#2202)
618fe07 [VNET]Fixing nexthop group delete during route change (sonic-net#2198)
91b66df [portsorch]: Prevent LAG member configuration when port has active ACL binding (sonic-net#2165)
29de9d0 Remove redundant and problematic code to skip "pool" field in buffer profile handling (sonic-net#2197)
ded0b45 [PBH] Implement Edit Flows (sonic-net#2169)
2ee0f49 [neighsyncd] increase neighsyncd timeout (sonic-net#2209)
a0160c0 [QosOrch] The notifications cannot be drained in QosOrch in case the first one needs to retry (sonic-net#2206)
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 this pull request may close these issues.

1 participant