Page MenuHomeVyOS Platform

Extend XML building blocks
Closed, ResolvedPublicFEATURE REQUEST

Description

As the VyOS CLI grows we need to maintain it.

One way to do so was the introduction in VyOS 1.3 to use #include statements well known from C. An #include block shares a common CLI node e.g. "source-address" with a simple implementation.
That building block is later reused across different sections inside the CLI.

From time to time new features are added the need the same - or almost the same CLI syntax. This task is more of a maintenance one to add missing include definitions and refactor the code consuming them,
to make every maintainers life a little less stressful.

Details

Version
-
Is it a breaking change?
Perfectly compatible
Issue type
Internal change (not visible to end users)

Related Objects

Mentioned In
rVYOSONEX27187e96127e: xml: T5738: reuse existing alpha-numeric-hyphen-underscore building block
rVYOSONEX7f7bb85fbc56: Merge pull request #4296 from c-po/xml-cleanup
rVYOSONEXfad878b1868a: Merge pull request #4294 from indrajitr/xml-reuse-rexexp
rVYOSONEXbe7fcf68576a: xml: T5738: Reuse predefined regex constraint
rVYOSONEXe9b1e26238c6: Merge pull request #4269 from indrajitr/dedu-pki-ssh
rVYOSONEX484e9b8f2e5d: xml: T5738: De-duplicate 'openssh' tagNode
rVYOSONEX214448b096cd: Merge pull request #4251 from c-po/xml-cleanup
rVYOSONEX020f217c2e4b: xml: T5738: re-use XML building blocks for interfaces as much as possible
rVYOSONEX92baf98bf806: Merge pull request #3344 from vyos/mergify/bp/sagitta/pr-3339
rVYOSONEX01f745f8d157: xml: T5738: fix typo in radius-additions.xml.i
rVYOSONEX8d0aa7bfb83a: xml: T5738: fix typo in radius-additions.xml.i
rVYOSONEXd22560b942cb: Merge pull request #3339 from andre-luiz-dos-santos/patch-1
rVYOSONEXb3fb51cd799d: utils: T5738: always use vyos.utils.network.interface_exists over os.path.exists
rVYOSONEX9f831714ef0f: xml: T5738: extend VRF building blocks with common constraint definition
rVYOSONEX5bb27f0c6220: utils: T5738: always use vyos.utils.network.interface_exists over os.path.exists
rVYOSONEX32d6a693de99: xml: T5738: extend VRF building blocks with common constraint definition
rVYOSONEX25891616fd74: Merge pull request #3183 from vyos/mergify/bp/sagitta/pr-3181
rVYOSONEX2a492bd9c701: xml: T5738: use common constraint include for container network
rVYOSONEX6be463fcca57: xml: T5738: use common constraint include for container network
rVYOSONEX20e069d83b4c: Merge pull request #3181 from c-po/xml-T5738-container
rVYOSONEX5472986c1fa7: Merge pull request #3116 from vyos/mergify/bp/sagitta/pr-3115
rVYOSONEX9ec374140aa2: xml: T5738: revert invalid change from lower character limit - 0 length must be…
rVYOSONEX724c685cba42: xml: T5738: revert invalid change from lower character limit - 0 length must be…
rVYOSONEX0eee7cf02e78: Merge pull request #3115 from c-po/T6071-part2
rVYOSONEXc70d6dec4f3a: xml: T5738: lower maximum description to 255 characters
rVYOSONEXa72ededa0b29: xml: T5738: lower maximum description to 255 characters
rVYOSONEX53eca9a3f9ea: Merge pull request #3092 from vyos/mergify/bp/sagitta/pr-3091
rVYOSONEX390e543780b5: xml: T5738: use generic-disable-node building block for "disable" CLI nodes
rVYOSONEX21b0bf016869: xml: T5738: use generic-disable-node building block for "disable" CLI nodes
rVYOSONEX51a0d5d9ed3f: Merge pull request #3091 from c-po/xml-T5738
rVYOSONEX3b383b52888a: Merge pull request #2984 from vyos/mergify/bp/sagitta/pr-2982
rVYOSONEXec049d192ddb: xml: T5738: improve PKI building blocks for CLI
rVYOSONEXd4278cde2b15: xml: T5738: improve PKI building blocks for CLI
rVYOSONEX3ae4c6236750: Merge pull request #2982 from c-po/pki-xml
rVYOSONEXac7ddf03149e: Merge pull request #2849 from vyos/mergify/bp/sagitta/pr-2848
rVYOSONEXd2a82619d5ad: xml: T5738: re-use source-address-ipv4-ipv6 building block for config-management
rVYOSONEX100c2393e873: xml: T5738: re-use source-address-ipv4-ipv6 building block for config-management
rVYOSONEX56a02a94a8b1: Merge pull request #2848 from c-po/T5738-config-management
rVYOSONEXdc1263677003: xml: T5738: add constraint building block with alphanumeric, hypen, underscore…
rVYOSONEX82b4b2db8fda: xml: T5738: add constraint building block with alphanumeric, hypen, underscore…
rVYOSONEX777a2a249294: xml: T5738: re-use "disable" building block
rVYOSONEX019723aaa421: xml: T5738: re-use "disable" building block
rVYOSONEX8c222c3848fe: Merge pull request #2484 from vyos/mergify/bp/sagitta/pr-2479
rVYOSONEXa818afbe81d1: xml: T5738: add source-address-ipv4-ipv6-multi building block
rVYOSONEX40c194c3a1ee: xml: T5738: add source-address-ipv4-ipv6-multi building block
rVYOSONEXdccca4307339: xml: T5738: add source-address-ipv4-ipv6-multi building block
rVYOSONEXaa4abbc73382: Merge pull request #2479 from c-po/t5738-xml

Event Timeline

c-po claimed this task.
c-po edited a custom field.
c-po changed Is it a breaking change? from Unspecified (possibly destroys the router) to Perfectly compatible.
c-po changed Issue type from Unspecified (please specify) to Internal change (not visible to end users).