validators: T6739: fix ipaddrcheck argument quoting

[dmbaturin]

Change Summary

Properly quote ipaddrcheck arguments in validator scripts to ensure they are handled correctly if they contain whitespace. Right now such arguments are correctly rejected but result in confusing error messages that contain ipaddrcheck's help:

vyos@vyos# set interfaces ethernet eth1 address "a valid IP address, I swear!"

  Error: wrong number of arguments, one argument required!
  Usage: ipaddrcheck <OPTIONS> [STRING]
  Address checking options:
    --is-valid                 Check if STRING is a valid IPv4 or IPv6 address
                                 with or without prefix length
    --is-any-cidr              Check if STRING is a valid IPv4 or IPv6 address
                                 with prefix length

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Code style update (formatting, renaming)
• Refactoring (no functional changes)
• Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
• Other (please describe):

Component(s) name

Validators.

How to test

Smoketest result

Checklist:

• I have read the CONTRIBUTING document
• I have linked this PR to one or more Phabricator Task(s)
• I have run the components SMOKETESTS if applicable
• My commit headlines contain a valid Task id
• My change requires a change to the documentation
• I have updated the documentation accordingly

[github-actions]

👍
No issues in PR Title / Commit Title

[github-actions]

✅ No issues found in unused-imports check.. Please refer the workflow run

[c-po]

@Mergifyio backport circinus sagitta

[mergify]

backport circinus sagitta

✅ Backports have been created

• #4100 validators: T6739: fix ipaddrcheck argument quoting (backport #4099) has been created for branch circinus
• #4101 validators: T6739: fix ipaddrcheck argument quoting (backport #4099) has been created for branch sagitta

[c-po]

@dmbaturin smoketests report:

DEBUG - ======================================================================
DEBUG - ERROR: test_cgnat (__main__.TestCGNAT.test_cgnat)
DEBUG - ----------------------------------------------------------------------
DEBUG - Traceback (most recent call last):
DEBUG -   File "/usr/libexec/vyos/tests/smoke/cli/test_cgnat.py", line 73, in test_cgnat
DEBUG -     self.cli_set(base_path + ['pool', 'external', external_name, 'range', external_net])
DEBUG -   File "/usr/libexec/vyos/tests/smoke/cli/base_vyostest_shim.py", line 71, in cli_set
DEBUG -     self._session.set(config)
DEBUG -   File "/usr/lib/python3/dist-packages/vyos/configsession.py", line 161, in set
DEBUG -     self.__run_command([SET] + path + value)
DEBUG -   File "/usr/lib/python3/dist-packages/vyos/configsession.py", line 150, in __run_command
DEBUG -     raise ConfigSessionError(output)
DEBUG - vyos.configsession.ConfigSessionError: Error: 192.0.2.1-192.0.2.2 is not a valid IPv4 address range
DEBUG -
DEBUG - Error: 192.0.2.1-192.0.2.2 is not a valid IPv4 host
DEBUG -
DEBUG - Error: 192.0.2.1-192.0.2.2 is not a valid IPv4 prefix

and it was missed here https://github.com/vyos/vyos-1x/actions/runs/11036683938/job/30656416756#step:5:1522