cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gaurav Aradhye" <>
Subject Re: Review Request 25642: CLOUDSTACK-7547: Brocade Device Data was hard coded, moved it to config file
Date Mon, 15 Sep 2014 12:39:02 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Sept. 15, 2014, 6:09 p.m.)

Review request for cloudstack, Ritu  Sabharwal, Santhosh Edukulla, and SrikanteswaraRao Talluri.


Repository: cloudstack-git

Description (updated)

Brocade device data was hard coded. 

1) Made changes so that it can be read from the config file. (Config tranlated to the config
file we provide to jenkins, not file, because dynamic data should always be placed
in config, not
2) Skipped the test case if the data is not present in config file.
3) Removed * from imports and made appropriate changes.


  test/integration/component/ a5eff53 




Tested all the scenarios of valid and invalid data.
Skipping of test case was proper.

When all the data was present, test case moved forward to adding the brocade device but test
case still failed because the details I put in config file were not valid (I don't have brocade
device details).


Gaurav Aradhye

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