Skip to content

Update cisco-8000-smartswitch.ini #23746

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kperumalbfn merged 1 commit into from
Aug 19, 2025
Merged

Update cisco-8000-smartswitch.ini #23746

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion platform/checkout/cisco-8000-smartswitch.ini
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
[module]
repo=git@github.com:Cisco-8000-sonic/platform-cisco-8000.git
ref=202411.1.0.12
Copy link
Contributor

@kperumalbfn kperumalbfn Aug 18, 2025

Choose a reason for hiding this comment

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

@Aravind-Subbaroyan Is this change only for Smartswitch or required for other SKUs?

Copy link
Contributor Author

@Aravind-Subbaroyan Aravind-Subbaroyan Aug 18, 2025

Choose a reason for hiding this comment

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

@kperumalbfn - This code drop validated only Smartswitch.

Copy link
Contributor

@rameshraghupathy rameshraghupathy Aug 18, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

@kperumalbfn Though the release is common, the other platform tests weren't done at this point and is qualified only on smartswitch topo, so as of now it applies only for smartswitch.

Copy link
Contributor

@prabhataravind prabhataravind Aug 18, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

@rameshraghupathy @Aravind-Subbaroyan I thought non-smartswitch is still using 202411.1.0.12 (as per platform/checkout/cisco-8000.ini), so this change should not impact non-smartswitch right? Also, can you confirm if the port naming issue is applicable for non-smartswitch to begin with?

Copy link
Contributor

@kperumalbfn kperumalbfn Aug 18, 2025

Choose a reason for hiding this comment

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

@kevinskwang for viz

Copy link
Contributor

@rameshraghupathy rameshraghupathy Aug 18, 2025

Choose a reason for hiding this comment

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

@kperumalbfn This fix is applicable only for smartswitch and other cisco 8k platforms don't need this change.

ref=202411.1.0.13

Loading