Bug #15681: Fog Openstack version should require at least version 0.1.7 to ensure compatibility with Openstack Liberty - Foreman

Actions

Copy link

Bug #15681

closed

Fog Openstack version should require at least version 0.1.7 to ensure compatibility with Openstack Liberty

Added by Imri Zvik almost 9 years ago. Updated almost 7 years ago.

Status:

Closed

Priority:

High

Assignee:

Imri Zvik

Category:

Compute resources - OpenStack

Target version:

1.13.0

Difficulty:

trivial

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/3648

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

Due to changes in Openstack, the Compute API in Liberty (and maybe other APIs) are no longer forgiving for empty values.
See the following commit: https://github.com/fog/fog-openstack/commit/13bfc96ba646c65487c8a619ee84ad5f6268e702

This totally breaks stuff like boot from volume.

Actions

Copy link

Updated by Imri Zvik almost 9 years ago

Should probably be

gem 'fog-openstack', '~> 0.1', '>= 0.1.7'

Actions

Copy link

Updated by Dominic Cleal almost 9 years ago

The pinning of fog-core to 1.36.0 currently prevents the ~> 0.1 matcher from matching 0.1.7. If fog-core is updated to over 1.40 (either specifically, or unpinned) then we'll automatically inherit a newer version of fog-openstack too.

Actions

Copy link

Updated by Imri Zvik almost 9 years ago

Well, since this breaks the functionality quite bad, I think this should get priority.

Note that Liberty is the minimal version of Openstack still supported (http://releases.openstack.org/), so Foreman should probably be compatibable with it.

Actions

Copy link

Updated by Imri Zvik almost 9 years ago

Dominic Cleal wrote:

The pinning of fog-core to 1.36.0 currently prevents the ~> 0.1 matcher from matching 0.1.7. If fog-core is updated to over 1.40 (either specifically, or unpinned) then we'll automatically inherit a newer version of fog-openstack too.

btw, another thing to note is that nothing ensures you will get the latest version of 0.1 - see https://github.com/bundler/bundler/issues/3089#issuecomment-48533609 for example.
Therefore, it is probably a good idea to change the requirement anyway.

Actions

Copy link