-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix for an issue introduced in this commit: 336df84f1787de962a67d0a34… #2141
Fix for an issue introduced in this commit: 336df84f1787de962a67d0a34… #2141
Conversation
All managed-storage tests pass successfully: TestAddRemoveHosts.py: test_add_remove_host_with_solidfire_plugin_1 (TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: test_add_remove_host_with_solidfire_plugin_1 | Status : SUCCESS === Ran 4 tests in 3036.238s OK TestManagedSystemVMs.py: test_01_create_system_vms_on_managed_storage (TestManagedSystemVMs.TestManagedSystemVMs) ... === TestName: test_01_create_system_vms_on_managed_storage | Status : SUCCESS === Ran 2 tests in 1864.418s OK TestVMMigrationWithStorage.py: test_01_storage_migrate_root_and_data_disks (TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName: test_01_storage_migrate_root_and_data_disks | Status : SUCCESS === Ran 5 tests in 2346.416s OK TestVMSnapshots.py: test_01_take_VM_snapshot (TestVMSnapshots.TestVMSnapshots) ... === TestName: test_01_take_VM_snapshot | Status : SUCCESS === Ran 2 tests in 869.646s OK TestVolumes.py (no resign): test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: test_00_check_template_cache | Status : SUCCESS === Ran 11 tests in 1326.838s OK TestVolumes.py (resign): test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: test_00_check_template_cache | Status : SUCCESS === Ran 11 tests in 1547.541s OK TestSnapshots.py (environment issued caused a failure in two tests, which I ran again and they passed): test_01_create_volume_snapshot_using_sf_snapshot (TestSnapshots.TestSnapshots) ... === TestName: test_01_create_volume_snapshot_using_sf_snapshot | Status : SUCCESS === Re-run of the two tests that failed for environmental reasons (now passed): test_03_create_volume_snapshot_using_sf_volume_and_sf_snapshot (TestSnapshots.TestSnapshots) ... === TestName: test_03_create_volume_snapshot_using_sf_volume_and_sf_snapshot | Status : SUCCESS === Ran 2 tests in 2952.535s OK |
Can't really say much about the code, but @karuturi , could you take a look? As this is a -1 by Mike now. |
@blueorangutan package |
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-764 |
@blueorangutan test |
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
Trillian test result (tid-1152)
|
need another LGTM. @borisstoyanov @rhtyd @anshul-gangwar Can you review? |
@mike-tutkowski |
code LGTM. |
Should n't this be targeted against master? /cc @mike-tutkowski |
Hi @karuturi - Yeah, I had noted those failed for environmental reasons. I had re-run them successfully. Here is a copy/paste from above: Re-run of the two tests that failed for environmental reasons (now passed): test_03_create_volume_snapshot_using_sf_volume_and_sf_snapshot (TestSnapshots.TestSnapshots) ... === TestName: test_03_create_volume_snapshot_using_sf_volume_and_sf_snapshot | Status : SUCCESS === |
@mike-tutkowski Can you rebase this to master? |
I created the following new PR against master (and referenced the old one in it): #2147
From: Rajani Karuturi <notifications@github.com>
Reply-To: apache/cloudstack <reply@reply.github.com>
Date: Wednesday, June 14, 2017 at 1:00 AM
To: apache/cloudstack <cloudstack@noreply.github.com>
Cc: "Tutkowski, Mike" <Mike.Tutkowski@netapp.com>, Mention <mention@noreply.github.com>
Subject: Re: [apache/cloudstack] Fix for an issue introduced in this commit: 336df84… (#2141)
@mike-tutkowski<https://github.com/mike-tutkowski> Can you rebase this to master?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2141 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AC4SHwQFSYcB6PbU195B9a6jRoqaiKW9ks5sD4UmgaJpZM4N2Mmc>.
|
…551f9027303040e and also a fix for VMware + managed storage