Skip to content

Fix buffer template for Arista SKU.#10663

Merged
sujinmkang merged 1 commit intosonic-net:masterfrom
ysmanman:fix-buffer-profile
May 2, 2022
Merged

Fix buffer template for Arista SKU.#10663
sujinmkang merged 1 commit intosonic-net:masterfrom
ysmanman:fix-buffer-profile

Conversation

@ysmanman
Copy link
Copy Markdown
Contributor

Why I did it

The buffer pool & profile setting in buffer template was not correct and caused the errors like the following:

ERR swss#orchagent: :- parseReference: malformed reference:[BUFFER_PROFILE|ingress_lossless_profile]. Must not be surrounded by [ ]

How I did it

Fix the buffer pool & profile setting by removing "[]".

How to verify it

Loaded image with this fix in a switch and made sure the error was not seen anymore.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@arlakshm
Copy link
Copy Markdown
Contributor

/Azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sujinmkang sujinmkang merged commit a9d5858 into sonic-net:master May 2, 2022
qiluo-msft added a commit that referenced this pull request May 3, 2022
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Jun 20, 2022
Why I did it
The buffer pool & profile setting in buffer template was not correct and caused the errors like the following:

ERR swss#orchagent: :- parseReference: malformed reference:[BUFFER_PROFILE|ingress_lossless_profile]. Must not be surrounded by [ ]

How I did it
Fix the buffer pool & profile setting by removing "[]".

How to verify it
Loaded image with this fix in a switch and made sure the error was not seen anymore.
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