Orchagent changes needed to support single ASIC VOQ Fixed-System#3847
Merged
prsunny merged 45 commits intosonic-net:masterfrom Nov 21, 2025
Merged
Orchagent changes needed to support single ASIC VOQ Fixed-System#3847prsunny merged 45 commits intosonic-net:masterfrom
prsunny merged 45 commits intosonic-net:masterfrom
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
lakshmi-nexthop
changed the title
Contributor
|
Please add UT for code-coverage. |
prsunny
reviewed
Oct 7, 2025
prsunny
reviewed
Oct 7, 2025
prsunny
reviewed
Oct 7, 2025
Collaborator
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
Collaborator
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
Collaborator
|
/azp run |
prsunny
approved these changes
Nov 21, 2025
yijingyan2
added a commit
to yijingyan2/sonic-swss
that referenced
this pull request
Dec 3, 2025
…tem (sonic-net#3847)" This reverts commit 5671e08.
saravanan-nexthop
pushed a commit
to saravanan-nexthop/sonic-swss
that referenced
this pull request
Dec 10, 2025
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality
saravanan-nexthop
pushed a commit
to saravanan-nexthop/sonic-swss
that referenced
this pull request
Dec 10, 2025
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality Signed-off-by: Lakshmi <[email protected]>
saravanan-nexthop
pushed a commit
to saravanan-nexthop/sonic-swss
that referenced
this pull request
Dec 10, 2025
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality Signed-off-by: Lakshmi <[email protected]> Signed-off-by: Saravanan <[email protected]>
kalash-nexthop
pushed a commit
to kalash-nexthop/sonic-swss
that referenced
this pull request
Dec 16, 2025
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality Signed-off-by: Kalash Nainwal <[email protected]>
kalash-nexthop
pushed a commit
to kalash-nexthop/sonic-swss
that referenced
this pull request
Dec 16, 2025
…tem (sonic-net#3847)" (sonic-net#4035) This reverts commit 5671e08. Signed-off-by: Kalash Nainwal <[email protected]>
saravanan-nexthop
pushed a commit
to saravanan-nexthop/sonic-swss
that referenced
this pull request
Dec 23, 2025
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality Signed-off-by: Lakshmi <[email protected]> Signed-off-by: Saravanan <[email protected]> Signed-off-by: Saravanan Sellappa <[email protected]>
prsunny
pushed a commit
that referenced
this pull request
Jan 5, 2026
What I did Orchagent changes needed to support single ASIC VOQ Fixed-System (#3847) was reverted via (#4035) because vms-kvm-t2 tests were failing. That was because the isChassisAppDbPresent() was using an incorrect path to database_config.json This PR brings back the original #3847 and fixes the isChassisAppDbPresent() to use the correct path. Why I did it Single-asic VOQ systems do need #3847
mssonicbld
added a commit
to mssonicbld/sonic-swss
that referenced
this pull request
Jan 5, 2026
<!-- Please make sure you have read and understood the contribution guildlines: https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md 1. Make sure your commit includes a signature generted with `git commit -s` 2. Make sure your commit title follows the correct format: [component]: description 3. Make sure your commit message contains enough details about the change and related tests 4. Make sure your pull request adds related reviewers, asignees, labels Please also provide the following information in this pull request: --> **What I did** Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) was reverted via (sonic-net#4035) because vms-kvm-t2 tests were failing. That was because the `isChassisAppDbPresent()` was using an incorrect path to `database_config.json` This PR brings back the original sonic-net#3847 and fixes the `isChassisAppDbPresent()` to use the correct path. **Why I did it** Single-asic VOQ systems do need sonic-net#3847 **How I verified it** Tests on vms-kvm-t2 and also single-asic voq systems Verified on vms-kvm-t2 the test_bgp_fact is passing ``` bgp/test_bgp_fact.py::test_bgp_facts[vlab-t2-1-1-0] PASSED [ 25%] bgp/test_bgp_fact.py::test_bgp_facts[vlab-t2-1-1-1] PASSED [ 50%] bgp/test_bgp_fact.py::test_bgp_facts[lvlab-t2-1-2-0] PASSED [ 75%] bgp/test_bgp_fact.py::test_bgp_facts[vlab-t2-1-2-1] PASSED [100%] ``` **Details if related**
mssonicbld
added a commit
that referenced
this pull request
Jan 5, 2026
<!-- Please make sure you have read and understood the contribution guildlines: https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md 1. Make sure your commit includes a signature generted with `git commit -s` 2. Make sure your commit title follows the correct format: [component]: description 3. Make sure your commit message contains enough details about the change and related tests 4. Make sure your pull request adds related reviewers, asignees, labels Please also provide the following information in this pull request: --> **What I did** Orchagent changes needed to support single ASIC VOQ Fixed-System (#3847) was reverted via (#4035) because vms-kvm-t2 tests were failing. That was because the `isChassisAppDbPresent()` was using an incorrect path to `database_config.json` This PR brings back the original #3847 and fixes the `isChassisAppDbPresent()` to use the correct path. **Why I did it** Single-asic VOQ systems do need #3847 **How I verified it** Tests on vms-kvm-t2 and also single-asic voq systems Verified on vms-kvm-t2 the test_bgp_fact is passing ``` bgp/test_bgp_fact.py::test_bgp_facts[vlab-t2-1-1-0] PASSED [ 25%] bgp/test_bgp_fact.py::test_bgp_facts[vlab-t2-1-1-1] PASSED [ 50%] bgp/test_bgp_fact.py::test_bgp_facts[lvlab-t2-1-2-0] PASSED [ 75%] bgp/test_bgp_fact.py::test_bgp_facts[vlab-t2-1-2-1] PASSED [100%] ``` **Details if related**
Pterosaur
pushed a commit
to Janetxxx/sonic-swss
that referenced
this pull request
Jan 6, 2026
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality
Pterosaur
pushed a commit
to Janetxxx/sonic-swss
that referenced
this pull request
Jan 6, 2026
…tem (sonic-net#3847)" (sonic-net#4035) This reverts commit 5671e08.
Pterosaur
pushed a commit
to Janetxxx/sonic-swss
that referenced
this pull request
Jan 6, 2026
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality
Pterosaur
pushed a commit
to Janetxxx/sonic-swss
that referenced
this pull request
Jan 6, 2026
…tem (sonic-net#3847)" (sonic-net#4035) This reverts commit 5671e08.
yehjunying
pushed a commit
to yehjunying/sonic-swss
that referenced
this pull request
Jan 16, 2026
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality
arpit-nexthop
pushed a commit
to nexthop-ai/sonic-swss
that referenced
this pull request
Jan 21, 2026
What I did Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) was reverted via (sonic-net#4035) because vms-kvm-t2 tests were failing. That was because the isChassisAppDbPresent() was using an incorrect path to database_config.json This PR brings back the original sonic-net#3847 and fixes the isChassisAppDbPresent() to use the correct path. Why I did it Single-asic VOQ systems do need sonic-net#3847
ganglyu
pushed a commit
to ganglyu/sonic-swss
that referenced
this pull request
Jan 26, 2026
What I did Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) was reverted via (sonic-net#4035) because vms-kvm-t2 tests were failing. That was because the isChassisAppDbPresent() was using an incorrect path to database_config.json This PR brings back the original sonic-net#3847 and fixes the isChassisAppDbPresent() to use the correct path. Why I did it Single-asic VOQ systems do need sonic-net#3847 Signed-off-by: ganglyu <[email protected]>
theasianpianist
pushed a commit
to theasianpianist/sonic-swss
that referenced
this pull request
Feb 4, 2026
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality Signed-off-by: Lawrence Lee <[email protected]>
baorliu
pushed a commit
to baorliu/sonic-swss
that referenced
this pull request
Feb 23, 2026
…ic-net#3847) What I did Portorch, Neighorch, Intfsorch are updated to not access chassis app DB. Chassis app DB is accessed only if the chassisdb.conf is present indicating its a real chassis (not a FS VOQ) For lag creation, system lags are created as the switch is a VOQ swtich. Why I did it For a fixed system, there is no chassis DB present How I verified it Ran sonic-mgmt tests to verify BGP, LAG, functionality Signed-off-by: Baorong Liu <[email protected]>
baorliu
pushed a commit
to baorliu/sonic-swss
that referenced
this pull request
Feb 23, 2026
…tem (sonic-net#3847)" (sonic-net#4035) This reverts commit 5671e08. Signed-off-by: Baorong Liu <[email protected]>
baorliu
pushed a commit
to baorliu/sonic-swss
that referenced
this pull request
Feb 23, 2026
What I did Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) was reverted via (sonic-net#4035) because vms-kvm-t2 tests were failing. That was because the isChassisAppDbPresent() was using an incorrect path to database_config.json This PR brings back the original sonic-net#3847 and fixes the isChassisAppDbPresent() to use the correct path. Why I did it Single-asic VOQ systems do need sonic-net#3847 Signed-off-by: Baorong Liu <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What I did
Why I did it
How I verified it
Details if related