Vincent/Octavian, Apologies- I gave the OK to merge these patches. I heard that a build with the patches merged passed the smoketests. As an expediency to move forward with clearing the pull-request backlog I asked Rajini to merge them in �C I am sorry, I jumped the gun on these. This was a bad call �C please do not blame Rajini as it was my bad. If it is a serious enough situation feel free to reverse those patches that need to be reworked.
- John T. From: Сергей Юдин [mailto:[email protected]] Sent: Thursday, September 26, 2013 6:27 AM To: Vincent Untz Cc: crowbar; Kanevsky, Arkady; Ram, Rajini; Terpstra, John Subject: Re: [Crowbar] RHEL/CentOS pull requests merged without any +1 from our side I'm still subscribed for those PRs and you can adress issues via github - i'm not going to ignore them and i'l fix them if we can find consensus. There was some unnoticed comments, i responded to them. Also sent few style fixes for nova https://github.com/crowbar/barclamp-nova/pull/229 according to yours comments. 2013/9/26 Vincent Untz <[email protected]<mailto:[email protected]>> Hi, The following pull requests got merged while they had no +1 from non-Dell people (and the +1 from Dell happened just before the merge, so with no time for anyone to react), and while there were comments that I feel were either not addressed or needed more discussion: https://github.com/crowbar/barclamp-crowbar/pull/675 https://github.com/crowbar/barclamp-keystone/pull/115 https://github.com/crowbar/barclamp-quantum/pull/118 https://github.com/crowbar/barclamp-nova/pull/221 I explicitly mentioned that some of the RHEL/CentOS pull requests were not ready in the call yesterday, and still this got merged. And of course, now it's unlikely that the comments will ever be addressed... How can we fix that? Vincent -- Les gens heureux ne sont pas pressés. _______________________________________________ Crowbar mailing list [email protected]<mailto:[email protected]> https://lists.us.dell.com/mailman/listinfo/crowbar For more information: http://crowbar.github.com/
_______________________________________________ Crowbar mailing list [email protected] https://lists.us.dell.com/mailman/listinfo/crowbar For more information: http://crowbar.github.com/
