Review Request #22863 , #23314 : Brocade Plugin review request...

2014-07-18 Thread Ritu Sabharwal
Hi Hugo, I have fixed the 3 items as proposed by you: 1. Added the unit tests 2. Added integration test 3. Removed the configuration from properties file to networks service provider configuration using APIs or GUI. Can you please review the plugin code so that we can make i

Brocade Plugin review..

2014-07-16 Thread Ritu Sabharwal
Hi Hugo, I have fixed the system.js file. Also, removed the configuration reading from properties files. Now the plugin configurations are read as network service provider configurations from GUI or APIs. The unit tests are also updated for the change. Please review it and do the needful so th