fuel-createmirror doesn't upgrade during upgrade of fuel master from 6.1 to 7.0

Bug #1496762 reported by Sergey Murashov
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Matthew Mosesohn
7.0.x
Fix Released
Critical
Matthew Mosesohn
8.0.x
Fix Released
High
Matthew Mosesohn

Bug Description

api: '1.0'
astute_sha: a717657232721a7fafc67ff5e1c696c9dbeb0b95
auth_required: true
build_id: '288'
build_number: '288'
feature_groups:
- mirantis
fuel-agent_sha: 082a47bf014002e515001be05f99040437281a2d
fuel-library_sha: 121016a09b0e889994118aa3ea42fa67eabb8f25
fuel-nailgun-agent_sha: d7027952870a35db8dc52f185bb1158cdd3d1ebd
fuel-ostf_sha: 1f08e6e71021179b9881a824d9c999957fcc7045
fuelmain_sha: 6b83d6a6a75bf7bca3177fcf63b2eebbf1ad0a85
nailgun_sha: 93477f9b42c5a5e0506248659f40bebc9ac23943
openstack_version: 2015.1.0-7.0
production: docker
python-fuelclient_sha: 1ce8ecd8beb640f2f62f73435f4e18d1469979ac
release: '7.0'
Steps to reproduce:
1) Install master 6.1
2) Download and untar upgrade tar ball
3) run upgrade.sh

Actual result:
we can find /opt/fuel-createmirror-6.1 but fuel-createmirror for 7.0 isn't presented

Changed in fuel:
importance: Undecided → High
assignee: nobody → Fuel Python Team (fuel-python)
milestone: none → 8.0
status: New → Confirmed
Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

The host-upgrade.pp upgrades all packages on master node. So move it to Fuel Library.

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fuel Library Team (fuel-library)
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

host-upgrade.pp ensures some packages are installed (and updated), but we can't ensure every single package installed is updated. We need a yum update -y in fuel-upgrade to accomplish this. yum update in puppet is a bad idea.

Output of yum check-update after upgrade:
http://paste.openstack.org/show/VDVyrUkVgCUEuSKPaG6X/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/224573

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/7.0)

Fix proposed to branch: stable/7.0
Review: https://review.openstack.org/224574

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

7.0 part is marked critical because otherwise it won't get included in the release

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

We should update all the OS packages during the Fuel upgrade process. This is important for security reasons and so that we don't find accidental regressions when running 7.0 with 6.1 OS packages.

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

While https://review.openstack.org/224573 and https://review.openstack.org/224574 are in progress, passing back to Igor K for fuel-upgrade part. Note critical state for 7.0.

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Adding feature-upgrade and non-release. Both my solution and the one intended for fuel-upgrade are not reflected in 7.0 ISO.

tags: added: feature-upgrade non-release
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Release notes details:
There are several packages that do not get updated during the Fuel upgrade process. Run yum -y update to install Mirantis OpenStack 7.0 system packages. Note there are approximately 190 packages to update and will take several minutes to complete. You will need to restart your Fuel Master host in order to apply the kernel update.

tags: added: release-notes
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (stable/7.0)

Change abandoned by Matthew Mosesohn (<email address hidden>) on branch: stable/7.0
Review: https://review.openstack.org/224574
Reason: Workaround in release notes for 7.0

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

Fix proposed to branch: master
Review: https://review.openstack.org/225627

tags: added: on-verification
tags: removed: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (master)

Change abandoned by Matthew Mosesohn (<email address hidden>) on branch: master
Review: https://review.openstack.org/224573
Reason: fixing in fuel-main instead

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/225627
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=4fb71f709a34712cfb37ea08a6f5525909dc18a4
Submitter: Jenkins
Branch: master

commit 4fb71f709a34712cfb37ea08a6f5525909dc18a4
Author: Matthew Mosesohn <email address hidden>
Date: Mon Sep 21 13:47:59 2015 +0300

    Add fuel-createmirror to fuel metapackage

    Adding fuel-createmirror as a dependency
    enables automatic upgrade when installing
    or updating Fuel from a previous release.

    Change-Id: Iaaaa728ed26a74b910039351b3b1bcf053fa387e
    Closes-Bug: #1496762

Changed in fuel:
status: In Progress → Fix Committed
tags: added: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-docs (master)

Reviewed: https://review.openstack.org/226839
Committed: https://git.openstack.org/cgit/stackforge/fuel-docs/commit/?id=85fe56d85541b5374b8c36fc52e6fa38136ef58b
Submitter: Jenkins
Branch: master

commit 85fe56d85541b5374b8c36fc52e6fa38136ef58b
Author: evkonstantinov <email address hidden>
Date: Wed Sep 23 17:53:33 2015 +0300

    Add fuel-createmirror issue to relnotes 7.0

    Related-Bug:#1496762
    Change-Id: I7640e6e90f46b302afa527027f2877a55b320ed1

Revision history for this message
Alexander Bochkarev (abochkarev) wrote :

Verified. After upgrading fuel 6.1 to fuel 7.0 (fuel-7.0-upgrade-298-2015-09-17_20-02-11.tar.lrz was used for upgrade) and executing 'yum -y update', fuel-createmirror for 7.0 was presented.

tags: removed: on-verification
Revision history for this message
Jason Venner (jvenner-e) wrote :

[root@fuel ~]# ls -l /usr/bin/fuel-createmirror
lrwxrwxrwx 1 root root 44 Oct 3 17:24 /usr/bin/fuel-createmirror -> /opt/fuel-createmirror-6.1/fuel-createmirror
[root@fuel ~]# ls -l /opt
total 8
drwxr-xr-x. 3 root root 4096 Oct 3 17:24 fuel-createmirror-6.1
drwxr-xr-x 4 root root 4096 Oct 3 17:23 fuel-createmirror-7.0
[root@fuel ~]#
[root@fuel ~]# ls -l /opt/fuel-createmirror-6.1
total 4
drwxr-xr-x. 2 root root 4096 Oct 3 17:24 config

6.1 fuel-createmirror partially removed and fuel 7.0 fuel-createmirror link in /usr/bin broken

tags: added: on-verification
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Revision history for this message
Veronica Krayneva (vkrayneva) wrote :
tags: removed: on-verification
tags: removed: release-notes
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.