4 changed files with 14 additions and 82 deletions
--- a/Changelog.md
+++ b/Changelog.md
@ -1,35 +0,0 @@
-# netbox-pve-sync
-
-## [Unreleased]
-
-## [0.2.2] - 07/05/2025
-
-### Fixed
-
- Use `NB_CLUSTER_ID` even for VM update.
-
-## [0.2.1] - 01/05/2025
-
-### Fixed
-
- [#7] Improve tag handling.
-
-## [0.2.0] - 21/04/2025
-
-### New
-
- Monitoring PVE HA/Replication.
-
-## [0.1.1] - 19/02/2025
-
-### New
-
- [#5] Allow to configure cluster ID.
-
-### Changed
-
- Add eth0 as raw_interface_name.
-
-## [0.1.0] - 19/02/2025
-
- Initial release.
--- a/README.md
+++ b/README.md
@ -9,12 +9,6 @@ This script work by pulling VMs information from the PVE API and create/update/d

 ## Installation

-This package is available on PyPi. You can install it using pip.
-
-```
-$ pip install netbox-pve-sync
-```
-
 ## Configuration

 ### On NetBox
@ -27,9 +21,6 @@ The following env variables will need to be set:
 - **NB_API_URL**: The URL to your NetBox instance. (ex: https://netbox.example.org)
 - **NB_API_TOKEN**: The token created previously. (ex: f74cb99cf552b7005fd1a616b53efba2ce0c9656)

-You can also set the `NB_CLUSTER_ID` env variable in order to indicate the ID of the cluster that will be used in
-NetBox.
-
 You'll also need to perform a minimal configuration on NetBox:

 - Create the physical nodes hosting the cluster. (The name should match the one on Proxmox, so that the script can
@ -37,19 +28,17 @@ You'll also need to perform a minimal configuration on NetBox:
 - Create the cluster.
 - Add the following Custom Fields:

-| Name       | Object types    | Label      | Type    |
-|------------|-----------------|------------|---------|
-| autostart  | Virtual Machine | Autostart  | Boolean |
-| replicated | Virtual Machine | Replicated | Boolean |
-| ha         | Virtual Machine | Failover   | Boolean |
-| backup     | Virtual Disk    | Backup     | Boolean |
-| dns_name   | Prefix          | DNS Name   | Text    |
+| Name      | Object types    | Label     | Type    |
+|-----------|-----------------|-----------|---------|
+| autostart | Virtual Machine | Autostart | Boolean |
+| backup    | Virtual Disk    | Backup    | Boolean |
+| dns_name  | Prefix          | DNS Name  | Text    |

 ### On the PVE API

 You'll need to create a dedicated user (ex: netsync) on your PVE cluster and then create an API token.

-The user needs to have access to the VM.Monitor, Pool.Audit, VM.Audit, Sys.Audit permissions.
+The user needs to have access to the VM.Monitor, Pool.Audit, VM.Audit permissions.

 The following env variables will need to be set:

@ -63,5 +52,5 @@ The following env variables will need to be set:
 You can then execute the script using the following command:

 ```
-PVE_API_HOST=xx PVE_API_USER=xx PVE_API_TOKEN=xx PVE_API_SECRET=xx NB_API_URL=xx NB_API_TOKEN=xx nbpxsync
-```
+PVE_API_HOST=xx PVE_API_USER=xx PVE_API_TOKEN=xx PVE_API_SECRET=xx NB_API_URL=xx NB_API_TOKEN=xx python3 -m netbox_pve_sync
+```
--- a/netbox_pve_sync/init.py
+++ b/netbox_pve_sync/init.py
@ -88,7 +88,6 @@ def _process_pve_tags(
                slug=f'pool-{_pve_pool["poolid"]}'.lower(),
                description=f'Proxmox pool {_pve_pool["poolid"]}',
            )
-            _nb_objects['tags'][_nb_tag.name] = _nb_tag

    return _nb_objects

@ -99,9 +98,7 @@ def _process_pve_virtual_machine(
        _nb_objects: dict,
        _nb_device: any,
        _pve_tags: [str],
-        _pve_virtual_machine: dict,
-        _is_replicated: bool,
-        _has_ha: bool,
+        _pve_virtual_machine: dict
 ) -> dict:
    _pve_node_name = _nb_device.name.lower()

@ -128,7 +125,7 @@ def _process_pve_virtual_machine(
            serial=_pve_virtual_machine['vmid'],
            name=_pve_virtual_machine['name'],
            site=_nb_device.site.id,
-            cluster=os.environ.get('NB_CLUSTER_ID', 1),
+            cluster=1,  # TODO
            device=_nb_device.id,
            vcpus=pve_virtual_machine_config['cores'],
            memory=int(pve_virtual_machine_config['memory']),
@ -136,22 +133,18 @@ def _process_pve_virtual_machine(
            tags=list(map(lambda _pve_tag_name: _nb_objects['tags'][_pve_tag_name].id, _pve_tags)),
            custom_fields={
                'autostart': pve_virtual_machine_config.get('onboot') == 1,
-                'replicated': _is_replicated,
-                'ha': _has_ha,
            }
        )
    else:
        nb_virtual_machine.name = _pve_virtual_machine['name']
        nb_virtual_machine.site = _nb_device.site.id
-        nb_virtual_machine.cluster = os.environ.get('NB_CLUSTER_ID', 1)
+        nb_virtual_machine.cluster = 1
        nb_virtual_machine.device = _nb_device.id
        nb_virtual_machine.vcpus = pve_virtual_machine_config['cores']
        nb_virtual_machine.memory = int(pve_virtual_machine_config['memory'])
        nb_virtual_machine.status = 'active' if _pve_virtual_machine['status'] == 'running' else 'offline'
        nb_virtual_machine.tags = list(map(lambda _pve_tag_name: _nb_objects['tags'][_pve_tag_name].id, _pve_tags))
        nb_virtual_machine.custom_fields['autostart'] = pve_virtual_machine_config.get('onboot') == 1
-        nb_virtual_machine.custom_fields['replicated'] = _is_replicated
-        nb_virtual_machine.custom_fields['ha'] = _has_ha
        nb_virtual_machine.save()

    # Handle the VM network interfaces
@ -253,7 +246,7 @@ def _process_pve_virtual_machine_network_interface(

    # TODO: Improve Multiple IP address handling
    _pve_virtual_machine_ip_address = None
-    for raw_interface_name in ['eth0', 'ens18', 'ens19']:
+    for raw_interface_name in ['ens18', 'ens19']:
        if raw_interface_name in _pve_virtual_machine_ip_addresses:
            _pve_virtual_machine_ip_address = _pve_virtual_machine_ip_addresses[raw_interface_name][0]
            break
@ -434,25 +427,13 @@ def main():
    for pve_vm_resource in pve_api.cluster.resources.get(type='vm'):
        pve_vm_tags[pve_vm_resource['vmid']] = []

-        if 'pool' in pve_vm_resource:
-            pve_vm_tags[pve_vm_resource['vmid']].append(f'Pool/{pve_vm_resource["pool"]}')
+        pve_vm_tags[pve_vm_resource['vmid']].append(f'Pool/{pve_vm_resource["pool"]}')

        if 'tags' in pve_vm_resource:
            pass  # TODO: pve_vm_tags[pve_vm_resource['vmid']].append(pve_vm_resource['tags'])

-    pve_ha_virtual_machine_ids = list(
-        map(
-            lambda r: int(r['sid'].split(':')[1]),
-            filter(lambda r: r['type'] == 'service', pve_api.cluster.ha.status.current.get())
-        )
-    )
-
    # Process Proxmox nodes
    for pve_node in pve_api.nodes.get():
-        pve_replicated_virtual_machine_ids = list(
-            map(lambda r: r['guest'], pve_api.nodes(pve_node['node']).replication.get())
-        )
-
        # This script does not create the hardware devices.
        nb_device = nb_objects['devices'].get(pve_node['node'].lower())
        if nb_device is None:
@ -471,8 +452,6 @@ def main():
                nb_device,
                pve_vm_tags.get(pve_virtual_machine['vmid'], []),
                pve_virtual_machine,
-                pve_virtual_machine['vmid'] in pve_replicated_virtual_machine_ids,
-                pve_virtual_machine['vmid'] in pve_ha_virtual_machine_ids,
            )


--- a/pyproject.toml
+++ b/pyproject.toml
@ -6,7 +6,7 @@ build-backend = "setuptools.build_meta"

 [project]
 name = "netbox-pve-sync"
-version = "0.2.2"
+version = "0.1.0"
 authors = [
    { name = "Aloïs Micard", email = "alois@micard.lu" },
 ]
@ -26,7 +26,6 @@ dependencies = [
 [project.urls]
 Homepage = "https://git.creekorful.cloud/creekorful/netbox-pve-sync"
 Issues = "https://git.creekorful.cloud/creekorful/netbox-pve-sync/issues"
-Changelog = "https://git.creekorful.cloud/creekorful/netbox-pve-sync/src/branch/main/Changelog.md"

 [project.scripts]
 nbpxsync = "netbox_pve_sync:main"