[RFE] Boot from Volume - Reference Drivers

Bug #1559691 reported by Julia Kreger on 2016-03-20
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Hironori Shiina

Bug Description

This bug is for tracking the Boot from Volume specification/RFE.

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

Changed in ironic:
assignee: nobody → Julia Kreger (juliaashleykreger)
status: New → In Progress
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Stephane Miller (stephaneeee)
tags: added: rfe
Changed in ironic:
importance: Undecided → Wishlist
Changed in ironic:
assignee: Stephane Miller (stephaneeee) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Stephane Miller (stephaneeee)
Changed in ironic:
assignee: Stephane Miller (stephaneeee) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Stephane Miller (stephaneeee)
Changed in ironic:
assignee: Stephane Miller (stephaneeee) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Stephane Miller (stephaneeee)
Changed in ironic:
assignee: Stephane Miller (stephaneeee) → Julia Kreger (juliaashleykreger)

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

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Stephane Miller (stephaneeee)

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

Changed in ironic:
assignee: Stephane Miller (stephaneeee) → Julia Kreger (juliaashleykreger)

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

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

Reviewed: https://review.openstack.org/294995
Committed: https://git.openstack.org/cgit/openstack/ironic-specs/commit/?id=cd17651481f1a52c665c7cce2a7e25f658b6b8b1
Submitter: Jenkins
Branch: master

commit cd17651481f1a52c665c7cce2a7e25f658b6b8b1
Author: Julia Kreger <email address hidden>
Date: Sun Mar 20 08:37:56 2016 -0400

    Boot from Volume - Reference Drivers

    During the Vancouver Summit, and most recently the ironic Mitaka
    midcycle, the question of booting from a remote volume arose.

    The decision from the ironic Mitaka midcycle was for a specification
    to be drafted for reference driver changes. As such, this change
    proposes that specification, in accordance with the prophecy.

    Co-Authored-By: Chris Krelle <email address hidden>
    Co-Authored-By: stephane <email address hidden>

    Partial-Bug: #1559691
    Change-Id: I2d7607d967b4191b94b261005519ae46637e790d

Ruby Loo (rloo) wrote :

The spec has been approved.

tags: added: rfe-approved
removed: rfe

Reviewed: https://review.openstack.org/366196
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=f11d3deff55ebb6bfd5a9d0db3257302bf7826f1
Submitter: Jenkins
Branch: master

commit f11d3deff55ebb6bfd5a9d0db3257302bf7826f1
Author: Julia Kreger <email address hidden>
Date: Mon Aug 22 16:02:17 2016 -0400

    Add volume_connectors and volume_targets to task

    Adding volume_connectors and volume_targets to the task object
    to allow for matching the existing model leveraged by ports and
    portgroups which allows for simplified storage_interface use.

    Change-Id: Iea1af5b939ba9374feef56ce22ae718eb6c7a672
    Partial-Bug: #1559691

Reviewed: https://review.openstack.org/348005
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=fa200aaeddb563fa9503489d876aaed1ee5955a8
Submitter: Jenkins
Branch: master

commit fa200aaeddb563fa9503489d876aaed1ee5955a8
Author: Julia Kreger <email address hidden>
Date: Tue Jul 26 11:39:30 2016 -0400

    Add storage_interface to node DB table

    Add storage_interface DB field, database migration,
    and object entry for storage_interface for boot from
    volume functionality.

    Initial versions of this revision added the field to
    the node object, but since then Ironic's code has
    drifted such that adding an object for an interface
    has to take place along with the entire driver interface,
    thus the object change is in the next revision.

    Co-Authored-By: Stephane Miller <email address hidden>
    Change-Id: Ib5d5de87163570949c12d9e95c73e17c1c12eec1
    Partial-Bug: #1559691

Reviewed: https://review.openstack.org/348006
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=c491e93a082ede262e617cebfd3022090e3e715b
Submitter: Jenkins
Branch: master

commit c491e93a082ede262e617cebfd3022090e3e715b
Author: Julia Kreger <email address hidden>
Date: Wed Jul 27 14:50:51 2016 -0400

    Add storage_interface to base driver class

    In order to properly support booting and maintenance of
    systems that boot from a remote storage device, we need an
    interface to associate the driver with.

    This commit adds a basic storage_interface and noop and fake
    interfaces along with the appropriate handling for configuration
    in the event that the driver list is blank, or is missing the
    noop driver.

    Co-Authored-By: Stephane Miller <email address hidden>
    Change-Id: Ib21eda88f207f18675c8580cd7fd37eab6fd70bf
    Partial-Bug: #1559691

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Joanna Taryma (jtaryma)
Changed in ironic:
assignee: Joanna Taryma (jtaryma) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Joanna Taryma (jtaryma)

