Fuel for OpenStack

[fuel-octane] Files should be deleted if package removed

Bug #1619319 reported by Dmitry Belyaninov on 2016-09-01

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Fuel for OpenStack	Fix Committed	High	Alexander Tsamutali	Fuel for OpenStack next
	Mitaka	Fix Released	High	Alexander Tsamutali	Fuel for OpenStack 9.1

Bug Description

Detailed bug description:
There are empty directories after removing fuel-octane package

Steps to reproduce:
yum install fuel-octane
yum remove fuel-octane
ls /usr/lib/python2.7/site-packages/octane
[root@nailgun ~]# ll /usr/lib/python2.7/site-packages/octane/
total 36
drwxr-xr-x. 2 root root 4096 Sep 1 13:49 commands
drwxr-xr-x. 2 root root 4096 Sep 1 13:49 fuelclient
drwxr-xr-x. 5 root root 4096 Sep 1 13:49 handlers
drwxr-xr-x. 2 root root 4096 Sep 1 13:49 helpers
drwxr-xr-x. 2 root root 4096 Sep 1 13:49 patches
drwxr-xr-x. 2 root root 12288 Sep 1 13:49 tests
drwxr-xr-x. 2 root root 4096 Sep 1 13:49 util

Expected results:
No octane folder
Actual result:
Few empty subfolders
Reproducibility:
yes
Workaround:
-
Impact:
-
Description of the environment:
-
Additional information:

the issue is the same as:

https://bugs.launchpad.net/fuel/+bug/1614926

See original description

Tags:

Ilya Kharin (akscram) on 2016-09-01

tags:

added: area-packaging

Dmitry Belyaninov (dbelyaninov) on 2016-09-01

description:

updated

Ilya Kharin (akscram) on 2016-09-01

Changed in fuel:
milestone:	none → 9.1
status:	New → Confirmed
importance:	Undecided → High
assignee:	nobody → MOS Packaging Team (mos-packaging)
milestone:	9.1 → next

Igor Yozhikov (iyozhikov) on 2016-09-05

Changed in fuel:
assignee:	MOS Packaging Team (mos-packaging) → Alexander Tsamutali (astsmtl)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix proposed to fuel-octane (master)

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

Changed in fuel:
status:	Confirmed → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix proposed to fuel-octane (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/367590

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix merged to fuel-octane (master)

Reviewed: https://review.openstack.org/367586
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=763b69b97645752b4b08253751962687d50cf1be
Submitter: Jenkins
Branch: master

commit 763b69b97645752b4b08253751962687d50cf1be
Author: Alexander Tsamutali <email address hidden>
Date: Thu Sep 8 22:03:24 2016 +0300

Cleanup %files section in spec

Don't use --record during install, don't use -f in %files. Specify
octane files/directories explicitly.

Change-Id: I84f0d71a2a582b3a23fc048a331d6caae775e38b
Closes-Bug: #1619319

Changed in fuel:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix merged to fuel-octane (stable/mitaka)

Reviewed: https://review.openstack.org/367590
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=ae03d07ba1a1d3c50f1bd356a34849c04b56df80
Submitter: Jenkins
Branch: stable/mitaka

commit ae03d07ba1a1d3c50f1bd356a34849c04b56df80
Author: Alexander Tsamutali <email address hidden>
Date: Thu Sep 8 22:03:24 2016 +0300

Cleanup %files section in spec

Don't use --record during install, don't use -f in %files. Specify
octane files/directories explicitly.

Change-Id: I84f0d71a2a582b3a23fc048a331d6caae775e38b
Closes-Bug: #1619319

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix proposed to fuel-octane (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/367652

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix merged to fuel-octane (stable/mitaka)

Download full text (4.3 KiB)

Reviewed: https://review.openstack.org/367652
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=293b79e6490605d9e74d2e2b6825146b49fdecfb
Submitter: Jenkins
Branch: stable/mitaka

commit 9a4817174ed8b0702564baadc1728eaf8b778e28
Author: Ilya Kharin <email address hidden>
Date: Sat Sep 3 01:04:10 2016 +0300

Move a set of nodes at once

Before this patch octane made separate calls to move nodes one by one.

Change-Id: I999a98d57b3184d35972e4862fcb4f284a066e9e
Related-Bug: #1616925

commit 763b69b97645752b4b08253751962687d50cf1be
Author: Alexander Tsamutali <email address hidden>
Date: Thu Sep 8 22:03:24 2016 +0300

Cleanup %files section in spec

Don't use --record during install, don't use -f in %files. Specify
octane files/directories explicitly.

Change-Id: I84f0d71a2a582b3a23fc048a331d6caae775e38b
Closes-Bug: #1619319

commit e7fad96f4125386df4e5c9b7f0744de185de6c33
Author: Sergey Abramov <email address hidden>
Date: Mon Sep 5 18:24:37 2016 +0300

Osd upgrade failed if version not changed

ceph deploy not raise exception if it doesn't upgrade osd version.

Change-Id: Ifcddd822228d78166d59b2ba49852be2e51c79fc
Closes-bug: 1620277

commit 349073cbe399184daee672f75b4fe9941ae3c5da
Author: Sergey Abramov <email address hidden>
Date: Wed Sep 7 18:20:33 2016 +0300

Remove patch pupet on upgrade osd

this is not required after merge
https://review.openstack.org/#/c/203639/

Change-Id: I67fbcd77ab3437443219c34ee3ddaf7895b068ce
Closes-bug: 1621436

commit 807c5166a8a0145623d41c3110007e33a1402c47
Author: Alexey Stepanov <email address hidden>
Date: Wed Sep 7 17:53:40 2016 +0300

Stop waiting status change on node with stopped status

Stopped status, if not expected, should be a reason for error on nodes
Closes-bug: #1621069

Change-Id: I0156d694ef20ece8603e3d840f085852a528e635

commit 5f689f3d6904e092998a0d8ba124280833452c30
Author: Anastasiya <email address hidden>
Date: Tue Sep 6 10:23:39 2016 +0300

Backup/restore for admin networks

* backup/restore for /etc/hiera/networks.yaml was added
* configure dhcp after restore was added

Change-Id: I5b1e3861589e1c56acbc37d0be569da5e55b8536
Closes-Bug: #1616998

commit e4820087678d83d9477db4c1688137ee5ff66c3f
Author: Pavel Chechetin <email address hidden>
Date: Thu Aug 18 14:46:59 2016 +0300

Graph-based upgrade-ceph. Python part.

Change-Id: Icb4d543bd6801f21c6aca57415105f88a601c0c2

commit 0af1b6517a97589a401328f09f3bf56d84db9dbd
Author: Pavel Chechetin <email address hidden>
Date: Tue Sep 6 13:13:42 2016 +0300

Graph-based upgrade-ceph. Puppet part.

     - Graphs and puppet manifests part
     - Delete lib from .gitignore
     - Augeas lens for Ceph is copied and pasted, should be switched to the
       version from the upstream when [0] is merged and published.

[0] https://github.com/hercules-team/augeas/pull/401

Change-Id: I639cbf786971fea8c56b4da6b2661477b3b12c41

commit 3315d741f295635303413ad839b3b66a0dac3282
Author: Alexey Stepanov <penguinolog@gmail....

Reviewed:  https://review.openstack.org/367652
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=293b79e6490605d9e74d2e2b6825146b49fdecfb
Submitter: Jenkins
Branch:    stable/mitaka

commit 9a4817174ed8b0702564baadc1728eaf8b778e28
Author: Ilya Kharin <akscram@gmail.com>
Date:   Sat Sep 3 01:04:10 2016 +0300

Move a set of nodes at once
    
    Before this patch octane made separate calls to move nodes one by one.
    
    Change-Id: I999a98d57b3184d35972e4862fcb4f284a066e9e
    Related-Bug: #1616925

commit 763b69b97645752b4b08253751962687d50cf1be
Author: Alexander Tsamutali <atsamutali@mirantis.com>
Date:   Thu Sep 8 22:03:24 2016 +0300

Cleanup %files section in spec
    
    Don't use --record during install, don't use -f in %files. Specify
    octane files/directories explicitly.
    
    Change-Id: I84f0d71a2a582b3a23fc048a331d6caae775e38b
    Closes-Bug: #1619319

commit e7fad96f4125386df4e5c9b7f0744de185de6c33
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Sep 5 18:24:37 2016 +0300

Osd upgrade failed if version not changed
    
    ceph deploy not raise exception if it doesn't upgrade osd version.
    
    Change-Id: Ifcddd822228d78166d59b2ba49852be2e51c79fc
    Closes-bug: 1620277

commit 349073cbe399184daee672f75b4fe9941ae3c5da
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Wed Sep 7 18:20:33 2016 +0300

Remove patch pupet on upgrade osd
    
    this is not required after merge
    https://review.openstack.org/#/c/203639/
    
    Change-Id: I67fbcd77ab3437443219c34ee3ddaf7895b068ce
    Closes-bug: 1621436

commit 807c5166a8a0145623d41c3110007e33a1402c47
Author: Alexey Stepanov <astepanov@mirantis.com>
Date:   Wed Sep 7 17:53:40 2016 +0300

Stop waiting status change on node with stopped status
    
    Stopped status, if not expected, should be a reason for error on nodes
    Closes-bug: #1621069
    
    Change-Id: I0156d694ef20ece8603e3d840f085852a528e635

commit 5f689f3d6904e092998a0d8ba124280833452c30
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Tue Sep 6 10:23:39 2016 +0300

Backup/restore for admin networks
    
    * backup/restore for /etc/hiera/networks.yaml was added
    * configure dhcp after restore was added
    
    Change-Id: I5b1e3861589e1c56acbc37d0be569da5e55b8536
    Closes-Bug: #1616998

commit e4820087678d83d9477db4c1688137ee5ff66c3f
Author: Pavel Chechetin <pchechetin@mirantis.com>
Date:   Thu Aug 18 14:46:59 2016 +0300

Graph-based upgrade-ceph. Python part.
    
    Change-Id: Icb4d543bd6801f21c6aca57415105f88a601c0c2

commit 0af1b6517a97589a401328f09f3bf56d84db9dbd
Author: Pavel Chechetin <pchechetin@mirantis.com>
Date:   Tue Sep 6 13:13:42 2016 +0300

Graph-based upgrade-ceph. Puppet part.
    
     - Graphs and puppet manifests part
     - Delete lib from .gitignore
     - Augeas lens for Ceph is copied and pasted, should be switched to the
       version from the upstream when [0] is merged and published.
    
    [0] https://github.com/hercules-team/augeas/pull/401
    
    Change-Id: I639cbf786971fea8c56b4da6b2661477b3b12c41

commit 3315d741f295635303413ad839b3b66a0dac3282
Author: Alexey Stepanov <penguinolog@gmail.com>
Date:   Wed Sep 7 09:38:04 2016 +0300

Colorize output on tty only
    
    Check ConsoleHandler stream for tty and colorize only if True.
    On CI colorized output is not processed and metacharacters
    simply reproduced in output as-is.
    
    Change-Id: I802e3d338c318d6fbb61dd556e13a2fad7eda121

commit 1d87cfefa8c7f93bd72ac8a0236d128ced1e8c0e
Author: Nguyen Hung Phuong <phuongnh@vn.fujitsu.com>
Date:   Wed Sep 7 11:46:45 2016 +0700

Clean imports in code
    
    In some part in the code we import objects. In the Openstack style
    guidelines they recommend to import only modules.
    
    http://docs.openstack.org/developer/hacking/#imports
    
    Change-Id: I5f5add3ad87c8e9b3a6d781c81c3bedf5b4e1c9c

commit dcb147e11b0fabcfe1acfbc44ca7503c5a717264
Author: Ilya Kharin <akscram@gmail.com>
Date:   Tue Sep 6 23:16:08 2016 +0300

Use graph_types to reflect changes in fuelclient
    
    The GraphClient.execute method was changed and this patch compensates
    such changes. The interface of GraphClient was changed by the next
    change request:
    
        Iafc59c60a1ebce4a360ce1faa00a27ab320fa90b
    
    Change-Id: I68523afe49de6da9f9792200419dcc35843b07d8
    Closes-Bug: #1620809

Dmitry Belyaninov (dbelyaninov) on 2016-09-09

tags:

added: on-verification

Revision history for this message

Dmitry Belyaninov (dbelyaninov) wrote on 2016-09-09:

Verified on snapshot #239

tags:

removed: on-verification

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2021-09-06: Fix included in openstack/fuel-octane ocata-eol

This issue was fixed in the openstack/fuel-octane ocata-eol release.

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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