]> review.fuel-infra Code Review - openstack-build/horizon-build.git/commit
Fix numerous issues in the Horizon package 45/8945/37
authorThomas Goirand <zigo@debian.org>
Thu, 2 Jul 2015 01:10:17 +0000 (03:10 +0200)
committerIgor Yozhikov <iyozhikov@mirantis.com>
Tue, 7 Jul 2015 19:30:25 +0000 (22:30 +0300)
commitb66c8946d35a15ca5d7b5e4e97ac90423ff037ab
tree982ed91ef26ace66b5bb369e4b72815e9a6ec48c
parent18d4877b7040ed37f4a1474084d3f4e05bf23ada
Fix numerous issues in the Horizon package

The Horizon package comming from Ubuntu has numerous issues, and this
patch attemps to fix some of them. There's some less important things
which could be fixed later on.

  * Adds missing symlinks to horizon to fix broken offline compress, which
    made the package uninstallable (LP: #1470143).
  * Ran wrap-and-sort -t -a to make dependencies readable.
  * Run compile-messages at build time, not in postinst.
  * Activates off-line compress.
  * Fix path of secret-key to be in /var (as /usr should be considered to be a
    read-only filesystem, as per Debian policy).
  * Copy the missing angular folder.
  * debian/openstack-dashboard.postinst
    - Re-wrapped the file which was really unreadable in its current state.
    - Redefines --home as /var/lib/horizon, and not /usr/share/horizon, which
      really is not policy compliant (in openstack-dashboard.postinst).
    - Completely remove the collecstatic which should *never* be done in this
      package: we're not using a virtualenv!
    - Completely remove chown of files in /usr. By policy of the FSHS, this
      should *never* happen.
    - Removed chown of /etc/openstack-dashboard. This is useless and
      dangerous in therms of security.
    - Avoids hacks with the secret-key folder in postinst (since we've set
      that by default to be /var in the default config file).
    - Add off-line compress if triggered.
  * Add trigger if the some javascripts are updated.
  * Fixes /horizon/static Alias directives (thanks to I. Yozhikov).
  * Fixes in _variables.scss for /horizon/ webroot
  * Activating OFFLINE_COMPRESS=True, which speeds up a lot Horizon.
  * Fix the way /var/lib/openstack-dashboard/secret_key is handled
    (previously a copy of /etc/openstack-dashboard, now an alias to it).
  * Fix tests to not try to overwrite the package apache config.

Closes-Bug: #1470143
Change-Id: I7de1968ff9c1ddd03a9965aa66fb9407d3f8d1dd
tests/runtests.sh
trusty/debian/changelog
trusty/debian/compile-messages [new file with mode: 0644]
trusty/debian/control
trusty/debian/openstack-dashboard.conf
trusty/debian/openstack-dashboard.links
trusty/debian/openstack-dashboard.postinst
trusty/debian/openstack-dashboard.postrm
trusty/debian/openstack-dashboard.triggers [new file with mode: 0644]
trusty/debian/rules