Skip to content
New issue

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

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

Already on GitHub? # to your account

[Centec ARM64]Upgrade Centec syncd docker to buster and Enable Telemetry on ARM64 #6386

Merged
merged 2 commits into from
Jan 9, 2021

Conversation

guxianghong
Copy link
Contributor

@guxianghong guxianghong commented Jan 8, 2021

- Why I did it

  1. Enable telemetry by default for ARM64 arch
  2. Upgrade Centec syncd docker to buster

- How I did it

  1. Remove the hardcoded disable of telemetry in slave.mk for ARM64 architecture
  2. Use docker-config-engine-buster for syncd docker
  3. Remove libjemalloc1 from Dockerfile.j2 for Centec ARM64 syncd docker . libjemalloc1 is replaced by libjemalloc2 in buster, and libjemalloc2 have been installed in docker-base-buster

- How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

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

Gu Xianghong added 2 commits January 8, 2021 02:59
… installed in docker-base-buster, remove libjemalloc1 from docker-syncd-centec's Dockerfile.j2
@guxianghong guxianghong changed the title Sonic master 202101070930 [Centec ARM64]Upgrade Centec syncd docker to buster and Enable Telemetry on ARM64 Jan 8, 2021
@lguohan lguohan merged commit c64052b into sonic-net:master Jan 9, 2021
lguohan pushed a commit that referenced this pull request Jan 9, 2021
…try on ARM64 (#6386)

* Enable telemetry for ARM64 by default

* [Centec]Upgrade Centec syncd docker to buster; libjemalloc2 have been installed in docker-base-buster, remove libjemalloc1 from docker-syncd-centec's Dockerfile.j2

Co-authored-by: Gu Xianghong <xgu@centecnetworks.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants