[Update] Murano Dashboard update fails

can't reproduce on my env
1. Deploy neuton vlan with murano (release 5.0-2014.1, iso 5.0-26)
2. Using 439 tarbal {"build_id": "2014-08-15_02-01-17", "ostf_sha": "d2a894d228c1f3c22595a77f04b1e00d09d8e463", "build_number": "439", "auth_required": true, "api": "1.0", "nailgun_sha": "7d7e862ccbdbaa36785b95028d37ac91d7aca440", "production": "docker", "fuelmain_sha": "4f7f45d6a020cc5d49ff5c7ee249522a6de7a096", "astute_sha": "b52910642d6de941444901b0f20e95ebbcb2b2e9", "feature_groups": ["mirantis"], "release": "5.1", "fuellib_sha": "d72cd84468ad9846d4768c11df039c43978a5028"}
3. cd /var; wget http://mc0n1-msk.msk.mirantis.net/fuelweb-iso/fuel-master-upgrade-439-2014-08-15_02-01-17.tar; tar -xvf fuel-master-upgrade-439-2014-08-15_02-01-17.tar; ./upgrade.sh

Actual result:
2014-08-15 10:01:40 INFO 20221 (upgrade) *** UPGRADE DONE SUCCESSFULLY
Ostf pass

Serge you need tun update after upgrade! :)

sorry, i've done update, and deployment failed

but reproduced

Testing shows that modify_horizon script is missing after murano_dashboard package in updated. Why do this happen?

murano_dashboard package has removal of modify_horizon script in its pre_uninstall script. Why there is no need for it.

BUT!

When the package is updated new package is installed new package is installed first and then the old package is uninstalled. PRe uninstall scrip removes the modify file and we are left with this file absent.

Please, remove this line from package pre_uninstall script on all murano_dashboard packages.

Dmitry T, please follow Dmitry I's advice.

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20742
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable-20742/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20743
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable-20743/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20747
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-20747/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20748
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.0.2-stable-20748/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20751
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-20751/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20758
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.0.1-stable-20758/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20751
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-20751/centos

Fix proposed to branch: master
Review: https://review.openstack.org/115264

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/115266

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20751
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-20751/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/17608
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-17608/centos

Reviewed: https://review.openstack.org/115264
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=08d965e3aa3d0660dbbb5c1bb3bee04500e41542
Submitter: Jenkins
Branch: master

commit 08d965e3aa3d0660dbbb5c1bb3bee04500e41542
Author: Dmitry Ilyin <email address hidden>
Date: Mon Aug 18 18:29:47 2014 +0400

    Add modify_horizon_config script

    * Force writing this script even if package
      have removed it.
    * Also set onlyif and unless for modify_config execs.

    Closes-Bug: 1356921
    Change-Id: I4d43706bc5335eb0a200264e83c154ceb2088bcc

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20748
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.0.2-stable/centos

Package murano-dashboard has been built from changeset: http://gerrit.mirantis.com/20742
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable/centos

Dmitry P, Dmitry I works on fix for 5.0.2, hence reassigning to him. Here is his CR into stable/5.0 - https://review.openstack.org/#/c/115266/

Reviewed: https://review.openstack.org/115266
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=c851882cc3f0039e35f366698ad9c3912aedc958
Submitter: Jenkins
Branch: stable/5.0

commit c851882cc3f0039e35f366698ad9c3912aedc958
Author: Dmitry Ilyin <email address hidden>
Date: Mon Aug 18 18:29:47 2014 +0400

    Add modify_horizon_config script

    * Force writing this script even if package
      have removed it.
    * Also set onlyif and unless for modify_config execs.

    Closes-Bug: 1356921
    Change-Id: I4d43706bc5335eb0a200264e83c154ceb2088bcc

Fix proposed to branch: master
Review: https://review.openstack.org/116227

Reviewed: https://review.openstack.org/116227
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=74953d17ea6c0a7f6630988b8dc3528c502b21f8
Submitter: Jenkins
Branch: master

commit 74953d17ea6c0a7f6630988b8dc3528c502b21f8
Author: Dmitry Ilyin <email address hidden>
Date: Fri Aug 22 15:12:16 2014 +0400

    Revert "Add modify_horizon_config script"

    This change is not needed in the master branch.
    Bug only affects updates from 5.0 to 5.0.2.
    Later packages will be fixed.

    This reverts commit 08d965e3aa3d0660dbbb5c1bb3bee04500e41542.

    Change-Id: I13213b67e0c94c1bcae78fe197a4f233f7392208
    Closes-Bug: 1356921

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "5.1.2"
  api: "1.0"
  build_number: "130"
  build_id: "2015-03-12_10-06-05"
  astute_sha: "ef8aa0fd0e3ce20709612906f1f0551b5682a6ce"
  fuellib_sha: "27fc0ad29c979e9e0b9de8987cf663ab253f7f8c"
  ostf_sha: "3447260f7f718e3e85d130b2d20a348b3b68b99d"
  nailgun_sha: "144465154516b81260a62afa0be7d121ab3503fe"
  fuelmain_sha: "6915517eb7ae4d7c3d132d1432fc5cc055257893"
verified