Skip to content

Conversation

@marek22k
Copy link

No description provided.

@marek22k marek22k changed the title Add rpf lookup for pim6 Draft. Add rpf lookup for pim6 Nov 28, 2025
@marek22k marek22k force-pushed the issue-20102 branch 2 times, most recently from 74c9672 to a5f4fdb Compare November 28, 2025 06:49
@marek22k marek22k marked this pull request as draft November 28, 2025 06:50
@marek22k marek22k changed the title Draft. Add rpf lookup for pim6 Add rpf lookup for pim6 Nov 28, 2025
@marek22k
Copy link
Author

I can't find the PIM6 tests. Are there no tests available for PIM6?

@marek22k marek22k marked this pull request as ready for review November 29, 2025 00:02
@donaldsharp
Copy link
Member

multicast_mld_join_topo1/test_multicast_mld_local_join.py:pytestmark = [pytest.mark.pim6d, pytest.mark.staticd]
multicast_pim6_sm_topo1/test_multicast_pim6_sm1.py:pytestmark = [pytest.mark.pim6d]
multicast_pim6_sm_topo1/test_multicast_pim6_sm2.py:pytestmark = [pytest.mark.pim6d]
multicast_pim6_static_rp_topo1/test_multicast_pim6_static_rp1.py:pytestmark = [pytest.mark.pim6d]
multicast_pim6_static_rp_topo1/test_multicast_pim6_static_rp2.py:pytestmark = [pytest.mark.pim6d]
pim_cand_rp_bsr/test_pim_cand_rp_bsr.py:    pytest.mark.pim6d,
pim_embedded_rp/test_pim_embedded_rp.py:pytestmark = [pytest.mark.pim6d]
sharpd@eva /w/h/s/f/t/topotests (master)>```

@donaldsharp
Copy link
Member

Can you also add the command to the doc/user/pim6.rst file?

@marek22k
Copy link
Author

multicast_pim6_sm_topo1

Must have been too stupid to see that - maybe I was too tired. Thanks for the answer.

@marek22k marek22k marked this pull request as draft December 3, 2025 15:54
@nabahr
Copy link
Contributor

nabahr commented Dec 3, 2025

I am curious to see the results from testing. I believe it should work with just adding the command that allows it to be configured in pim6. Let me know if you run into any issues. See the pim_mrib topotest that tests the IPv4 version.

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.

3 participants