Fuel for OpenStack

[Ironic] Deployment failed: Missing a required parameter uids

Bug #1541256 reported by Dmitry Belyaninov on 2016-02-03

This bug affects 2 people

	Status	Importance	Assigned to	Milestone
Fuel for OpenStack	Fix Committed	High	Maciej Kwiek	Fuel for OpenStack 9.0
8.0.x	Fix Released	High	Maciej Kwiek	Fuel for OpenStack 8.0-mu-2
Mitaka	Fix Released	High	Maciej Kwiek	Fuel for OpenStack 9.0

Bug Description

Scenario:
Deploy cluster with default settings: 1 controller and 1 ironic.
Add 1 compute and redeploy <- FAIL

Error running RPC method granular_deploy: Missing a required parameter uids

But if you want to deploy full cluster(1 controller, 1 ironic, 1 compute) - it will be successful.

[root@nailgun ~]# cat /etc/fuel/version.yaml
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "507"
  build_id: "507"
  fuel-nailgun_sha: "8e954abd70ef0083109f34289de2553dcda544d4"
  python-fuelclient_sha: "4f234669cfe88a9406f4e438b1e1f74f1ef484a5"
  fuel-agent_sha: "658be72c4b42d3e1436b86ac4567ab914bfb451b"
  fuel-nailgun-agent_sha: "b2bb466fd5bd92da614cdbd819d6999c510ebfb1"
  astute_sha: "b81577a5b7857c4be8748492bae1dec2fa89b446"
  fuel-library_sha: "ec7e212972ead554f21b52b9e165156665f659df"
  fuel-ostf_sha: "ab5fd151fc6c1aa0b35bc2023631b1f4836ecd61"
  fuel-mirror_sha: "351d568fa3b3e4dd062054b91d766aa54d379867"
  fuelmenu_sha: "234cb4cbb30fbd2df00f388c28f31606d9cae15f"
  shotgun_sha: "63645dea384a37dde5c01d4f8905566978e5d906"
  network-checker_sha: "a43cf96cd9532f10794dce736350bf5bed350e9d"
  fuel-upgrade_sha: "616a7490ec7199f69759e97e42f9b97dfc87e85b"
  fuelmain_sha: "94507c5e4dad6d8cfbd8f5d41aa8389d5335990a"
[root@nailgun ~]

Note: there is the 1530291 bug with the same scenario and different result.

Tags:

Revision history for this message

Dmitry Belyaninov (dbelyaninov) wrote on 2016-02-03:

fuel-snapshot-2016-02-02_18-50-45.tar.xz Edit (29.2 MiB, application/octet-stream)

Dmitry Pyzhov (dpyzhov) on 2016-02-03

Changed in fuel:
milestone:	none → 9.0
assignee:	nobody → Fuel Python Team (fuel-python)
importance:	Undecided → High
tags:	added: area-python

Maciej Kwiek (maciej-iai) on 2016-02-03

Changed in fuel:
assignee:	Fuel Python Team (fuel-python) → Maciej Kwiek (maciej-iai)

Dmitry Pyzhov (dpyzhov) on 2016-02-03

Changed in fuel:
status:	New → Confirmed

Bulat Gaifullin (bulat.gaifullin) on 2016-02-03

Changed in fuel:
assignee:	Maciej Kwiek (maciej-iai) → Bulat Gaifullin (bgaifullin)
assignee:	Bulat Gaifullin (bgaifullin) → Maciej Kwiek (maciej-iai)

Revision history for this message

Maciej Kwiek (maciej-iai) wrote on 2016-02-03:

I was not able to reproduce this locally (I followed steps in description) on ISO 518. Also, astute.log in diagnostic snapshot doesn't show the trace, line containing

"2016-02-02 18:32:37 ERROR [778] Error running RPC method granular_deploy: Missing a required parameter uids, trace:"

is the last line of the log. Marking as incomplete.

Revision history for this message

Maciej Kwiek (maciej-iai) wrote on 2016-02-04:

