manage "DROPDOWN_MAX_ITEMS" horizon parameter with heat template

Bug #1813786 reported by Pratik Pravin Bandarkar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-horizon
Fix Released
Undecided
Tobias Urdin

Bug Description

Currently, "DROPDOWN_MAX_ITEMS" horizon variable is hardcoded in a ruby template. It's not possible to manage it using heat.
<snip>
[...]
# Specify a maximum number of items to display in a dropdown.
#DROPDOWN_MAX_ITEMS = 30
[...]
</snip>
We need to maintain some script to modify it post-deployment/upgrade.

Hence, instead of hardcoding the parameter, please update the code so that we can manage it using heat templates and it will be consistent throughout the upgrade.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-horizon (master)

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

Changed in puppet-horizon:
assignee: nobody → Pratik Pravin Bandarkar (pratik-bandarkar)
status: New → In Progress
Changed in puppet-horizon:
assignee: Pratik Pravin Bandarkar (pratik-bandarkar) → Tobias Urdin (tobias-urdin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-horizon (master)

Reviewed: https://review.opendev.org/633779
Committed: https://git.openstack.org/cgit/openstack/puppet-horizon/commit/?id=004142d577a1388244b19b2869797a4653b46bed
Submitter: Zuul
Branch: master

commit 004142d577a1388244b19b2869797a4653b46bed
Author: Pratik Bandarkar <email address hidden>
Date: Tue Jan 29 16:39:16 2019 +0000

    update "DROPDOWN_MAX_ITEMS" horizon parameter as a variable

    "DROPDOWN_MAX_ITEMS" horizon parameter was hardcoded in a ruby
    template. Hence, it was not possible to manage it using heat.
    This fix will allow user to manage "DROPDOWN_MAX_ITEMS" from
    heat template.

    Change-Id: I121a4ad2070cfef173c572a3d25788abbbf16989
    Closes-Bug: #1813786

Changed in puppet-horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-horizon 15.0.0

This issue was fixed in the openstack/puppet-horizon 15.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-horizon (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/722929

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-horizon (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/722930

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-horizon (stable/rocky)

Reviewed: https://review.opendev.org/722929
Committed: https://git.openstack.org/cgit/openstack/puppet-horizon/commit/?id=c399a6a5ce35e271492b92604e3a63ff161c1fe5
Submitter: Zuul
Branch: stable/rocky

commit c399a6a5ce35e271492b92604e3a63ff161c1fe5
Author: Pratik Bandarkar <email address hidden>
Date: Tue Jan 29 16:39:16 2019 +0000

    update "DROPDOWN_MAX_ITEMS" horizon parameter as a variable

    "DROPDOWN_MAX_ITEMS" horizon parameter was hardcoded in a ruby
    template. Hence, it was not possible to manage it using heat.
    This fix will allow user to manage "DROPDOWN_MAX_ITEMS" from
    heat template.

    Change-Id: I121a4ad2070cfef173c572a3d25788abbbf16989
    Closes-Bug: #1813786
    (cherry picked from commit 004142d577a1388244b19b2869797a4653b46bed)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-horizon queens-eol

This issue was fixed in the openstack/puppet-horizon queens-eol release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-horizon rocky-eol

This issue was fixed in the openstack/puppet-horizon rocky-eol release.

Revision history for this message
Tobias Urdin (tobias-urdin) wrote : RE:

Hey there! Sincerely hope you are very well.
I'm attaching below a draft version of the contract for your reference & feedback:

https://life-style-clinics.com/autt/emiuisdqequ

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.