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

Update sonic-host-services submodule #12635

Merged
merged 1 commit into from
Nov 8, 2022

Conversation

judyjoseph
Copy link
Contributor

@judyjoseph judyjoseph commented Nov 8, 2022

Why I did it

Update sonic-host-services submodule to include PR: sonic-net/sonic-host-services#19

The following commits are taken in.

6eac2d3 Merge pull request #19 from judyjoseph/macsec_feature_enable
31c6108 Sync has_per_asic_scope attribute to config_db in all namespaces for multi-asic
185547f Add UT to improve coverage, for namespace config update
27bdc02 Merge branch 'master' into macsec_feature_enable
7c3aca0 macsec_supported info is part of DEVICE_RUNTIME_METADATA itself.

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

6eac2d3 (HEAD -> master, origin/master, origin/HEAD) Merge pull request sonic-net#19
Copy link
Contributor

@abdosi abdosi left a comment

Choose a reason for hiding this comment

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

Can you add commit history in PR description

@judyjoseph
Copy link
Contributor Author

Can you add commit history in PR description

Done

@judyjoseph judyjoseph merged commit 1320319 into sonic-net:master Nov 8, 2022
@judyjoseph judyjoseph deleted the sonic_host_update branch November 29, 2022 18:36
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.

2 participants