Maniphest T5319

Remove remaining workarounds for incorrect defaults
Closed, ResolvedPublicENHANCEMENT
Actions

Assigned To

Authored By

	jestabro
	Jun 28 2023, 3:17 PM

Tags

Referenced Files

None

Subscribers

Description

This is a parent task for removing remaining workarounds due to incorrect defaults, replacing them with the tools developed in T5218, T5228.

Details

Difficulty level: Normal (likely a few hours)
Version: vyos-1.4
Why the issue appeared?: Will be filled on close
Is it a breaking change?: Perfectly compatible
Issue type: Improvement (missing useful functionality)

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved	BUG	jestabro	T2665 vyos.xml.defaults for tag nodes
		Resolved	ENHANCEMENT	jestabro	T5218 Revise vyos xml lib for bug fixes and extensions
		Resolved	ENHANCEMENT	jestabro	T5228 Simplify get_config_dict and add argument with_defaults
		Resolved	ENHANCEMENT	jestabro	T5308 Remove workarounds for incorrect defaults in get_interface_dict
		Resolved	ENHANCEMENT	jestabro	T5319 Remove remaining workarounds for incorrect defaults
		Resolved	BUG	jestabro	T5350 Confusing warning message when committing VRRP config
		Resolved	ENHANCEMENT	jestabro	T5443 Add merge_defaults as Config method
		Resolved	ENHANCEMENT	jestabro	T6203 Remove obsoleted xml lib

Event Timeline

jestabro triaged this task as Normal priority.Jun 28 2023, 3:17 PM

jestabro created this task.

jestabro created this object in space S1 VyOS Public.

jestabro added a parent task: T5308: Remove workarounds for incorrect defaults in get_interface_dict.

pasik added a subscriber: pasik.Jun 28 2023, 4:27 PM

jestabro added a subtask: T5350: Confusing warning message when committing VRRP config.Jul 11 2023, 1:51 PM

jestabro mentioned this in T5350: Confusing warning message when committing VRRP config.Aug 2 2023, 6:20 PM

jestabro closed subtask T5350: Confusing warning message when committing VRRP config as Resolved.Aug 2 2023, 7:38 PM

jestabro mentioned this in T5434: Replace remaining calls of vyos.xml library.Aug 3 2023, 1:31 PM

jestabro mentioned this in T5443: Add merge_defaults as Config method.Aug 6 2023, 1:18 AM

jestabro added a subtask: T5443: Add merge_defaults as Config method.Aug 6 2023, 2:07 PM

PR:
https://github.com/vyos/vyos-1x/pull/2136

jestabro closed subtask T5443: Add merge_defaults as Config method as Resolved.Aug 9 2023, 2:04 PM

jestabro closed this task as Resolved.Aug 11 2023, 2:30 PM

jestabro moved this task from Need Triage to Finished on the VyOS 1.4 Sagitta board.

jestabro mentioned this in T6203: Remove obsoleted xml lib.Thu, Apr 4, 7:57 PM

jestabro added a subtask: T6203: Remove obsoleted xml lib.Thu, Apr 4, 8:01 PM

jestabro changed the status of subtask T6203: Remove obsoleted xml lib from Open to Backport candidate.Fri, Apr 5, 7:46 PM

jestabro closed subtask T6203: Remove obsoleted xml lib as Resolved.Sat, Apr 6, 2:54 PM