Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Proxmox provider #169

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

giodamelio
Copy link

@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

hashicorp-cla commented Feb 28, 2021

CLA assistant check
All committers have signed the CLA.

@JalfResi
Copy link

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

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

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

@prologic
Copy link

Yes please! 🙏

@giodamelio
Copy link
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

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

@mikemorris
Copy link
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