pim(6): rewrite FRR PIM daemon configuration to get_config_dict() and add missing IGMP features
Closed, ResolvedPublicFEATURE REQUEST
Actions

Assigned To

Authored By

	c-po
	Nov 12 2023, 9:26 AM

Description

Remove the old way of how single config entries are retrieved using

conf.list_nodes() or config.exists() - switch to get_config_dict() instead which fetches the entire configuration at once.

In addition to the rewrite to get_config_dict() all the FRR PIM and UGMP options should be implemented.

Details

Version: -
Is it a breaking change?: Perfectly compatible
Issue type: Internal change (not visible to end users)

Related Objects
Search...

Status	Subtype	Assigned	Task
Resolved	FEATURE REQUEST	c-po	T5733 pim(6): rewrite FRR PIM daemon configuration to get_config_dict() and add missing IGMP features
Resolved	FEATURE REQUEST	c-po	T5595 Multicast - PIM bfd feature enable
Resolved	FEATURE REQUEST	c-po	T5736 igmp: migrate "protocols igmp" to "protocols pim"

Event Timeline

c-po created this task.Nov 12 2023, 9:26 AM

c-po changed the task status from Open to In progress.

c-po claimed this task.

pasik subscribed.Nov 12 2023, 9:34 AM

This also implements https://github.com/vyos/vyos-1x/pull/2411

c-po added a subtask: T5595: Multicast - PIM bfd feature enable .Nov 12 2023, 12:05 PM

c-po renamed this task from pim: rewrite FRR PIM daemon configuration to get_config_dict() to pim: rewrite FRR PIM daemon configuration to get_config_dict() and add missing IGMP features.Nov 12 2023, 1:50 PM

c-po updated the task description. (Show Details)

c-po mentioned this in T5595: Multicast - PIM bfd feature enable .Nov 12 2023, 2:04 PM

PR for VyOS 1.5 https://github.com/vyos/vyos-1x/pull/2476

c-po renamed this task from pim: rewrite FRR PIM daemon configuration to get_config_dict() and add missing IGMP features to pim(6): rewrite FRR PIM daemon configuration to get_config_dict() and add missing IGMP features.Nov 12 2023, 5:45 PM

Restricted Repository Identity mentioned this in rVYOSONEXe085f3e6c21a: Merge pull request #2476 from c-po/frr-pim-T5733.Nov 15 2023, 5:21 PM

c-po mentioned this in rVYOSONEX64b4cfc71d40: pim: T5733: incorporate feedback from peer review.Nov 15 2023, 5:21 PM

c-po mentioned this in rVYOSONEX403d2ffd6e46: pim6: T5733: add missing FRR PIM6 related features.

c-po mentioned this in rVYOSONEXdd13213ae94f: pim: T5733: fix CLI level of global PIM commands.

c-po mentioned this in rVYOSONEX45ea9ed72ee1: pim: T5733: rename watermark-warn -> watermark-warning.

c-po mentioned this in rVYOSONEXc5e2c25f8968: pim: T5733: split out XML definitions to be re-used by pim6.

c-po mentioned this in rVYOSONEX9abc02edcc23: pim: T5733: add missing FRR PIM related features.

Restricted Repository Identity mentioned this in rVYOSONEXa78037d0c3eb: pim: T5733: add missing FRR PIM related features.Nov 15 2023, 7:22 PM

Restricted Repository Identity mentioned this in rVYOSONEXe68d8d7b69d5: pim: T5733: split out XML definitions to be re-used by pim6.

Restricted Repository Identity mentioned this in rVYOSONEXf42ae2c900a3: pim: T5733: rename watermark-warn -> watermark-warning.

Restricted Repository Identity mentioned this in rVYOSONEXc1279a0faf64: pim: T5733: fix CLI level of global PIM commands.

Restricted Repository Identity mentioned this in rVYOSONEXb2d446963680: pim: T5733: incorporate feedback from peer review.