VyOS command definitions, scripts, and utilities (mirror of https://github.com/vyos/vyos-1x.git) https://git.amelek.net/vyos/vyos-1x.git/atom?h=feature%2FT9082-codeql-cpp 2025-01-24T06:29:30+00:00 2025-01-24T06:29:30+00:00 Christian Breunig christian@breunig.cc 2025-01-24T06:29:30+00:00 urn:sha1:2fb52485d412cc3874e544f6b0011a51af28b897 Change some values from the defaults in the used smoketest config for basic-vyos-no-ntp to make it a litte bit more interesting for the underlaying config load subsystem. 2025-01-23T20:12:16+00:00 Christian Breunig christian@breunig.cc 2025-01-23T19:46:59+00:00 urn:sha1:12616b3da02600a27922dc1e275886c3b80ca1f8 With T3008 we moved from ntpd to chrony. This came with a restructuring of the CLI (mainly moving ntp out of system to services). In addition the definition of a server was made mandatory. The bug itself manifests at a more crucial point - config migration vyos-router[1265]: Migration script error: /opt/vyatta/etc/config-migrate/migrate/ntp/1-to-2: [Errno 1] failed to run command: ['/opt/vyatta/etc/config-migrate/migrate/ntp/1-to-2', '/opt/vyatta/etc/config/config.boot'] vyos-router[1265]: returned: - op: copy old_path: ['system', 'ntp'] new_path: ['service', 'ntp'] vyos-router[1265]: - op: delete path: ['system', 'ntp'] The fix is that we will no longer migrate an empty ntp CLI node from the old syntax to the new.