Page MenuHomeVyOS Platform
Create Task
Maniphest T4645

show nat source statistics lack argument --family
Closed, ResolvedPublicBUG
Actions

Description

show nat source statistics lack argument family
As we use in the commit argument --family for function def _get_raw_data_rules(direction, family):
We require this argument also for function nat.py show_statistics

vyos@r14:~$ show nat source statistics 
Traceback (most recent call last):
  File "/usr/libexec/vyos/op_mode/nat.py", line 201, in <module>
    res = vyos.opmode.run(sys.modules[__name__])
  File "/usr/lib/python3/dist-packages/vyos/opmode.py", line 145, in run
    res = func(**args)
  File "/usr/libexec/vyos/op_mode/nat.py", line 192, in show_statistics
    nat_statistics = _get_raw_data_rules(direction)
TypeError: _get_raw_data_rules() missing 1 required positional argument: 'family'
vyos@r14:~$

Details

Version
VyOS 1.4-rolling-202208220217
Is it a breaking change?
Perfectly compatible
Issue type
Bug (incorrect behavior)

Related Objects

Event Timeline

Viacheslav created this task.Aug 25 2022, 12:15 PM
Viacheslav changed the task status from Open to In progress.
Viacheslav claimed this task.
Viacheslav added a comment.Aug 25 2022, 12:32 PM

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

vyos@r14:~$ show nat source statistics 
Rule    Packets    Bytes    Interface
------  ---------  -------  -----------
100     1279       107896   eth0
120     1          60       eth1
vyos@r14:~$
Restricted Repository Identity mentioned this in rVYOSONEX6d82dab3c2e8: Merge pull request #1497 from sever-sever/T4645.Aug 25 2022, 4:55 PM
Viacheslav mentioned this in rVYOSONEX53bc8022e3be: op-mode: T4645: Show nat source stat missing argument --family.Aug 25 2022, 4:55 PM
Viacheslav closed this task as Resolved.Aug 29 2022, 6:21 AM
Viacheslav moved this task from Open to Finished on the VyOS 1.4 Sagitta board.
pasik subscribed.Oct 18 2022, 6:07 PM