modify aggregates missing

Bug #1261932 reported by Matthias Runge
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Wishlist
Alejandro Emanuel Paredes

Bug Description

Since aggregates are now added, we should be able to modify them, add hosts, etc.

nova has the ability to do so.

Matthias Runge (mrunge)
Changed in horizon:
milestone: none → icehouse-3
Changed in horizon:
status: New → Confirmed
David Lyle (david-lyle)
Changed in horizon:
importance: Undecided → Wishlist
Changed in horizon:
assignee: nobody → Alejandro Emanuel Paredes (alejandro-e-paredes)
Revision history for this message
Alejandro Emanuel Paredes (alejandro-e-paredes) wrote :

Hi all, just to clarify a little bit about this bug, I traced the changes about this issue and found this:
- The fix for Bug #1197775 (adding aggregates panel to admin dashboard) showed the aggregates in the admin panel...
- The implementation of the blueprint "Show zone information in admin panel" (https://blueprints.launchpad.net/horizon/+spec/show-zone-for-admin), moved the aggregates to System Info panel since it's static information...

Then to create and modify aggregates, I think we should move back the aggregates section to the admin panel (as it was originally before the blueprint I'm mentioning above).
IMHO, we should have the aggregates in the admin panel just like "Flavors" , where they can be added, edited and deleted.

Any other thoughts about this issue?
Thanks!!!

Revision history for this message
Julie Pichon (jpichon) wrote :

Hi Alejandro. How is the progress for this going? I'd really like to see this one happen in Icehouse, so please let me know if I can offer any help with the code or reviewing patches!

I agree with your comment, once Host Aggregates is more than a read-only information panel it should move out of System Info and into the main panel list.

Revision history for this message
Alejandro Emanuel Paredes (alejandro-e-paredes) wrote :

Hi Julie!, thanks for replying and your help...
I was waiting for an external opinion about this issue.
I've a draft, and as soon is ready to be shared, I'll contact you to to see it together.
Have a nice day.
ale.

Revision history for this message
Julie Pichon (jpichon) wrote :

Thanks for the reply Alejandro. I find that in general, patches are more likely to get feedback/answers than questions in the bug tracker comments :)

Revision history for this message
Alejandro Emanuel Paredes (alejandro-e-paredes) wrote :

work in proggress in...
https://review.openstack.org/#/c/71061
Any sugestion and feedback are welcome!

Changed in horizon:
assignee: Alejandro Emanuel Paredes (alejandro-e-paredes) → Facundo Farias (facundo-farias)
status: Confirmed → In Progress
Changed in horizon:
assignee: Facundo Farias (facundo-farias) → Santiago Baldassin (santiago-b-baldassin)
Changed in horizon:
assignee: Santiago Baldassin (santiago-b-baldassin) → Facundo Farias (facundo-farias)
Changed in horizon:
assignee: Facundo Farias (facundo-farias) → Julie Pichon (jpichon)
Julie Pichon (jpichon)
Changed in horizon:
assignee: Julie Pichon (jpichon) → Alejandro Emanuel Paredes (alejandro-e-paredes)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/71061
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=ed1525bc91e1ca68a33117f0dbef4a241f33971f
Submitter: Jenkins
Branch: master

commit ed1525bc91e1ca68a33117f0dbef4a241f33971f
Author: Facundo Farias <email address hidden>
Date: Fri Feb 7 18:05:27 2014 -0300

    Host aggregates panel.

    On this panel, aggregates could be added, deleted and edited.
    This patch takes the aggregates panel out of System Info and
    puts it back in the main admin panel list, as now, aggregates
    are not static information. The host can be associated to a
    host aggregate on this panel as well.

    Change-Id: I4ef2d87c33981db36d4ebd3de2f4841cdfa9dbfd
    Closes-Bug: #1261932
    Implements: blueprint manage-host-aggregates
    Co-Authored-By: Santiago Baldassin <email address hidden>
    Co-Authored-By: Alejandro Paredes <email address hidden>

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-3 → 2014.1
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.