Fuel CLI failed to execute 'fuel --fuel-version' if keystone password was changed from default

Bug #1399608 reported by Dennis Dmitriev on 2014-12-05
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Medium
Vladimir Sharshov
Mitaka
Medium
Vladimir Sharshov
Newton
Medium
Vladimir Sharshov

Bug Description

If keystone password was changed, CLI works fine, but one option failed:

=========== password changed to 'adminn'
[root@nailgun ~]# fuel --user admin --password adminn --fuel-version

            Unauthorized: need authentication!
            Please provide user and password via client
             fuel --user=user --password=pass [action]
            or modify "KEYSTONE_USER" and "KEYSTONE_PASS" in
            /etc/fuel/client/config.yaml

=========== password changed to 'adminn'
[root@nailgun ~]# fuel --user admin --password adminn release
id | name | state | operating_system | version
---|------------------------|-----------|------------------|-----------
2 | Juno on Ubuntu 12.04.4 | available | Ubuntu | 2014.2-6.0
1 | Juno on CentOS 6.5 | available | CentOS | 2014.2-6.0

{"build_id": "2014-11-23_10-54-46", "ostf_sha": "a35f516f1606b0d03d51ff63bfe3fbe23de4b622", "build_number": "130", "auth_required": true, "api": "1.0", "nailgun_sha": "7196c478bfe0b5cff97077c1829009b9dbc3ee92", "production": "docker", "fuelmain_sha": "0dd338b6280843b4c45b46635528a6c43939c772", "astute_sha": "c15623d05ccdf7ac10873e7a90df954de8726280", "feature_groups": ["mirantis"], "release": "6.0", "release_versions": {"2014.2-6.0": {"VERSION": {"build_id": "2014-11-23_10-54-46", "ostf_sha": "a35f516f1606b0d03d51ff63bfe3fbe23de4b622", "build_number": "130", "api": "1.0", "nailgun_sha": "7196c478bfe0b5cff97077c1829009b9dbc3ee92", "production": "docker", "fuelmain_sha": "0dd338b6280843b4c45b46635528a6c43939c772", "astute_sha": "c15623d05ccdf7ac10873e7a90df954de8726280", "feature_groups": ["mirantis"], "release": "6.0", "fuellib_sha": "2bfa9431a4839efae9c75d6b133df24c0f11c868"}}}, "fuellib_sha": "2bfa9431a4839efae9c75d6b133df24c0f11c868"}

Nastya Urlapova (aurlapova) wrote :

Does it mean that all actions through fuel client are failed if user changed password?

Changed in fuel:
milestone: 6.1 → none
summary: - Fuel CLI faield to execute 'fuel --fuel-version' if keystone password
+ Fuel CLI failed to execute 'fuel --fuel-version' if keystone password
was changed from default
Stanislaw Bogatkin (sbogatkin) wrote :

Reproduced on fuel 6.0 build 554.

Nastya, nope.
I can reproduce this only for one command too: "fuel --fuel-version". Seems that other commands work well.

Changed in fuel:
importance: Undecided → Medium
status: New → Confirmed
milestone: none → 6.1
Dmitry Pyzhov (dpyzhov) on 2015-03-27
tags: added: module-client
removed: cli
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Vladimir Sharshov (vsharshov)
Vladimir Sharshov (vsharshov) wrote :

Moved to 7.0.

--fuel-version release as separate call instead of any other actions. That is why it do not work with non-default credentials.

Changed in fuel:
assignee: Vladimir Sharshov (vsharshov) → Fuel Python Team (fuel-python)
milestone: 6.1 → 7.0
tags: added: release-notes
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Vladimir Sharshov (vsharshov)

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

Changed in fuel:
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/202049
Committed: https://git.openstack.org/cgit/stackforge/python-fuelclient/commit/?id=6deb1c7daf8038f52d71ddcc8688583ccb0d7882
Submitter: Jenkins
Branch: master

commit 6deb1c7daf8038f52d71ddcc8688583ccb0d7882
Author: Vladimir Sharshov (warpc) <email address hidden>
Date: Wed Jul 15 13:56:19 2015 +0300

    Fuel version as full action instead of option

    Implementation for both cases:

    - as action for fuel v1;
    - as command for fuel v2.

    Add deprecation message for --fuel-version
    in 7.0 release.

    Also solved several problems:

    - support custom credential;
    - support interective mode;
    - support unified scheme of api queries.

    Change-Id: I915d3b5ff265883883fbf92786c3cbc860ec4128
    Closes-Bug: #1474767
    Closes-Bug: #1399608
    DocImpact