Reviewed: https://review.openstack.org/355625
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=459fe314fa65111d4b1fd4f8210117903a778093
Submitter: Jenkins
Branch: master

commit 459fe314fa65111d4b1fd4f8210117903a778093
Author: Joanna Taryma <email address hidden>
Date: Mon Apr 3 09:37:50 2017 -0700

    Add base cinder common interface

    Addition of common cinder related code that a storage interface
    driver will be able to leverage for interacting with cinder.

    Co-Authored-By: Joanna Taryma <email address hidden>
    Partial-Bug: #1559691
    Change-Id: I4aed895e52fcefb61dd0751e8bc9b39a26706276

Reviewed: https://review.openstack.org/453839
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=3b5aefa0b64e05395ff7907025ac22b6dd5f2a0d
Submitter: Jenkins
Branch: master

commit 3b5aefa0b64e05395ff7907025ac22b6dd5f2a0d
Author: Joanna Taryma <email address hidden>
Date: Wed Apr 12 12:47:13 2017 -0700

    Common cinder interface additional improvements

    This is a follow up to 459fe314fa65111d4b1fd4f8210117903a778093

    Minor issues in the code and tests are fixed, the release note is added,
    configuration options are described as part of not yet exposed feature.

    Change-Id: Ib466ee3970dcc1e141ddac38a54544f6a011549e
    Partial-Bug: #1559691

Changed in ironic:
assignee: Joanna Taryma (jtaryma) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)

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

Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)

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

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

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Michael Turek (mjturek)

Reviewed: https://review.openstack.org/464125
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=703b8898e38837519c0aaff6967cf359058ba699
Submitter: Jenkins
Branch: master

commit 703b8898e38837519c0aaff6967cf359058ba699
Author: Hironori Shiina <email address hidden>
Date: Fri May 12 14:19:23 2017 +0900

    Fix attribute name of cinder volume

    This patch fixes the name of UUID of a cinder volume object. The
    correct attribute is not 'uuid', but 'id'.

    Change-Id: Ie054be643e1925bd8c2333b3b1ecd2988cc9eeaa
    Partial-Bug: #1559691

Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)

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

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)

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

Reviewed: https://review.openstack.org/460250
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=2a9013599f6abd9a30cea207aa97637599a9f26e
Submitter: Jenkins
Branch: master

commit 2a9013599f6abd9a30cea207aa97637599a9f26e
Author: Michael Turek <email address hidden>
Date: Wed Apr 26 13:45:38 2017 -0400

    Enable getting volume targets by their volume_id

    This patch introduces methods for retrieving volume targets by
    their ID. As this field does not have a unique constraint, getting
    volume targets by their volume_id field will return a list. This
    will allow us to simplify attaching volumes in the cinder driver
    as we can now easily query the database rather than looping
    through all connected volumes.

    Change-Id: I05709fd2ab09bd127c0d202ddcd4863077db3d5b
    Partial-Bug: #1559691

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Dmitry Tantsur (divius)

Reviewed: https://review.openstack.org/366197
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b23a176a737f217e9573d6f10c3059d22813b159
Submitter: Jenkins
Branch: master

commit b23a176a737f217e9573d6f10c3059d22813b159
Author: Michael Turek <email address hidden>
Date: Wed Apr 19 13:26:22 2017 -0400

    Add Cinder storage driver

    A cinder interface driver that can be called in order to signal
    cinder to attach and detach volumes.

    Authored-By: Julia Kreger <email address hidden>
    Co-Authored-By: Joanna Taryma <email address hidden>
    Co-Authored-By: Mike Turek <email address hidden>
    Change-Id: I3d6bb126ef34b30f316ed5a10fd13a3106d1400b
    Partial-Bug: #1559691

Changed in ironic:
assignee: Dmitry Tantsur (divius) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Michael Turek (mjturek)

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

Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)

Reviewed: https://review.openstack.org/467930
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=9c45a284d08bbe68dedc33c8cb85d8b60b0a46ec
Submitter: Jenkins
Branch: master

