[GitHub] cloudstack pull request: Sorting of security groups

2015-10-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/937#issuecomment-149943690 thanks @atrbgithub , please add the string "CLOUDSTACK-8976" to the title of the PR. It will probably be merged in after 4.6 release. --- If your project is se

[GitHub] cloudstack pull request: Sorting of security groups

2015-10-21 Thread atrbgithub
Github user atrbgithub commented on the pull request: https://github.com/apache/cloudstack/pull/937#issuecomment-149888063 Hi @DaanHoogland , I've raised jira ticket for this request: https://issues.apache.org/jira/browse/CLOUDSTACK-8976 Cheers! --- If your project is set up

[GitHub] cloudstack pull request: Sorting of security groups

2015-10-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/937#issuecomment-149817205 @atrbgithub @rafaelweingartner LGTM with only code inspection. @remibergsma @karuturi one to put on the great pile for batch testing. It is trivial and a great

[GitHub] cloudstack pull request: Sorting of security groups

2015-10-20 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/937#issuecomment-149743155 Nice work LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] cloudstack pull request: Sorting of security groups

2015-10-16 Thread atrbgithub
GitHub user atrbgithub opened a pull request: https://github.com/apache/cloudstack/pull/937 Sorting of security groups Simple change to sort the security groups in alphabetical order within the instance creation wizard. This makes it much easier to find a security group when th