Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1511#issuecomment-220595800
Im not sure what to think of those failures. I will run again in clean
setup to see if I can get rid of those.
---
If your project is set up for it, you can reply
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-220595991
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 2
Errors: 0
Duration: 4h 12m 50s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-220596270
I am getting similar errors in other PRs, so I don't think they are related
to this PR.
---
If your project is set up for it, you can reply to this email and
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1518#issuecomment-220619741
For some reason Travis is unhappy. Would you mind trying again. Sorry for
the inconvenience...
---
If your project is set up for it, you can reply to this email
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1545#issuecomment-220621430
@DaanHoogland yes, I will retest now. Thanks guys...
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-220622074
Can I get some code review on this one. It is looking to be in pretty good
shape otherwise. Thanks...
---
If your project is set up for it, you can reply to this
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1511#issuecomment-220624878
I have seen similar issues in a couple other PRs. Since this is a PR
related to connectivity, I do want to try to get a cleaner CI run if I can.
Also, travis is
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1551#issuecomment-220626559
@rhtyd thank you. I will review what happened there. This is something
that has started showing up intermittently, but only started happening in the
last few weeks
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1251#issuecomment-220627549
Thanks @koushik-das, I need to track down what changed to cause that
intermittent problem of the `template.status`. I need some code review on this
one. Thanks
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220627907
### CI RESULTS
```
Tests Run: 83
Skipped: 0
Failed: 0
Errors: 2
Duration: 8h 27m 27s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220628087
Two clean runs in a row. I think this one is ready to put back in now...
---
If your project is set up for it, you can reply to this email and have your
reply
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220628402
BTW, thank you for your quick response time to work through issues on your
PRs and get things stabilized. I really do appreciate it.
---
If your project is set up
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1251#issuecomment-220628656
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 4
Duration: 8h 31m 38s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1251#issuecomment-220628965
Similar issues on this run. They are not related to this PR. Still need
some review on this one...
---
If your project is set up for it, you can reply to this
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1514#issuecomment-220629958
This one is coming back clean. If we can get Jenkins and Travis green I
think this one is ready...
---
If your project is set up for it, you can reply to this
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1514#issuecomment-220629640
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 8h 44m 23s
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1417#issuecomment-220631924
Ok, thanks @nitt10prashant. ð
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1540#issuecomment-220632778
Ya, sorry, I had a backlog of commits that were ready while I sorted out
some merge conflicts doing a forward merge from 4.8 and then testing the
resulting master to
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1533#issuecomment-220633650
@sverrirab thank you. Would you mind adding the revert of #1495 into this
commit in order to clean up unnecessary packages once this PR is accepted? I
think we are
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1511#issuecomment-220640589
Ok. Travis has been pretty consistently green for PRs recently, so I have
not seen this problem elsewhere. I am not sure why there are sometimes more
than once
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1360#issuecomment-220656462
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 1
Errors: 4
Duration: 10h 22m 36s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1360#issuecomment-220656774
Ran again because the error was not something I had seen often. The errors
in the new run are not related to this PR.
---
If your project is set up for it, you can
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220715003
And of course the first build I do after I merge it into master I get the
`addHost` error and I did not get it once after I pulled it out. Haha... I
will build
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220715486
@rhtyd can you specify the logs you need if I run into this issue so you
can track down why it is happening? I will leave it in for now but I would
like to send you
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1551#issuecomment-220715917
I am getting it on almost every PR right now (if you notice the last 6 or 7
CI runs), so I will have to review this as it seems to have been introduced
recently
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1551#issuecomment-220716022
I think since it is an intermittent issue, we did not catch it on the
actual PR it was introduced in.
---
If your project is set up for it, you can reply to this
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220735448
@rhtyd uggg... The first two builds on master after merging this back in
have both resulted in the `addHost` error. :( Let me know what you need to
troubleshoot
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-220737825
This is one of the only PRs I have seen consistently failing on the scale
vm test. I am not sure what to think about that. Can you rebase and force
push or close
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1482#issuecomment-220738113
@remibergsma can you give me a status on this PR. I know you reverted it
because of missing licenses. Should I be trying to wait for this one to get
back in for
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1514#issuecomment-220738431
I would like to get this one re-pushed or closed and reopened so we can
kick the CI again to see if we can get things green. I am getting very close
to freezing and
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1410#issuecomment-220738747
Getting close to freeze, please repush to see if we can get travis green.
Thx...
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1410#issuecomment-220738889
@ustcweizhou ^
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1424#issuecomment-220738801
@syed: Getting close to freeze, please repush to see if we can get jenkins
green. Thx...
---
If your project is set up for it, you can reply to this email and have
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1423#issuecomment-220739003
@syed please close and reopen or repush. we are close to freeze and I want
to get some of these PRs in...
---
If your project is set up for it, you can reply to
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1464#issuecomment-220739360
@sanju1010 I don't know what to do with this one. I don't have any
verification that this PR works, so I am not sure what information I am
supposed to b
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-220739661
@DaanHoogland you think the issue that travis is having is a genuine issue,
is that what you mean?
---
If your project is set up for it, you can reply to this email
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-220741573
OK. I don't really have the bandwidth to troubleshoot this (unfortunately).
This is one of the many PRs that are on the bubble which I am trying to figure
out
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-220745686
Have a nice weekend. :)
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1424#issuecomment-220755087
the sweet sweet color green... ð
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1533#issuecomment-220810829
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 8h 55m 29s
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1533#issuecomment-220810862
This is coming back clean. Would you mind trying to rebase and re-push or
close and reopen to see if we can kick off travis one more time. Thanks...
---
If your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1556#issuecomment-220811376
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 1
Errors: 1
Duration: 11h 00m 25s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1406#issuecomment-220813294
@ProjectMoon can you reopen this against at least 4.7, maybe even master.
Thanks...
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1552#issuecomment-220813347
Does this need CI?
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1423#issuecomment-220813537
@syed can you provide details for how to add `ike-scan` in order to run the
test and verify via CI? Not sure how I feel about adding a test that can't be
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1482#issuecomment-221151970
@remibergsma ok, no problem. Thanks for the heads up. :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1552#issuecomment-221155776
Ok, so I have two LGTM both with validation that the code behaves as
expected. I think that is enough for this to get in. Thanks...
---
If your project is set up
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1545#issuecomment-221158751
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 4h 25m 18s
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1545#issuecomment-221158910
This is coming back clean. Can I get a second LGTM? Maybe @rhtyd?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1511#issuecomment-221159015
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 1
Duration: 4h 20m 05s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1511#issuecomment-221159200
@DaanHoogland can you rebase and push again to try to kick off Travis and
see if we can clear this one up? Thanks...
---
If your project is set up for it, you can
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1556#issuecomment-221159616
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 1
Errors: 1
Duration: 10h 51m 43s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1556#issuecomment-221159792
I ran CI again hoping to clear up those errors and I got the same ones
again. I don't think they are related to this PR, but twice in a row concerns
me a bi
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221159912
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 8h 55m 13s
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221159974
I think this one is ready...
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-221160227
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 35m 03s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-221160368
This is coming back clean now. Those failures are not related to this PR.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1550#issuecomment-221160504
### CI RESULTS
```
Tests Run: 78
Skipped: 0
Failed: 0
Errors: 4
Duration: 7h 14m 41s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1550#issuecomment-221160650
This is a clean CI run. The failures are not related to this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-221162055
@rhtyd fingers crossed, things seem to be in pretty good shape. The first
two runs after the merge had `addHost` errors, but I have done about 6-7 CI
runs since and
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1514#issuecomment-221162753
This one is green. I think this one is ready??? Please verify if there is
anything outstanding on this one. Thanks... :)
---
If your project is set up for it
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1533#issuecomment-221162872
Yes, ready now. Thank you. :)
---
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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1545#issuecomment-221166274
I missed that this one was going to need a new systemvmtemplate. Given
that I am already over a week late on this release and I have not done any
testing or work
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1556#issuecomment-221277292
This is a clean run. The errors here are not related to this PR. I will
merge this. Thanks...
---
If your project is set up for it, you can reply to this email
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-221278053
Apparently `status` is missing from the `listTemplates` response in some
cases. I will investigate after we freeze and before we cut an RC to see if we
can resolve
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1556#issuecomment-221277026
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 11m 34s
```
**Summary of the
Github user swill commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1530#discussion_r64395993
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/mo/VmdkFileDescriptor.java ---
@@ -27,55 +27,53 @@
import org.apache.log4j.Logger
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221278559
@dsclose if you can fix the merge conflicts I can run this again and see
what is outstanding. Thanks...
---
If your project is set up for it, you can reply to this
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1548#issuecomment-221284250
@moul can you close and reopen the PR or rebase and force push in order to
kick off Jenkins again so we can get this all green?
@pdion891 I think you have
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221324840
Thanks for the update. :)
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221409302
Was the only change since the last CI run the updating of the marvin test
(which I can't run anyway)?
---
If your project is set up for it, you can reply to
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221409650
Oh, I think you were just resolving conflicts. I will run CI again just to
verify... Otherwise this is looking good. Thanks everyone...
---
If your project is
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1423#issuecomment-221631046
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 1
Duration: 4h 14m 49s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1423#issuecomment-221631305
@syed can you review this error? It seems to be related to your test.
Thanks...
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/846#issuecomment-221633655
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 2
Errors: 1
Duration: 10h 41m 55s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221631430
### CI RESULTS
```
Tests Run: 84
Skipped: 0
Failed: 0
Errors: 1
Duration: 5h 26m 59s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1559#issuecomment-221633251
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 1
Errors: 1
Duration: 10h 41m 20s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1553#issuecomment-221634279
### CI RESULTS
```
Tests Run: 83
Skipped: 0
Failed: 1
Errors: 2
Duration: 8h 32m 15s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221631597
This error is unrelated to this PR. This is ready to merge...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221637516
@dsclose can you kick it off again. It is just a timeout which caused it
to fail, so kicking it off again could fix it. `No output has been received in
the last
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/846#issuecomment-221757982
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 33m 42s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/846#issuecomment-221758314
This is a cleaner run because this issue is a known issue. I think this
one is ready now...
---
If your project is set up for it, you can reply to this email and
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1553#issuecomment-221758690
### CI RESULTS
```
Tests Run: 83
Skipped: 0
Failed: 0
Errors: 2
Duration: 8h 39m 49s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1553#issuecomment-221758855
This is a known issue, so this one should be ready now. Thx...
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221759081
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 37m 16s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221759242
This is ready to merge now. The issue we are seeing is a known issue not
related to this PR.
---
If your project is set up for it, you can reply to this email and
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1450#issuecomment-221762730
CI is clean and everything is green. I need some code review on this one.
Thanks...
---
If your project is set up for it, you can reply to this email and have
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1471#issuecomment-221762946
Can I get some code review on this one? 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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1491#issuecomment-221763412
This one needs 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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1499#issuecomment-221763535
Bump...
@alexandrelimassantana this one has merge conflicts. Can you rebase and fix
the conflicts? Thanks...
---
If your project is set up for it, you can
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/669#issuecomment-221763988
This one needs some work. We have merge conflicts and we need another code
review.
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1251#issuecomment-221764160
Need code review on this one. Thx...
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/669#issuecomment-221861017
Can I get one more review on this one? I think we are basically set
otherwise. Thx...
---
If your project is set up for it, you can reply to this email and have
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/866#issuecomment-221862006
@bvbharatk can you close and reopen this PR to kick off jenkins again to
see if we can get this green?
I will rerun CI on it to make sure we are still good
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221862724
@dsclose sorry to do this to you. Can you close and reopen again to kick
off the jobs again? Thanks...
---
If your project is set up for it, you can reply to this
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1511#issuecomment-221863141
this one just does not want to turn green. :( wanna give it another shot
@rhtyd?
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1561#issuecomment-221863597
@sanju1010 thanks for posting the test results with this change. This one
should be good now. Thanks everyone...
---
If your project is set up for it, you can
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1437#issuecomment-221871305
I think the only thing we are missing with this one is verification that
the change works and does not break anything. @pedro-martins is there anything
you can
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1565#issuecomment-221872868
@ProjectMoon please close and reopen to kick off travis again. Thanks...
---
If your project is set up for it, you can reply to this email and have your
reply
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1471#issuecomment-221873120
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 8h 23m 38s
301 - 400 of 1169 matches
Mail list logo