glassfish
  1. glassfish
  2. GLASSFISH-17880

Unable to successfully start a suspended vm in kvm virtualization

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.0_b31
    • Component/s: iaas
    • Labels:
      None

      Description

      This is the scenario that will happen when an application goes through deploy, disable, enable lifecycle:

      1. iaas.allocate(); // happens during 'asadmin deploy paas_app' -- respective SPEs will do this to create their VM.
      
      2. vmLifecycle.stop(vm); // during  'asadmin disable paas_app' -- respective SPEs will invoke this on their VM
      
      3. vmLifecycle.start(vm); // during  'asadmin enable paas_app' -- respective SPEs will invoke this on their VM
      
      

      At step 3, looks like the VM is started without remounting the disk with new keys. Hence 'register-startup' fails when the VM comes up. VM console shows the error as attached (vm_restart.png)

      Also, vmLifecycle.start(vm) seems to come out immediately without waiting for the VM to come up completely (this is to do with waiting for startupLatch).

      This issue can be reproduced by creating a template for Database ServiceType (refer attached kvm_setup.sh) and run the attached 'create-simple-service' command.

        Issue Links

          Activity

          Hide
          Bhavanishankar added a comment -

          Run the testcase with : ./kvm_setup.sh && asadmin create-simple-service --templateid javadb test

          Show
          Bhavanishankar added a comment - Run the testcase with : ./kvm_setup.sh && asadmin create-simple-service --templateid javadb test
          Hide
          Bhavanishankar added a comment -

          I have already communicated to Jerome about this issue a while back.

          Also, as an experiment, tried the attached fix (virt-core.diff, source-files.zip). Although the change works, I would request Jerome to make the right fix.

          Show
          Bhavanishankar added a comment - I have already communicated to Jerome about this issue a while back. Also, as an experiment, tried the attached fix (virt-core.diff, source-files.zip). Although the change works, I would request Jerome to make the right fix.
          Hide
          Bhavanishankar added a comment -

          experimental fix.

          Show
          Bhavanishankar added a comment - experimental fix.
          Hide
          Tom Mueller added a comment -

          Bulk update to reassign dochez issue to component lead.

          Show
          Tom Mueller added a comment - Bulk update to reassign dochez issue to component lead.
          Hide
          Yamini K B added a comment -

          Suggested fix looks good, made some minor changes. Checked in r53152

          Show
          Yamini K B added a comment - Suggested fix looks good, made some minor changes. Checked in r53152

            People

            • Assignee:
              Yamini K B
              Reporter:
              Bhavanishankar
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: