Stengrammic comments inline:

On Sun, Jun 17, 2012 at 09:04:28PM +0300, Dan Kenigsberg wrote:
> Hi!
> 
> tomorrow I would like to discuss:
> 
> - the abysmal review condition of the rest api patches

Ryan + Adam: future of vdsm API. Saggi is going to send his proposal.

> - vdsm status for ovirt-3.1
>   I know networking requires a heavy cherry-pick from upstream. There
>   is probably more.
>   Everybody invited to care for vdsm bugs that blocks Bug 822145 -
>   Tracker: oVirt 3.1 release.

Adam to add blocking bugs to the tracker.

> - plenty pep8 patches applied, but there is plenty more.

> 
> - Patches with pending verification. I see 11 of those now
>   
> http://gerrit.ovirt.org/#/q/status:open+project:vdsm+verified%253D0+codereview%253E%253D%252B2+-codereview%253C%253D-1,n,z
>   Please do not send your patches out to the cold and desert them there.
>   Pet them, nag folks to review and verify them, and rebase (only!) when
>   required.

e.g. http://gerrit.ovirt.org/#/c/191/ should be rebased

> - Your issue comes here (or above, if it's more urgent).

Ryan: would like to see Gal's commits in Gerrit. I think it's in
      Gerrit-2.5. Hopefully we can upgrade soon. Itamar?

Ayal: suggest to separate +2 and push rights in Gerrit. This will help
      in prolifirating +2, while maintaining code coherence.
Danken: Fine by me, we'd need the help of our Gerrit master, Itamar.

Federico: suggests to not rebase on top of master branch, but only on top of
          your topic branch. this does not clear the grades of untouched
          commits.

Saggi: crab-rpc still out. Danken: I have few non-blocking comments to
       it. The problem is that it sits unverified.

Adam: Wants to formalize the content of vm.create params dictionary and
      similar elemets of the API.

Deepack: please review libstorage writeup. Saggi, Adam: will do.
Deepack: is there an irc bot logging #vdsm? No but I'd love if you
         configure one ;-)
Deepack to research if there's an available bot to do this. It can be
        run on a private server, and then post to somewhere like
        you.fedorapeople.org.

See you in two weeks!

Dan.
_______________________________________________
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/vdsm-devel

Reply via email to