Skipping test temporarily for caclmgrd updates#17231
Skipping test temporarily for caclmgrd updates#17231gupurush wants to merge 1 commit intosonic-net:masterfrom
Conversation
|
/azp run |
|
|
|
Azure Pipelines successfully started running 1 pipeline(s). |
299fa9d to
7a437ea
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
tests/cacl/test_cacl_application.py
Outdated
| pytest.mark.topology('any') | ||
| ] | ||
|
|
||
| pytest.skip("Skipping this test file temporarily for caclmgrd updates", allow_module_level=True) |
There was a problem hiding this comment.
No, it's not a good way to skip it for all test environment.
We can remove it from .azure-pipelines/pr_test_scripts.yaml
After image fix gets merged, add it back then
There was a problem hiding this comment.
Updated as suggested.
62144d4 to
3f52c54
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
ZhaohuiS
left a comment
There was a problem hiding this comment.
Need to add it back and update the test case according to the changes in sonic-net/sonic-host-services#197
|
@gupurush could you please also prepare PR for cacl test case update for the PR sonic-net/sonic-host-services#197 |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@gupurush please update the test_cacl_application and enable it for PR test. |
|
reopen it to retrigger PR test |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Yes, we can close it. Thank you |
Description of PR
Skipping this test file temporarily for caclmgrd updates - sonic-net/sonic-host-services#197