commit 9c45a284d08bbe68dedc33c8cb85d8b60b0a46ec
Author: Hironori Shiina <email address hidden>
Date: Mon Jun 5 10:28:32 2017 +0900

    Change volume metadata not to use nested dicts

    Ironic sets metadata to a volume when attaching and detaching. However,
    Cinder doesn't accept metadata which consists of nested dictionaries.
    This patch fixes the issue by passing metadata as string without
    nested dictionaries.

    Change-Id: Ibe0a752a923ff52526a7e04374e61c14f6c4f0b0
    Partial-Bug: #1559691

Reviewed: https://review.openstack.org/472342
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=18f3c48ca47209f01c298d2572900e116f32308c
Submitter: Jenkins
Branch: master

commit 18f3c48ca47209f01c298d2572900e116f32308c
Author: Julia Kreger <email address hidden>
Date: Thu Jun 8 16:41:54 2017 +0000

    Follow-up docstring revision

    Follow-up to Ibe0a752a923ff52526a7e04374e61c14f6c4f0b0
    to revise the doc string based on review feedback.

    Change-Id: I38294b5ea4279b09e05da55198d5a2d09c5f1a71
    Partial-Bug: #1559691

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)

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

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Hironori Shiina (shiina-hironori)

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

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Dao Cong Tien (tiendc)

Reviewed: https://review.openstack.org/406290
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=104000549eb542db25f780279c885c6a55623e98
Submitter: Jenkins
Branch: master

commit 104000549eb542db25f780279c885c6a55623e98
Author: Joanna Taryma <email address hidden>
Date: Mon Apr 10 15:46:52 2017 -0700

    Wire in storage interface attach/detach operations

    Addition of storage interface attachment and detachment
    operations when:

    * Node power is turned on/off, with a storage_interface
      configured, and when the node is in ACTIVE state.
    * Node deployment and node tear_down operations.

    In addition to attachment and detachment, driver_internal_info
    is now populated with a boot from volume target uuid, if a
    volume is defined for the node.

    Additionally, upon tear_down, the drivers now call a helper
    to remove storage related dictionaries and destroy
    volume target records.

    The "cinder" storage interface has been enabled by default,
    and further details on the storage interface's use are in
    later patchsets for this feature.

    Authored-By: Julia Kreger <email address hidden>
    Co-Authored-By: Joanna Taryma <email address hidden>
    Co-Authored-By: Michael Turek <email address hidden>
    Change-Id: I0e22312e8cebb37b8f025da2baeca8eb635f35b7
    Partial-Bug: #1559691

Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Dmitry Tantsur (divius)
Changed in ironic:
assignee: Dmitry Tantsur (divius) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)

Reviewed: https://review.openstack.org/467766
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=41c0a31bb771326350c5481256b68f6bccf58a21
Submitter: Jenkins
Branch: master

commit 41c0a31bb771326350c5481256b68f6bccf58a21
Author: Mike Turek <email address hidden>
Date: Thu Jun 15 11:06:46 2017 -0400

    Move _abort_attach_volumes functionality to detach_volumes

    This is a followup to the Cinder driver patch [0]. There were several
    comments during review that asked why we weren't using
    detach_volumes instead of _abort_attach_volumes. This patch is a
    proposal to remove the method and merge it's functionality with
    the detach_volumes method.

    This patch also addresses some nits found in the cinder driver.

    [0] b23a176a737f217e9573d6f10c3059d22813b159

    Change-Id: Ic5584bc1380a65fdd9a74c174bee63bae1c6a4b3
    Parital-bug: #1559691

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Hironori Shiina (shiina-hironori)

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

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Michael Turek (mjturek)

Reviewed: https://review.openstack.org/413324
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=6883674b3c4b016ba36b73bb385ec8813d88f93f
Submitter: Jenkins
Branch: master

commit 6883674b3c4b016ba36b73bb385ec8813d88f93f
Author: Joanna Taryma <email address hidden>
Date: Mon Apr 10 16:01:44 2017 -0700

    iPXE template support for iSCSI

    Added support for iPXE template output of a template
    containing iscsi based URL sanhook entries to enable
    boot as well as additional attachments.

    Authored-By: Julia Kreger <email address hidden>
    Co-Authored-By: Joanna Taryma <email address hidden>
    Co-Authored-By: Michael Turek <email address hidden>
    Change-Id: I75869262dbfd1caa779fa21e93cdb31f193cb829
    Partial-Bug: #1559691

Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)

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

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Dao Cong Tien (tiendc)

Reviewed: https://review.openstack.org/454243
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=152a3654ce0b03c70a489919611eeaec343cc8ea
Submitter: Jenkins
Branch: master

commit 152a3654ce0b03c70a489919611eeaec343cc8ea
Author: Julia Kreger <email address hidden>
Date: Thu Apr 6 15:48:54 2017 +0000

    Logic for skipping deployment with BFV

    In order to boot from volume, the deploy driver needs to
    know when not to actually deploy. This change wires in the
    checks to skip deployment activities if it appears that we
    have a valid root volume target defined.

    Co-Authored-By: Hironori Shiina <email address hidden>
    Partial-Bug: #1559691
    Change-Id: I62e622a2b053f685c2da42ca5106bdb9bdd22dc6

Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)

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

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Dao Cong Tien (tiendc)

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

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

Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Hironori Shiina (shiina-hironori)

Change abandoned by Dao Cong Tien (<email address hidden>) on branch: master
Review: https://review.openstack.org/480907
Reason: Shiina will merge this and https://review.openstack.org/#/c/463908 into one patch.

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → John L. Villalovos (happycamp)
Changed in ironic:
assignee: John L. Villalovos (happycamp) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)

Reviewed: https://review.openstack.org/463908
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b90f7a15fb3a8b69f725c685108d351d17b0e4d8
Submitter: Jenkins
Branch: master

commit b90f7a15fb3a8b69f725c685108d351d17b0e4d8
Author: Hironori Shiina <email address hidden>
Date: Thu May 11 14:18:39 2017 +0900

    Enable cinder storage interface for generic hardware

    This patch enables cinder storage interface for generic hardware. It
    also adds storage_interface field to node resource and driver resource
    in API and bumps API version to 1.33 so that storage interface can be
    set and shown via API.

    Change-Id: I2c74f386291e588a25612f73de08e8367795acff
    Partial-Bug: #1559691

Reviewed: https://review.openstack.org/463972
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=961f046a4a5c1e1a2a43cc1d495b9be2c46b7e28
Submitter: Jenkins
Branch: master

commit 961f046a4a5c1e1a2a43cc1d495b9be2c46b7e28
Author: Hironori Shiina <email address hidden>
Date: Thu May 11 20:37:59 2017 +0900

    Add storage interface field to node-related notifications

    This patch adds storage_interface field to node-related notification
    objects.

    Change-Id: I1d67075a2c29d24d6321fd9989fbe4ee9f8f513c
    Partial-Bug: #1559691

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

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Ruby Loo (rloo)

Reviewed: https://review.openstack.org/482800
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=ba52e406b94c508c8ade3d23cccabe4793b46430
Submitter: Jenkins
Branch: master

commit ba52e406b94c508c8ade3d23cccabe4793b46430
Author: Hironori Shiina <email address hidden>
Date: Wed Jul 12 13:49:13 2017 +0900

    Add Driver API change in 1.33 to history

    API Version 1.33 introduced fields related to storage interface to
    Driver API as well as Node API. This patch adds missing description
    for Driver API to REST API Version History documentation.

    Change-Id: I2fd0709865adc1d02cf3800aa11497d97ab0b14b
    Partial-Bug: #1559691

Changed in ironic:
assignee: Ruby Loo (rloo) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Michael Turek (mjturek)

Reviewed: https://review.openstack.org/478690
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=2c18f22f9976870b71bfd1e471afa18fd50a7a25
Submitter: Jenkins
Branch: master

commit 2c18f22f9976870b71bfd1e471afa18fd50a7a25
Author: Mike Turek <email address hidden>
Date: Wed Jun 28 20:20:42 2017 -0400

    Generate iPXE boot script when deploying with boot from volume

    This patch moves the boot from volume skip logic for the prepare
    step of deployment into the boot interface, allowing the template
    to get generated before skipping the remainder of the step.

    Partial-Bug: #1559691
    Change-Id: Icfea16c62a753c77942107af287880f35f28c404

Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → John L. Villalovos (happycamp)
Changed in ironic:
assignee: John L. Villalovos (happycamp) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → John L. Villalovos (happycamp)
Changed in ironic:
assignee: John L. Villalovos (happycamp) → Julia Kreger (juliaashleykreger)

Reviewed: https://review.openstack.org/466333
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=2c134ac4c599b8964320893216dd907e0ea90f1f
Submitter: Jenkins
Branch: master

commit 2c134ac4c599b8964320893216dd907e0ea90f1f
Author: Hironori Shiina <email address hidden>
Date: Sat May 20 00:33:32 2017 +0900

    DevStack: Add configuration for boot-from-volume

    This patch enables boot-from-volume feature for DevStack with VMs.
    - Add parameters for setting storage interface.
    - Set node property for iscsi boot.
    - Create volume connectors.

    Change-Id: I1085bb8d44e3dbae61243dd9b65d17db33c505be
    Partial-Bug: #1559691

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Julia Kreger (juliaashleykreger)
Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Dao Cong Tien (tiendc)
Changed in ironic:
assignee: Dao Cong Tien (tiendc) → Dmitry Tantsur (divius)

Reviewed: https://review.openstack.org/472740
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b05a6a64b128ca6dc2fad2f88651d12544d30174
Submitter: Jenkins
Branch: master

commit b05a6a64b128ca6dc2fad2f88651d12544d30174
Author: Dao Cong Tien <email address hidden>
Date: Fri Jun 9 22:51:34 2017 +0700

    Tempest scenario test for boot-from-volume

    Added a tempest test to create a cinder volume, then create a
    server with appropriate config options for block device mapping.
    That will result in the baremetal node to boot from the cinder
    volume.

    Change-Id: Ife0db2b8a1e639ac1e3d617b981eb81f793a71f1
    Partial-bug: #1559691

Reviewed: https://review.openstack.org/486853
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=fb5def4ebc89ebee327a6b06ef8340903259b4c1
Submitter: Jenkins
Branch: master

commit fb5def4ebc89ebee327a6b06ef8340903259b4c1
Author: Julia Kreger <email address hidden>
Date: Tue Jul 25 03:59:44 2017 +0000

    Address review feedback for ipxe boot file fix

    Address feedback in review Id545d6cf93227cf1fc2ff0c05dbdceb8fb6aa5b9
    as all requested changes were basically limited to tests.

    Partial-bug: #1559691
    Change-Id: Ic414581c145d0e4b94163c9bd2dd4ae160196b28

Changed in ironic:
assignee: Dmitry Tantsur (divius) → Hironori Shiina (shiina-hironori)

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

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → John L. Villalovos (happycamp)
Changed in ironic:
assignee: John L. Villalovos (happycamp) → Hironori Shiina (shiina-hironori)
Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Dmitry Tantsur (divius)
Changed in ironic:
assignee: Dmitry Tantsur (divius) → Julia Kreger (juliaashleykreger)

Reviewed: https://review.openstack.org/480975
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=05d85dd8bb6f1de76d37a331c35134dc4e52e724
Submitter: Jenkins
Branch: master

commit 05d85dd8bb6f1de76d37a331c35134dc4e52e724
Author: Hironori Shiina <email address hidden>
Date: Thu Jul 6 19:48:02 2017 +0900

    Add storage interface to api-ref

    This patch adds these storage interface fields to api-ref:
      - storage_interface of Node
      - default_storage_interface of Driver
      - enabled_storage_interfaces of Driver
    These fields were introdeced with API version 1.33.

    Change-Id: Iac73d7ae4976dfd437b9821a2221dd842be24b55
    Partial-Bug: #1559691

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Nguyen Hung Phuong (phuongnh)

Change abandoned by John L. Villalovos (<email address hidden>) on branch: master
Review: https://review.openstack.org/489810
Reason: Will work with infra to do a batch upload.

Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Hironori Shiina (shiina-hironori)

Reviewed: https://review.openstack.org/479326
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=c0ce6ebf1988c29cc68d16fc2e9142a022f7024b
Submitter: Jenkins
Branch: master

commit c0ce6ebf1988c29cc68d16fc2e9142a022f7024b
Author: Julia Kreger <email address hidden>
Date: Fri Jun 30 14:14:15 2017 +0000

    BFV Deploy skip minor logging, logic, and test fixes

    This is a follow-up to I62e622a2b053f685c2da42ca5106bdb9bdd22dc6
    addressing some minor logging, test fixes/additions that were requested
    as well as correcting a minor logic discrepency between iscsi and agent
    deploy preparation methods as it makes no sense to re-plug networks in
    the iscsi deployment preparation method if we are not writing an image
    since we will ultimately skip deployment.

    Additionally, as a result of review feedback, the FlatNetwork mocks
    were updated to be more consistent.

    Change-Id: If6bd7e34ea92add15c3c7d8b94a2efbaf882d6ff
    Partial-Bug: #1559691

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Nguyen Hung Phuong (phuongnh)
Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Hironori Shiina (shiina-hironori)

Reviewed: https://review.openstack.org/466568
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=90d1ac43c20f80b18f4a6c3807f8842cfe5c4970
Submitter: Jenkins
Branch: master

commit 90d1ac43c20f80b18f4a6c3807f8842cfe5c4970
Author: Hironori Shiina <email address hidden>
Date: Mon May 22 11:25:15 2017 +0900

    Add a guide for Devstack configuration for boot-from-volume

    This patch adds a guide for Devstack configuration to build a
    environment to test boot-from-volume feature with VMs.

    Change-Id: If1a2f1a712af61618ec6e96ce3bad67295ddcff3
    Partial-Bug: #1559691

Changed in ironic:
assignee: Hironori Shiina (shiina-hironori) → Nguyen Hung Phuong (phuongnh)

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

Changed in ironic:
assignee: Nguyen Hung Phuong (phuongnh) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Ruby Loo (rloo)
Changed in ironic:
assignee: Ruby Loo (rloo) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)
assignee: Julia Kreger (juliaashleykreger) → Michael Turek (mjturek)
Changed in ironic:
assignee: Michael Turek (mjturek) → Julia Kreger (juliaashleykreger)

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

Reviewed: https://review.openstack.org/494016
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=a397a84e65d32a52e253c64652501dcd3565d659
Submitter: Jenkins
Branch: master

commit a397a84e65d32a52e253c64652501dcd3565d659
Author: Michael Turek <email address hidden>
Date: Tue Aug 15 16:52:57 2017 -0400

    Add admin guide for boot from volume

    This patch introduces a guide for admin configuration to use
    the boot from volume feature.

    Change-Id: I70bd54191ad53b6048a915e5e8451a05464d190a
    Partial-Bug: #1559691

Reviewed: https://review.openstack.org/494283
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=f471601f7bfe4b33a6e9c1c55c45f1eb3337f876
Submitter: Jenkins
Branch: master

commit f471601f7bfe4b33a6e9c1c55c45f1eb3337f876
Author: Julia Kreger <email address hidden>
Date: Wed Aug 16 17:54:41 2017 +0000

    Add storage interface to enabling-drivers doc

    Adds the storage interface to the enabling-drivers documentation
    with example configuration, and notes that the interface is
    explicitly an "opt-in" interface.

    Change-Id: If471708c7827954a1935b4f6c0a312b9fa258c43
    Partial-Bug: #1559691

Reviewed: https://review.openstack.org/473717
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=74a126ab0eda334f9ca358ed0320226752d21e31
Submitter: Jenkins
Branch: master

commit 74a126ab0eda334f9ca358ed0320226752d21e31
Author: Nguyen Hung Phuong <email address hidden>
Date: Tue Jun 13 14:22:10 2017 +0700

    Add functional API tests for volume connector and volume target

    Extend baremetal json client with volume connector and volume target.
    Add basic positive tests for volume connector and volume target:

        test_create_volume_connector_error
        test_delete_volume_connector
        test_delete_volume_connector_error
        test_show_volume_connector
        test_list_volume_connectors
        test_list_with_limit
        test_update_volume_connector_replace
        test_update_volume_connector_replace_error
        test_update_volume_connector_remove_item
        test_update_volume_connector_remove_collection
        test_update_volume_connector_add

        test_create_volume_target_error
        test_delete_volume_target
        test_delete_volume_target_error
        test_show_volume_target
        test_list_volume_targets
        test_list_with_limit
        test_update_volume_target_replace
        test_update_volume_target_replace_error
        test_update_volume_target_remove_item
        test_update_volume_target_remove_collection
        test_update_volume_target_add

    Change-Id: I642a02cff2afe7f89b6800d69e14d05e04e6a59c
    Partial-bug: #1559691

Reviewed: https://review.openstack.org/494785
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=554c2dabfc45c6ee91fd6e3f6913f5849e17dbd8
Submitter: Jenkins
Branch: stable/pike

