Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1023
---
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 remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153726658
LGTM, the test from this branch now succeed:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153717404
Ping @borisroman @remibergsma @DaanHoogland @miguelaferreira @karuturi
Test results:
```
Test redundant router internals ... === TestNa
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153713989
@borisroman , could you please retest? It worked for me and for you, but
not for @remibergsma due to a resolving thing on RVR test. I then ran it again
and
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153707458
A ticket is out for this at infra: INFRA-10703
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user miguelaferreira commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153701940
JVM died:
https://builds.apache.org/job/cloudstack-pull-analysis/1150/org.apache.cloudstack$cloud-server/console
---
If your project is set up for it, you
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153701619
Jenkins error is not related. It says "no failures" but still shows a red
button on Cloud Server. By the way, I pushed a python file.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153596954
Hi @karuturi,
Sorry, perhaps too earlier here, but I don't follow completely. :)
How did you setup your environment in order to test the cas
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153573013
@wilderrodrigues
I think this has to do with restarting a egress default allow network with
existing egress rules.
When my network had an existing egress
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153572067
testing steps same as above.
iptables rules on default egress ALLOW router
```
Chain FORWARD (policy DROP 0 packets, 0 bytes)
pkts bytes target
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153525534
Updated test results:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component/test_vpc_r
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153376651
The last test failed because the connection timed out without printing
"Giving up.", as I expected in the test. You can see the output below:
```
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153373615
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first):
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153372412
Thanks @wilderrodrigues I will test this tomorrow morning IST
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153371614
@wilderrodrigues my bubble is very busy but ping if you are ready and need
testing
---
If your project is set up for it, you can reply to this email and have
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153370030
Ping @remibergsma @karuturi @borisroman @DaanHoogland
The test is ready! It now does the following:
1. Creates network + VM + FW + PF + Def
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153363457
Ping @remibergsma @karuturi @DaanHoogland @borisroman
It's working already, but I'm modifying the tests. I pushed by mistake. If
you want to start
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153309653
More test results. It did not break existing functionality, but still I
will fix the one @karuturi found.
Cheers,
Wilder
```
test_p
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153305656
Thanks for the details, @karuturi ! I will fix that.
Cheers,
Wilder
---
If your project is set up for it, you can reply to this email and have
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153301644
@wilderrodrigues apart from the issue mentioned in CLOUDSTACK-9018, I found
the below issue.
The egress rule added in a default egress ALLOW network doesnt bloc
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153286583
@wilderrodrigues Thanks for the fix, tests are running.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153285784
Ping @remibergsma @borisroman
Partial results (tests still running)
```
Test start/stop of router after addition of one guest network
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153274765
did the following to test it on an existing XenServer setup (It has two
networks egress_allow with default egress allow and isolated2 with default
egress DENY):
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1023#issuecomment-153257739
Ping @DaanHoogland @borisroman @remibergsma @miguelaferreira @wido
@karuturi
There is another issue about the VR/egress, but since that's a differe
GitHub user wilderrodrigues opened a pull request:
https://github.com/apache/cloudstack/pull/1023
CLOUDSTACK-8925 - Default allow for Egress rules is not being configured
properly in VR iptables rules
This PR fixes the router default policy for egress. When the default is
DENY, the
34 matches
Mail list logo
