UCA specific data must be moved into separate release in openstack.yaml

Bug #1556011 reported by Ihor Kalnytskyi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Matthew Mosesohn

Bug Description

Detailed bug description:

Since https://review.openstack.org/#/c/279542/ we start supporting Ubuntu Cloud Archive repos for deploying openstack. However, it was implemented as part of Nailgun's Ubuntu release which is wrong in terms of separating concerns.

Expected results:

UCA must be implemented as separate release in openstack.yaml, derived from base Ubuntu release and that adds additional pin attributes.

Actual result:

UCA is a switch with ugly conditions in base Ubuntu release.

The fix must be done on both sides: nailgun and library.

Changed in fuel:
status: New → Confirmed
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Matthew Mosesohn (raytrac3r)
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Igor, how should it be represented in openstack.yaml? I see three options, but I'm not sure if some are possible:
1 (worst): Copy and paste ubuntu release and add new parameters
2 (simplest): Extend ubuntu release and add new parameters
3 (probably best, but most work): Create new base ubuntu release and then make a Fuel + Ubuntu and UCA + Ubuntu release that extends ubuntu_base.

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/292340

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

This task seems to require a member of fuel-python group. It looks like this sort of task will require restructuring of plugin support and fixing how we extend releases. Furthermore, I think it will break all compatibility with plugins when trying to deploy with UCA release.

I'm not sure how much effort is required in fuel-qa to add another release value.

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

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

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

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

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

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

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

Reviewed: https://review.openstack.org/292340
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=60ad1a9ad8de5de18d806c4c373b327cfb0127f9
Submitter: Jenkins
Branch: master

commit 60ad1a9ad8de5de18d806c4c373b327cfb0127f9
Author: Matthew Mosesohn <email address hidden>
Date: Mon Mar 14 15:17:20 2016 +0300

    Move UCA configuration to separate release

    Change-Id: Ife805af287b0923d6fca74552e8d74f7d6330ec0
    blueprint: deploy-with-uca-packages
    Partial-Bug: #1556011

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/293967

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

Reviewed: https://review.openstack.org/293480
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=5b67d1743946d4d8ab37befc683f82e7ec206cfc
Submitter: Jenkins
Branch: master

commit 5b67d1743946d4d8ab37befc683f82e7ec206cfc
Author: Matthew Mosesohn <email address hidden>
Date: Wed Mar 16 17:55:09 2016 +0300

    Refactor UCA deployment to separate release

    Renamed Ubuntu release to Ubuntu 14.04.
    UCA release is now called Ubuntu+UCA 14.04 so it
    can be identified with find() in get_release method.

    Fixed several lookups for lookups of OPENSTACK_RELEASE.

    Change-Id: I998ff288282db94bfbc7bbe5bedb1c420b09f452
    Partial-Bug: #1556011

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

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

commit e01cd884f684c9c2e36cde43ca8f6be00a08af34
Author: Matthew Mosesohn <email address hidden>
Date: Thu Mar 17 11:33:22 2016 +0000

    Move UCA configuration to separate release

    Depends-On: I998ff288282db94bfbc7bbe5bedb1c420b09f452
    Change-Id: I08336266f7bfb2d6ebd3b881dc0c1bc5e91fbd98
    blueprint: deploy-with-uca-packages
    Partial-Bug: #1556011

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

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

commit b7e333c70fd150628b20c6c6741de6a5db5ca933
Author: Matthew Mosesohn <email address hidden>
Date: Wed Mar 16 16:48:13 2016 +0300

    Enable trusty-proposed/mitaka repo for UCA

    Change-Id: I4275fb79c21f02cf6ed7dbb3d10bec65fd3a2699
    Related-Bug: #1556011

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

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

commit 416db0f3cc9a1df871a3ec84a4c0a034b8ef25d9
Author: Matthew Mosesohn <email address hidden>
Date: Tue Mar 15 19:52:45 2016 +0300

    Refactor UCA pins and package configuration

    After moving UCA to a separate release, repo config
    can be managed by nailgun in the release repo list.

    Repo configuration and repo pin itself are now removed, but
    package pins remain.

    Also removed pin workaround for keystoneclient.

    Change-Id: I3a03e2c02074fe14c85666def121c5c36dda9f74
    Partial-Bug: #1556011

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Tatyanka (tatyana-leontovich) wrote :

verified 250 iso

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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