Fuel for OpenStack

Astute.yaml should be splitted up to two parts for eliminate long serialization

Bug #1660308 reported by Stanislaw Bogatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Stanislaw Bogatkin
Fuel for OpenStack 11.0
Newton
Fix Released
High
Stanislaw Bogatkin
Fuel for OpenStack 10.1

Bug Description

Today we have too long serialization process for big amount of nodes. It leads us to deploy failures. Needs to be fixed. It is a derived bug from https://bugs.launchpad.net/fuel/+bug/1596987

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-qa (master)

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

OpenStack Infra (hudson-openstack)
Changed in fuel:
status: New → In Progress
Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

Fuel-library part addressed by https://review.openstack.org/#/c/366844/

Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

Fuel-QA part should be merged as two commits. In first we will introduce workaround for proper reviews testing. In second we'll remove opening of astute.yaml file and only cluster.yaml will stay.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/426741
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=62ec18011fbf2ba4428c4f68f45ab8630dd55baf
Submitter: Jenkins
Branch: master

commit 62ec18011fbf2ba4428c4f68f45ab8630dd55baf
Author: Stanislaw Bogatkin <email address hidden>
Date: Mon Jan 30 15:46:37 2017 +0300

    Change path for astute.yaml

    As astute.yaml splits for 2 parts, rename file in tests accordingly.

    Change-Id: If7c6f26fe2a1835b2e0824b8a9cba261fddd709c
    Related-Bug: #1660308

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/431415

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

Reviewed: https://review.openstack.org/366844
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=cfd2588728e855a6b4aee7b1c7516a0aefa26b42
Submitter: Jenkins
Branch: master

commit cfd2588728e855a6b4aee7b1c7516a0aefa26b42
Author: Dmitry Guryanov <email address hidden>
Date: Wed Sep 21 11:34:26 2016 +0300

    Split configuration astute.yaml into common and node parts

    This file is huge if there are many nodes in cluster. Now
    we generate it and store in memory for each node and it
    takes a lot of time and memory.

    Let's split it into common and node-specific parts so that
    we can generate common part once and copy it from master
    node to each node affected by deployment.

    Change-Id: I114a8598460bef9063e43505f46fe38cd564d8c7
    Closes-Bug: #1660308

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 11.0.0.0rc1

This issue was fixed in the openstack/fuel-library 11.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-qa (stable/newton)

Related fix proposed to branch: stable/newton
Review: https://review.openstack.org/441066

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-qa (stable/newton)

Reviewed: https://review.openstack.org/441066
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=96f000fa5140874ae3da63a264988c6e58db000c
Submitter: Jenkins
Branch: stable/newton

commit 96f000fa5140874ae3da63a264988c6e58db000c
Author: Stanislaw Bogatkin <email address hidden>
Date: Mon Jan 30 15:46:37 2017 +0300

    Change path for astute.yaml

    As astute.yaml splits for 2 parts, rename file in tests accordingly.

    Change-Id: If7c6f26fe2a1835b2e0824b8a9cba261fddd709c
    Related-Bug: #1660308
    (cherry picked from commit 62ec18011fbf2ba4428c4f68f45ab8630dd55baf)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/newton)

Reviewed: https://review.openstack.org/431415
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=88bfbd924137cd2aa0ed7517029abcfddd48d0c1
Submitter: Jenkins
Branch: stable/newton

commit 88bfbd924137cd2aa0ed7517029abcfddd48d0c1
Author: Dmitry Guryanov <email address hidden>
Date: Wed Sep 21 11:34:26 2016 +0300

    Split configuration astute.yaml into common and node parts

    This file is huge if there are many nodes in cluster. Now
    we generate it and store in memory for each node and it
    takes a lot of time and memory.

    Let's split it into common and node-specific parts so that
    we can generate common part once and copy it from master
    node to each node affected by deployment.

    Change-Id: I114a8598460bef9063e43505f46fe38cd564d8c7
    Closes-Bug: #1660308
    (cherry-picked from commit id cfd2588728e855a6b4aee7b1c7516a0aefa26b42)

Revision history for this message
Ilya Bumarskov (ibumarskov) wrote :

Verified on Fuel 10 build #1575

[root@nailgun ~]# ll /etc/fuel/cluster/1/cluster.yaml
-rw-r--r--. 1 root root 30281 Apr 18 07:38 /etc/fuel/cluster/1/cluster.yaml

root@node-1:~# ll /etc/fuel/cluster/1/cluster.yaml
-rw-r--r-- 1 root root 30281 Apr 18 07:38 /etc/fuel/cluster/1/cluster.yaml

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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