Driver functions are missing in rethtool packages

Bug #1423150 reported by Aviram Bar-Haim
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Aleksander Mogylchenko

Bug Description

According to blueprint https://blueprints.launchpad.net/fuel/+spec/support-infiniband-network , driver and bus_info functions needed to be merged to the current rethtool repositories:
* packages/centos6/rubygem-rethtool
* packages/precise/rethtool

And to the future repositories:
* packages/centos6/ruby21-rubygem-rethtool
* packages/trusty/rethtool

The patch is already pushed to precise repository for review: https://review.fuel-infra.org/#/c/3633/ .
The patch is also attached here.

Revision history for this message
Aviram Bar-Haim (aviramb) wrote :
Changed in fuel:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Fuel OSCI Team (fuel-osci)
milestone: none → 6.1
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

I think ruby21-rubygem-rethtool is't used in the iso. So it doesn't need to patch it

Changed in fuel:
assignee: Fuel OSCI Team (fuel-osci) → MOS Linux (mos-linux)
description: updated
tags: added: mellanox
tags: added: partner
Changed in fuel:
assignee: MOS Linux (mos-linux) → Aleksander Mogylchenko (amogylchenko)
Pavel Chekin (pchekin)
description: updated
Revision history for this message
Aleksander Mogylchenko (amogylchenko) wrote :

packages/trusty/rethtool is ready to accept requests.

Revision history for this message
OSCI Robot (oscirobot) wrote :

package rethtool has been built for project packages/precise/rethtool
Package version == 0.0.3, package release == ubuntu1

Changeset: https://review.fuel-infra.org/3633
project: packages/precise/rethtool
branch: 6.1
author: Aviram Bar-Haim
committer: Aleksandr Mogylchenko
subject: Added interface driver and bus_info support
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /ubuntu-fuel-6.1-stable-3633/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package rubygem-rethtool has been built for project packages/centos6/rubygem-rethtool
Package version == 0.0.3, package release == 3

Changeset: https://review.fuel-infra.org/3876
project: packages/centos6/rubygem-rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Backport of upstream commit: https://github.com/mpalmer/rethtool/commit/bed7c657f77a4b51aae5afbff8a2fca32d9f4f56
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-3876/

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package rethtool has been built for project packages/trusty/rethtool
Package version == 0.0.3, package release == ubuntu1

Changeset: https://review.fuel-infra.org/4051
project: packages/trusty/rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Add rethtool ruby package 0.0.3
status: patchset-created

Files placed on repository:
ruby-rethtool_0.0.3-ubuntu0~mos6.1+1_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://obs-1.mirantis.com:82/trusty-fuel-6.1-stable-4051/ubuntu

Revision history for this message
Aviram Bar-Haim (aviramb) wrote :

The new packages (rpm and deb) worked for me over Bootstrap, CentOS 6.5 and Ubuntu 12.04.4 in Fuel 6.1 (master).
After adding them to the repositories and building ISO I was able to get driver and bus_info data in the Fuel master.

The packages I've got from AleksanderM to verify:
rubygem-rethtool-0.0.3-3.mira2.noarch.rpm
rubygem-rethtool-doc-0.0.3-3.mira2.noarch.rpm
ruby-rethtool_0.0.3-ubuntu1_all.deb

Revision history for this message
OSCI Robot (oscirobot) wrote :

package rethtool has been built for project packages/precise/rethtool
Package version == 0.0.3, package release == ubuntu1

Changeset: https://review.fuel-infra.org/3633
project: packages/precise/rethtool
branch: 6.1
author: Aviram Bar-Haim
committer: Aleksandr Mogylchenko
subject: Added interface driver and bus_info support
status: change-merged

Files placed on repository:

Changeset merged. Package placed on primary repository
 repository URL: /ubuntu-fuel-6.1-stable/

Changed in fuel:
assignee: Aleksander Mogylchenko (amogylchenko) → asyriy (asyriy)
status: Triaged → In Progress
Albert Syriy (asyriy)
Changed in fuel:
assignee: asyriy (asyriy) → Alexei Sheplyakov (asheplyakov)
Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package rethtool has been built for project packages/trusty/rethtool
Package version == 0.0.3~mos6.1+1, package release == ubuntu1

Changeset: https://review.fuel-infra.org/4051
project: packages/trusty/rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Add rethtool ruby package 0.0.3
status: patchset-created

Files placed on repository:
ruby-rethtool_0.0.3~mos6.1+1_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://obs-1.mirantis.com:82/trusty-fuel-6.1-stable-4051/ubuntu

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package rethtool has been built for project packages/trusty/rethtool
Package version == 0.0.3, package release == ubuntu1

Changeset: https://review.fuel-infra.org/4388
project: packages/trusty/rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: ruby-rethtool 0.0.3 with patch from Mellanox
status: patchset-created

Files placed on repository:
ruby-rethtool_0.0.3-1~mos6.1+1_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://obs-1.mirantis.com:82/trusty-fuel-6.1-stable-4388/ubuntu

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package rethtool has been built for project packages/trusty/rethtool
Package version == 0.0.5, package release == ubuntu1

Changeset: https://review.fuel-infra.org/4445
project: packages/trusty/rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Update rethtool to most recent 0.0.5 version
status: patchset-created

Files placed on repository:
ruby-rethtool_0.0.5-1~mos6.1_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://obs-1.mirantis.com:82/trusty-fuel-6.1-stable-4445/ubuntu

Revision history for this message
OSCI Robot (oscirobot) wrote :

package rethtool has been built for project packages/trusty/rethtool
Package version == 0.0.5, package release == ubuntu1

Changeset: https://review.fuel-infra.org/4445
project: packages/trusty/rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Update rethtool to most recent 0.0.5 version
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4445/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package rethtool has been built for project packages/trusty/rethtool
Package version == 0.0.5, package release == ubuntu1

Changeset: https://review.fuel-infra.org/4445
project: packages/trusty/rethtool
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Update rethtool to most recent 0.0.5 version
status: change-merged

Files placed on repository:

Changeset merged. Package placed on primary repository
 repository URL: /trusty-fuel-6.1-stable/

Changed in fuel:
assignee: Alexei Sheplyakov (asheplyakov) → Aleksander Mogylchenko (amogylchenko)
Revision history for this message
Aleksander Mogylchenko (amogylchenko) wrote :
Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Aviram Bar-Haim (aviramb) wrote :

Is there a reason why rubygem-rethtool-0.0.3-3.mira2.noarch.rpm was downgraded to http://fuel-repository.mirantis.com/fwm/6.1/centos/os/x86_64/Packages/rubygem-rethtool-0.0.3-2.mira1.noarch.rpm ?

Changed in fuel:
status: Fix Committed → Confirmed
Changed in fuel:
importance: Medium → High
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

Downgraded packages was fixed at OBS host. Waiting for BVTs.

Revision history for this message
Aleksander Mogylchenko (amogylchenko) wrote :
Changed in fuel:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.