-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
Permit to query a proxmox with invalid certificate #17247
Permit to query a proxmox with invalid certificate #17247
Conversation
Not everybody has a valid certificate in its proxmox installation
Not everybody has a valid certificate in its proxmox installation
parser = OptionParser(usage='%prog [options] --list | --host HOSTNAME') | ||
parser.add_option('--list', action="store_true", default=False, dest="list") | ||
parser.add_option('--host', dest="host") | ||
parser.add_option('--url', default=os.environ.get('PROXMOX_URL'), dest='url') | ||
parser.add_option('--username', default=os.environ.get('PROXMOX_USERNAME'), dest='username') | ||
parser.add_option('--password', default=os.environ.get('PROXMOX_PASSWORD'), dest='password') | ||
parser.add_option('--pretty', action="store_true", default=False, dest='pretty') | ||
parser.add_option('--trust-invalid-certs', action="store_true", default=bool_validate_cert, dest='validate') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, if I am not mistaken, if I do not use the option, valide is True. If I use the option, then store_true will mean that validate will also be True. Shouldn't it be the reverse ?
@bmillemathias Your branch does not contain a shippable.yml file. Please rebase your branch to trigger running of current tests. |
@bmillemathias this needs to be rebased in order for it to use the new testing infrastructure. could you do so and we'll finish the review. |
@bmillemathias I'm closing this PR as it's so old it doesn't have support for Shippable. If you are interested in this still please rebase your branch and reopen this PR If anyone else is interested in this, please
Rebase guide: https://docs.ansible.com/ansible/latest/dev_guide/index.html |
ISSUE TYPE
COMPONENT NAME
contrib/inventory/proxmox.py
ANSIBLE VERSION
SUMMARY
Permit to query a proxmox server which doesn't have a proper certificate.
This is a patch which adds an argument to the inventory script for proxmox to permit to query the API even in the absence of valid certificate.
Fixes #15888