Skip to content

feat(nocloud): add optional network file name#164

Open
CorentinPtrl wants to merge 1 commit intocloudbase:masterfrom
CorentinPtrl:master
Open

feat(nocloud): add optional network file name#164
CorentinPtrl wants to merge 1 commit intocloudbase:masterfrom
CorentinPtrl:master

Conversation

@CorentinPtrl
Copy link

This fixes the network config for proxmox since the naming is different for the network config (NETWORK_CONFIG)
related to #89

@plouton24
Copy link

Would be nice if we could merge this. As it would fix a couple issues when trying to add auto deploy Windows Nodes to Proxmox plus some kubernetes flavors dont merge network configs into meta data. So you end up needing to manually configure networking.

ader1990 added a commit to ader1990/cloudbase-init-test-resources-1 that referenced this pull request Feb 16, 2026
Add required Proxmox cidata NoCloud config drive support.

Proxmox has two types for ISO based configdrives that can be created,
one for Windows, and one for non-Windows OS types.

The Windows type is a configdrivev2 ISO type, that is similar but not
quite the same with the OpenStack configdrive v2: the label is config-2.

This change focuses on the NoCloud ISO type, that is similar but not the
same with the NoCloud implementation: the label is cidata, and the
file structure is as follows:

```
META_DATA -> file containing at most 3 entries, in the format key:
value - instance-id, network-interfaces, hostname.
NETWORK_CONFIG -> network configuration, standard format.
USER_DATA -> user data, standard format.

```

The currently created Proxmox cidata ISO usage with NoCloud provider
shows the following errors:

```
ERROR cloudbaseinit.plugins.common.userdataplugins.cloudconfig [-] Plugin 'manage_etc_hosts' is currently not supported
ERROR cloudbaseinit.plugins.common.userdataplugins.cloudconfig [-] Plugin 'fqdn' is currently not supported
ERROR cloudbaseinit.plugins.common.userdataplugins.cloudconfig [-] Plugin 'user' is currently not supported
ERROR cloudbaseinit.plugins.common.userdataplugins.cloudconfig [-] Plugin 'password' is currently not supported
ERROR cloudbaseinit.plugins.common.userdataplugins.cloudconfig [-] Plugin 'chpasswd' is currently not supported
ERROR cloudbaseinit.plugins.common.userdataplugins.cloudconfig [-] Plugin 'package_upgrade' is currently not supported

INFO cloudbaseinit.metadata.services.nocloudservice [-] V2 network metadata not found
```

The support for `manage_etc_hosts` cannot be added, as it cannot be mapped to an existing Windows feature.

The support for `fqdn` can be added easily, as it is the same as the `set_hostname ` one.

The support for `user` with the  `default` entry cannot be added, as it cannot be mapped to an existing Windows feature. This feature is in some sense implicit.

The support for `password` can be added as per https://cloudbase-init.readthedocs.io/en/latest/plugins.html#setting-password-main.
The support for `chpasswd` can be added as per https://cloudbase-init.readthedocs.io/en/latest/plugins.html#setting-password-main.

The support for `package_upgrade` can be added as per the https://cloudbase-init.readthedocs.io/en/latest/plugins.html#windows-auto-updates-main.

The support for v2 network metadata is in progress here: cloudbase/cloudbase-init#164

Signed-off-by: Adrian Vladu <avladu@cloudbasesolutions.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants