[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/703 --- 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 have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/703#issuecomment-132234105 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 have this feature e

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-18 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/703#issuecomment-132145192 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 have this feature enabled

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-17 Thread ksowmya
Github user ksowmya commented on the pull request: https://github.com/apache/cloudstack/pull/703#issuecomment-131794674 @remibergsma , Thanks! I've updated this one to include both commits. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-17 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/703#issuecomment-131792278 Hi @ksowmya Thanks for the PR! It seems this one contains the same commit (2a0eaaebd301c623879384801261c39ea73da305) as in PR #702? Could you please close that o

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-17 Thread ksowmya
GitHub user ksowmya opened a pull request: https://github.com/apache/cloudstack/pull/703 Disablestorage pep8 Fixed pep8 issues in script, most of them for E501 (long lines) with max-line-length set to default Test Result: Test Steps: ... === TestName: test_01_disable_en