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

topotests: avoid console errors if no mpls - do not merge #15736

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vjardin
Copy link
Contributor

@vjardin vjardin commented Apr 13, 2024

On some environments, when running topotests, we get:

modprobe: FATAL: Module mpls-router not found in directory /lib/modules/6.5.0-1017-azure 
  ...
/usr/bin/nsenter --mount=/proc/27565/ns/mnt --net=/proc/27565/ns/net --uts=/proc/27565/ns/uts -F /bin/bash -c echo 100000 > /proc/sys/net/mpls/platform_labels
/proc/sys/net/mpls/platform_labels: No such file or directory

@ton31337
Copy link
Member

Seems this is failing for all the tests that do not have ldpd as a daemon, while hasmpls is True when ldpd is used, right?

@vjardin vjardin changed the title topotests: avoid console errors if no mpls topotests: avoid console errors if no mpls - do not merge Apr 26, 2024
@vjardin vjardin self-assigned this Apr 26, 2024
@vjardin vjardin marked this pull request as draft April 26, 2024 09:29
@vjardin vjardin force-pushed the mpls_topotests branch 2 times, most recently from 5e54d0b to d62fe39 Compare April 30, 2024 05:50
On some environments, when running topotests, we get:
modprobe: FATAL: Module mpls-router not found in directory /lib/modules/6.5.0-1017-azure
...
/usr/bin/nsenter --mount=/proc/27565/ns/mnt --net=/proc/27565/ns/net --uts=/proc/27565/ns/uts -F /bin/bash -c echo 100000 > /proc/sys/net/mpls/platform_labels
/proc/sys/net/mpls/platform_labels: No such file or directory

Signed-off-by: Vincent Jardin <[email protected]>
Please, ignore it, currently it is only for a comparaison of traces.

Signed-off-by: Vincent Jardin <[email protected]>
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.

None yet

2 participants