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 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 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 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 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 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 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
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Good feedback @ustcweizhou, addressed that.
Thanks @The-Loeki for the report, this is indeed a upstream issue somewhere
else, but if we can mitigate in the meantime that would be great.
-
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1743
@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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Packaging result: âcentos6 âcentos7 âdebian. JID-130
---
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/1743
@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/1743
@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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1743
@The-Loeki as I remember, that is a bug in dhcp3-client. The issue does
exist in Ubuntu 12.04 (guest vm), but fixed in Ubuntu 12.04.3. (I can not find
the related link now). Anyway, the ipta
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Previous Trillian test job failed due to backend issue, I've rekicked them
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Trillian test result (tid-254)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 27962 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Trillian test result (tid-262)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26169 seconds
Marvin logs:
https://github.com/blueoranguta
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Is this one ready to merge? It fixes a rather nasty bug in the VR which
causes big problems when no DHCP offer is received by the Instance(s).
---
If your project is set up for it, you can reply t
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1743
@ustcweizhou @The-Loeki do we have LGTM from you guys on this? Thanks.
LGTM.
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1743
@rhtyd @wido I would suggest one more change: move line 378-380 after line
360.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1743
Good suggestion @ustcweizhou, done that!
---
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 ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1743
LGTM !
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1743
LGTM on code review, merging this based on review and test lgtms. Few
errors around VPC/VR are known intermittent issues.
---
If your project is set up for it, you can reply to this email and hav
23 matches
Mail list logo