fuel-ostf isn't ready for updated master node / OpenStack

Bug #1516621 reported by Dmitry Teselkin
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Artem Roma

Bug Description

During update of master node to newer version of CentOS / OpenStack we found that fuel-ostf isn't ready for it.

We found the following issues:
* it uses old oslo namespaces:
  * oslo.config
  * olso.serialization
* Incorrect project name specified in cfg.CONF (fuel_plugin/ostf_adapter/config.py:101)
* fuel-ostf uses deprecated _handle_get call (class RestController) in pecan library. According to global-requirements pecan can be at least of version 1.0.0
* python dependencies include obsolete components that should be removed:
  * argparse
* RPM spec contains upperbounds for some requirements, which isn't good from upgradability point ov view.

We've collected all fixes and hacks in one code review - https://review.openstack.org/#/c/237491/

Changed in fuel:
assignee: Fuel for Openstack (fuel) → nobody
Ilya Kutukov (ikutukov)
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
importance: Undecided → Wishlist
status: New → Confirmed
status: Confirmed → New
Revision history for this message
Dmitry Teselkin (teselkin-d) wrote :

It's not a whishlist, it's a *blocker* for CentOS7 master node.

Changed in fuel:
importance: Wishlist → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-ostf (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/245863

Revision history for this message
Dmitry Teselkin (teselkin-d) wrote :
Changed in fuel:
milestone: none → 8.0
Ilya Kutukov (ikutukov)
Changed in fuel:
status: New → Confirmed
Dmitry Pyzhov (dpyzhov)
tags: added: regression-8.0
tags: added: area-python
Artem Roma (aroma-x)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Artem Roma (aroma-x)
Dmitry Pyzhov (dpyzhov)
tags: added: team-linux
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: master
Review: https://review.openstack.org/248800

Mike Scherbakov (mihgen)
tags: added: tech-debt
Dmitry Pyzhov (dpyzhov)
tags: removed: tech-debt
tags: added: tech-debt
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-ostf (master)

Reviewed: https://review.openstack.org/245863
Committed: https://git.openstack.org/cgit/openstack/fuel-ostf/commit/?id=fc6648be21f26fac4f66ef18ba36f521960d554b
Submitter: Jenkins
Branch: master

commit fc6648be21f26fac4f66ef18ba36f521960d554b
Author: Dmitry Teselkin <email address hidden>
Date: Mon Nov 16 19:12:24 2015 +0300

    Fix project name

    Blueprint: master-on-centos7

    Compatible with CentOS6 and CentOS7 master node

    Related-Bug: #1516621

    Closes-Bug: #1520634

    Change-Id: Ifc378a70b8997330754f4faca4fa66808849fef3

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/245850
Committed: https://git.openstack.org/cgit/openstack/fuel-ostf/commit/?id=a98973482f839554d90cc1c071d625a01e018cfe
Submitter: Jenkins
Branch: master

commit a98973482f839554d90cc1c071d625a01e018cfe
Author: Dmitry Teselkin <email address hidden>
Date: Mon Nov 16 18:55:02 2015 +0300

    Support for new oslo namespaces

    Compatible with CentOS6 & CentOS7 master node

    Blueprint: master-on-centos7

    Related-Bug: #1516621

    Change-Id: I06bf0dc8406ac6551632702f90b427f5a42f8088

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/248800
Committed: https://git.openstack.org/cgit/openstack/fuel-ostf/commit/?id=bfa90e5caaad7264830991458f88b8015339c687
Submitter: Jenkins
Branch: master

commit bfa90e5caaad7264830991458f88b8015339c687
Author: Artem Roma <email address hidden>
Date: Mon Nov 23 17:32:11 2015 +0200

    Adhere python package requirements to Liberty global-requirements

    What was done:
    * versions of python packages that are requirements for the project
      changed to those from Liberty release;
    * custom BaseRestController class changed to correlate with
      corresponding code of used pecan package version;
    * fixed code of the orm models that become broken after installing
      SQLAlchemy package of different version

    Blueprint: master-on-centos7

    Is NOT compatible with CentOS6 master node

    Related-Bug: #1516621

    Change-Id: I9f5e600a0189d0d9fc0ae5b154254dc3a32a86c1

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: feature-centos7
Revision history for this message
Alexandr Kostrikov (akostrikov-mirantis) wrote :

Fuel-ostf is passing.

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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