Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1503
---
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/1503#issuecomment-220038585
Im not sure. I need to check the mailing list to see if there are other
PRs with similar errors. It seems to have the same errors for the past 3 runs,
so it seems p
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-220019839
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 1
Errors: 1
Duration: 11h 12m 43s
```
**Summary of the
Github user olivierlemasle commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219980996
Ok Still the same error.
I still don't think it's related to my change. The same error occured
before, for example for https://github.com/apache/
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219748341
### CI RESULTS
```
Tests Run: 126
Skipped: 0
Failed: 2
Errors: 2
Duration: 13h 11m 51s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219708414
retesting the failure 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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219704785
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 1
Errors: 1
Duration: 10h 57m 29s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219126476
I am running my CI against it again to make sure this does not pop again.
If it doesn't then I will merge it right away as Jenkins and Travis are green.
Thx...
--
Github user olivierlemasle commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219121751
In the three tracebacks, the error is located in
https://github.com/apache/cloudstack/blob/821b2da1267c486bb1f5c9e90d3d1969b6fe947f/tools/marvin/marvin/lib/ba
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219083712
I have never seen these test failures before, so I believe they are somehow
related to this change. @olivierlemasle can you review the CI results and post
your feedb
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-219082695
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 28m 52s
```
**Summary of the p
Github user olivierlemasle commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-218893171
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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-218745932
@DaanHoogland & @olivierlemasle thanks for the heads up guys. I will get
CI run against this today...
---
If your project is set up for it, you can reply to this em
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-218716828
LGTM, always better to have real json handling as opposed to string
parsing. @swill, can you schedule the integration suite on this?
---
If your project is se
Github user olivierlemasle commented on the pull request:
https://github.com/apache/cloudstack/pull/1503#issuecomment-218715075
Any chance reviewing this pull request?
The error is quite critical because it causes HTTP 530 errors in API
requests.
Thanks!
CC @swill
Github user olivierlemasle commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1503#discussion_r61726860
--- Diff: server/src/com/cloud/api/ApiServer.java ---
@@ -264,10 +266,11 @@ public void handleAsyncJobPublishEvent(String
subject, String senderAd
Github user alexandrelimassantana commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1503#discussion_r61723859
--- Diff: server/src/com/cloud/api/ApiServer.java ---
@@ -264,10 +266,11 @@ public void handleAsyncJobPublishEvent(String
subject, String s
GitHub user olivierlemasle opened a pull request:
https://github.com/apache/cloudstack/pull/1503
CLOUDSTACK-9358: StringIndexOutOfBoundsException on events
Fixes JSON deserialization of `cmdInfo` (current process fails with
`StringIndexOutOfBoundsException` when `cmdEventType` is th
18 matches
Mail list logo
