Re: Commit broke vm_data for Xen

2013-07-26 Thread Marcus Sorensen
I went looking for other uses of vm_data.sh in the CitrixResourceBase.java and didn't find it, hence the reason why I removed it. I already knew vmware used their own method. In looking more closely, I see that CitrixResourceBase calls a host plugin called "vm_data", that must later depend on vm_da

Re: Commit broke vm_data for Xen

2013-07-26 Thread Prasanna Santhanam
Seems like we'll have to take care of both when changing userdata in the future - shell + python. Any chance we can make this work for Xen too using python or is that not possible? On Thu, Jul 25, 2013 at 05:52:59PM -0600, Marcus Sorensen wrote: > Thanks for taking care of that. > > On Thu, Jul

Re: Commit broke vm_data for Xen

2013-07-25 Thread Marcus Sorensen
Thanks for taking care of that. On Thu, Jul 25, 2013 at 5:30 PM, Sheng Yang wrote: > Hi Marcus, > > Xen's vm_data has been broken since this commit, because the file > scripts/network/domr/vm_data.sh you deleted is shared by Xen as well. > > commit 28855b4987c9274d15a539b9d7ae26c0073b0651 > Autho

Commit broke vm_data for Xen

2013-07-25 Thread Sheng Yang
Hi Marcus, Xen's vm_data has been broken since this commit, because the file scripts/network/domr/vm_data.sh you deleted is shared by Xen as well. commit 28855b4987c9274d15a539b9d7ae26c0073b0651 Author: Marcus Sorensen Date: Wed Jul 24 13:58:17 2013 -0600 Summary: Get away from dozens of