Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy @abhinandanprateek @DaanHoogland can you comment on 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 proj
Github user milamberspace commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy @rhtyd Possible regression introduce by the PR1659 see
https://issues.apache.org/jira/browse/CLOUDSTACK-9770
---
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/1659
Travis failure is to do with a test_volumes failures in one of the
component tests, not related to this PR. Will merge, and handle the Travis
failure separately.
---
If your project is set up fo
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
component test results. If any one wish to run component test
test_multiple_public_interfaces.py, add a new public ip range in the
test_daya.py, in the 'publiciprange' dict object.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-597)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36723 seconds
Marvin logs:
https://github.com/blueoranguta
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@rhtyd please proceed with merge. I have not added the test
suite(test_multiple_public_interfaces.py) part of this PR as smoke tests,
because it takes around an hour for execution so they a
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Thanks @murali-reddy @borisstoyanov I'll proceed with merging this now.
@murali-reddy do you want to run the component test on this PR or on 1753/1754?
---
If your project is set up for it, you c
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Yes I think test failures does not seem to be caused by this PR, LGTM based
on code review.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@rhtyd I understand these words: "help", "hello", "thanks", "package",
"test"
Test command usage: test [mgmt os] [hypervisor] [additional tests]
Mgmt OS options: ['centos6', 'centos
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@blueorangutan help
---
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 murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Can this merged now? test failures are specific to redundent VR/VPC VR
which have been failing in 4.8/4.9/master.
---
If your project is set up for it, you can reply to this email and have
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1659
LGTM on code review and testing @murali-reddy @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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-591)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6
Total time taken: 34361 seconds
Marvin logs:
https://github.com/blueoran
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-581)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28738 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt
+ vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
---
If your project is set up for it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@blueorangutan test matrix
---
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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-323
---
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/1659
@murali-reddy @abhinandanprateek let me know any help needed from my end?
---
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 pr
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
@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/1659
Test lgtm, @murali-reddy are we good on this PR? I'm seeing some failures
though not sure if they related to your changes. /cc @jburwell
@abhinandanprateek
---
If your project is set up for it,
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-532)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35829 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-533)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35604 seconds
Marvin logs:
https://github.com/blueoran
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@blueorangutan test centos7 xenserver-65sp1
---
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 fe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@jburwell a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
r
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Tests look good on KVM. However, it seems like a Good Thing(tm) to test on
KVM and XenServer as well.
@blueorangutan test centos7 vmware-55u3
@blueorangutan test centos7 xenserver-
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@jburwell 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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-483)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 27669 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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
r
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-280
---
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/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-279
---
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/1659
@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
r
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@blueorangutan test centos7 kvm-centos7
---
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 fe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-278
---
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/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-277
---
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/1659
Trillian test result (tid-464)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28781 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
@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/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-263
---
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/1659
@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/1659
@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/1659
Trillian test result (tid-440)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33701 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
@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/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-254
---
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/1659
@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/1659
@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/1659
@murali-reddy can you check why we've test failures for KVM? Also, if the
integration tests don't take a lot of time (few minutes to half hour) can you
move it to smoke tests?
---
If your projec
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-393)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 29875 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
@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/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-201
---
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/1659
@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/1659
@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/1659
Trillian test result (tid-195)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26462 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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
r
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-99
---
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 do
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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
rep
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
Trillian test result (tid-168)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24863 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
@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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-87
---
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 do
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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
rep
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy can you fix the merge conflict, and rebase against latest
base branch?
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
LGTM. @murali-reddy can you squash the commits to a single commit and
rebase against latest 4.9 branch. Following merge of PR #1703 I'll kick a new
trillian test run and merge based on the test re
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Did a code walk-thru and review with @murali-reddy, 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 projec
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-32
---
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 do
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy there is a potential to move some of the code into net
utility classes, think about it.
---
If your project is set up for it, you can reply to this email and have your
r
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
#1692 has been opened that replaces #1683 to address the ping test issue on
the 4.8, 4.9, and master branches rather just master for the original PR.
Testing the PR now.
---
If your project
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@ustcweizhou thanks, i have fixed it earlier today and tested, am trying to
fix other test failures and re-run the CI
---
If your project is set up for it, you can reply to this email and
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy FYI
```
diff --git a/systemvm/patches/debian/config/opt/cloud/bin/merge.py
b/systemvm/patches/debian/config/opt/cloud/bin/merge.py
index c6ed190..7f3dbbc 100755
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@jburwell i am investigating them. Just so you know, blueorangutan is
running full smoke test suite. Similar failures are seen in other PR as well.
So some of the tests may have been failin
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy 8 errors occurred in the VMware test run. Could you please
investigate these failures?
---
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/1659
Trillian test result (#21)
Environment: vmware-55u3 (x2), Advanced Networking
Total time taken: 27908s
Marvin logs:
https://github.com/blueorangutan/acs-prs/releases/download/tr
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy 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
r
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Packaging result: âcentos6 âcentos7 âdebian. JID-20
---
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 do
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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
rep
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@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 jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy what is the status of addressing the review feedback. Also,
have addressed the VPC-related issues as well?
---
If your project is set up for it, you can reply to this email and
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy Looking through the Marvin tests, there is a lot of
boilerplate code and duplicated setup in test cases. It feels like there is an
opportunity to introduce one or more base class
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@dsclose what is your opinion of the changes in this PR relative to your
work? Do you see any gaps or missing pieces?
---
If your project is set up for it, you can reply to this email and hav
Github user dsclose commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy - thank you for picking this up. Let me know when you think
it's appropriate to close PR #1519 and I shall do so.
---
If your project is set up for it, you can reply to this email
88 matches
Mail list logo