Congratulations to Alexandre Mattioli for Joining the CloudStack PMC

2024-12-04 Thread Mike Tutkowski
Hi everyone, Please join me in congratulating Alexandre Mattioli for becoming a member of the CloudStack PMC. Congratulations, Alex! :) Thanks, Mike https://www.linkedin.com/in/mike-tutkowski-5880286/

Re: Congratulations to Gabriel - CloudStack PMC Chair

2021-03-17 Thread Mike Tutkowski
Congratulations, Gabriel! On Wed, Mar 17, 2021 at 3:12 PM Paul Angus wrote: > Hi All CloudStack enthusiasts! > > > > Please join me in congratulating Gabriel for becoming the next CloudStack > PMC Chair. > > Congratulations Gabriel, very well deserved! > > > > I would also like to thank Sven for

[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1994 OK, this LGTM (as long as Travis shows green before we merge). 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

[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1994 I should have a chance to look through this later tonight. In the meanwhile, perhaps we can re-push the code to kick off Travis again (since its most recent run has a failure). Thanks

[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-10 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1994 I have run the following use cases successfully: PS = Primary Storage ST = Storage Tag CO = Compute Offering DO = Disk Offering V = Volume Create PS_1

[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-10 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1994 I should be able to do so over the weekend. On Mar 10, 2017, at 11:26 AM, serg38 mailto:notificati...@github.com>> wrote: @mike-tutkowski<https://github

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-03-02 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1813 This code LGTM. I reviewed the code, but don't have the ability to run tests at the moment as I'm in the process of testing RC1 for 4.10. We have updates to two unit tes

[GitHub] cloudstack issue #1962: CLOUDSTACK-9798 - Resizing root disk fails when usin...

2017-02-27 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1962 @syed Yes, I can include you as a reviewer for the relevant PR. --- 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 #1962: CLOUDSTACK-9798 - Resizing root disk fails when usin...

2017-02-23 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1962 @syed Hey - I was also working on a feature like this, but related to both XenServer and VMware (adding in VMware-specific code on the resource side). Do you need these changes for 4.10

[GitHub] cloudstack pull request #1963: CLOUDSTACK-9799 - Unable to create volume fro...

2017-02-23 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1963#discussion_r102793622 --- Diff: engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategy.java --- @@ -711,6 +711,7

[GitHub] cloudstack issue #1963: CLOUDSTACK-9799 - Unable to create volume from snaps...

2017-02-23 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1963 Yes, but the PR is scheduled for 4.11. On Feb 23, 2017, at 10:58 AM, Syed Mushtaq Ahmed mailto:notificati...@github.com>> wrote: @mike-tutkowski<https://g

[GitHub] cloudstack issue #1963: CLOUDSTACK-9799 - Unable to create volume from snaps...

2017-02-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1963 @syed Are you looking to get this into 4.10? If you are not, I have this corrected in a PR scheduled for 4.11. If you are, then I think you're missing a file (SolidFireUtil). -

[GitHub] cloudstack issue #1962: CLOUDSTACK-9798 - Resizing root disk fails when usin...

2017-02-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1962 @syed This LGTM. I wonder, though, if we can only check in the necessary parts (i.e. not the massive rearrangements to the imports)? --- If your project is set up for it, you can reply

[GitHub] cloudstack issue #1952: CLOUDSTACK-9790: fix NPE in case of Basic zone.

2017-02-17 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1952 LGTM I have also incorporated this code and created a Basic Zone with it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request #1579: CLOUDSTACK-9403 : Support for shared networks...

2017-02-16 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1579#discussion_r101596817 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -2909,51 +2911,21 @@ public Vlan doInTransaction(final

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2017-01-11 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1735#discussion_r94659104 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -526,16 +529,18 @@ public static DataStoreRole getDataStoreRole(Snapshot

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2017-01-11 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1735#discussion_r94661240 --- Diff: api/src/org/apache/cloudstack/api/command/user/snapshot/ListSnapshotsCmd.java --- @@ -115,8 +115,10 @@ public void execute

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2017-01-11 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1735#discussion_r94660627 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -493,7 +494,9 @@ public SnapshotResponse createSnapshotResponse(Snapshot

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2017-01-11 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1735#discussion_r94660806 --- Diff: engine/storage/volume/src/org/apache/cloudstack/storage/datastore/manager/PrimaryDataStoreProviderManagerImpl.java --- @@ -56,7 +55,7

[GitHub] cloudstack issue #1900: CLOUDSTACK-8862: Introduced new state attaching for ...

2017-01-10 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1900 I don't have the resources in my lab available to perform tests on this at this time, but the code itself LGTM. Thanks for working on this JIRA ticket! --- If your proje

[GitHub] cloudstack issue #1735: CLOUDSTACK-9570: Bug in listSnapshots for snapshots ...

2017-01-10 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1735 @serg38 @anshul1886 @rhtyd Maybe someone could take a look at my comments before we check this in and see if they think any of those should be implemented here. Thanks --- If your

[GitHub] cloudstack issue #1888: CLOUDSTACK-9710: Switch to JRE1.8

2017-01-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1888 I walked through the diffs and they LGTM. I did not actually run the code, however, because I'm in the middle of another CloudStack project and I don't have the resources av

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2017-01-04 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1735#discussion_r94657561 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -526,16 +529,18 @@ public static DataStoreRole getDataStoreRole(Snapshot

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-01-04 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1847 Code = LGTM --- 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

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-01-04 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1847 To clarify this line: "If that comes back null, then we apparently have removed primary storage, which can only be done if your snapshots don't reside on it. If

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-01-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1847 Let me provide a bit of background on this and then we can decide which way we want to correct this side effect. Here is the PR that went in a while ago that enabled CloudStack

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-01-02 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1847 Can you tell me which line was throwing a NullPointerException in the code that you fixed? Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-27 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1854 @rhtyd I think we can deal with this in another PR, as you suggest. @syed Does any of that code look like something that might have been changed for archiving managed snapshots

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 The semi-unusual part about this PR, @rhtyd, is that is contains code jointly developed by me and @syed. He has reviewed my contributions and I have reviewed his. We

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 Hi @rhtyd We have one LTGM (from @syed) and I have run and posted all regression tests for managed storage. If we could have one more person take a look, that would be great. I don&#

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @syed Would you be able to review this code when you have a little time? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-06 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd It took a few force pushes, but it worked. :) --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd Are we having trouble with Travis? I've amended my SHA a few times and get different Travis failures. Thanks! --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1748: CLOUDSTACK-9620: KVM enhancements for managed storag...

2016-12-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 I've updated the commit summary, @rhtyd. --- 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 issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 I've updated the commit summary, @rhtyd --- 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

[GitHub] cloudstack issue #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

2016-12-01 Thread mike-tutkowski
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 t

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd @jburwell I recently ran all of the managed-storage-related integration tests successfully (referenced above). I think we're in a good position to get this into 4.1

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @rhtyd I added that JIRA ticket and referenced it in the PR description. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd I added that JIRA ticket and referenced it in the PR description. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 TestAddRemoveHosts (1, 2, and 4 were successful and 3 failed for an environmental reason): [results.txt](https://github.com/apache/cloudstack/files/613869/results.txt

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 TestVolumes (resign = true): [results.txt](https://github.com/apache/cloudstack/files/613863/results.txt) TestVolumes (resign = false): [results.txt](https://github.com

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-24 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1754 Hi Rohit, Today is a holiday in the US and many people are also off work tomorrow. I have a bunch of holiday-related activities going on today and tomorrow, but should

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-23 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1710 This code LGTM. --- 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 #1749: Updates for SAN-assisted snapshots

2016-11-08 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @syed I think any failures/errors listed for those regression tests might be erroneous. I double checked the canHandle method of StorageSystemSnapshotStrategy and our snapshot

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @jburwell @karuturi Although I won't be able to run regression tests on #1749 anytime soon (due to being out of the country for a while), @syed has volunteered to do so. --- If

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @jburwell @karuturi I went ahead and opened #1749. That one just has the fixes for #1600. That should go into 4.10 because it fixes pretty important issues and is low risk. --- If your

[GitHub] cloudstack pull request #1749: Updates for SAN-assisted snapshots

2016-11-07 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1749 Updates for SAN-assisted snapshots This PR is to address a few issues in #1600 (which was recently merged to master for 4.10). In

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @karuturi @jburwell Oh, and just to point out, the reason I embedded those changes in this PR at the time was because I couldn't finish running my regression tests on this PR until

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @karuturi @jburwell Hi I was recently working on some enhancements to 4.5 for a prospect. Once I completed that work in their custom branch, I wanted to forward port the

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 All tests for managed storage have been run and passed: [results.txt](https://github.com/apache/cloudstack/files/576566/results.txt) [results.txt](https://github.com/apache

[GitHub] cloudstack pull request #1748: KVM enhancements for managed storage

2016-11-07 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1748 KVM enhancements for managed storage This PR contains changes mainly related to enhancing managed storage on KVM. I recently added these features to a custom build of 4.5 for a prospect

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-11-01 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @rhtyd I don't think either of those two errors are related to this PR (probably environment issues). I think we are good to go here. What do you think? --- If your project is s

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-11-01 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 Thanks, @rhtyd! --- 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 #1732: Switched to the official SolidFire SDK for Java

2016-10-31 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 @rhtyd Do you know what the results were of those regression tests you kicked off for this PR a few days ago? Just checking in to see if we are able to merge this. Thanks! --- If your

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-30 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed Just an FYI that this branch had two conflicts with master (after recent changes to master). I went ahead and fixed the conflicts and re-pushed the commit. --- If your project is

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-28 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 @rhtyd Do you know what the results were of those regression tests you kicked off for this PR a few days ago? Thanks! --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @karuturi Anything else @syed needs to do for this one? 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

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVolumes.py (no resigning) test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: test_00_check_template_cache | Status : SUCCESS === ok Attach a

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVolumes (resigning) test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: test_00_check_template_cache | Status : SUCCESS === ok Attach a volume to

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVMSnapshots.py test_01_take_VM_snapshot (TestVMSnapshots.TestVMSnapshots) ... === TestName: test_01_take_VM_snapshot | Status : SUCCESS === ok

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVMMigrationWithStorage test_01_storage_migrate_root_and_data_disks (TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestSnapshots.py: The failure in test 04 makes sense as that test case is actually dependent upon #1600, which has not yet been merged in

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestAddRemoveHosts.py test_add_remove_host_with_solidfire_plugin_1 (TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: test_add_remove_host_with_solidfire_plugin_1

[GitHub] cloudstack pull request #1732: Switched to the official SolidFire SDK for Ja...

2016-10-25 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1732 Switched to the official SolidFire SDK for Java SolidFire has recently released an official SDK for Java. I have converted the SolidFire storage plug-ins to make use of this new

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @rhtyd @karuturi I think we are good to go on this PR now. Do you agree? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-24 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Any way to easily just re-run test_05_stop_ssvm and test_07_reboot_ssvm? I'm not sure how this PR would cause such failures. The only changed Java code does the foll

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-23 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 2016-10-23 19:32:49,315 - CRITICAL - EXCEPTION: test_05_stop_ssvm: ['Traceback (most recent call last):\n', ' File "/usr/lib64/python2.7/unittest/case.py&

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-23 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 == FAIL: Create a redundant VPC with two networks with two VMs in each network

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-23 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Tests out-of-band management change password feature ... === TestName: test_oobm_zchange_password | Status : FAILED === FAIL That might be a problem with the test. --- If

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Looks like re-pushing the commit did the trick, @rhtyd. I think we are good to go here. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-21 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @rhtyd I changed the SHA and re-pushed the commit. Hopefully that will clear up Travis. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 Thanks for the clarification, @jburwell. I had the same question for #1642, but your answer here works for both PRs. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Just following up on this one. Is there any pending work item that remains or are we able to merge? Thanks! --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 Just following up on this one. Is there any pending work item that remains or are we able to merge? Thanks! --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed Thanks! I had a similar issue with #1642 and fixed it the same way. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell Excellent...closing and re-opening the ticket fixed the CI issue. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Closing and re-opening the PR fixed the Travis issue. --- 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

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 I still got a weird Travis failure, so I decided to close and re-open this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request #1642: CLOUDSTACK-9504: System VMs on Managed Storag...

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski closed the pull request at: https://github.com/apache/cloudstack/pull/1642 --- 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 #1642: CLOUDSTACK-9504: System VMs on Managed Storag...

2016-10-03 Thread mike-tutkowski
GitHub user mike-tutkowski reopened a pull request: https://github.com/apache/cloudstack/pull/1642 CLOUDSTACK-9504: System VMs on Managed Storage This PR makes it easier to spin up system VMs on managed storage. Managed storage is when you have a dedicated volume on a SAN

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Travis, which was failing for a similar reason for #1600, passed, so I will (once again) change the commit SHA and re-push. In fact, while I'm at it, I'll rebase it against

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell I think it's Travis. I've changed the commit SHA twice now and it failed both times plus it fails with #1600, as well (even though I pushed that one multiple times,

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-03 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Travis issue? --- 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 #1600: Support Backup of Snapshots for Managed Storage

2016-10-02 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 It looks like we're having an issue with Travis? --- 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 doe

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-02 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 I have run all of the relevant managed-storage tests and all passed (details below). I did encounter one issue: When the fourth test of TestSnapshots ran, it failed

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @jburwell Yes, the managed-storage tests should be re-executed. I can do this over the weekend as my CloudStack systems in the lab here are being fully utilized for the time being with

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-29 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 Sounds good, @syed Since what you added is the same as what's currently in master, it looks like your list of changed files for this PR is down to 35 (only one of which is a Python

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-28 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed It actually looks like the rebase lead to the integration tests being put back in their pre-official SolidFire SDK for Python state. I recommend you just remove all

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-28 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed You can actually just remove the integration tests from this PR as they were added (along with updates to use the new SolidFire SDK for Python) in #1689. --- If your project is

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed @jburwell I should be able to run the tests toward the end of the week or during the weekend. At the moment, my resources in the lab are devoted to other CloudStack dev activities

[GitHub] cloudstack issue #1689: Switched to the official SolidFire SDK for Python

2016-09-27 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1689 @jburwell I just added that to requirements.txt. I also put a link to solidfire-sdk-python 1.1.0.92 in the description of this PR. --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1689: Switched to the official SolidFire SDK for Python

2016-09-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1689 TestAddRemoveHosts.py: test_add_remove_host_with_solidfire_plugin_1 (TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: test_add_remove_host_with_solidfire_plugin_1

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed @karuturi @jburwell Just wondering how we're doing on this one. Is there anything else that Syed needs to do here? Thanks! --- If your project is set up for it, you can rep

[GitHub] cloudstack pull request #1689: Switched to the official SolidFire SDK for Py...

2016-09-23 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1689 Switched to the official SolidFire SDK for Python SolidFire has recently released an official SDK for Python. I have converted all of the SolidFire integration tests over to

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-09-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Thanks for running that, @serg38! --- 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

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-09-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell Let me run those tests next week (probably Monday) as I'm just about ready to head out on a flight to a customer site for an upcoming meeting and won't be back

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-09-21 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Thank you, @karuturi! Does anyone know who can run the VMware test that @jburwell asked for? I can't due to this issue: https://issues.apache.org/jira/browse/CLOUDSTACK

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-09-20 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi That config is actually one of the ones John is asking for. :) Smoke test suite + system VM integration test suites (SSVM, CPVM, and VR) against XenServer using

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-20 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi Do you happen to have any resources we can use to fulfill John's test requests? Thanks --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-20 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 How are we doing on this one? Do my comments above make sense to everyone? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

  1   2   3   4   5   6   7   8   9   10   >