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

CP-23093: Fill new field 'igmp_snooping' with default value #3153

Merged
merged 1 commit into from
Aug 1, 2017

Conversation

fillzero
Copy link
Contributor

Signed-off-by: Liang Dai liang.dai1@citrix.com

Signed-off-by: Liang Dai <liang.dai1@citrix.com>
@fillzero
Copy link
Contributor Author

fillzero commented Jul 31, 2017

@robhoes
As mentioned on xapi-project/xcp-networkd#107, this PR just fill new field 'igmp_snooping' with default value. Regarding IGMP Snooping toggle implementation, @YarsinCitrix will send PR later.

Thanks.

Copy @huizh to be aware.

@huizh huizh requested a review from robhoes July 31, 2017 07:31
@huizh
Copy link
Contributor

huizh commented Jul 31, 2017

Thanks @fillzero.

Hi @robhoes, please could you take a look at this and see if we can merge this along with xapi-project/xcp-idl#164 and xapi-project/xcp-networkd#107?
Many thanks.

@robhoes robhoes merged commit 94fa6e9 into xapi-project:multicast Aug 1, 2017
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.

None yet

3 participants