I don’t think it currently is possible to disable syslog interval markers. Could this be added? Also I noticed some minor typos in the text:
Can be up to 86400.
vyos@vyos# set system syslog marker interval
Possible completions:
<1-65535> Time in seconds
And of course, it isn’t a port number:
vyos@vyos# set system syslog marker interval 0
Number is not in any of allowed ranges
Port number must be in range 1 to 86400
Value validation failed
Set failed
c-po
March 4, 2025, 1:09pm
2
Hi @jauling ,
thanks for mentioning the type. I have addressed both in
current ← syslog-typos-T6989
opened 01:07PM - 04 Mar 25 UTC
## Change summary
Fix CLI validator and error message for marker interval.
…
Add possibility to disable the mark module.
## Types of changes
- [x] Bug fix (non-breaking change which fixes an issue)
- [x] 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):
## Related Task(s)
* https://vyos.dev/T6989
## Related PR(s)
* https://github.com/vyos/vyos-1x/pull/4305
* https://github.com/vyos/vyos-documentation/pull/1604
## How to test / Smoketest result
```
vyos@vyos:~$ /usr/libexec/vyos/tests/smoke/cli/test_system_syslog.py
test_basic (__main__.TestRSYSLOGService.test_basic) ... ok
test_console (__main__.TestRSYSLOGService.test_console) ... ok
test_remote (__main__.TestRSYSLOGService.test_remote) ... ok
test_vrf_source_address (__main__.TestRSYSLOGService.test_vrf_source_address) ... ok
----------------------------------------------------------------------
Ran 4 tests in 10.833s
OK
```
## Checklist:
- [x] I have read the [**CONTRIBUTING**](https://github.com/vyos/vyos-1x/blob/current/CONTRIBUTING.md) document
- [x] I have linked this PR to one or more Phabricator Task(s)
- [x] I have run the components [**SMOKETESTS**](https://github.com/vyos/vyos-1x/tree/current/smoketest/scripts/cli) if applicable
- [x] My commit headlines contain a valid Task id
- [x] My change requires a change to the documentation
- [x] I have updated the documentation accordingly
1 Like
I noticed in the pull request that lines 23, 30, 34, and 49 are missing the first word ‘set’ in docs/configuration/system/syslog.rst
Thank you for notifying, will correct it.
1 Like
system
Closed
May 13, 2025, 11:05am
5
This topic was automatically closed 60 days after the last reply. New replies are no longer allowed.