Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/957
---
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 user michaelandersen commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149805301
yes @wilderrodrigues it is most likely an environment issue. don't wait on
me for this PR.
---
If your project is set up for it, you can reply to this ema
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149799787
@remibergsma @michaelandersen
New results:
```
test_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL)
... === TestN
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149791585
@michaelandersen
Could you give some details on how you executed the tests? It looks like
env issues.
Did you...
* checked out the
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149718198
It works fine for me:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=false smoke/test_privategw_acl.p
Github user michaelandersen commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149706996
from the mgt server log:
WARN [c.c.a.d.ParamGenericValidationWorker]
(709693495@qtp-265638379-0:ctx-dfba94e8 ctx-8090674b ctx-c8da340d) Received
unknown
Github user michaelandersen commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149706469
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP
connection (1): cs2.cloud.lan
requests.packages.urllib3.connectionpool: DEBUG: "GET
/cl
Github user michaelandersen commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149706192
@remibergsma @wilderrodrigues LGTM'ed this one too soon, was a bit too
enthusiastic. sorry about that. Ran the test just now, but it's failing!
nos
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/957#issuecomment-149573159
@remibergsma @michaelandersen
I added the wrong ACS issue to the PR title and the commit. I already
edited the title, will now amend the commit and