[openstack|compute] authenticate_v2 fixes #890

Merged
merged 1 commit into from May 4, 2012

Conversation

Projects
None yet
4 participants
@rubiojr
Member

rubiojr commented May 4, 2012

  • Add connection_options to Fog::Connection when
    authenticating so we can disable SSL cert verification for example.
  • Do not use hardcoded keystone 5000 port

Sample code:

conn = Fog::Compute.new({
  :provider => 'OpenStack',
  :openstack_api_key => "changeme",
  :openstack_username => "user@bvox.net",
  :openstack_auth_url => "https://auth-server/v2.0/tokens",
  :connection_options => { :ssl_verify_peer => false }
})
@Keoven

This comment has been minimized.

Show comment Hide comment
@Keoven

Keoven May 4, 2012

Member

This is good for me, could you just please update your Git commit's subject. :)

Member

Keoven commented May 4, 2012

This is good for me, could you just please update your Git commit's subject. :)

@rubiojr

This comment has been minimized.

Show comment Hide comment
@rubiojr

rubiojr May 4, 2012

Member

@Keoven, let me know if that works for you

Member

rubiojr commented May 4, 2012

@Keoven, let me know if that works for you

@Keoven

This comment has been minimized.

Show comment Hide comment
@Keoven

Keoven May 4, 2012

Member

You might want to clean that up a bit by squashing it to just one commit and force upload to your remote on that branch to update this pull request :)

Member

Keoven commented May 4, 2012

You might want to clean that up a bit by squashing it to just one commit and force upload to your remote on that branch to update this pull request :)

[openstack|compute] authenticate_v2 fixes
* Add connection_options to Fog::Connection when
authenticating so we can disable SSL cert verification for example.
* Do not use hardcoded keystone 5000 port

Sample code:

    conn = Fog::Compute.new({
      :provider => 'OpenStack',
      :openstack_api_key => "changeme",
      :openstack_username => "user@bvox.net",
      :openstack_auth_url => "https://auth-server/v2.0/tokens",
      :connection_options => { :ssl_verify_peer => false }
    })
@rubiojr

This comment has been minimized.

Show comment Hide comment
@rubiojr

rubiojr May 4, 2012

Member

Thanks @Keoven, done as suggested.

Member

rubiojr commented May 4, 2012

Thanks @Keoven, done as suggested.

@xtoddx

This comment has been minimized.

Show comment Hide comment
@xtoddx

xtoddx May 4, 2012

Member

lgtm

Member

xtoddx commented May 4, 2012

lgtm

Keoven added a commit that referenced this pull request May 4, 2012

Merge pull request #890 from bvox/openstack_conn_options
[openstack|compute] authenticate_v2 fixes

@Keoven Keoven merged commit ad39fd2 into fog:master May 4, 2012

@Keoven

This comment has been minimized.

Show comment Hide comment
@Keoven

Keoven May 4, 2012

Member

merged, thanks!

Member

Keoven commented May 4, 2012

merged, thanks!

@geemus

This comment has been minimized.

Show comment Hide comment
@geemus

geemus May 4, 2012

Member

Thanks guys!

Member

geemus commented May 4, 2012

Thanks guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment