Skip to content
This repository has been archived by the owner on Sep 26, 2021. It is now read-only.

Add support for OS_CACERT in Openstack driver #3472

Conversation

mathias-nyman
Copy link

This enables using docker-machine's openstack driver without having to
resort to using the --insecure flag.

This enables using docker-machine's openstack driver without having to
resort to using the --insecure flag.

Signed-off-by: Mathias Nyman <mathias.nyman@iki.fi>
@mathias-nyman mathias-nyman force-pushed the add_openstack_driver_cacert_support branch from c25338b to d0c92a3 Compare June 2, 2016 13:52
@mathias-nyman
Copy link
Author

I could use some help finding the needle in this haystack:
https://jenkins.dockerproject.org/job/docs-machine-pr/1425/console

To me it looks like the source of the 255 reported errors are not from my diff.

@nathanleclaire
Copy link
Contributor

@fsoppelsa WDYT

@nathanleclaire
Copy link
Contributor

The docs sometimes have intermittent failures

@nathanleclaire
Copy link
Contributor

Retest this please

@nathanleclaire
Copy link
Contributor

(The above re-triggers a docs build.)

@fsoppelsa
Copy link
Contributor

I need to try this, ETA by this week.

@SvenDowideit
Copy link
Contributor

@nathanleclaire is this for v0.8.1 or v0.9.0? - if you set the milestone docs'll know when to publish it (and if its a fix for older releases, use that as the milestone?)

@nathanleclaire
Copy link
Contributor

@SvenDowideit It depends, @fsoppelsa @mathias-nyman are you still interested in this? I'll merge if it looks good to you @fsoppelsa

@mathias-nyman
Copy link
Author

@nathanleclaire, yup, still very much desired.

@nathanleclaire nathanleclaire added this to the 0.9.0 milestone Aug 5, 2016
@nathanleclaire
Copy link
Contributor

It LGTM -- we can revert if we encounter issues

@nathanleclaire nathanleclaire merged commit d0e0310 into docker:master Aug 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants