Skip to content

[sonic-cfggen] add option for redis connection#1213

Merged
lguohan merged 1 commit intosonic-net:masterfrom
ishidawataru:dev
Dec 7, 2017
Merged

[sonic-cfggen] add option for redis connection#1213
lguohan merged 1 commit intosonic-net:masterfrom
ishidawataru:dev

Conversation

@ishidawataru
Copy link
Collaborator

@ishidawataru ishidawataru commented Dec 7, 2017

- What I did

add an option for redis connection to sonic-cfggen

- How I did it

Used sonic-net/sonic-py-swsssdk#22

- How to verify it

sonic-cfggen -s /var/run/redis/redis.sock -j /root/sonic/config/config_db.json --write-to-db

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Wataru Ishida <ishida.wataru@lab.ntt.co.jp>
@lguohan lguohan requested a review from taoyl-ms December 7, 2017 03:02
Copy link
Contributor

@taoyl-ms taoyl-ms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to update submodule pointer before merging.

@lguohan lguohan merged commit 3d70b71 into sonic-net:master Dec 7, 2017
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022
…nic-net#1213)

* Introduce new DVSDatabase abstraction and SONiC redis fixtures
* Refactor ACL tests to use redis fixtures

Signed-off-by: Danny Allen <daall@microsoft.com>
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Mar 10, 2023
Update sonic-sairedis submodule pointer to include the following:
* 4bd1dc5 Fast reboot finalizer ([sonic-net#1213](sonic-net/sonic-sairedis#1213))
* 749b393 [ci] Fix apt-get install unable locate package issue. ([sonic-net#1212](sonic-net/sonic-sairedis#1212))
* 886875b [Dual-ToR] update sai.profile with SAI_ADDITIONAL_MAC_ENABLED attribute if corresponding arg passed to syncd ([sonic-net#1201](sonic-net/sonic-sairedis#1201))
* c58d259 Use new value of STATE_DB FAST_REBOOT entry ([sonic-net#1196](sonic-net/sonic-sairedis#1196))
* 3808e4c Fix issue: bulk counter feature is disabled ([sonic-net#1205](sonic-net/sonic-sairedis#1205))

Signed-off-by: dprital <drorp@nvidia.com>
liat-grozovik pushed a commit that referenced this pull request Mar 12, 2023
Update sonic-sairedis submodule pointer to include the following:
* 4bd1dc5 Fast reboot finalizer ([#1213](sonic-net/sonic-sairedis#1213))
* 749b393 [ci] Fix apt-get install unable locate package issue. ([#1212](sonic-net/sonic-sairedis#1212))
* 886875b [Dual-ToR] update sai.profile with SAI_ADDITIONAL_MAC_ENABLED attribute if corresponding arg passed to syncd ([#1201](sonic-net/sonic-sairedis#1201))
* c58d259 Use new value of STATE_DB FAST_REBOOT entry ([#1196](sonic-net/sonic-sairedis#1196))
* 3808e4c Fix issue: bulk counter feature is disabled ([#1205](sonic-net/sonic-sairedis#1205))

Signed-off-by: dprital <drorp@nvidia.com>
vmittal-msft pushed a commit to vmittal-msft/sonic-buildimage that referenced this pull request Oct 20, 2025
…tomatically (sonic-net#1213)

#### Why I did it
src/sonic-swss
```
* c203cec - (HEAD -> 202503, origin/202503) Merge pull request sonic-net#89 from mssonicbld/sonicbld/202503-merge (21 hours ago) [mssonicbld]
* c48655b - Merge branch '202412' of https://github.com/Azure/sonic-swss.msft into 202503 (23 hours ago) [Sonic Automation]
* 735a52f - Merge pull request sonic-net#87 from mssonicbld/sonicbld/202412-merge (2 days ago) [mssonicbld]
* 2761106 - Merge branch '202411' of https://github.com/sonic-net/sonic-swss into 202412 (2 days ago) [Sonic Automation]
* 9c3daae - [202411][muxorch] Catch error when checking active state of missing neighbor (2 days ago) [Kumaresh Perumal]
|\ 
| failure_prs.log skip_prs.log 0a1e41e - [muxorch] Catch error when checking active state of missing neighbor (3 days ago) [Nikola Dancejic]
* cc0255b - Update swss pipeline to use ubuntu 22.04 agent pool (sonic-net#3677) (3 days 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants