cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nvazquez <...@git.apache.org>
Subject [GitHub] cloudstack issue #871: CLOUDSTACK-8893: Fixing script as per the latest func...
Date Wed, 21 Sep 2016 18:15:04 GMT
Github user nvazquez commented on the issue:

    https://github.com/apache/cloudstack/pull/871
  
    Hi @sanju1010,
    
    @serg38 and I are working on this PR #1677 which solves vm snapshot creation issue. After
running `test_vm_snapshots.py` we're getting exception on `test_01_test_vm_volume_snapshot`
as it was expected before your changes in this PR.
    
    We're getting: `'CloudstackAPIException: Execute cmd: createsnapshot failed, due to: errorCode:
431, errorText:Volume snapshot is not allowed, please detach it from VM with VM Snapshots\n']`
    
    This is consistent with `VolumeApiServiceImpl.allocSnapshot` method, in which it is:
    ```
            if (volume.getInstanceId() != null) {
                // Check that Vm to which this volume is attached does not have VM Snapshots
                if (_vmSnapshotDao.findByVm(volume.getInstanceId()).size() > 0) {
                    throw new InvalidParameterValueException("Volume snapshot is not allowed,
please detach it from VM with VM Snapshots");
                }
            }
    ```
    
    Do you want us to adapt `test_vm_snapshots.py` in mentioned PR to support this or would
you like to address it yourself?
    
    Thanks


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message