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.

Sign up for GitHub

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

Jump to bottom

[action] [PR:13822] [dhcp_relay] Remove exist check while adding dhcpv6 relay #13830

Merged
merged 1 commit into from
Feb 17, 2023
Merged

[action] [PR:13822] [dhcp_relay] Remove exist check while adding dhcpv6 relay #13830

merged 1 commit into from
Feb 17, 2023

Conversation

mssonicbld
Copy link
Collaborator

No description provided.

@yaqiangz @mssonicbld
[dhcp_relay] Remove exist check while adding dhcpv6 relay (sonic-net#…
256d20e 
…13822)

Why I did it
DHCPv6 relay config entry is not useful while del dhcpv6 relay config.

How I did it
Remove dhcpv6_relay entry if it is empty and not check entry exist while adding dhcpv6 relay
@mssonicbld mssonicbld requested a review from lguohan as a code owner February 15, 2023 18:24
@mssonicbld mssonicbld mentioned this pull request Feb 15, 2023
Merged
8 tasks
@mssonicbld
Copy link
Collaborator Author

Original PR: #13822

@yaqiangz
Copy link
Contributor 

The following packages have unmet dependencies:
 libssl-dev : Depends: libssl1.1 (= 1.1.1n-0+deb11u4) but 1.1.1n-0+deb11u3 is to be installed
E: Unable to correct problems, you have held broken packages.

@yxieca The failure is not related with this PR's code change, and I have tried re-run, could you give any suggestions how to proceed?

@yaqiangz
Copy link
Contributor

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 928aad1 into sonic-net:202205 Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

Assignees
No one assigned
Labels
automerge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mssonicbld @yaqiangz