Changed in fuel:
status: In Progress → Fix Committed
tags: added: on-verification
Changed in fuel:
status: Fix Committed → Confirmed

verified on ISO # 301

Steps to reproduce
1)change admin password in UI from "admin" to "adminn"
2)
[root@fuel-lab-cz5558 ~]# fuel --user admin --password adminn release
DEPRECATION WARNING: /etc/fuel/client/config.yaml exists and will be used as the source for settings. This behavior is deprecated. Please specify the path to your custom settings file in the FUELCLIENT_CUSTOM_SETTINGS environment variable.
id | name | state | operating_system | version
---|----------------------|-------------|------------------|-------------
2 | Kilo on Ubuntu 14.04 | available | Ubuntu | 2015.1.0-7.0
1 | Kilo on CentOS 6.5 | unavailable | CentOS | 2015.1.0-7.0

3)
[root@fuel-lab-cz5558 ~]# fuel --user admin --password adminn --fuel-version
DEPRECATION WARNING: /etc/fuel/client/config.yaml exists and will be used as the source for settings. This behavior is deprecated. Please specify the path to your custom settings file in the FUELCLIENT_CUSTOM_SETTINGS environment variable.

            Unauthorized: need authentication!
            Please provide user and password via client
             fuel --user=user --password=pass [action]
            or modify "KEYSTONE_USER" and "KEYSTONE_PASS" in
            /etc/fuel/client/config.yaml

Also present next issue

Available to execute fuel --fuel-version command with wrong password , when it doesn't changed from default

[root@fuel-lab-cz5558 ~]# fuel --user admin --password wrong --fuel-version
DEPRECATION WARNING: /etc/fuel/client/config.yaml exists and will be used as the source for settings. This behavior is deprecated. Please specify the path to your custom settings file in the FUELCLIENT_CUSTOM_SETTINGS environment variable.
api: '1.0'
astute_sha: 6c5b73f93e24cc781c809db9159927655ced5012
auth_required: true
build_id: '301'
build_number: '301'
feature_groups:
- mirantis
fuel-agent_sha: 50e90af6e3d560e9085ff71d2950cfbcca91af67
fuel-library_sha: 5d50055aeca1dd0dc53b43825dc4c8f7780be9dd
fuel-nailgun-agent_sha: d7027952870a35db8dc52f185bb1158cdd3d1ebd
fuel-ostf_sha: 2cd967dccd66cfc3a0abd6af9f31e5b4d150a11c
fuelmain_sha: a65d453215edb0284a2e4761be7a156bb5627677
nailgun_sha: 4162b0c15adb425b37608c787944d1983f543aa8
openstack_version: 2015.1.0-7.0
production: docker
python-fuelclient_sha: 486bde57cda1badb68f915f66c61b544108606f3
release: '7.0'
release_versions:
  2015.1.0-7.0:
    VERSION:
      api: '1.0'
      astute_sha: 6c5b73f93e24cc781c809db9159927655ced5012
      build_id: '301'
      build_number: '301'
      feature_groups:
      - mirantis
      fuel-agent_sha: 50e90af6e3d560e9085ff71d2950cfbcca91af67
      fuel-library_sha: 5d50055aeca1dd0dc53b43825dc4c8f7780be9dd
      fuel-nailgun-agent_sha: d7027952870a35db8dc52f185bb1158cdd3d1ebd
      fuel-ostf_sha: 2cd967dccd66cfc3a0abd6af9f31e5b4d150a11c
      fuelmain_sha: a65d453215edb0284a2e4761be7a156bb5627677
      nailgun_sha: 4162b0c15adb425b37608c787944d1983f543aa8
      openstack_version: 2015.1.0-7.0
      production: docker
      python-fuelclient_sha: 486bde57cda1badb68f915f66c61b544108606f3
      release: '7.0'

tags: removed: on-verification
Igor Shishkin (teran) on 2015-10-08
Changed in fuel:
milestone: 7.0 → 8.0
Dmitry Pyzhov (dpyzhov) on 2015-10-22
tags: added: area-python
Dmitry Pyzhov (dpyzhov) wrote :

We passed SCF in 8.0. Moving the bug to 9.0.

Changed in fuel:
milestone: 8.0 → 9.0

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info

Related fix proposed to branch: master
Change author: Evgeny Konstantinov <email address hidden>
Review: https://review.fuel-infra.org/22324

tags: added: release-notes-done
removed: release-notes
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Related blueprints