Neutron: package improvement
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mirantis OpenStack |
Fix Released
|
Medium
|
Sergey Kolekonov |
Bug Description
This is a general review of the package.
debian/control
--------------
- Missing build-depends: dh-python, dh-systemd, openstack-pkg-tools (>= 23~), python-pbr, python-setuptools, python-sphinx
- Missing build-depends-
- Useless build-depends-
- Missing dependency: iproute2, ipset, python-gflags, python-mysqldb, python-psycopg2, python-pyudev
- Useless depends: python-anyjson, python-babel, python-kombu
- Useless: XS-Python-Version: >= 2.7
- conntrack dependency missplaced (should be in neutron-common)
- Why is neutron-common depending on python-
debian/copyright:
-----------------
- No mention of the licensing for debian/*
debian/changelog:
-----------------
- only 2 entries!
debian/
-------
- Same as for other packages: lots (if not all) of this file should go away.
debian/*.logrotate
------------------
- Advise: scrap all of these, and use a single neutron-
- Not using copytruncate: bad for production
plugin configuration
-------
- In Debian, the startup scripts are looking at the core_plugin values.
We should move to that, and scrap the /etc/default/
debian/*.install
----------------
- Advise: scap all this, and install all in neutron-common. It's prone to lots of errors.
init scripts
------------
- Only support for upstart
- Missing chgid on all daemons
- neutron-
- lack of consistency for log files (ie: missing neutron- before <something.log>
naming conventions:
-------------------
- In Debian, we removed "plugin" from the name of the neutron-
General:
--------
- All of these plugin packages are an invention of Canonical. I'd advise to scrap all of it, and switch to a more simple way: put everything in neutron-common, available all the time.
Changed in mos: | |
milestone: | none → 7.0 |
assignee: | nobody → MOS Deployment Team (mos-deployment) |
importance: | Undecided → Medium |
Changed in mos: | |
status: | New → Confirmed |
Changed in mos: | |
assignee: | MOS Deployment Team (mos-deployment) → Sergey Kolekonov (skolekonov) |
tags: | added: neutron |
Changed in mos: | |
status: | Confirmed → Fix Committed |
tags: | added: on-verification |
tags: | removed: on-verification |
Changed in mos: | |
status: | Fix Committed → Fix Released |
Reviewed: https:/ /review. fuel-infra. org/9526 ci/fuel- 7.0/2015. 1.0
Submitter: Igor Yozhikov <email address hidden>
Branch: openstack-
Commit: a2762ccf67e2d52 ccf212495c35a06 40c2563bdb
Author: Sergey Kolekonov <email address hidden>
Date: Fri Jul 17 15:42:12 2015
Improve Neutron packages
- fix build and runtime dependencies
- fix a problem with Mellanox agent upstart script
- use universal logrotate configuration
- use chgid on all daemons
Related-bug: #1462251 79b83b6f9e5d367 6c256d587e7
Change-Id: I44bc699a9a6ac3