lae

Oh! Okay, yeah that does seem to indicate my understanding for osds_from_devices from just the docs was incorrect. (I have like, no environment (or funds to make one to be...

Thanks for taking on this.

> `NETBOX_DELETE_LEGACY_DATA` Historically, I've done a minor release bump (second number) for each minor version of NetBox. It's expected that admins upgrading read the release notes for those updates before...

@Andre15711 Is this something you can help test? If I can get a confirmation that it works properly for someone, I guess I can go ahead and merge it.

@Andre15711 Normally, you can install a specific branch (like this PR) using a URL like the following: ``` ansible-galaxy install https://github.com/lae/ansible-role-proxmox/archive/feature/pbs-storage.tar.gz,pr-169,lae.proxmox --force ``` This grabs the archive from the branch...

> If I see this correct, the entire syntax of the storage declaration is totally diffrent. > Is this correct? No. > ``` > File "/tmp/ansible_proxmox_storage_payload_3vsafbr6/ansible_proxmox_storage_payload.zip/ansible/modules/proxmox_storage.py", line 219, in __init__...

Okay, now it's failing for the PBS definition itself so that's progress. It looks like some people actually brought up the master_pubkey issue in that PR, and I just haven't...

Yes, the encryption is optional. That doesn't mean I should leave it out of the module for users who want to use it. Invalid syntax sounds like you may have...

> error from the API No, it's an error with Python importing the `proxmox_storage` module. So something had to have broken during editing. I can't provide much help right now,...

Rebased against develop and fixed the logic issue with the fingerprint param. Can someone test this for me? Will merge afterwards.