[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-13 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1996 In the travis ci the tests are failing it could be because of timeout. But when I run some of the test locally it is passing. The same test test_invalid_gw_nm is failing in travis

[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-12 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1996 @koushik-das I have updated for your review comments. --- 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 issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-07 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1996 @jayapalu can you push -f to kick Travis? --- 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

[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-07 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1996 It seems there is issue in CI due to that test are failing. --- 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

[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-04 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1996 @rhtyd Once this PR got the LGTMs, I can rebase it on 4.9. Can you please review this PR --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-03-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1996 @jayapalu this is a useful security fix for 4.9 as well, can you please rebase against the 4.9 branch and edit the base branch of the PR to 4.9? --- If your project is set up for it, you can