Fuel for OpenStack

Add macro replacement for release's orchestrator data

  1. Series 5.1.x
  2. Bug #1378901
Bug #1378901 reported by Ihor Kalnytskyi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Ihor Kalnytskyi
Fuel for OpenStack 6.0
5.1.x
Fix Committed
High
Ihor Kalnytskyi
Fuel for OpenStack 5.1.1
6.0.x
Fix Committed
Medium
Ihor Kalnytskyi
Fuel for OpenStack 6.0

Bug Description

We should be able to add release with an orchestrator data which contains some macro replacements.

For example:

    "puppet_modules_source": "rsync://{MASTER_IP}:/path/to"

has to be converted by Nailgun into:

    "puppet_modules_source": "rsync://10.20.0.2:/path/to"

Tags: nailgun
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

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

Reviewed: https://review.openstack.org/126943
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=2488bffd55baee84b33366ac9e4eef62dbd70607
Submitter: Jenkins
Branch: master

commit 2488bffd55baee84b33366ac9e4eef62dbd70607
Author: Igor Kalnitsky <email address hidden>
Date: Wed Oct 8 18:41:10 2014 +0300

    Use masks in release orchestrator_data

    Since Fuel Upgrade feature, it's a common practice to add new
    openstack release, so it'd be very useful to add new release
    with a kind of macro replacement in paths to repos and puppets.

    Since this commit, the macro replacement for MASTER_IP was
    introduced.

    Change-Id: Ie3a79ab105461f10cbabcc5353ca952946200f32
    Closes-Bug: #1378901

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

Since we're going to deliver diff-based repos in the upgrade tarball for 5.1.1, we need to resolve this issue in stable/5.1 since fuel_upgrade script are depended on it.

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

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/132845

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/5.1)

Reviewed: https://review.openstack.org/132845
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=8e919c35fee65e19b161a751952d5296899a5ee6
Submitter: Jenkins
Branch: stable/5.1

commit 8e919c35fee65e19b161a751952d5296899a5ee6
Author: Igor Kalnitsky <email address hidden>
Date: Wed Oct 8 18:41:10 2014 +0300

    Use masks in release orchestrator_data

    Since Fuel Upgrade feature, it's a common practice to add new
    openstack release, so it'd be very useful to add new release
    with a kind of macro replacement in paths to repos and puppets.

    Since this commit, the macro replacement for MASTER_IP was
    introduced.

    (cherry picked from commit 2488bffd55baee84b33366ac9e4eef62dbd70607)

    Closes-Bug: #1378901
    Change-Id: Ie3a79ab105461f10cbabcc5353ca952946200f32

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.