Output for a standard check with section ['nat'] and for the test case above, with debug logging on; changes are correctly mirrored on secondary node:

Mar 23 2024, 2:03 AM · VyOS 1.4 Sagitta (1.4.0-epa3)

Mar 22 2024

jestabro added a comment to T6145: Service config-sync does not rely on priorities.

Note that the subtask T6146 is useful in itself, as a modernization of the legacy tool priority.pl, but manual ordering of config-sync sections to address this issue is fragile, given the subtleties (== design constraints) of the legacy commit algorithm (cf. T5492). It is preferable to hand the full commit proposal of the primary node to the secondary node, and allow the underlying configsession to manage priority ordering --- this may be easy to do with a simple refactor in the post-commit hook and (if needed) adjustment in the http-api request handler.

Mar 22 2024, 2:58 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro claimed T6145: Service config-sync does not rely on priorities.

Mar 22 2024, 2:50 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro closed T6146: Add python script to get all priorities of service or section from XML, a subtask of T6145: Service config-sync does not rely on priorities, as Unknown Status.

Mar 22 2024, 12:56 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro closed T6146: Add python script to get all priorities of service or section from XML as Unknown Status.

Mar 22 2024, 12:56 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro committed rVYOSONEXe915900bfec8: xml: T6146: add utils and helper to provide priority data.

Mar 22 2024, 10:33 AM

Mar 21 2024

jestabro closed T6149: Update node_data when merging nodes in reference tree generation, a subtask of T6146: Add python script to get all priorities of service or section from XML, as Unknown Status.

Mar 21 2024, 8:46 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro closed T6149: Update node_data when merging nodes in reference tree generation as Unknown Status.

Mar 21 2024, 8:46 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro added a comment to T6146: Add python script to get all priorities of service or section from XML.

Comparison with legacy priority.pl can be seen with priority.py --legacy-format, below; note that priority of 0 is default for interface-definitions lacking a priority element, as is assigned in legacy commit algorithm.

Mar 21 2024, 4:55 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro added a comment to T6146: Add python script to get all priorities of service or section from XML.

PR for subtask bug merged:
https://github.com/vyos/vyos1x-config/pull/24
PR for this task will follow the update/rebuild of libvyosconfig.

Mar 21 2024, 1:45 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro renamed T6149: Update node_data when merging nodes in reference tree generation from Upadate node_data when merging nodes in reference tree generation to Update node_data when merging nodes in reference tree generation.

Mar 21 2024, 1:19 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro added a subtask for T6146: Add python script to get all priorities of service or section from XML: T6149: Update node_data when merging nodes in reference tree generation.

Mar 21 2024, 1:15 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro added a parent task for T6149: Update node_data when merging nodes in reference tree generation: T6146: Add python script to get all priorities of service or section from XML.

Mar 21 2024, 1:15 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro created T6149: Update node_data when merging nodes in reference tree generation.

Mar 21 2024, 1:14 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

Mar 20 2024

jestabro claimed T6146: Add python script to get all priorities of service or section from XML.

Mar 20 2024, 4:01 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

Mar 19 2024

jestabro added a comment to T6103: DHCP-server bootfile-name double slash syntax weird behaviour.

The fix for values containing single backslashes has been merged for 1.5 in
https://github.com/vyos/vyos1x-config/pull/23
https://github.com/vyos/vyos-1x/pull/3035

Mar 19 2024, 2:01 PM · VyOS 1.4 Sagitta (1.4.0-epa3), vyatta-cfg-dhcp-server

jestabro closed T5996: Incorrect behavior for backslash escapes in config save and compare commands, a subtask of T5939: [1.3.5 -> 1.4.0-RC1 Migration] as-path-list Entries Get Messed Up, as Unknown Status.

Mar 19 2024, 1:58 PM · VyOS 1.4 Sagitta (1.4.0-GA)

jestabro closed T5996: Incorrect behavior for backslash escapes in config save and compare commands as Unknown Status.

As discussed, this will wait before being backported to 1.4.

Mar 19 2024, 1:58 PM · VyOS 1.4 Sagitta (1.4.0-epa3)

jestabro committed rVYOSONEXb16c5fbbcb10: T5996: selectively escape and restore single backslashes in config.

Mar 19 2024, 1:56 PM

jestabro committed rVYOSONEXd2df08856ddc: T5996: add smoketest to check translation of backslash character.

Mar 19 2024, 1:56 PM

Mar 18 2024

jestabro added a comment to T6135: HTTPS API endpoint to check if an empty config exists.

Fix to docs pending:
https://github.com/vyos/vyos-documentation/pull/1331

Mar 18 2024, 7:21 PM · VyOS 1.5 Circinus

jestabro added a comment to T6135: HTTPS API endpoint to check if an empty config exists.

Well that is a fault of the docs; I will add now. Thanks !

Mar 18 2024, 6:10 PM · VyOS 1.5 Circinus

jestabro added a comment to T6135: HTTPS API endpoint to check if an empty config exists.

curl -k -X POST -Fkey=baz -Fdata='{"op": "exists", "path": ["service","no","such","subpath"]}' https://192.168.122.238/retrieve|jq 
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   340  100    47  100   293    547   3414 --:--:-- --:--:-- --:--:--  4000
{
  "success": true,
  "data": false,
  "error": null
}

Mar 18 2024, 6:09 PM · VyOS 1.5 Circinus

jestabro claimed T6135: HTTPS API endpoint to check if an empty config exists.

Mar 18 2024, 6:02 PM · VyOS 1.5 Circinus

jestabro added a comment to T6135: HTTPS API endpoint to check if an empty config exists.

@penetal does the operation 'exists' not suffice for your needs ?

Mar 18 2024, 6:02 PM · VyOS 1.5 Circinus