Skip to content

Update DHCP_PACKET_MARK schema in state_db#9077

Merged
1 commit merged intosonic-net:masterfrom
trzhang11:mark_dhcp
Nov 2, 2021
Merged

Update DHCP_PACKET_MARK schema in state_db#9077
1 commit merged intosonic-net:masterfrom
trzhang11:mark_dhcp

Conversation

@ghost
Copy link

@ghost ghost commented Oct 27, 2021

Example output of DHCP_PACKET_MARK table in state_db:

state_db_mark

@prsunny
Copy link
Contributor

prsunny commented Oct 27, 2021

Is this a bug from #9015? I think, its a test gap and need some unit-tests

@prsunny
Copy link
Contributor

prsunny commented Oct 27, 2021

Please provide proper description following the template

@ghost
Copy link
Author

ghost commented Oct 27, 2021

Is this a bug from #9015? I think, its a test gap and need some unit-tests

No bug here, just change the schema.

@ghost
Copy link
Author

ghost commented Oct 27, 2021

Please provide proper description following the template

for simple changes, i don't use the template which contains too much unnecessary description.

@ghost ghost merged commit 689c101 into sonic-net:master Nov 2, 2021
@qiluo-msft
Copy link
Collaborator

qiluo-msft commented Nov 5, 2021

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

@ghost
Copy link
Author

ghost commented Nov 8, 2021

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

I don't see mux docker template file in 202012. Please solve that problem first. "mark_dhcp_packet" is a service inside mux container.

@ghost
Copy link
Author

ghost commented Nov 12, 2021

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

Please check if this pr now can be cherry-picked. If not, I will submit another.

qiluo-msft pushed a commit that referenced this pull request Nov 15, 2021
- update DHCP_PACKET_MARK schema in state_db
- this is an update over PR: Add service mark_dhcp_packet to mux container #9015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants