fuelclient: ha and ha_compact misunderstanding

Bug #1441129 reported by Vladimir Sharshov
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Roman Prykhodchenko

Bug Description

Default env mode 'ha_compact'

[root@fuel ~]# fuel env create --name MyEnv --rel 1
Environment 'MyEnv' with id=1, mode=ha_compact and network-mode=nova_network was created!

But fuel as cli expected only 2 possible mode: ha or multinode. We know that ha_compact is ha in terms of fuel cli, but such historical naming is misunderstanding.

4 possible options:

- rename ha_compact to ha in Nailgun;
- add ha_compact as synonym for ha in Fuel CLI;
- rename ha to ha_compact in Fuel CLI;
- do nothing.

Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Seems it's easier to rename it in CLI to not change UI and to not write migrations in Nailgun.

Changed in fuel:
status: New → Confirmed
Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Also, it's not needed to set it for 6.1 environments as 'multinode' is not supported anymore. So, it's relevant in case of upgrade only.

tags: added: qa-agree-7.0
Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

I don't want to have "ha_compact" naming everywhere. I'd prefer to have "ha", since we support only one "ha" now.

Still, taking into account that we do not support other modes and multinode is deprecated in 6.1, I believe that we should not over-engineer here. Let's keep things "As Is" for now and remove "mode" column in 7.0 at all.

Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

Tech debt. Does not affect deployment. Moving to 8.0

tags: added: tech-debt
Changed in fuel:
status: Confirmed → Won't Fix
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
status: Won't Fix → Confirmed
no longer affects: fuel/8.0.x
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Revision history for this message
Roman Prykhodchenko (romcheg) wrote :

IIRC setting a mode for either new or an existing environment was removed from fuel2. I propose to remove all reference to that from the new fuel client because it will not work with Fuel 6.1 due to changes in the API.

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Roman Prykhodchenko (romcheg)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-fuelclient (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-fuelclient (master)

Reviewed: https://review.openstack.org/242860
Committed: https://git.openstack.org/cgit/openstack/python-fuelclient/commit/?id=ee31f93f047a54b3bd2ab5d281c214bbd53f9156
Submitter: Jenkins
Branch: master

commit ee31f93f047a54b3bd2ab5d281c214bbd53f9156
Author: Roman Prykhodchenko <email address hidden>
Date: Sun Nov 8 17:45:38 2015 +0100

    Remove deployment mode support

    Different deployment mode was deprecated in Fuel 6.1. The
    current and newer versions of python-fuelclient are not
    compatible with that version of Fuel.

    This patch removes all options and mentions of the deprecated
    feature.

    Change-Id: Ieeadc43592bd72cb813a0aa914779bd4cfe13586
    Closes-bug: #1441129

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Tatyanka (tatyana-leontovich) wrote :

verified 509 iso

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.