Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1710
@jburwell Could you please review this PR ? Thanks, @ozhanrk wants to make
it in 4.9.1.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1706
this is our change for similar issue
```
diff --git a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py
b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py
i
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@blueorangutan package
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1681
Invalid params were provided, the hypervisor to be passed was vmware-55u3.
I'll fix this in blueorangutan as it currently does not check/validate the
hypervisor name properly. Kicked a new job man
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1680
The job failed due to systemvms failed to come up, I'll re-kick this.
---
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 projec
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1676
@rhtyd I understand these words: "help", "hello", "thanks", "package",
"test"
Test run usage: test (mgmt server, one of: centos6, centos7, ubuntu)
(hypervisor, one of: kvm-centos6, kvm
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1676
@blueorangutan help
@jburwell incorrect hypervisor name provided, as replied earlier bo does
not check the passed arguments. I'll fix this.
---
If your project is set up for it, you can
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1674
LGTM. I'll re-kick tests.
@blueorangutan package
---
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 t
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1674
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1712
Merging this since it's a pure UI change, validated with manual test LGTMs
and code review LGTMs.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1701
Since this is a pure UI change, merging based on manual test LGTMs and code
review.
---
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 asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1701
---
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 asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1712
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1701
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1745
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1743
LGTM, not sure if this can lead to any side-effects.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1745
@blueorangutan package
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1743
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1542
@blueorangutan package
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@wido I can help run tests if you can rebase/fix-conflicts on this PR,
thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1577
@nlivens I can help run tests on this if you can squash your changes,
thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1542
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1578
@nlivens I can run tests on this PR, if you can squash your changes. Thanks.
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1579
@nlivens I can run tests on this PR if you can rebase, fix conflicts and
squash your changes. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1600
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1600
@blueorangutan package
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1676
Tests have been manually kicked on this now.
---
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 feat
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1743
in 4.9, ALL iptables rules will be saved in /tmp/rules.save
I think it is better to add the rules in
systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py
```
+
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1684
### ACS CI BVT Run
**Sumarry:**
Build Number 133
Hypervisor xenserver
NetworkType Advanced
Passed=98
Failed=5
Skipped=6
_Link to logs Folder (search
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1677
Packaging result: âcentos6 âcentos7 âdebian. JID-115
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1674
Packaging result: âcentos6 âcentos7 âdebian. JID-116
---
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 d
There are some PRs whose author need to fix merge conflicts, or squash their
commits, or participate in the discussion:
1545
1577
1578
1579
1580
Due to issues at the backend, I've manually kicked Trillian tests on following
PRs:
1673
1676
1680
1681
1732
The following PRs in the queue
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1729
LGTM from code change.
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1729
Thanks @murali-reddy
I've rebase against latest base branch, kicking one final round of tests.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1718
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1719
---
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 rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1717
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1713
@blueorangutan package
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1717
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1718
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1719
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1713
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1714
LGTM. @jburwell we can pick this for 4.8+ releases.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1714
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1674
@blueorangutan test
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1674
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@blueorangutan centos7 vmware-55u3
---
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
enable
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/977
@ustcweizhou can you rebase against latest master, fix and merge conflicts
and squash your changes, thanks.
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1701
Packaging result: âcentos6 âcentos7 âdebian. JID-117
---
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 d
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1578
@jburwell @rhtyd Ok, we will rebase the PR and squash our changes/commits,
and let you know here. Moreover, we can re-run the added Marvin tests and share
the results after the re-base.
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1580
@jburwell @rhtyd Ok, we will rebase the PR, and let you know here.
Moreover, we can re-run the added Marvin tests and share the results after the
re-base.
This PR has dependency
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1579
@jburwell @rhtyd Ok, we will rebase the PR, and let you know here.
Moreover, we can re-run the added Marvin tests and share the results after the
re-base.
This PR has dependency
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1577
@jburwell @rhtyd Ok, we will rebase the PR, and let you know here.
Moreover, we can re-run the added Marvin tests and share the results after the
re-base.
Generally, we have two
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1745
Packaging result: âcentos6 âcentos7 âdebian. JID-118
---
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 d
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1745
@blueorangutan test
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1745
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1673
Trillian test result (tid-230)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26194 seconds
Marvin logs:
https://github.com/blueoranguta
Github user abhinandanprateek commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1729#discussion_r85901583
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareContextPool.java ---
@@ -45,76 +43,74 @@ public VmwareContextPool() {
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1729#discussion_r85902083
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareContextPool.java ---
@@ -45,76 +43,74 @@ public VmwareContextPool() {
this(DE
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1729#discussion_r85902494
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareContextPool.java ---
@@ -45,76 +43,74 @@ public VmwareContextPool() {
this(DE
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1729#discussion_r85903435
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareContextPool.java ---
@@ -45,76 +43,74 @@ public VmwareContextPool() {
this(DE
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1542
Packaging result: âcentos6 âcentos7 âdebian. JID-120
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Packaging result: âcentos6 âcentos7 âdebian. JID-119
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1600
Packaging result: âcentos6 âcentos7 âdebian. JID-121
---
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 d
Github user abhinandanprateek commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1729#discussion_r85904768
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareContextPool.java ---
@@ -45,76 +43,74 @@ public VmwareContextPool() {
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
Packaging result: âcentos6 âcentos7 âdebian. JID-122
---
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 d
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1745
@abhinandanprateek the Jenkins build failed. Could you please investigate?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@blueorangutan test centos7 vmware-55u3
---
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 user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1745
@jburwell The reason is ""ERROR: Step âArchive the artifactsâ failed:
no workspace for cloudstack-pr-analysis #2184""
I will force push again.
---
If your project is set up fo
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1600
@blueorangutan test
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1600
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1542
The debian pkg failed due to an intermittent unit test failure (ignorable,
env issue).
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
repl
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1542
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1694
@murali-reddy @jburwell can we have lgtm/review on this, thanks.
---
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 doe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1673
The test results have some intermittent failures around VPC VR related
tests. The PR does not break anything, LGTM on both tests and code review.
---
If your project is set up for it, you can rep
GitHub user ProjectMoon opened a pull request:
https://github.com/apache/cloudstack/pull/1746
SSVM downloader now handles redirects properly.
New version of #1607, opened against 4.9 so it can be forward merged.
**Original Description**
Previously it was using the HttpCl
Github user ProjectMoon commented on the issue:
https://github.com/apache/cloudstack/pull/1607
Closed in favor of #1746.
---
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 a
Github user ProjectMoon closed the pull request at:
https://github.com/apache/cloudstack/pull/1607
---
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 featu
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1713
Packaging result: âcentos6 âcentos7 âdebian. JID-123
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1719
Packaging result: âcentos6 âcentos7 âdebian. JID-126
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1718
Packaging result: âcentos6 âcentos7 âdebian. JID-125
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1717
Packaging result: âcentos6 âcentos7 âdebian. JID-124
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1714
Packaging result: âcentos6 âcentos7 âdebian. JID-127
---
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 d
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Thanks @ustcweizhou for the pointer. It was already fixed in fw_router, but
not in fw_vpcrouter there.
How does this look?
---
If your project is set up for it, you can reply to this emai
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1746
LGTM.
@blueorangutan package
---
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 user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1746
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@jburwell and @rhtyd: Done, conflicts have been resolved!
---
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 ha
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
Thanks @wido can you also squash them?
---
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
en
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1746
Packaging result: âcentos6 âcentos7 âdebian. JID-128
---
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 d
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1746
@blueorangutan test
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1746
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1732
Trillian test result (tid-231)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26649 seconds
Marvin logs:
https://github.com/blueoranguta
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1545
Done @rhtyd
---
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
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1542
Thanks @jburwell! Rebased
---
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1743
@wido maybe the guy who made the change forgot to add it in fw_vpcrouter.
In fw_router, it should be moved out from "if self.get_type() in
["public"]", because the network type is 'guest'
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1577
@jburwell @rhtyd I have rebased this PR with latest master, and squashed
the Marvin test commits.
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user The-Loeki commented on the issue:
https://github.com/apache/cloudstack/pull/1743
See https://bugzilla.redhat.com/show_bug.cgi?id=910619 for what's actually
going on (and what actually should be going on)
---
If your project is set up for it, you can reply to this email a
1 - 100 of 147 matches
Mail list logo
