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

[Barefoot] [202006] Fix default profile for Newport #6308

Open
wants to merge 2 commits into
base: 202006
Choose a base branch
from

Conversation

msosyak
Copy link
Contributor

@msosyak msosyak commented Dec 28, 2020

- Why I did it
To fix the default profile used for that platform

- How I did it
Cherry-pick fixes from master

- How to verify it

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

…orm/ (sonic-net#5585)

Fix symlinks and syncd.conf for Newport in platform/ directory

Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>
@msosyak
Copy link
Contributor Author

msosyak commented Dec 28, 2020

@lguohan Please merge this and same for 201911 #6307

@msosyak
Copy link
Contributor Author

msosyak commented Dec 29, 2020

retest this please

@msosyak
Copy link
Contributor Author

msosyak commented Dec 29, 2020

Looks like CI is broken. All builds failed with:
Unable to find image 'sonic-slave-jessie-johnar:48b2b7de4f6' locally

@msosyak
Copy link
Contributor Author

msosyak commented Jan 4, 2021

retest this please

1 similar comment
@msosyak
Copy link
Contributor Author

msosyak commented Jan 5, 2021

retest this please

@msosyak
Copy link
Contributor Author

msosyak commented Jan 5, 2021

@lguohan @jleveque Could we marge this?

@jleveque
Copy link
Contributor

jleveque commented Jan 5, 2021

Retest this please

1 similar comment
@msosyak
Copy link
Contributor Author

msosyak commented Jan 11, 2021

Retest this please

@msosyak
Copy link
Contributor Author

msosyak commented Jan 18, 2021

@lguohan @jleveque Could you have a look at the CI error?

@msosyak msosyak force-pushed the myron/y_fix_202006 branch from 69d734b to 73e28f9 Compare March 29, 2022 12:22
Y* profile is the name pattern for p4 programs that developed for the current platform. The difference between them is features enabled and resource reservation.

For this platform, it is expected to work on any Y profile. but after the latest changes, the first Y profile is always used.

Signed-off-by: Myron Sosyak <myronx.sosyak@intel.com>
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