Skip to content

Commit

Permalink
Fixed a minor bug and added a missing settings.conf.example parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
Nicolás Kovac committed Jul 14, 2019
1 parent 6a0bc63 commit b7da10f
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
2 changes: 1 addition & 1 deletion ovirtclient.py
Expand Up @@ -968,7 +968,7 @@ def checkConfig():
sys.exit("[ERROR] Configuration file (%s) is missing a mandatory parameter: Section: ovirt, parameter: cafile. Check config." % (conf.CONFIGFILE))

if not isfile(cafile):
sys.exit("[ERROR] Cannot find the CA file (%s). Check if file exists and if so, check if you have reading permissions in your config file." % (cafile, conf.CONFIGFILE))
sys.exit("[ERROR] Cannot find the CA file (%s). Check if file exists and if so, check if you have reading permissions in your config file." % (cafile))

try:
ovirtdomain = config.get('ovirt', 'domain')
Expand Down
8 changes: 8 additions & 0 deletions settings.conf.example
Expand Up @@ -5,6 +5,14 @@
; like: `https://myovirt.mydomain.com/ovirt-engine/api`.
url = https://ovirt.yourdomain.com/api

; cafile: Since oVirt 4.x, including the certificate of the oVirt instance
; is mandatory. For that, you have to take and export your oVirt
; certificate (you can do this connecting with your web browser to
; your URL and saving the certificate). It doesn't matter whether the
; CA is self-signed or not: Since oVirt 4.x this parameter is
; mandatory.
cafile = /your-ovirt-project/ca.crt

; domain: The domain under which your users will authenticate. When you create
; an AAA authenticator (LDAP, Kerberos, ...), a domain name is created
; to match it. This value goes here, so users will authenticate as
Expand Down

0 comments on commit b7da10f

Please sign in to comment.