| From | Sent On | Attachments |
|---|---|---|
| Andrew Bogott | Jul 2, 2012 12:16 pm | |
| Russell Bryant | Jul 2, 2012 12:26 pm | |
| Joshua Harlow | Jul 2, 2012 2:41 pm | |
| Joshua Harlow | Jul 2, 2012 2:54 pm | |
| Gabriel Hurley | Jul 2, 2012 3:18 pm | |
| John Postlethwait | Jul 2, 2012 4:42 pm | |
| Christopher B Ferris | Jul 2, 2012 5:41 pm | |
| Thierry Carrez | Jul 3, 2012 2:31 am | |
| Thierry Carrez | Jul 3, 2012 2:34 am | |
| Doug Hellmann | Jul 3, 2012 5:37 am | |
| James E. Blair | Jul 3, 2012 6:55 am | |
| Joshua Harlow | Jul 3, 2012 10:46 am | |
| Dan Prince | Jul 3, 2012 10:59 am | |
| Gabriel Hurley | Jul 3, 2012 11:59 am | |
| Andrew Bogott | Jul 3, 2012 12:47 pm | |
| Joshua Harlow | Jul 3, 2012 2:09 pm | |
| James E. Blair | Jul 3, 2012 2:54 pm | |
| Eric Windisch | Jul 3, 2012 3:47 pm | |
| Andrew Bogott | Jul 3, 2012 3:54 pm | |
| Gabriel Hurley | Jul 3, 2012 4:53 pm | |
| Timothy Daly | Jul 3, 2012 5:27 pm | |
| Monty Taylor | Jul 3, 2012 6:17 pm | |
| Thierry Carrez | Jul 4, 2012 2:56 am | |
| Gabriel Hurley | Jul 4, 2012 3:17 pm | |
| Eric Windisch | Jul 4, 2012 4:11 pm | |
| Christopher B Ferris | Jul 5, 2012 4:29 am | |
| Sean Dague | Jul 5, 2012 6:55 am | |
| Doug Hellmann | Jul 5, 2012 7:16 am | |
| Joshua Harlow | Jul 5, 2012 10:21 am | |
| Mark McLoughlin | Jul 18, 2012 2:01 am | |
| Mark McLoughlin | Jul 18, 2012 2:13 am | |
| Mark McLoughlin | Jul 18, 2012 2:16 am | |
| Mark McLoughlin | Jul 18, 2012 2:23 am | |
| Thierry Carrez | Jul 18, 2012 4:00 pm | |
| Doug Hellmann | Jul 23, 2012 8:50 am | |
| Thierry Carrez | Jul 23, 2012 8:59 am | |
| Doug Hellmann | Jul 23, 2012 9:04 am | |
| Eric Windisch | Aug 2, 2012 1:05 pm | |
| Christopher B Ferris | Aug 2, 2012 2:08 pm | |
| Vishvananda Ishaya | Aug 2, 2012 3:47 pm | |
| Jay Pipes | Aug 2, 2012 5:18 pm | |
| Zhongyue Luo | Aug 2, 2012 5:24 pm | |
| Eric Windisch | Aug 2, 2012 5:51 pm | |
| Mark McLoughlin | Aug 2, 2012 10:26 pm | |
| Thierry Carrez | Aug 3, 2012 2:49 am | |
| Thierry Carrez | Aug 3, 2012 4:02 am | |
| Jay Pipes | Aug 3, 2012 9:25 am | |
| Eric Windisch | Aug 3, 2012 9:34 am |
| Subject: | Re: [Openstack] best practices for merging common into specific projects | |
|---|---|---|
| From: | Gabriel Hurley (Gabr...@nebula.com) | |
| Date: | Jul 3, 2012 11:59:12 am | |
| List: | net.launchpad.lists.openstack | |
The notion that copying code is any protection against APIs that may change is a
red herring. It's the exact same effect as pegging a version of a dependency
(whether it's a commit hash or a real version number), except now you have code
duplication. An unstable upgrade path is an unstable upgrade path, and copying
the code into the project doesn't alleviate the pain for the project if the
upstream library decides to change its APIs.
Also, we're really calling something used by more or less all the core projects
"incubated"? ;-) Seems like it's past the proof-of-concept phase now, at least
for many parts of common. Questions of API stability are an issue unto
themselves.
Anyhow, I'm +1 on turning it into a real library of its own, as a couple people
suggested already.
- Gabriel
-----Original Message----- From: openstack-bounces+gabriel.hurley=nebu...@lists.launchpad.net [mailto:openstack- bounces+gabriel.hurley=nebu...@lists.launchpad.net] On Behalf Of James E. Blair Sent: Tuesday, July 03, 2012 6:56 AM To: andr...@gmail.com Cc: open...@lists.launchpad.net Subject: Re: [Openstack] best practices for merging common into specific projects
Andrew Bogott <abog...@wikimedia.org> writes:
I. As soon as a patch drops into common, the patch author should submit merge-from-common patches to all affected projects. A. (This should really be done by a bot, but that's not going to happen overnight)
Actually, I think with our current level of tooling, we can have Jenkins do this (run by Zuul as a post-merge job on openstack-common).
I very much believe that the long-term goal should be to make openstack- common a library -- so nothing I say here should be construed against that. But as long as it's in an incubation phase, if doing something like this would help move things along, we can certainly implement it, and fairly easily.
Note that a naive implementation might generate quite a bit of review spam if several small changes land to openstack-common (there would then be changes*projects number of reviews in gerrit). We have some code laying around which might be useful here that looks for an existing open change and amends it; at least that would let us have at most only one open merge- from-common-change per-project.
Okay, that's all on that; I don't want to derail the main conversation, and I'd much rather it just be a library if we're close to being ready for that.
-Jim
_______________________________________________ Mailing list: https://launchpad.net/~openstack Post to : open...@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack More help : https://help.launchpad.net/ListHelp
_______________________________________________ Mailing list: https://launchpad.net/~openstack Post to : open...@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack More help : https://help.launchpad.net/ListHelp





