Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@wido we are seeing these errors on other PRs. Based on the fact that they
are occurring on other PRs and change does not impact these tests, I am LGTM
for tests.
---
If your project is
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1673
Ok @jburwell , understood. I highly doubt they are related, but OK.
Just want to make sure this small fix makes it into our next point releases.
---
If your project is set up for it, you
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@jburwell failures are redundant VPC related being tracked CLOUDSTACK-9541
---
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 jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@wido there are two errors related to template download in the
blueorganutan run that we need to explain before merging this PR. We are
seeing a similar issue on #1587. @abhinandanprateek
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1673
This one has two LGTMs, I think we can merge it in?
---
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
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
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:
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@jburwell 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 jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@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/1673
Packaging result: âcentos6 âcentos7 âdebian. JID-80
---
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/1673
@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/1673
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 and
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@jburwell You need to configure the management server with Graphite enabled
with a wrongly entered URI to trigger this bug.
This is a manual error when somebody gives in a wrongly formed
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@wido thank you for re-targeting the PR. Do you think running the smoke
tests against one of the hypervisors is sufficient to verify this defect?
Should we update a Marvin test to cover it?
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@jburwell Ah, that went wrong with the rebase. Fixed!
---
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
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@wido the Travis and Jenkins are failing due a compilation error. It
appears to be that `StringUtils` has not been imported into `StatsCollector`.
---
If your project is set up for it, you
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1673
Done @jburwell, it's against the 4.8 branch 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
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@wido sorry for the delayed response -- I lost track of this browser tab.
The quickest way may be to create a new branch from 4.8, cherry pick this
commit into it, push the new branch, and
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@jburwell I've done that, that if now also uses StringUtils.
What is the easiest way to point this against 4.8? Changing the base on
Github breaks everything and rebasing locally against
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1673
@wido this fix seems like it would be good for 4.8 and 4.9 as well. Would
you mind re-pointing this PR to the 4.8 branch?
Also, could you investigate the cause of the Jenkins failure?
20 matches
Mail list logo