I got access to env on which this bug is reproduced, investigating.

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-02-04: Fix proposed to fuel-web (master)

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-02-04: Fix proposed to fuel-web (stable/8.0)

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/276270

Revision history for this message

Dmitry Pyzhov (dpyzhov) wrote on 2016-02-05:

We in HCF state for 8.0 and don't accept High priority fixes. Moving bug to mu-1

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-02-05: Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/276165
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=2a806db75133717eee98f8ec35791d12305d01a6
Submitter: Jenkins
Branch: master

commit 2a806db75133717eee98f8ec35791d12305d01a6
Author: Maciej Kwiek <email address hidden>
Date: Thu Feb 4 12:16:51 2016 +0100

Check for uids before yielding tasks in CopyKeys hook

CopyKeys task won't be yielded if get_uids returns empty list.

Change-Id: I44c088b79bfb392d88174f8101e38a60ac6a6c67
Closes-Bug: 1541256

Changed in fuel:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-05-31: Fix merged to fuel-web (stable/8.0)

Reviewed: https://review.openstack.org/276270
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=aeea1a8a5be8bc1b22c2f7c95991cd29fdabc991
Submitter: Jenkins
Branch: stable/8.0

commit aeea1a8a5be8bc1b22c2f7c95991cd29fdabc991
Author: Maciej Kwiek <email address hidden>
Date: Thu Feb 4 12:16:51 2016 +0100

Check for uids before yielding tasks in CopyKeys hook

CopyKeys task won't be yielded if get_uids returns empty list.

Change-Id: I44c088b79bfb392d88174f8101e38a60ac6a6c67
Closes-Bug: 1541256

Kyrylo Romanenko (kromanenko) on 2016-06-14

tags:

added: on-verification

Revision history for this message

Kyrylo Romanenko (kromanenko) wrote on 2016-06-14:

Fix verified on ISO:
cat /etc/fuel_build_id:
465
cat /etc/fuel_build_number:
465
cat /etc/fuel_release:
9.0
cat /etc/fuel_openstack_version:
mitaka-9.0
rpm -qa | egrep 'fuel|astute|network-checker|nailgun|packetary|shotgun':
fuel-release-9.0.0-1.mos6349.noarch
fuel-misc-9.0.0-1.mos8454.noarch
python-packetary-9.0.0-1.mos140.noarch
fuel-bootstrap-cli-9.0.0-1.mos285.noarch
fuel-migrate-9.0.0-1.mos8454.noarch
rubygem-astute-9.0.0-1.mos750.noarch
fuel-mirror-9.0.0-1.mos140.noarch
shotgun-9.0.0-1.mos90.noarch
fuel-openstack-metadata-9.0.0-1.mos8742.noarch
fuel-notify-9.0.0-1.mos8454.noarch
nailgun-mcagents-9.0.0-1.mos750.noarch
python-fuelclient-9.0.0-1.mos325.noarch
fuel-9.0.0-1.mos6349.noarch
fuel-utils-9.0.0-1.mos8454.noarch
fuel-setup-9.0.0-1.mos6349.noarch
fuel-provisioning-scripts-9.0.0-1.mos8742.noarch
fuel-library9.0-9.0.0-1.mos8454.noarch
network-checker-9.0.0-1.mos74.x86_64
fuel-agent-9.0.0-1.mos285.noarch
fuel-ui-9.0.0-1.mos2717.noarch
fuel-ostf-9.0.0-1.mos935.noarch
fuelmenu-9.0.0-1.mos274.noarch
fuel-nailgun-9.0.0-1.mos8742.noarch

tags:

removed: on-verification

Vladimir Jigulin (vjigulin) on 2016-06-20

tags:

added: on-verification

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-06-29: Related fix merged to fuel-octane (master)

#10

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

commit fdafa2666dd2948175c8d5297727d42472dc6537
Author: Sergey Abramov <email address hidden>
Date: Mon May 30 17:40:57 2016 +0300

Patch nailgun serializer on cluster upgrade handlers

