[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2017-02-07 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1644 A separate PR for master is not required. #1697 will be forward merged. --- 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] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2017-02-03 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1644 @karuturi Just checking in on this, at one point it looked like we were going to possibly roll this out as a maintenance release of 4.8, and there is another PR open with the same

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-11-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1644 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-202 --- 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] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-11-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1644 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-11-20 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1644 @blueorangutan package --- 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

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-11-03 Thread ozhanrk
Github user ozhanrk commented on the issue: https://github.com/apache/cloudstack/pull/1644 This is a really good enhancement for Ceph users, thanks to all contributors 👍 :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-09-29 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1644 @jburwell @kiwiflyer @bvbharatk I've gotten this to a place where I'm feeling more comfortable with how it works, would you all like me to close this PR, reopen it against 4.9 then?

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-09-18 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1644 @jburwell We found an issue on Friday related to a use case related to snapshot to template. We're working on it. More soon. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-09-18 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1644 @kiwiflyer could you please post the detailed test results included the environment specification? @nathanejohnson have you had a chance to investigate the failures from the

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-31 Thread bvbharatk
Github user bvbharatk commented on the issue: https://github.com/apache/cloudstack/pull/1644 ### ACS CI BVT Run **Sumarry:** Build Number 86 Hypervisor xenserver NetworkType Advanced Passed=99 Failed=4 Skipped=3 _Link to logs Folder (search

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-25 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1644 Pure RBD based revert is not supported today (plus it's really slow). The way we do it is we snap to volume (clone), shut the vm down, then detach the old disk and reattach the new disk to

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-25 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1644 Seems logical to me. But we can't revert a RBD snapshot yet, can we? Sorry, it's getting a bit rusty for me as I haven't looked at that code for a while. Still, sane feature.

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-18 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1644 @wido Nice feature for Ceph users. --- 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

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-18 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1644 LGTM, tested in our lab. --- 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

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-17 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1644 Here is a snippet of the logs with storage.snapshot.backup.rightafter set to false. ```2016-08-17 15:34:38,530 DEBUG [c.c.a.ApiServlet] (catalina-exec-22:ctx-b9fb719f)

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2016-08-17 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1644 `2016-08-17 15:34:38,530 DEBUG [c.c.a.ApiServlet] (catalina-exec-22:ctx-b9fb719f) (logid:389a008f) ===START=== 10.16.0.21 -- GET