diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-08-15 16:24:03 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-08-15 16:24:03 +0200 |
| commit | 0dd9037036c6b4909bfcb10c50b587a4456a9c27 (patch) | |
| tree | 54ed51e7b47e9c990b34cc774d8c08d6891304fd | |
| parent | c244c2e99519f9b453adf9a61d03488b09cb3ca2 (diff) | |
| parent | 1a62587a57d034681bd728b10b13a5ae5d7d2d4d (diff) | |
| download | vyos-1x-0dd9037036c6b4909bfcb10c50b587a4456a9c27.tar.gz vyos-1x-0dd9037036c6b4909bfcb10c50b587a4456a9c27.zip | |
Merge pull request #973 from sever-sever/T3702
pbr: T3702: Fix incorrect splits for fwmark
| -rwxr-xr-x | src/conf_mode/policy-local-route.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/conf_mode/policy-local-route.py b/src/conf_mode/policy-local-route.py index 0b7ceedeb..539189442 100755 --- a/src/conf_mode/policy-local-route.py +++ b/src/conf_mode/policy-local-route.py @@ -114,15 +114,15 @@ def apply(pbr): # Only fwmark in the rule # set policy local-route rule 101 fwmark '23' if 'fwmark' in pbr['rule'][rule] and not 'source' in pbr['rule'][rule]: - for fwmk in pbr['rule'][rule]['fwmark']: - call(f'ip rule add prio {rule} from all fwmark {fwmk} lookup {table}') + fwmk = pbr['rule'][rule]['fwmark'] + call(f'ip rule add prio {rule} from all fwmark {fwmk} lookup {table}') # Source and fwmark in the rule # set policy local-route rule 100 source '203.0.113.1' # set policy local-route rule 100 fwmark '23' if 'source' in pbr['rule'][rule] and 'fwmark' in pbr['rule'][rule]: + fwmk = pbr['rule'][rule]['fwmark'] for src in pbr['rule'][rule]['source']: - for fwmk in pbr['rule'][rule]['fwmark']: - call(f'ip rule add prio {rule} from {src} fwmark {fwmk} lookup {table}') + call(f'ip rule add prio {rule} from {src} fwmark {fwmk} lookup {table}') return None |