Skip to content
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.

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

Add buffers_defaults_t1 for Arista-7260CX3-D108C8 #18845

Merged
merged 1 commit into from
May 24, 2024

Conversation

andywongarista
Copy link
Contributor

Why I did it

Add symlink so that config-setup passes

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • 202311

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

This symlink is required else config-setup fails
@lguohan
Copy link
Collaborator

lguohan commented May 19, 2024

@prsunny , can you take a look at this one?

@lguohan lguohan requested a review from prsunny May 19, 2024 06:52
Copy link
Contributor

@prsunny prsunny left a comment

Choose a reason for hiding this comment

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

lgtm. @neethajohn

@bingwang-ms
Copy link
Contributor

@andywongarista Please file a direct PR to 2022405 to address the conflict.

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #20061

yxieca pushed a commit that referenced this pull request Aug 28, 2024
This symlink is required else config-setup fails

Co-authored-by: andywongarista <78833093+andywongarista@users.noreply.github.com>
@bingwang-ms
Copy link
Contributor

@andywongarista Please file a direct PR to 2022405 to address the conflict.

@andywongarista
Copy link
Contributor Author

@andywongarista Please file a direct PR to 2022405 to address the conflict.

This PR is already in 202405 (it was already in master when 202405 branched off).

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.

6 participants