[GitHub] cloudstack issue #2008: [CLOUDSTACK-9840] Fix datetime format of snapshots e...

2017-04-05 Thread olivierlemasle
Github user olivierlemasle commented on the issue: https://github.com/apache/cloudstack/pull/2008 The Travis tests have finally succeeded. @borisstoyanov Any chance merging this pull request? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request #2008: [CLOUDSTACK-9840] Fix datetime format of snap...

2017-04-03 Thread olivierlemasle
GitHub user olivierlemasle reopened a pull request: https://github.com/apache/cloudstack/pull/2008 [CLOUDSTACK-9840] Fix datetime format of snapshots events Include the timezone in datetime format of snapshot events, to be consistent with every other events. "eventDateTime

[GitHub] cloudstack pull request #2008: [CLOUDSTACK-9840] Fix datetime format of snap...

2017-04-03 Thread olivierlemasle
Github user olivierlemasle closed the pull request at: https://github.com/apache/cloudstack/pull/2008 --- 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

[GitHub] cloudstack issue #2008: [CLOUDSTACK-9840] Fix datetime format of snapshots e...

2017-03-20 Thread olivierlemasle
Github user olivierlemasle commented on the issue: https://github.com/apache/cloudstack/pull/2008 @borisstoyanov Thanks, I've renamed the pull request. Do you want me to rename to commit as well? Additionally, the travis checks hit a timeout; I've closed and op

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-17 Thread olivierlemasle
Github user olivierlemasle closed the pull request at: https://github.com/apache/cloudstack/pull/2008 --- 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

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-17 Thread olivierlemasle
GitHub user olivierlemasle reopened a pull request: https://github.com/apache/cloudstack/pull/2008 Fix datetime format of snapshots events Include the timezone in datetime format of snapshot events, to be consistent with every other events. "eventDateTime" wa

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-16 Thread olivierlemasle
GitHub user olivierlemasle reopened a pull request: https://github.com/apache/cloudstack/pull/2008 Fix datetime format of snapshots events Include the timezone in datetime format of snapshot events, to be consistent with every other events. "eventDateTime" wa

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-16 Thread olivierlemasle
Github user olivierlemasle closed the pull request at: https://github.com/apache/cloudstack/pull/2008 --- 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

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-15 Thread olivierlemasle
GitHub user olivierlemasle opened a pull request: https://github.com/apache/cloudstack/pull/2008 Fix datetime format of snapshots events Include the timezone in datetime format of snapshot events, to be consistent with every other events. "eventDateTime" wa

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-18 Thread olivierlemasle
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.

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-13 Thread olivierlemasle
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

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-12 Thread olivierlemasle
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

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-12 Thread olivierlemasle
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

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-02 Thread olivierlemasle
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

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-04-20 Thread olivierlemasle
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