Bug 212444 - fence_xvmd reboot does not work
Summary: fence_xvmd reboot does not work
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: cman
Version: 5.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: Lon Hohberger
QA Contact: Cluster QE
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-10-26 19:55 UTC by Lon Hohberger
Modified: 2009-04-16 22:36 UTC (History)
2 users (show)

Fixed In Version: beta2
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-23 01:44:35 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
makes reboot work (2.49 KB, patch)
2006-10-26 20:29 UTC, Lon Hohberger
no flags Details | Diff
Use 'reboot' as default instead of 'off' since reboot works; requires above patch (733 bytes, patch)
2006-10-26 21:57 UTC, Lon Hohberger
no flags Details | Diff
handle 0.1.9 status from libvirt (VIR_DOMAIN_SHUTOFF) as a successful operation (4.11 KB, patch)
2006-12-01 22:15 UTC, Lon Hohberger
no flags Details | Diff

Description Lon Hohberger 2006-10-26 19:55:30 UTC
Description of problem:

The 'reboot' operation for xen fencing does not work.  This has two causes:

(a) libvirt does not have a guaranteed immediate synchronous reboot option.  The
only "reboot" operation it has is worthless, because it is equivalent to a
soft-shutdown / init 6 / ACPI soft-powerdown event.  Supposedly, this could be
implemented with the xen 3.0.3 APIs, but it does not appear to be in libvirt, at
least not according to the documentation here:

  http://libvirt.org/html/libvirt-libvirt.html#virDomainReboot

(b) xend unlinks the temporary images set up by pygrub, making the
virDomainGetXMLDesc() information about the kernel and initrd.


A solution to (b) above is to kill the entire <os> tag.  Here's an example XML
desc from virDomainGetXMLDesc:

<domain type='xen' id='10'>
  <name>kanderso-xen-02</name>
  <uuid>165b5ef7e37713750409e70cf9cbd43c</uuid>
  <bootloader>/usr/bin/pygrub</bootloader>
  <os>
    <type>linux</type>
    <kernel>/var/lib/xen/vmlinuz.CRKPf-</kernel>
    <initrd>/var/lib/xen/initrd.kcwg-N</initrd>
    <cmdline>ro root=/dev/VolGroup00/LogVol00 console=xvc0 rhgb quiet</cmdline>
  </os>
  <memory>393216</memory>
  <vcpu>1</vcpu>
  <on_poweroff>destroy</on_poweroff>
  <on_reboot>restart</on_reboot>
  <on_crash>restart</on_crash>
  <devices>
    <interface type='bridge'>
      <source bridge='xenbr1'/>
      <mac address='00:16:3e:37:02:d9'/>
      <script path='vif-bridge'/>
    </interface>
    <disk type='file' device='disk'>
      <driver name='tap' type='aio'/>
      <source file='/tmp/futon2.dsk'/>
      <target dev='xvda'/>
    </disk>
    <console tty='/dev/pts/4'/>
  </devices>
</domain>


All we need to do is blank-out the <os> tag and all children, and it will fall
back to pygrub.

Comment 1 Lon Hohberger 2006-10-26 20:29:38 UTC
Created attachment 139524 [details]
makes reboot work

Comment 2 Kiersten (Kerri) Anderson 2006-10-26 21:37:22 UTC
GA Blocker request - Xen hasn't been stable enough during this cycle to address
some of the interesting scenarios, so wouldn't hold up beta2 for this support.

Comment 3 RHEL Program Management 2006-10-26 21:46:05 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux release.  Product Management has requested further review
of this request by Red Hat Engineering.  This request is not yet committed for
inclusion in release.

Comment 4 Lon Hohberger 2006-10-26 21:57:04 UTC
Created attachment 139532 [details]
Use 'reboot' as default instead of 'off' since reboot works; requires above patch

Comment 5 Kiersten (Kerri) Anderson 2006-12-01 21:08:00 UTC
Fails initial testing with the updating library so marking it FAILS_QA

Comment 6 Lon Hohberger 2006-12-01 22:15:46 UTC
Created attachment 142630 [details]
handle 0.1.9 status from libvirt (VIR_DOMAIN_SHUTOFF) as a successful operation

incremental; requires previous 2 patches

Comment 7 Benjamin Kahn 2006-12-12 16:33:33 UTC
Lon:  This should have been in modified for weeks.

Comment 8 RHEL Program Management 2006-12-23 01:44:35 UTC
A package has been built which should help the problem described in 
this bug report. This report is therefore being closed with a resolution 
of CURRENTRELEASE. You may reopen this bug report if the solution does 
not work for you.


Comment 9 Nate Straz 2007-12-13 17:21:47 UTC
Moving all RHCS ver 5 bugs to RHEL 5 so we can remove RHCS v5 which never existed.


Note You need to log in before you can comment on or make changes to this bug.