We can't. This patch will never see 4.2. You can still start working on your plugin on 4.2, but the change represented by this patch can only go into master. On Oct 7, 2013 5:01 PM, "Mike Tutkowski" <mike.tutkow...@solidfire.com> wrote:
> So, now that I'm getting back to this, do you think I should just try to > make this work with 4.2 (like we originally talked about)? > > I updated again from master, rebuilt, redeployed DEBs and still get this > JNA error message: > > log4j:WARN No appenders could be found for logger > (org.apache.commons.httpclient.params.DefaultHttpParams). > log4j:WARN Please initialize the log4j system properly. > log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for > more info. > java.lang.reflect.InvocationTargetException > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.lang.reflect.Method.invoke(Method.java:606) > at > org.apache.commons.daemon.support.DaemonLoader.start(DaemonLoader.java:243) > Caused by: java.lang.UnsatisfiedLinkError: Can't obtain updateLastError > method for class com.sun.jna.Native > at com.sun.jna.Native.initIDs(Native Method) > at com.sun.jna.Native.<clinit>(Native.java:139) > at org.libvirt.jna.Libvirt.<clinit>(Unknown Source) > at org.libvirt.Library.<clinit>(Unknown Source) > at org.libvirt.Connect.<init>(Unknown Source) > at > com.cloud.hypervisor.kvm.resource.LibvirtConnection.getConnection(LibvirtConnection.java:44) > at > com.cloud.hypervisor.kvm.resource.LibvirtConnection.getConnection(LibvirtConnection.java:37) > at > com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.configure(LibvirtComputingResource.java:733) > at com.cloud.agent.Agent.<init>(Agent.java:161) > at com.cloud.agent.AgentShell.launchAgent(AgentShell.java:415) > at com.cloud.agent.AgentShell.launchAgentFromClassInfo(AgentShell.java:370) > at com.cloud.agent.AgentShell.launchAgent(AgentShell.java:351) > at com.cloud.agent.AgentShell.start(AgentShell.java:448) > ... 5 more > Cannot start daemon > Service exit with a return value of 5 > > > On Mon, Oct 7, 2013 at 2:31 PM, Mike Tutkowski < > mike.tutkow...@solidfire.com> wrote: > >> Sure, that's a good plan. >> >> I'll get to it. >> >> >> On Mon, Oct 7, 2013 at 2:29 PM, Marcus Sorensen <shadow...@gmail.com>wrote: >> >>> I know you mentioned you might need some minor changes to it, as well as >>> other minor changes just for master (attach volume switched to pool vs >>> adapter or something). My hope was that you would be able to send an update >>> that works for your plugin on master, I'll test against existing libvirtd >>> storage and apply it. >>> On Oct 7, 2013 1:49 PM, "Mike Tutkowski" <mike.tutkow...@solidfire.com> >>> wrote: >>> >>>> This is an automatically generated e-mail. To reply, visit: >>>> https://reviews.apache.org/r/14381/ >>>> >>>> This looks reasonable to me, Marcus. >>>> >>>> When do you think you might start the process of getting this into master? >>>> >>>> >>>> - Mike Tutkowski >>>> >>>> On September 30th, 2013, 5:14 p.m. UTC, Marcus Sorensen wrote: >>>> Review request for cloudstack, edison su and Mike Tutkowski. >>>> By Marcus Sorensen. >>>> >>>> *Updated Sept. 30, 2013, 5:14 p.m.* >>>> *Repository: * cloudstack-git >>>> Description >>>> >>>> With custom storage plugins comes the need to prep the KVM host prior to >>>> utilizing the disks. e.g. an iscsi initiator needs to log into the target >>>> and scan for the lun before it can be used on the host. This patch is an >>>> example I developed against 4.2, minor changes may be necessary to apply >>>> to master, but I want to share with others who are working on storage so >>>> they can ensure it works for them. Please tweak as you see fit. >>>> >>>> MigrateCommand: pass vmTO object so we can see which disks/storage pool >>>> types belong to the vm when migrating a VM. This facilitates being able to >>>> call disconnectPhysicalDisksViaVmSpec >>>> >>>> VirtualMachineManagerImpl: pass VirtualMachineTO when migrating so that we >>>> can see which disks belong to the VM and what storage pools/adaptors >>>> should be used >>>> >>>> LibvirtComputingResource: add calls KVMStoragePoolManager's >>>> connectPhysicalDiskViaVmSpec and disconnectPhysicalDiskViaVmSpec calls >>>> where appropriate (when starting a vm, migrating a vm). Ensure that we >>>> create 'raw' format XML disk definitions when the storage format is RAW. >>>> Move cleanupDisk logic to storage adaptors so that each adaptor type can >>>> clean up its disks in is own way. >>>> >>>> KVMStoragePoolManager: add connectPhysicalDisk, disconnectPhysicalDisk, >>>> connectPhysicalDiskViaVmSpec, disconnectPhysicalDiskViaVmSpec, >>>> disconnectPhysicalDiskByPath. These all call the specific StorageAdaptor's >>>> connectPhysicalDisk, disconnectPhysicalDisk, or >>>> disconnectPhysicalDiskByPath calls. >>>> >>>> KVMStorageProcessor: Call connectPhysicalDisk/disconnectPhysicalDisk on >>>> the storage adaptor. Whether or not this is implemented is up to the >>>> storage adaptor. >>>> >>>> LibvirtStorageAdaptor: implement dummy >>>> connectPhysicalDisk/disconnectPhysicalDisk, move cleanupDisk logic from >>>> LibvirtComputingResource to disconnectPhysicalDiskByPath >>>> >>>> StorageAdaptor: define >>>> connectPhysicalDisk/disconnectPhysicalDisk/disconnectPhysicalDiskByPath in >>>> the interface >>>> >>>> >>>> Testing >>>> >>>> Basic testing with my storage adaptor >>>> >>>> Diffs >>>> >>>> - core/src/com/cloud/agent/api/MigrateCommand.java (5042b8c) >>>> - >>>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java >>>> (3ee811f) >>>> - >>>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java >>>> (e09c9ba) >>>> - >>>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java >>>> (c69f9b0) >>>> - >>>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java >>>> (123a9f1) >>>> - >>>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/StorageAdaptor.java >>>> (4956d8d) >>>> - server/src/com/cloud/vm/VirtualMachineManagerImpl.java (d46bbb0) >>>> >>>> View Diff <https://reviews.apache.org/r/14381/diff/> >>>> >>> >> >> >> -- >> *Mike Tutkowski* >> *Senior CloudStack Developer, SolidFire Inc.* >> e: mike.tutkow...@solidfire.com >> o: 303.746.7302 >> Advancing the way the world uses the >> cloud<http://solidfire.com/solution/overview/?video=play> >> *™* >> > > > > -- > *Mike Tutkowski* > *Senior CloudStack Developer, SolidFire Inc.* > e: mike.tutkow...@solidfire.com > o: 303.746.7302 > Advancing the way the world uses the > cloud<http://solidfire.com/solution/overview/?video=play> > *™* >
