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 rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220893971
@swill I ran tests on latest master using packages:
http://packages.shapeblue.com/cloudstack/custom/master-latest/
I was able to deploy a zone with two KVM
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220766648
@swill if it's failing for addHost, can you send the mgmt server log,
marvin log and the agent log, and details of the environment (as much as
possible). I can have
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/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/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 rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220675936
@swill thank you for your time in testing 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
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1549
---
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 the feature
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/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-220627907
### CI RESULTS
```
Tests Run: 83
Skipped: 0
Failed: 0
Errors: 2
Duration: 8h 27m 27s
```
**Summary of the
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220545220
Thanks @swill
---
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/1549#issuecomment-220511825
I am going to run the test again because it was an intermittent issue, so a
couple tests is not a bad idea.
---
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/1549#issuecomment-220511743
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 8h 51m 55s
```
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1549
CLOUDSTACK-9348: NioConnection improvements
Reopened PR with squashed changes for a re-review and testing after
https://github.com/apache/cloudstack/pull/1493 and sub-sequent PRs got reverted
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1549
---
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 the feature is
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220380081
@swill yes, this PR includes all the reverted changes and the addHost fix.
---
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-220362988
I will get this run through CI. Just to be clear, this PR includes all of
the changes that were included in 1543, 1538, 1534, 1493 as well as a fix for
the
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1549
CLOUDSTACK-9348: NioConnection improvements
Reopened PR with squashed changes for a re-review and testing after
https://github.com/apache/cloudstack/pull/1493 and sub-sequent PRs got reverted
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1549
---
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 the feature is
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1549#issuecomment-220289585
@swill I was able to reproduce the addHost issue and fixed it. I was able
to deploy against a KVM env with two clusters with Ubuntu and CentOS based
hosts in each:
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1549
CLOUDSTACK-9348: NioConnection improvements
Reopened PR with squashed changes for a re-review and testing after
https://github.com/apache/cloudstack/pull/1493 and sub-sequent PRs got reverted
22 matches
Mail list logo