I'll take a look at this. It's been some moths but afaik I added that extension because previously that code wasn't working. I'll take a look in detail and let you guys know On Thu, Dec 1, 2016 at 11:34 mike-tutkowski <g...@git.apache.org> wrote:
> Github user mike-tutkowski commented on the issue: > > https://github.com/apache/cloudstack/pull/1800 > > Thanks, @rhtyd I've been trying to get test_snapshots.py to fail in my > environment, but haven't been able to. > > I'll let @syed answer your question, Rohit, since that is a change he > put in. > > If we do need/want to change that, let's do it in #1749 as I have that > PR open to fix a couple issues I found with regards to snapshots on managed > storage (specifically with regards to storing them on secondary storage, if > that option is passed into the command). > > > --- > 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 enabled but not working, please > contact infrastructure at infrastruct...@apache.org or file a JIRA ticket > with INFRA. > --- >
