Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Thanks all for reviewing and testing. I'll merge 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
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy @rhtyd looks good to me on code review. Marvin test is neat.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@rhtyd reviewing...
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1766
LGTM. @abhinandanprateek can you review as well?
---
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
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@rhtyd it was packaging issue, in failed run my change was not picked
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy lgtm, can you see why the same test passed again given no
change to the PR was made? Do you think this new test could become a source of
intermittent failure?
---
If your project
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@rhtyd @abhinandanprateek @ustcweizhou can you please review, we have
clean test pass and new test test_router_dhcp_opts passing too.
---
If your project is set up for it, you can reply
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Trillian test result (tid-402)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28886 seconds
Marvin logs:
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Packaging result: âcentos6 âcentos7 âdebian. JID-229
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy can you check why `test_router_dhcp_opts` failed?
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Trillian test result (tid-396)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26115 seconds
Marvin logs:
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy 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
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Packaging result: âcentos6 âcentos7 âdebian. JID-228
---
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
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Added test case TestRouterDHCPOpts, that will test the VR behaviour of not
sending gateway and DNS server information while server DHCP response on
non-default network NIC's
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy 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
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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/1766
Packaging result: âcentos6 âcentos7 âdebian. JID-171
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Trillian test result (tid-344)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 23535 seconds
Marvin logs:
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1766
I suggest to use default_entry in the script.
see
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Packaging result: âcentos6 âcentos7 âdebian. JID-161
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy 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
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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
28 matches
Mail list logo
