[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220716022 I think since it is an intermittent issue, we did not catch it on the actual PR it was introduced in. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220715917 I am getting it on almost every PR right now (if you notice the last 6 or 7 CI runs), so I will have to review this as it seems to have been introduced recently...

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220675652 Thanks @swill if we keep getting this error, I'll try to find some time to see what's causing this as well --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220626559 @rhtyd thank you. I will review what happened there. This is something that has started showing up intermittently, but only started happening in the last few

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1551 --- 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

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220533786 @swill the error suggests the template response is missing the key 'status' in its json response. It may happen if there has been any change around template

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220512003 These errors are not related to this PR, but if you know where these errors are coming from I would be interested in some input. :) --- If your project is set up

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220511935 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 29m 09s ``` **Summary of the

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220395388 I already started another run against this PR, so that should also validate nothing was accidentally broken. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220380639 @swill thanks, though the latter changes are all packaging related. I've built rpm/deb repository to show packaging works:

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220379062 This was run prior to your changes. I will need to rerun to incorporate the recent changes... --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220378822 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 4h 15m 16s ```

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220273077 👍 (code review only) --- 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: dynamic-roles: packaging improvements

2016-05-19 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220262092 Ok, so this PR seems to fix multiple things. I would LGTM on the packaging fixes as well as the Python part. The MySQL thing I'm not sure about. I