Done. Thanks again vinzenz Uploaded spec without the duplication. Ofer, did you take a look?
----- Original Message ----- > From: "Vinzenz Feenstra" <vfeen...@redhat.com> > To: vdsm-devel@lists.fedorahosted.org > Sent: Thursday, May 9, 2013 1:04:19 PM > Subject: Re: [vdsm] sync meeting minutes > > On 05/08/2013 11:04 AM, Yaniv Bronheim wrote: > > Hey, > > Sorry for the delay, all morning I've been working to verify again the > > package and upload new version that fits the changes we did for > > vdsm-python-cpopen. > > We tried that until this package will be accepted to fedora, we'll make it > > as part of vdsm's rpms (http://gerrit.ovirt.org/#/c/13685/). > > > > I ran rpmlint, koji build, and rpmbuild verification on the spec and source > > file, all looks fine. > > Please tell me if i need to run more tests before nudge you... (upload new > > spec and srpm https://bugzilla.redhat.com/show_bug.cgi?id=903246) > fedora-review gave these issues: > Issues: > ======= > - Permissions on files are set properly. > Note: See rpmlint output > See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions > - Package does not contain duplicates in %files. > Note: warning: File listed twice: /usr/lib64/python2.7/site- > packages/cpopen.so > See: http://fedoraproject.org/wiki/Packaging/Guidelines#DuplicateFiles > > You can ignore the first permissions related issue because that's a FP. > Not sure why however this is wrong. 755 is perfectly ok for .so files > > The second however needs to be addressed. You put there cpopen.so in the > %files section twice. One time with attr and one time without. Remove > the one without as this is a duplicate. > > > > Thanks! > > And thanks Dan for pushing it forward.. > > > > ----- Original Message ----- > >> From: "Ofer Schreiber" <oschr...@redhat.com> > >> To: "Dan Kenigsberg" <dan...@redhat.com> > >> Cc: "VDSM Project Development" <vdsm-devel@lists.fedorahosted.org> > >> Sent: Tuesday, May 7, 2013 8:04:55 AM > >> Subject: Re: [vdsm] sync meeting minutes > >> > >> Sure, I can help. > >> I'll do it tomorrow when I'll arrive to the office. > >> (I haven't looked into the package itself, please make sure that it > >> follows > >> all fedora guidelines, and that you have an official tarball published > >> somewhere, it will save some time) > >> > >> > >> ב-7 May 2013, בשעה 07:54, Dan Kenigsberg < dan...@redhat.com > כתב/ה: > >> > >> > >> > >> > >> On Mon, May 06, 2013 at 08:12:50PM -0400, Antoni Segura Puimedon wrote: > >> > >> > >> VDSM sync meeting May 6th 13:30 UTC > >> > >> > >> > >> > >> > >> > >> > >> > >> Federico says that he can't see anything pending for 3.2. Tomorrow 3.2.2 > >> will > >> > >> > >> be released. If anybody has anything pending push and ping Federico today. > >> > >> > >> > >> > >> > >> Storage is working on completing the feature pages, after that they will > >> link > >> > >> > >> them in the oVirt release management page. > >> > >> > >> http://wiki.ovirt.org/OVirt_3.3_release-management > >> > >> > >> Dan suggests to link the feature pages in their incomplete state to get > >> more > >> > >> > >> feedback. > >> > >> > >> > >> > >> > >> Yaniv states features for oVirt-3.3 to be reported. Splitting supervdsm is > >> > >> > >> going to be one of them. Needs reviews. > >> > >> > >> > >> > >> > >> Yaniv discussed the libvirt connection getting wrapper. We are still > >> looking > >> > >> > >> for a better solution that doesn't involve a special cif instance. > >> > >> > >> > >> > >> > >> Vinzenz hints Yaniv that for his cpopen that he needs to talk in > >> fedora-devel > >> > >> > >> for the package to be reviewed there. > >> > >> Ofer, maybe you too can help us out with the review of > >> Bug 903246 - Review Request: cpopen - Creates a subprocess in > >> simpler safer manner > >> and its acceptence into Fedora? > >> > >> > >> > >> > >> > >> > >> Infra has been working on improving vdsm utils for out of process > >> execution. > >> > >> > >> > >> > >> > >> The git master breakage from Mark Wu's patch was reverted and Giuseppe > >> > >> > >> Vallarelli has made a unit test for preventing future patches from > >> breaking > >> > >> > >> master by importing vdsm/ modules from lib/vdsm. > >> > >> > >> > >> > >> > >> Vinzenz to ask on the list about the jsonrpc model. > >> > >> > >> > >> > >> > >> Libvirtvm.py and vm.py to be verified by Vinzenz and QE to be thoroughly > >> tested > >> > >> > >> this week. > >> > >> > >> > >> > >> > >> Vinzenz points out that vdsClient is completely out of sync with the > >> schema. > >> > >> > >> Dan mentions that Zhou Zheng Sheng has probably had some success with > >> that. > >> To > >> > >> > >> be presented in the Shangai workshop. > >> > >> > >> > >> > >> > >> Adam has automatically generated bindings that need review and should be > >> given > >> > >> > >> a push. Also vdsm schema verification has some work already done that > >> should > >> be > >> > >> > >> resumed. > >> > >> > >> > >> > >> > >> We should put effort in bringing up possible changes/inconsistencies in > >> the > >> > >> > >> schema relative to the way the api is evolving. > >> > >> > >> > >> > >> > >> Toni to copy the agenda of the workshop in the mailing list. (DONE) > >> > >> Thanks! > >> > >> _______________________________________________ > >> vdsm-devel mailing list > >> vdsm-devel@lists.fedorahosted.org > >> https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel > >> > > _______________________________________________ > > vdsm-devel mailing list > > vdsm-devel@lists.fedorahosted.org > > https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel > > > -- > Regards, > > Vinzenz Feenstra | Senior Software Engineer > RedHat Engineering Virtualization R & D > Phone: +420 532 294 625 > IRC: vfeenstr or evilissimo > > Better technology. Faster innovation. Powered by community collaboration. > See how it works at redhat.com > > > _______________________________________________ > vdsm-devel mailing list > vdsm-devel@lists.fedorahosted.org > https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel > _______________________________________________ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
