ReproNim / reproman

ReproMan (AKA NICEMAN, AKA ReproNim TRD3)
https://reproman.readthedocs.io
Other
24 stars 14 forks source link

replace tracing /sbin/iptables with smth else (/bin/bash?) #423

Closed yarikoptic closed 5 years ago

yarikoptic commented 5 years ago

Since we are not tracking symlinks/alternatives (#102 ) ATM, tests started to fail for me locally due to

$> ls -l {/etc/alternatives,{/usr,}/sbin}/iptables 
lrwxrwxrwx 1 root root 25 Feb 23 10:58 /etc/alternatives/iptables -> /usr/sbin/iptables-legacy*
lrwxrwxrwx 1 root root 18 Dec 26 11:19 /sbin/iptables -> /usr/sbin/iptables*
lrwxrwxrwx 1 root root 26 Dec 26 11:19 /usr/sbin/iptables -> /etc/alternatives/iptables*

and then dpkg -S knowing only about the actual one

$> dpkg -S {/etc/alternatives,{/usr,}/sbin}/iptables /usr/sbin/iptables-legacy 
dpkg-query: no path found matching pattern /etc/alternatives/iptables
dpkg-query: no path found matching pattern /usr/sbin/iptables
dpkg-query: no path found matching pattern /sbin/iptables
iptables: /usr/sbin/iptables-legacy
kyleam commented 5 years ago

Following 6b495486d (BF(TST): /sbin/iptables is no longer belonging to a package - /bin/ed, 2019-04-11), these could be switched to /bin/ed, but that of course leads to gh-420. If we decide on something else that we can assume is present, we could change /bin/ed over to that and drop gh-420.

yarikoptic commented 5 years ago

oh, interesting -- so I guess that https://github.com/ReproNim/reproman/commit/6b495486dea3e2a6213066b75badd6d5752339ab didn't replace it thoroughly -- will do