CopyKeys task won't be yielded if get_uids returns empty list.

    Change-Id: Ifd4589b6a0e9a8e8636fea9a36afa091b230ab11
    Closes-bug: 1584658
    Closes-bug: 1584041
    Related-bug: 1541256

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-06-30: Related fix proposed to fuel-octane (stable/8.0)

#11

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/335860

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-07-01: Related fix merged to fuel-octane (stable/8.0)

#12

Reviewed: https://review.openstack.org/335860
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=b991d9996f64301d3308a03dfb1bb2f00aec580e
Submitter: Jenkins
Branch: stable/8.0

commit b991d9996f64301d3308a03dfb1bb2f00aec580e
Author: Sergey Abramov <email address hidden>
Date: Mon May 30 17:40:57 2016 +0300

Patch nailgun serializer on cluster upgrade handlers

CopyKeys task won't be yielded if get_uids returns empty list.

    Change-Id: Ifd4589b6a0e9a8e8636fea9a36afa091b230ab11
    Closes-bug: 1584658
    Closes-bug: 1584041
    Related-bug: 1541256
    (cherry picked from commit fdafa2666dd2948175c8d5297727d42472dc6537)

Revision history for this message

Vladimir Jigulin (vjigulin) wrote on 2016-07-03:

#13

Verified on mos 8.0mu2

tags:

removed: on-verification

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-08-05: Related fix proposed to fuel-octane (master)

#14

Related fix proposed to branch: master
Review: https://review.openstack.org/351721

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-08-05: Change abandoned on fuel-octane (master)

#15

Change abandoned by Ilya Kharin (<email address hidden>) on branch: master
Review: https://review.openstack.org/351721
Reason: This patch have to go in stable/mitaka instead of master.

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-08-05: Related fix proposed to fuel-octane (stable/mitaka)

#16

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/351728

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-08-05: Related fix merged to fuel-octane (stable/mitaka)

#17

Download full text (17.8 KiB)

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

commit a05d2ecf13e15c12f10e1517fae425e583e0d714
Author: Nikita Zubkov <email address hidden>
Date: Thu Aug 4 17:51:56 2016 +0300

fix ssh connection auth

According to spec [1] service user must use key authentication for ssh

[1] https://github.com/openstack/fuel-specs/blame/ba4504/specs/9.0/fuel-nonroot-openstack-nodes.rst#L61-L64

Change-Id: I461f659d4f0985283a43ec6a702bdd78af91a072

commit 7140398e501879a4e1f5605f3d1cc99195464850
Author: Sergey Abramov <email address hidden>
Date: Tue Aug 2 12:12:57 2016 +0300

Setup upgrade levels for each fuel version

Setup upgrade levels block in nova.conf for each fuel version,
this is required for backward compatibility on RPC.

Change-Id: I6ccbc34457d11616b015cdb1e46e733b0b49ce58

commit 9e2a419ac79ad591faee9afe16419bf4497eacad
Author: Vladimir Khlyunev <email address hidden>
Date: Mon Aug 1 21:31:56 2016 +0300

Fix several typos

Copypasting is evil, lets remove typos

Change-Id: Icf4bec1a3de954911a469fa98380af2b6175da08

commit 013a7ded2275245a38ee58289dac2bc8b90fa37e
Author: Sergey Abramov <email address hidden>
Date: Wed Jul 20 16:05:10 2016 +0300

Host evacuation using nova

Evacuate instance from compute node using
nova host-evacuate-live except host_evacuation.sh script

Change-Id: I01a04cfa1ed2aafce7987e523e6b589dea21fbb0

commit 00a1d0dce1f7ae5db6706a59353403516f7913dc
Author: Sergey Abramov <email address hidden>
Date: Tue Jul 19 17:10:44 2016 +0300

Change shutoff vm command

    * run nova commands using nova util function
    * host filter using get nova node handle method(
        hostname for fuel < 6.1 and fqdn for newer version)

Change-Id: Ib5c373714e359982b13910a225787167db7b35e9

commit 49357c98313c5f551b13d0877581c5fb5e108db1
Author: Nikita Zubkov <email address hidden>
Date: Mon Jun 27 18:40:13 2016 +0300

Add `stderr_log_level` parameter to popen

