Comment 3 for bug 1267557

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

MIR review:
* Does it FTBFS currently? no
* Does it have a test suite? yes, though a handful of tests are skipped in debian/patches/skip-tests.patch
* Does it have a team bug subscriber? yes, ubuntu-server
* If it's a Python package, does it use dh_python? yes
* If it's a Python package going on the desktop CD, will it pull in Python 2? python2, but not desktop
* Does Ubuntu carry a delta? Ubuntu maintains its own OpenStack packages and is ahead of Debian
* Does it have a watch file? yes
* Is its update history slow or sporadic? it is updated with the rest of OpenStack (ie, Ubuntu and OpenStack release schedules are in sync)
* Is the current release packaged? yes
* Will entering main make it harder for the people currently keeping it up to date? no
* Lintian warnings: source and binaries have ignorable lintian issues
* Is debian/rules a mess? modern dh
* Errors/warnings during the build: quite a few DeprecationWarnings, but since this is py2 and there shouldn't be a py2.8, this shouldn't be a problem for maintenance

Would be nice to have a man page for /usr/bin/heat-manage, but this doesn't block the MIR
/etc/heat is 755 with files that may contain passwords. Should this be 0750 instead like with other OpenStack packages?

MIR team conditional ACK provided python-sendfile dependency is removed.