FFU: openstack overcloud ffwd-upgrade run doesn't work with custom non-default --ssh-user option

Bug #1769867 reported by Marios Andreou on 2018-05-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
High
Carlos Camacho

Bug Description

(first discussed in https://bugzilla.redhat.com/show_bug.cgi?id=1575445)

 Marius Cornea 2018-05-07 03:36:07 EEST

Description of problem:
FFU: openstack overcloud ffwd-upgrade run doesn't work with custom non-default --ssh-user:

openstack overcloud ffwd-upgrade run --ssh-user admin --yes

Warning! The TripleO Fast Forward Upgrade workflow is currently considered under development. In particular invocations of the ffwd-upgrade cli should be initially limited to development/test environments. Once and if you decide to use ffwd-upgrade in production, ensure you are adequately prepared with valid backup of your current deployment state.
Continuing fast forward upgrade
Started Mistral Workflow tripleo.package_update.v1.update_nodes. Execution ID: 13c40cc7-41d3-428e-ac79-96f4d64580f5
Waiting for messages on queue 'ffwdupgrade' with no timeout.
[u'Using /tmp/ansible-mistral-actionhN4lEx/ansible.cfg as config file',
 u' [WARNING]: Skipping unexpected key (hostvars) in group (_meta), only "vars",',
 u'"children" and "hosts" are valid',
 u'',
 u'PLAY [overcloud] ***************************************************************',
 u'',
 u'TASK [Gathering Facts] *********************************************************',
 u'fatal: [192.168.0.14]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh: Could not create directory \'/home/mistral/.ssh\'.\\r\\nWarning: Permanently added \'192.168.0.14\' (ECDSA) to the list of known hosts.\\r\\nPermission denied (publickey,gssapi-keyex,gssapi-with-mic).\\r\\

Additional info:
We can actually see in the generated inventory.yaml that the default 'heat-admin' was used not the custom one:

[root@undercloud-0 stack]# grep ansible_ssh_user /tmp/ansible-mistral-action815Z96/inventory.yaml
    ansible_ssh_user: heat-admin
    ansible_ssh_user: heat-admin
    ansible_ssh_user: heat-admin

Changed in tripleo:
assignee: Marios Andreou (marios-b) → Carlos Camacho (ccamacho)
Changed in tripleo:
assignee: Carlos Camacho (ccamacho) → Marios Andreou (marios-b)
Changed in tripleo:
assignee: Marios Andreou (marios-b) → Carlos Camacho (ccamacho)

Reviewed: https://review.openstack.org/566859
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=8863b5d679e54742b0d5efb6ea3637432521061d
Submitter: Zuul
Branch: master

commit 8863b5d679e54742b0d5efb6ea3637432521061d
Author: mandreou <email address hidden>
Date: Tue May 8 14:16:08 2018 +0300

    Refactor, fix ansible inventory method parameters for ffwd-upgrade run

    This submission had the correct fix for the bug 1769867.

    [0] was accidentaly stepping over this submission
    to fix the parameter configuration. Still we need
    the fix for configuring the parameters correctly
    also fixing the unit tests.

    The method parameters wasn't being passed into the
    tripleo ansible inventory generation.
    Also discussed in [1].

    Related-Bug: 1769867
    [1]: https://bugzilla.redhat.com/show_bug.cgi?id=1575445
    [0]: https://review.openstack.org/#/c/566960/

    Change-Id: I7c5347ef31aa002f22bcaa60fcbfff058ae8acb9

Changed in tripleo:
importance: Undecided → High

Reviewed: https://review.openstack.org/567233
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=245eb695fb15c1cd0beea1c2cd91aa6de789f14b
Submitter: Zuul
Branch: stable/queens

commit 245eb695fb15c1cd0beea1c2cd91aa6de789f14b
Author: mandreou <email address hidden>
Date: Tue May 8 14:16:08 2018 +0300

    Refactor, fix ansible inventory method parameters for ffwd-upgrade run

    This submission had the correct fix for the bug 1769867.

    [0] was accidentaly stepping over this submission
    to fix the parameter configuration. Still we need
    the fix for configuring the parameters correctly
    also fixing the unit tests.

    The method parameters wasn't being passed into the
    tripleo ansible inventory generation.
    Also discussed in [1].

    Related-Bug: 1769867
    [1]: https://bugzilla.redhat.com/show_bug.cgi?id=1575445
    [0]: https://review.openstack.org/#/c/566960/
    (cherry picked from commit d0673159e0b412c051abfdf0aa4b91944211e270)
    Change-Id: I7c5347ef31aa002f22bcaa60fcbfff058ae8acb9

tags: added: in-stable-queens
Changed in tripleo:
milestone: rocky-2 → rocky-3
Changed in tripleo:
milestone: rocky-3 → rocky-rc1
Changed in tripleo:
milestone: rocky-rc1 → stein-1
Changed in tripleo:
milestone: stein-1 → stein-2
Changed in tripleo:
milestone: stein-2 → stein-3
Changed in tripleo:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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