Now posible to set logging level in popen for logging stderr

fuel-bootstrap build comand stderr now logged with INFO level

Change-Id: Ibaa2be54c68ce7a428e07052c917f54e99b95a5e

commit fb582b271dab4bdd10d27149a77c0e1f6a580b9e
Author: Sergey Abramov <email address hidden>
Date: Thu Jul 21 17:56:41 2016 +0300

Add run_nova_cmd function

This is required for running nova command on contoller without
manipulating manipulating with command.

It makes code easier and more readable

Usage:

from fuelclient.objects import environment as environment_obj

from octane.util import env
from octane.util import nova

        env_id = ...
        my_env = environment_obj.Environment(env_id)
        controller = env.get_one_controller(my_env)
        nova_cmd_just_run = [...]
        nova_cmd_with_output = [...]
        nova.run_nova_cmd(nova_cmd_just_run, controller, False)
        result = nova.run_nova_cmd(nova_cmd_w...

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

commit a05d2ecf13e15c12f10e1517fae425e583e0d714
Author: Nikita Zubkov <nzubkov@mirantis.com>
Date:   Thu Aug 4 17:51:56 2016 +0300

fix ssh connection auth
    
    According to spec [1] service user must use key authentication for ssh
    
    [1] https://github.com/openstack/fuel-specs/blame/ba4504/specs/9.0/fuel-nonroot-openstack-nodes.rst#L61-L64
    
    Change-Id: I461f659d4f0985283a43ec6a702bdd78af91a072

commit 7140398e501879a4e1f5605f3d1cc99195464850
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Tue Aug 2 12:12:57 2016 +0300

Setup upgrade levels for each fuel version
    
    Setup upgrade levels block in nova.conf for each fuel version,
    this is required for backward compatibility on RPC.
    
    Change-Id: I6ccbc34457d11616b015cdb1e46e733b0b49ce58

commit 9e2a419ac79ad591faee9afe16419bf4497eacad
Author: Vladimir Khlyunev <vkhlyunev@mirantis.com>
Date:   Mon Aug 1 21:31:56 2016 +0300

Fix several typos
    
    Copypasting is evil, lets remove typos
    
    Change-Id: Icf4bec1a3de954911a469fa98380af2b6175da08

commit 013a7ded2275245a38ee58289dac2bc8b90fa37e
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Wed Jul 20 16:05:10 2016 +0300

Host evacuation using nova
    
    Evacuate instance from compute node using
    nova host-evacuate-live except host_evacuation.sh script
    
    Change-Id: I01a04cfa1ed2aafce7987e523e6b589dea21fbb0

commit 00a1d0dce1f7ae5db6706a59353403516f7913dc
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Tue Jul 19 17:10:44 2016 +0300

Change shutoff vm command
    
    * run nova commands using nova util function
    * host filter using get nova node handle method(
        hostname for fuel < 6.1 and fqdn for newer version)
    
    Change-Id: Ib5c373714e359982b13910a225787167db7b35e9

commit 49357c98313c5f551b13d0877581c5fb5e108db1
Author: Nikita Zubkov <nzubkov@mirantis.com>
Date:   Mon Jun 27 18:40:13 2016 +0300

Add `stderr_log_level` parameter to popen
    
    Now posible to set logging level in popen for logging stderr
    
    fuel-bootstrap build comand stderr now logged with INFO level
    
    Change-Id: Ibaa2be54c68ce7a428e07052c917f54e99b95a5e

commit fb582b271dab4bdd10d27149a77c0e1f6a580b9e
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Thu Jul 21 17:56:41 2016 +0300

Add run_nova_cmd function
    
    This is required for running nova command on contoller without
    manipulating manipulating with command.
    
    It makes code easier and more readable
    
    Usage:
    
        from fuelclient.objects import environment as environment_obj
    
        from octane.util import env
        from octane.util import nova
    
        env_id = ...
        my_env = environment_obj.Environment(env_id)
        controller = env.get_one_controller(my_env)
        nova_cmd_just_run = [...]
        nova_cmd_with_output = [...]
        nova.run_nova_cmd(nova_cmd_just_run, controller, False)
        result = nova.run_nova_cmd(nova_cmd_with_output, controller, True)
    
    Change-Id: Icf14b968a372e07bab9517191380e67074aae4ce

commit 350768b5a2c5625183fc479c4b4962875e14cc75
Author: Nikita Zubkov <nzubkov@mirantis.com>
Date:   Thu Jul 28 18:57:17 2016 +0300

Add dependency on package fuel-nailgun-extension-cluster-upgrade
    
    fuel-nailgun-extension-cluster-upgrade provide nailgun extension that
    used by octane cli
    
    Related-Bug: #1604492
    Change-Id: Ib5a0d165614cd7d8626f3715b0c682efe1c4a684

commit 18c276d33ad0b247e9cd461e03512ea3a3637aad
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Sun Jul 24 12:41:10 2016 +0300

Don't overwrite ips in monmap on upgrade ceph
    
    Change only fsid in monmap on upgrade ceph. This is required if user set
    up sepcial network setting in network template.
    
    Closes-bug: 1605655
    Change-Id: Ib7b10ce604e1c16960a70a74a1cabcd983cca7ab

commit 4f70c9bf7fa07e246d9d9dbba9f6dac52943c736
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Tue Jun 28 09:59:47 2016 +0300

Move the flag --template to UpgradeEnvCommand
    
    The flag --template was returned
    from UpgradeNodeCommand to UpgradeEnvCommand,
    because the problem with assigning node to cluster
    with network template is resolved
    
    Depends-On: Ibd06de87964bf7a2038899d32e3d8a0189b9cbbd
    Change-Id: Ia470f674a6d6690f12d086af0ecbbdc8bc297fdd

commit 56fc094b2a80a74404c168101a5e6062fc8684f7
Author: Oleg Gelbukh <ogelbukh@mirantis.com>
Date:   Fri Jul 22 10:17:20 2016 +0000

Update README file for version 9.1
    
    Describe the current workflows with octane in README.rst file.
    
    Change-Id: Ic51af24fba70d0e2ebb3378472547b124c9d1b95

commit 41941d50071308a3b9d9a4d345dc553d408699b2
Author: Pavel Chechetin <pchechetin@mirantis.com>
Date:   Mon Jun 27 11:54:41 2016 +0300

Copy Fuel's keys in upgrade-env command
    
      Those keys are generated on the first "deploy_changes" for
      an environtment.
    
      It's ok to regenerate them in the majority cases,
      but, for example, in case of TLS + self-signed certificate
      it would be a problem. MongoDB replication also depends on
      those keys.
    
    Change-Id: I6e5e65feff2b662336028ca5114f7245147d664f
    Closes-Bug: 1595499

commit a2c46d5db218fdc542a5e284617197205fc4843b
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Fri Jul 22 11:30:50 2016 +0300

Replace run psql in container to run in host
    
    run_psql_in_container function was replaced to run_psql
    because Fuel 9.0 doesn't use docker containers
    
    Change-Id: I9e57aea7f55a6c48cfebbe93c76690dbbfa8224e
    Partial-bug:1602590

commit dbcc7eec0e6594431f4fe8b438ee543643bddbda
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Fri Jul 22 11:52:12 2016 +0300

Remove copy folder with patches to nailgun container
    
    because Fuel 9.0 doesn't use docker containers
    
    Change-Id: Ie25f31f1bfde73e9d9ace3a76a9789b76ddbf255
    Partial-bug:1602590

commit f817210c52761f3e0ddb4f283aef5773279dbcac
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Fri Jul 22 11:46:12 2016 +0300

Runing update_node_partition_info was replaced to host
    
    because Fuel 9.0 doesn't use docker containers
    
    Change-Id: Ic2f7d02e87be799269f8d654306a07de0dc80f67
    Partial-bug:1602590

commit 44870ae0ac963cd5209980e123952b0e2f450fbd
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Fri Jul 15 17:58:16 2016 +0300

remove overwrite net04_ext params
    
    Nailgun set up network_type to flat and
    physnet to physnet1 for fuel 8.0.
    They shouldn't been overwrite by octane on upgrade procedure.
    
    Related-bug: 1601895
    Change-Id: I0802cc96bfab8d5d153c5893e5f04df53942af47

commit 4aa5b77f663d13377fb675570c8ae11afad9ea10
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jul 11 16:50:57 2016 +0300

Add nova-manage db contract
    
    nova-manage db contract is the migration command.
    It apply changes, that are compatible with new running code.
    
    Change-Id: I2ac6f95a0e1ba6f5f0204b47d11d5c40936a2b8d

commit e4c12fc006a084870b4c127ab0841632b401abb7
Author: Nikita Zubkov <nzubkov@mirantis.com>
Date:   Wed Jul 6 14:01:30 2016 +0300

Add non-root ssh connection
    
    Since fuel 9.0 clusters have non-root service user. This patch adds
    logic that allows connecting to slaves with such credentials if cluster
    have non-root support.
    
    Change-Id: Iac4ac168022de7e60c40494841ac34d70f6f7986
    Partial-Bug: #1575477

commit 19d4b508ba642abcfea14b3ac4090204fc2cdb03
Author: Nikita Zubkov <nzubkov@mirantis.com>
Date:   Thu Jul 14 16:54:54 2016 +0300

For non-root user run sftp server as root
    
    To be able to get access to all files on fs.
    
    Change-Id: I3a106ff75023acfc9ce79c90efde5d07ba0dce3f
    Partial-Bug: #1575477

commit c01c4d92d343745e136cbafc233e336f5fcef5f2
Author: Oleg Gelbukh <ogelbukh@mirantis.com>
Date:   Mon Jul 4 14:33:24 2016 +0000

Remove leftover neutron restart from controller post-deploy stage
    
    Since neutron.conf file is not updated anymore with valid value
    of nova_admin_tenant_id, the restart of neutron server in post
    deployment stage of upgrade operation is not needed.
    
    Remove command that restarts neutron server.
    
    Change-Id: I4e2ebe49221c0d0f02609d68cb343e2a707c5bad
    Related-bug: 1584746

commit 44beb3932b8416ff4df7c9c9caf9206ecebe89f9
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Tue Jul 5 16:45:50 2016 +0300

Add destroyed container context manager
    
    Required for running destroy docker contaier before code block and
    startup it after block
    
    Usage:
    
        from octane.util import docker
    
        with docker.destoyed_container("container_name"):
           ...
    
    Change-Id: Iab57665d2b345bc19f04fad4c0cd4a4e461edd89

commit cc2ad7781599d6f8923fd434cfc7065daf589993
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Thu Jul 7 11:42:30 2016 +0000

Revert "Patch nailgun serializer on cluster upgrade handlers"
    
    This reverts commit fdafa2666dd2948175c8d5297727d42472dc6537.
    
    Patch doesn't required after merge
    https://review.openstack.org/#/c/276165/
    
    Change-Id: If121ebf723f6d2f7b5e4058098578eee483a4e2c

commit 56017df7f7d0733bb804ea8ed2a1253f2ac5170d
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Wed Jul 6 13:33:56 2016 +0300

Fix path problem on patch docker container files
    
    Closes-bug: 1599258
    Change-Id: Id5ec802552256b8b1d684cc169a2cd86a6657980

commit 60386fe00c3497311e121614dbf1bf84464ae2f5
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Thu Jun 30 11:05:09 2016 +0300

Revert patches in reverse order
    
    Change-Id: I8f3431691d93043fb49570f68417cf3011b5d806

commit 9912cdc6030d13f140c3f63cd1c34ec497404e07
Author: Oleg Gelbukh <ogelbukh@mirantis.com>
Date:   Wed Jun 22 10:11:20 2016 +0000

Save port membership for patch interface
    
    Patch interface added to Linux bridge during the control plane
    cutover, but not saved to configuration file for persistence.
    
    Add function to add port to bridge's configuration.
    
    Change-Id: If40bebf384b24d6af5e42798fbf6bc2993dc4795
    Closes-bug: 1576150

commit fdafa2666dd2948175c8d5297727d42472dc6537
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon May 30 17:40:57 2016 +0300

Patch nailgun serializer on cluster upgrade handlers
    
    CopyKeys task won't be yielded if get_uids returns empty list.
    
    Change-Id: Ifd4589b6a0e9a8e8636fea9a36afa091b230ab11
    Closes-bug: 1584658
    Closes-bug: 1584041
    Related-bug: 1541256

commit b1fc22b69f8badf2b64f64f82efdc56e28133c8f
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 16:52:29 2016 +0300

Add patch docker container service function
    
    this function setup patches to container and restart service
    After code block it reverts changes and restart service
    
    Usage:
    
    from octane.util import docker
    
    with docker.patch_container_service(
            "container", "service", "prefix", "patch_1", ..., "patch_N"):
        ...
    
    Change-Id: I34f4ff3356e92371ff5c1bcfbacd2d47b832b78d

commit 65856e6615e3b87324c553953b753cdc55201163
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 16:41:39 2016 +0300

Add applied_patches context manager to docker
    
    Required for patch docker container files before
    and revert patched changes after code block
    
    Usage:
    
    from octane.util import docker
    
    with docker.applied_patches(
            "container", "prefix", "patch_1", ..., "patch_N"):
        ...
    
    Change-Id: I10e5b8917166572ed6920f49b05e65db7b8cbbe8

commit cdcd1ffa35555880382d7eb65a1308b829aa23a4
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 16:39:15 2016 +0300

Patch docker container files using patch_apply function from patch module
    
    Change-Id: I7dab89037acfaa5f4590b0dcfc2d10fd26c91611

commit 578efa9a7764cf9428e619fce5555865f4140e36
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 16:37:16 2016 +0300

Add prefix in tempfile temp_dir util function
    
    Change-Id: I54a79e497addc8f2bf26d2cf9a1d6410db0b58f8

commit 9c1615b88deb240d8264519bc3f69bf27dd5382c
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 16:31:02 2016 +0300

Allocated file names from patches in a separate function
    
    It returns files from all sent patch files with prefix removed from
    their paths
    
    Change-Id: I564cf909f653be6703931dbe942170431325ee87
    Partial-bug: 1584658
    Partial-bug: 1584041

commit 2aad59a7da45db99ad544a47161a3c8e8eafe051
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 12:15:24 2016 +0300

Change ceph.conf editing procedure
    
    Update fsid value generated by new release to fsid value from old
    realese.
    
    Change-Id: I963aa0a0e0825ed55e767160eea3cb2f83f6ace1

commit 74990f1f2eec00b167df08da516270e7bc131baa
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Tue Jun 28 14:43:26 2016 +0300

Add host in mysql cmd
    
    This is required for run mysql script on env with disabled apis.
    
    Closes-bug: 1596909
    Change-Id: I18562444b9eea01c37e6a8a4b06ff91d6ce0d8c4

commit 603e028b5ff7bdaf03b38a3ff931c91b885a90a2
Author: Pavel Chechetin <pchechetin@mirantis.com>
Date:   Fri Jun 3 18:03:44 2016 +0300

Extract fuel2 client calling to separate function
    
      Also --debug flag has been added for more meaningful error reporting.
      Without --debug flag fuel2 client doesn't provide full error messsage
      in case of error.
    
    Closes-Bug: 1588955
    Change-Id: I53873fdb745d7cc368040a72496f7ccfa5e3a8b4

commit 159a00fcd8b9cb60a77fb5e14dba85ceeecc3fc1
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Tue Jun 28 16:58:51 2016 +0300

Revert test_copy_vips in test_env
    
    test_copy_vips was improperly removed in the commit
    919cceeba486c1457d10bb0264ec435b7f0d0568
    
    Change-Id: I74f4f8ce6eb5af8baa8288a5445341ff45a3399b

commit c9470a928ad84fc09adf8e25fd3128f6918d2405
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 16:22:52 2016 +0300

Move get_files_from_patch from docker to patch util module
    
    Change-Id: I324b5a371023df22254d631f9bc4b07778b21505

commit a2671ff513704f13c43b67b0d33491bbc1608389
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Tue Jun 7 15:51:11 2016 +0300

Fix migration in upgrade db from Juno to Liberty
    
    * patch nova (revert migrate_flavor_data delete)
    * soft delete instance_system_metadata where key = instance_type_id
    * split nova-manage db sync to 2 patch
    * call migrate_flavor_data between nova-manage db sync
    * call nova-manage db expand after sync
    
    required for upgrade db from 6.0 cluster to 8.0
    
    Change-Id: I7c3cd2812df7559af800febb34b558731d5bd9e9
    Closes-bug: 1573653

commit 1a7411322355cf8a835f5be00e9505988b3b15c6
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Mon Jun 27 13:32:06 2016 +0300

Add ssh patch context manager
    
    Context manager setup patch on remote host before block and
    revert patch on remote host upon completion of the block.
    
    Usage:
    
    from octane.util import ssh
    
    with ssh.applied_patches(
            "/remote_path", node, "patch_1", ..., "patch_N"):
        ...
    
    Change-Id: If84db0f2da5644d114e2fac0f98be4ee5297cff1

commit 919cceeba486c1457d10bb0264ec435b7f0d0568
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Mon Jun 27 14:51:36 2016 +0300

Removing following functions:
    
     * cache_service_tenant_id
     * get_service_tenant_id
     * get_openstack_project_value
     * get_openstack_project_dict
     * get_keystone_tenants
     * get_openstack_projects
    
    Tenant id doesn't exist in neutron.conf since version 8.0
    
    Change-Id: Ia79582de03e6783bdfec2052b0308f668fd4a311

commit c3ba24c8d5fab55410791c064d6ade49a25a1f57
Author: Anastasiya <atolochkova@mirantis.com>
Date:   Fri Jun 24 17:59:06 2016 +0300

Delete update_neutron_config function
    
    nova_admin_tenant_id doesn't need to update
    because it doesn't exist in neutron.conf since version 8.0
    
    Change-Id: Ifa906d6fe34b4ce8cdb5fab5d76a0138469101f1
    Closes-bug: 1584746

commit 3a432b63c9ab21b97810896696d212328d738bc4
Author: Sergey Abramov <sabramov@mirantis.com>
Date:   Thu Jun 9 13:08:46 2016 +0300

Add patch-active-img command
    
    Required for creating new patched bootstrap image
    based on active bootstrap image.
    
    It patches active bootstrap image to skip the step
    that add UEFI.
    
    Note:
    
    * You should active your patched image using command:
    
        fuel-bootstrap activate <img_uuid>
    
    * squashfs-tools should be installed on master node
    
        yum install squashfs-tools
    
    Usage:
    
        octane patch-active-img
    
    Closes-bug: 1575054
    
    Change-Id: I7b12db209b5d2db158d4c26bc162bab504aa1590

commit 4459e156e00adfee279760d2167109718ffd6ccf
Author: Ilya Kharin <akscram@gmail.com>
Date:   Wed Jun 15 16:18:43 2016 +0300

Fix logging for backup archivators
    
    The backup_restore.Base class has the archivator_name property that is
    used to log messages before and after the backup call. This property
    should be used only for instance-bound objects and not classes.
    
    Closes-Bug: #1592969
    Change-Id: Ib3b4542a5882cba3ac4e0468ccd8d407d8694162

commit 6504fb25b6b525e0d43d375f9daa4c43a87c5385
Author: Oleg Gelbukh <ogelbukh@mirantis.com>
Date:   Wed Jun 22 10:08:16 2016 +0000

Move test for update file to module tests/util.py
    
    Refactor tests for updating files on the node for later
    reuse in testing of update of /etc/network/interfaces
    file for persisting the interface created by the
    control plane cut-over.
    
    Change-Id: Ic65a69a0c2ef442b8d0e3d20b05db639a63fca7c
    Related-bug: 1576150

tags:

added: in-stable-mitaka