User Details
- User Since
- Apr 12 2019, 4:27 PM (266 w, 1 d)
Yesterday
Thu, May 16
Wed, May 15
Note that T5996 has been backported and a full explanation is to be found in the previous comment. Close as resolved, with the needed workaround for earlier images.
This was a worthwhile investigation, with the goal of using Cython as a method of easily wrapping the standard netlink headers. Although workable, it appeared more trouble than it's worth, as one still has to cheat to navigate e.g. pass by reference, and limitations of Cython at the time. A much better approach would be a standard Python extension module ... set to 'on hold' for reference and until we have a plan for development, as has been recently discussed.
As c-po mentioned above, this was an interesting idea that was not workable; it was subsumed by the investigations in T3876.
@tjh in fact, this was a design decision, discussed with @Viacheslav at the time, and agreed upon, although neither of use are fully satisfied with the decision (and I'll let him amend these claims, if needed):
Mon, May 13
Checking a recent 1.5 build (1.5-rolling-202405090225), I do not see the discrepancy mentioned in the description: the result for 1.5 is also (just) missing vif entries, and one can observe that the syntax error in vif 2 causes cascading syntax errors in the other vif sections.
The following script recreates the effect of the XorpConfigParser:
Sun, May 12
Migration requires the config file to be in the standard syntax, such as produced by the system itself (set/del; commit; save) --- in this case, a configuration that can be loaded and saved on VyOS 1.3.5, would not be missing braces as above. If one needs to hand edit a config file, at the very least one needs to load/save on the source system before migration. The only weird corner case, linked above, is when a change of mac address triggers a rewrite by the legacy parser; that case is also restored by saving the config before migration.
Fri, May 10
This will be possible after subtask T6006, for 1.5; backport can be considered at that time, but for now limit tags to Circinus.
Thu, May 9
See details and explanation in subtask T6327; notable, one can configure as system console, but setting at boot has limited use.
Wed, May 8
Note that the work in the link above, but for the get_commit_schedule function itself, has already been added in subtasks T6319/T6146. As mentioned above, the get_commit_schedule function is not useful (and potentially misleading) until we replace the legacy commit algorithm in 1.5. Remove 1.4.0 tag.
Sat, May 4
Mon, Apr 29
Sun, Apr 28
PR for 1.5:
https://github.com/vyos/vyos-1x/pull/3372
Combined PRs for backport to 1.4 of T5839, T5660, T6276 pending.
Thu, Apr 25
Wed, Apr 24
Reopen to investigate.
Tue, Apr 23
Mon, Apr 22
The fix mentioned above has been backported to 1.4: T5996.