cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yichi Lu" <yichi...@sungard.com>
Subject Review Request 15566: Remove the comparison between GuestOSCategoryVO.CATEGORY_NONE and guestOSCategory.getName() to make OS category of 'None' to be a valid option.
Date Fri, 15 Nov 2013 05:37:54 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15566/
-----------------------------------------------------------

Review request for cloudstack.


Bugs: CLOUDSTACK-4969
    https://issues.apache.org/jira/browse/CLOUDSTACK-4969


Repository: cloudstack-git


Description
-------

CLOUDSTACK-4969
In server/src/com/cloud/resource/ResourceManagerImpl.java near line 1264, there is a line:
 if (guestOSCategory != null && !GuestOSCategoryVO.CATEGORY_NONE.equalsIgnoreCase(guestOSCategory.getName())).
But since GuestOSCategoryVO.CATEGORY_NONE = 'None' and guestOSCategory.getName() = 'None',
this would cause OS category of 'None' to not be selected. Removing it does the job.
Notice the OS category of 'None' at the top of the dropdown list does not have a valid guestOSCategoryid
(and would be null).


Diffs
-----

  server/src/com/cloud/resource/ResourceManagerImpl.java 428a57b 

Diff: https://reviews.apache.org/r/15566/diff/


Testing
-------

Tested against cloudstack GUI. Was able to change OS category to 'None', the bottom one, and
change it back to other OS categories. Selecting the top 'None' would have no effect.


Thanks,

Yichi Lu


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message