Ironic template does not work

Bug #1454895 reported by yuanying
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Wishlist
Spyros Trigazis

Bug Description

Template definition doesn't support ironic template.
So we should add new template definition class for ironic.

yuanying (yuanying)
Changed in magnum:
assignee: nobody → yuanying (ootsuka)
Steven Dake (sdake)
Changed in magnum:
status: New → Confirmed
importance: Undecided → Wishlist
Revision history for this message
Haomeng,Wang (whaom) wrote :

Hi YuanYing,

I understand magnum call heat to deploy vm/bm, and heat will call nova to boot bm via ironic, so dont think we need an special heat template to boot bare metal for magnum, correct me if I am wrong understanding, any comments?

Thank you.

Revision history for this message
yuanying (yuanying) wrote :

Right, but currently it doesn't work.

Changed in magnum:
assignee: yuanying (ootsuka) → Kai Qiang Wu(Kennan) (wkqwu)
Changed in magnum:
status: Confirmed → In Progress
Adrian Otto (aotto)
summary: - Ironic template does not worked
+ Ironic template does not work
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/198596
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=52268629c159e50e7070ddf177a2f4709ef34fbb
Submitter: Jenkins
Branch: master

commit 52268629c159e50e7070ddf177a2f4709ef34fbb
Author: Kai Qiang Wu(Kennan) <email address hidden>
Date: Mon Jul 6 04:36:22 2015 +0000

    Make ironic templates working

    This first fix make ironic template working to boot kubernetes cluster.

    Change-Id: Ia54bf9ef647d623b5f832ec1069cd6a037c2b6c1
    Partial-Bug: 1454895

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on magnum (master)

Change abandoned by Adrian Otto (<email address hidden>) on branch: master
Review: https://review.openstack.org/198984
Reason: Kennan,

This patch has not seen any revisions in over two weeks. We want our review queue to contain active work-in-progress and code that reviewers are considering for merge. You are welcome to un-abandon this at any time, and continue making revisions.

Thanks,

Adrian

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Adrian Otto (<email address hidden>) on branch: master
Review: https://review.openstack.org/199478
Reason: Kennan,

This patch has not seen any revisions in over two weeks. We want our review queue to contain active work-in-progress and code that reviewers are considering for merge. You are welcome to un-abandon this at any time, and continue making revisions.

Thanks,

Adrian

Revision history for this message
Kai Qiang Wu(Kennan) (wkqwu) wrote :

Use this to track that. As it is a not simple issue.

https://blueprints.launchpad.net/magnum/+spec/magnum-baremetal-full-support

Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
yuanying (yuanying)
Changed in magnum:
assignee: Kai Qiang Wu(Kennan) (wkqwu) → yuanying (ootsuka)
Revision history for this message
Murali Allada (murali-allada) wrote :

This is not a bug. It's a feature request.

Changed in magnum:
status: In Progress → Invalid
status: Invalid → Confirmed
status: Confirmed → In Progress
Revision history for this message
Murali Allada (murali-allada) wrote :

Please add 'closes-bug' to your review commit message.

Changed in magnum:
assignee: yuanying (ootsuka) → Spyros Trigazis (strigazi)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/320968
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=08531c170293bf13ec0c77bc396c1150998d39eb
Submitter: Jenkins
Branch: master

commit 08531c170293bf13ec0c77bc396c1150998d39eb
Author: OTSUKA, Yuanying <email address hidden>
Date: Thu Jul 14 21:07:06 2016 +0200

    Fix ironic template

    This is workaround fix to support baremetal.
    Following items are remained to support.
    * Documents
    * Functional test

    To test this template, there are some requirements and problem as below.

    Requirements:
    * `ephemeral_disk` on ironic baremetal flavor
      `ephemeral_disk` is used for docker storage instead of cinder volume.

    * `fixed_subnet` must be setup with dns_nameservers like following.
        * `neutron subnet-update private-subnet --dns-nameserver 8.8.8.8`

    * `fixed_subnet` must be IP version 4.
      if you use devstack, please add following configuration.
        * `IP_VERSION=4`

    * Fedora 23 image including kubernetes, etcd, flannel.

    Problem:
    Ironic stores `instance_info` about nova instance.
    `instance_info` contains config_drive data, but this data can be
    too large to store ironic.nodes table.
    Magnum uses large config drive data to setup k8s.
    It means, we can not start ironic instance by Magnum.

    Workaround fix is changing column type of ironic.nodes.instance_info.
    Following sql will help you.

      `alter table ironic.nodes modify instance_info LONGTEXT;`

    Partial-Implements: blueprint magnum-baremetal-full-support
    Change-Id: Ica87610b9114bff4277b492de8fe528fe2860108
    Closes-Bug: #1454895
    Closes-Bug: #1472938
    Co-Authored-By: Spyros Trigazis <email address hidden>

Changed in magnum:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/magnum 3.0.0

This issue was fixed in the openstack/magnum 3.0.0 release.

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.