Skip to content

[service_acl] Don't fail if acl.json is not already present on device#669

Merged
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:service_acl_fix
Jul 31, 2018
Merged

[service_acl] Don't fail if acl.json is not already present on device#669
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:service_acl_fix

Conversation

@jleveque
Copy link
Copy Markdown
Contributor

Service ACL test no longer expects acl.json to be present on the device before executing the test. This was causing erroneous test failures on devices which did not have an acl.json file present.

@jleveque jleveque self-assigned this Jul 31, 2018
@jleveque jleveque requested a review from maggiemsft July 31, 2018 17:02
@jleveque jleveque merged commit 69f7254 into sonic-net:master Jul 31, 2018
@jleveque jleveque deleted the service_acl_fix branch July 31, 2018 18:30
wangxin pushed a commit to wangxin/sonic-mgmt that referenced this pull request Oct 27, 2025
…net#669)

Sync PR sonic-net#17700 from sonic-net/sonic-mgmt to 202412.
Original PR: sonic-net#17700

Co-authored-by: Nana@Nvidia <78413612+nhe-NV@users.noreply.github.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
#### Why I did it

Update sonic-swss-common submodule pointer to include the following:
* [202012] Add docker-mux related table names  ([sonic-net#669](sonic-net/sonic-swss-common#669))
* Transfer organization from Azure to sonic-net ([sonic-net#658](sonic-net/sonic-swss-common#658))
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…D automatically (#23816)

#### Why I did it
src/sonic-platform-daemons
```
* 65a5c75 - (HEAD -> 202505, origin/202505) [xcvrd] Create TRANSCEIVER_FIRMWARE_INFO table for all subports (sonic-net#669) (2 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants