inject missing env vars in configd to support configfs util
Closed, ResolvedPublicBUG
Actions

Assigned To

Authored By

	jestabro
	Aug 2 2024, 4:27 PM

Description

The configfs utility introduced in T6489 requires two environment variables to successfully run under configd; provide from config session. Without those available, the script was executed within the CLI context, as a result of T6608, defeating the purpose of introduction of configfs.

Details

Difficulty level: Easy (less than an hour)
Version: 1.5
Why the issue appeared?: Will be filled on close
Is it a breaking change?: Perfectly compatible
Issue type: Bug (incorrect behavior)

Related Objects
Search...

Status	Subtype	Assigned	Task
Resolved	FEATURE REQUEST	c-po	T6489 Add/Improve support for CLI config scripts that change the underlayin actual configuration and make them work with vyos-configd
Open	BUG	jestabro	T6608 configd should report exceptions uncaught by conf_mode scripts as commit error
Resolved	BUG	jestabro	T6633 inject missing env vars in configd to support configfs util
Open	BUG	jestabro	T6639 Resolve undesired interactions configverify <-> configfs <-> configd
Resolved	BUG	jestabro	T6638 QoS CAKE config with PPPoE interface does not load after reboot
Resolved	BUG	jestabro	T6642 verify_interface_exists should not instantiate its own Config object
Resolved	ENHANCEMENT	jestabro	T6640 Return False for boolean check in_session when running under configd