Code review comment for lp:~flacoste/launchpad/bug-129943

Revision history for this message
Robert Collins (lifeless) wrote :

> Even though python-feedparser is a pretty static dependancy, I'd really not want to block on a l-d-d update to land this branch. If you insist, I could add it to the launchpad-dependencies, so that the next deployment of it, would pull it off and use it instead of the one in the download-cache. I would not ask LOSA to go to the trouble of building new buildbot images and deploy these two packages across all app servers for that dependency.

I'm fine with not blocking.

If we need new buildbot images when a standard packaged item is added
to the deps, thats a bug in our use of buildbot we should just fix. Is
that historical or current experience?

James and Tom have indicated in the past that new deps in the packages
are trivial for them to accomodate when they are of standard packages,
so I wouldn't expect any difficulty in having it deployed promptly.

But again, I'm fine with not blocking, doing the l-prod-d dependency
change and when that has been fully actioned dropping the buildout
stanza.

« Back to merge proposal