Page MenuHomeVyOS Platform
Create Task
Maniphest T6431

monitor traceroute broken VRF support
Closed, ResolvedPublicBUG
Actions

Description

You cannot set the VRF with monitor traceroute, because the CLI parser does not accept an additional argument after the VRF keyword

admin@vyos:~$ monitor traceroute 1.1.1.1 vrf 
Possible completions:
  <Enter>               Execute the current command

      
admin@vyos:~$ monitor traceroute 1.1.1.1 vrf 
mtr: missing argument for vrf option
admin@vyos:~$ monitor traceroute 1.1.1.1 vrf MWN

  Invalid command: monitor traceroute 1.1.1.1 vrf [MWN]

it looks like monitor traceroute is just an alias for mtr these days, where setting the VRF does work fine

admin@vyos:~$ mtr 1.1.1.1 vrf MWN

Details

Version
1.5-rolling-202405310019
Is it a breaking change?
Perfectly compatible
Issue type
Bug (incorrect behavior)

Related Objects

Event Timeline

bernhardschmidt created this task.May 31 2024, 8:48 PM
pasik subscribed.May 31 2024, 9:02 PM
Viacheslav triaged this task as Normal priority.Jun 1 2024, 11:16 AM
talmakion subscribed.Jun 4 2024, 10:35 AM

PR created: https://github.com/vyos/vyos-1x/pull/3582

That was a fun one to figure out.

Restricted Repository Identity mentioned this in rVYOSONEX184da97efbc1: Merge pull request #3582 from talmakion/bugfix/T6431.Jun 5 2024, 7:02 AM
Restricted Repository Identity mentioned this in rVYOSONEXde1479b06cb9: T6431: op-mode command monitor traceroute missing recursive symlink.Jun 5 2024, 7:02 AM
Restricted Repository Identity mentioned this in rVYOSONEXe4d1d4a319ee: T6431: op-mode command monitor traceroute missing recursive symlink.Jun 5 2024, 7:45 AM
Restricted Repository Identity mentioned this in rVYOSONEX170901bdcae6: Merge pull request #3583 from vyos/mergify/bp/sagitta/pr-3582.Jun 5 2024, 3:14 PM
Viacheslav closed this task as Resolved.EditedJun 5 2024, 3:36 PM
Viacheslav assigned this task to talmakion.
Viacheslav added a project: VyOS 1.4 Sagitta (1.4.1).
Viacheslav moved this task from Open to Finished on the VyOS 1.5 Circinus board.
Viacheslav subscribed.

@talmakion Good catch! Thanks for contributing!

Viacheslav moved this task from Backlog to Finished on the VyOS 1.4 Sagitta (1.4.1) board.Jun 5 2024, 3:36 PM
dmbaturin removed a project: VyOS 1.5 Circinus.Jul 1 2024, 2:23 PM
dmbaturin edited a custom field.
dmbaturin changed Is it a breaking change? from Unspecified (possibly destroys the router) to Perfectly compatible.