+1 for removing "changes" attribute. It's useless now. If there are no plans to add something else there, let's remove it.
2015-01-26 11:39 GMT+03:00 Julia Aranovich <jkirnos...@mirantis.com>: > Hi All, > > Since we changed Deploy Changes pop-up and added processing of role > limits and restrictions <https://review.openstack.org/#/c/126930/> I > would like to raise a question of it's subsequent refactoring. > > In particular, I mean 'changes' attribute of cluster model. It's displayed > in Deploy Changes dialog in the following format > <http://s2.postimg.org/ak9inonhl/deploy_changes_dialog.png>: > > - Changed disks configuration on the following nodes: > - <node_name_list> > - Changed interfaces configuration on the following nodes: > - <node_name_list> > - Changed network settings > - Changed OpenStack settings > > This list looks absolutely useless. > > It doesn't make any sense to display lists of new, not deployed nodes with > changed disks/interfaces. It's obvious I think that new nodes attributes > await deployment. At the same time user isn't able to change > disks/interfaces on deployed nodes (at least in UI). So, such node name > lists are definitely redundant. > Networks and settings are also locked after deployment finished. > > > I tend to get rid of cluster model 'changes' attribute at all. > > It is important for me to know your opinion, to make a final decision. > Please feel free and share your ideas and concerns if any. > > > Regards, > Julia > > -- > Kind Regards, > Julia Aranovich, > Software Engineer, > Mirantis, Inc > +7 (905) 388-82-61 (cell) > Skype: juliakirnosova > www.mirantis.ru > jaranov...@mirantis.com <jkirnos...@mirantis.com> > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > -- Vitaly Kramskikh, Fuel UI Tech Lead, Mirantis, Inc.
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev