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

fixes #4245 'New window' button for SPICE console does not work #1211

Closed
wants to merge 1 commit into from
Closed

fixes #4245 'New window' button for SPICE console does not work #1211

wants to merge 1 commit into from

Conversation

jmontleon
Copy link
Member

With to_s this is returning something like '#Net::HTTPOK:0x007fbd31708f28' instead of the cert and the connection fails when using the 'New window' button.

@lzap
Copy link
Member

lzap commented Feb 10, 2014

I am installing oVirt Live DVD to try this. Did not succeed with available oVirt instances here :-P

@jmontleon
Copy link
Member Author

Does /ca.crt exist on the oVirt instance? I think this used to be a default, though maybe not anymore. Anyway, we still rely on it for pulling the CA Cert down so if not you'll have to copy it there. If we can get it via the API or some other smarter method it might be preferable.

@jmontleon
Copy link
Member Author

You should also be able to see (at least with the version of virt-viewer in Fedora 20) if it had trouble getting the CA Cert with a line like the following in /var/log/messages:

spice: (remote-viewer:2227): GSpice-WARNING **: loading ca certs from /var/tmp/trustore.pem-677YAX failed

And /var/tmp/trustore.pem-XXXXXX gets left around so you can easily inspect what you're getting for a cert to compare and make sure it is what it should be.

If the connection succeeds I don't believe you'll see the message in /var/log/messages though you'll still see connection information and the trustore.pem- file will still get left behind.

@lzap
Copy link
Member

lzap commented Feb 11, 2014

I have no oVirt to play with, instances I normally use do not work for me.

@domcleal
Copy link
Contributor

domcleal commented Mar 4, 2014

Merged as 2e417b5 for Foreman 1.4.2, thanks @jmontleon!

@domcleal domcleal closed this Mar 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants