[minigraph-gen]Find port alias column by header name if exist#262
Merged
maggiemsft merged 2 commits intosonic-net:masterfrom Oct 2, 2017
Merged
[minigraph-gen]Find port alias column by header name if exist#262maggiemsft merged 2 commits intosonic-net:masterfrom
maggiemsft merged 2 commits intosonic-net:masterfrom
Conversation
stcheng
approved these changes
Sep 11, 2017
ansible/library/port_alias.py
Outdated
| else: | ||
| self.portmap.append(mapping[2]) | ||
| index = 0 | ||
| if re.match('^#', lines[0].strip()): |
Contributor
There was a problem hiding this comment.
could you refer to this pull request and store all the information in the port_config.ini file accordingly?
Contributor
Author
There was a problem hiding this comment.
when building docker-sonic-mgmt, it will install the device package which include all port_config.ini files to the correct location in docker-sonic-mgmt. Is this what you want?
Contributor
There was a problem hiding this comment.
sorry I didn't add the link to my previous comment: https://github.com/Azure/sonic-swss/pull/287/files this pull request has a better logic to parse the port_config.ini file.
stcheng
approved these changes
Oct 2, 2017
Contributor
|
I think the title of this PR would be more accurate as, "[minigraph-gen] Find port alias column by header name if exist" |
maggiemsft
changed the title
sdszhang
pushed a commit
to sdszhang/sonic-mgmt
that referenced
this pull request
May 14, 2025
…net#262) Cherry-pick PR for sonic-net#17861 in sonic-mgmt. Summary: This is to add a test case to verify that after enabling proxy_arp, there is no L2 flooding in downstream VLAN even though the switch does not have a FDB entry to forward a downstream SRv6 packet.
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…atically (sonic-net#16576) #### Why I did it src/sonic-snmpagent ``` * 64f0def - (HEAD -> 202012, origin/202012) Support interface speed for PortChannels (sonic-net#262) (2 days ago) [Lukas Stockner] ```
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…atically (sonic-net#16877) src/sonic-snmpagent * 17a8bb2 - (HEAD -> 202205, origin/202205) Add ifhighspeed UT (sonic-net#296) (5 minutes ago) [Hua Liu] * b5a52ff - Fix key missing exception when invalied transiver info in STATE_DB (sonic-net#289) (5 minutes ago) [Hua Liu] * 09bb0c2 - Fix FdbUpdater crash when SAI_FDB_ENTRY_ATTR_BRIDGE_PORT_ID attribute missing. (sonic-net#286) (5 minutes ago) [Hua Liu] * 792e403 - Support interface speed for PortChannels (sonic-net#262) (5 minutes ago) [Lukas Stockner]
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…verlay SNMP (sonic-net#2721) * [snmpagent][swss-common] advance sub module head to include overlay SNMP Submodule src/sonic-snmpagent 0f2bbd7..26f0495: > Remove verbose feature missing logs (sonic-net#102) > Enable overriding interface counters OIDs (sonic-net#98) Submodule src/sonic-swss-common 5f4abd9..36fd5e9: > Add new DB index for SNMP_OVERLAY_DB (sonic-net#262) Note: overlay DB also requires change in swss-common, which has been moved ahead. Signed-off-by: Ying Xie <ying.xie@microsoft.com> * [utilities] advance utilities sub-module head Submodule src/sonic-utilities 9d9aaa0..d1070b2: > [warm-reboot] initialize warm reboot state table before warm rebooting (sonic-net#492) > Allow config shutdown and startup operations on valid PortChannel interface names (sonic-net#474)
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
8b9cab7 2023-10-26 [201911] Fix IfHighSpeed UT issue on 201911 (sonic-net#299) 622b771 2023-10-13 | Fix backup port rfc2863 UT to 202012 branch issue (sonic-net#298) [Hua Liu] fa94798 2023-10-11 | Add ifhighspeed UT (sonic-net#296) [Hua Liu] 41789ca 2023-09-14 | Support interface speed for PortChannels (sonic-net#262) [Lukas Stockner]
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
5 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.
We are adding more columns in port_config.ini file, line port speed, and the location of alias name may change.