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 ho
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 a
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 th
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 agai
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 your
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 is
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 f
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 appear
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 p
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 feat
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
```
**Associ
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 appea
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 `addHost`
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: (a
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
Yo
22 matches
Mail list logo