Skip to content

Add Proxmox provider#169

Open
giodamelio wants to merge 10 commits intohashicorp:masterfrom
giodamelio:proxmox-provider
Open

Add Proxmox provider#169
giodamelio wants to merge 10 commits intohashicorp:masterfrom
giodamelio:proxmox-provider

Conversation

@giodamelio
Copy link
Copy Markdown

@giodamelio giodamelio commented Feb 28, 2021

This is a simple Proxmox provider that allows you to discover all the IPs from the VMs in a Proxmox pool

@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Feb 28, 2021

CLA assistant check
All committers have signed the CLA.

@JalfResi
Copy link
Copy Markdown

Is there any chance we can get this merged? I'd love to be able to use go-discover on my Proxmox cluster!

@aduzsardi
Copy link
Copy Markdown

@hashicorp-cla Bot please merge this 😄
thank you!

Jokes aside , any ideas when or if this is going to be merged ?

@prologic
Copy link
Copy Markdown

Yes please! 🙏

@giodamelio
Copy link
Copy Markdown
Author

Just resolved the conflicts.

Is this something that the maintainers of this project would be interested in? Not sure who to tag here, so @mikemorris as the last committer as of the time of writing, you are in luck 😆 .

@aduzsardi
Copy link
Copy Markdown

thank you @giodamelio for your time and effort in adding this feature

@mikemorris
Copy link
Copy Markdown
Contributor

👋 We're in a difficult position currently as maintainers currently in that it's not possible for us to validate or test most of the less-common cloud providers that contributors are interested in adding support for - we've discussed an alternative proposed structure to make these lookups more pluggable in hashicorp/consul#9100 and have started some work towards enabling that in a new library that we've recently made public at https://github.com/hashicorp/go-netaddrs, please take a look and let us know if this might work for your needs!

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.

6 participants