commit 554c2dabfc45c6ee91fd6e3f6913f5849e17dbd8
Author: Nguyen Hung Phuong <email address hidden>
Date: Tue Jun 13 14:22:10 2017 +0700

    Add functional API tests for volume connector and volume target

    Extend baremetal json client with volume connector and volume target.
    Add basic positive tests for volume connector and volume target:

        test_create_volume_connector_error
        test_delete_volume_connector
        test_delete_volume_connector_error
        test_show_volume_connector
        test_list_volume_connectors
        test_list_with_limit
        test_update_volume_connector_replace
        test_update_volume_connector_replace_error
        test_update_volume_connector_remove_item
        test_update_volume_connector_remove_collection
        test_update_volume_connector_add

        test_create_volume_target_error
        test_delete_volume_target
        test_delete_volume_target_error
        test_show_volume_target
        test_list_volume_targets
        test_list_with_limit
        test_update_volume_target_replace
        test_update_volume_target_replace_error
        test_update_volume_target_remove_item
        test_update_volume_target_remove_collection
        test_update_volume_target_add

    Change-Id: I642a02cff2afe7f89b6800d69e14d05e04e6a59c
    Partial-bug: #1559691
    (cherry picked from commit 74a126ab0eda334f9ca358ed0320226752d21e31)

tags: added: in-stable-pike

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

Changed in ironic:
assignee: Julia Kreger (juliaashleykreger) → Hironori Shiina (shiina-hironori)

Reviewed: https://review.openstack.org/495736
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=e6f09618a58f9af8671d0ba6bda6f3ee92a90efa
Submitter: Jenkins
Branch: master

commit e6f09618a58f9af8671d0ba6bda6f3ee92a90efa
Author: Hironori Shiina <email address hidden>
Date: Mon Aug 21 15:15:34 2017 +0900

    Fix names of capabilities for FibreChannel volume boot

    Cinder uses 'fibre_channel' to for the volume type for FibreChannel
    volumes. This value is returned as 'driver_volume_type' when
    initialize_connection API. Then, nova sets the value to a volume target
    of ironic.

    The capabilities names of interfaces to allow FC boot are expected to
    use the value in populating[1]. This patch fixes the capabilities names
    from 'fc' to 'fibre_channel'.

    [1] https://github.com/openstack/ironic/blob/c0ce6ebf1988c29cc68d16fc2e9142a022f7024b/ironic/drivers/modules/deploy_utils.py#L1263,L1268

    Change-Id: I9ff3c55cff035b0ff87b36da820f8129a9ffcbe2
    Partial-bug: #1559691

Dmitry Tantsur (divius) on 2017-08-22
Changed in ironic:
status: In Progress → Fix Released

Reviewed: https://review.openstack.org/496246
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=9305aedf83ef87ad6f33ff1e2e7f7a64d508b030
Submitter: Jenkins
Branch: stable/pike

commit 9305aedf83ef87ad6f33ff1e2e7f7a64d508b030
Author: Hironori Shiina <email address hidden>
Date: Mon Aug 21 15:15:34 2017 +0900

    Fix names of capabilities for FibreChannel volume boot

    Cinder uses 'fibre_channel' to for the volume type for FibreChannel
    volumes. This value is returned as 'driver_volume_type' when
    initialize_connection API. Then, nova sets the value to a volume target
    of ironic.

    The capabilities names of interfaces to allow FC boot are expected to
    use the value in populating[1]. This patch fixes the capabilities names
    from 'fc' to 'fibre_channel'.

    [1] https://github.com/openstack/ironic/blob/c0ce6ebf1988c29cc68d16fc2e9142a022f7024b/ironic/drivers/modules/deploy_utils.py#L1263,L1268

    Change-Id: I9ff3c55cff035b0ff87b36da820f8129a9ffcbe2
    Partial-bug: #1559691
    (cherry picked from commit e6f09618a58f9af8671d0ba6bda6f3ee92a90efa)

Reviewed: https://review.openstack.org/516192
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b709f5e9097e439d49821eb862a8c2f2b2a67c9d
Submitter: Zuul
Branch: master

commit b709f5e9097e439d49821eb862a8c2f2b2a67c9d
Author: Hironori Shiina <email address hidden>
Date: Mon Oct 30 15:31:04 2017 +0900

    Remove unnesessary description for config parameters in cinder group

    Now that the BFV implementation with iPXE was completed, the
    parameters in [cinder] group can be exposed. This patch fixes the help
    messages.

    Change-Id: I9f20d478334901140f7c1639d8f0bb0612ae5de0
    Related-Bug: #1559691

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

Other bug subscribers