[GitHub] [cloudstack] blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR

blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-520116244 Trillian test result (tid-298) Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33359 seconds

[GitHub] [cloudstack] blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion

blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion URL: https://github.com/apache/cloudstack/pull/3540#issuecomment-520090178 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-243 This is an automated messa

[GitHub] [cloudstack] blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion

blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion URL: https://github.com/apache/cloudstack/pull/3540#issuecomment-520078220 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-242 This is an automated messa

[GitHub] [cloudstack] blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR

blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-520067801 Trillian test result (tid-296) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32562 seconds

[GitHub] [cloudstack] kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange

kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange URL: https://github.com/apache/cloudstack/pull/3522#discussion_r312646913 ## File path: server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java ## @@ -3958,6 +3961,11

[GitHub] [cloudstack] kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange

kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange URL: https://github.com/apache/cloudstack/pull/3522#discussion_r312646304 ## File path: api/src/org/apache/cloudstack/api/command/admin/vlan/UpdateVlanIpRangeCmd.java ## @@ -0,0 +1,137

[GitHub] [cloudstack] blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion

blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion URL: https://github.com/apache/cloudstack/pull/3540#issuecomment-520057053 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-241 This is an automated messa

[GitHub] [cloudstack] blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR

blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-520014689 Trillian test result (tid-295) Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 23065 seconds

[GitHub] [cloudstack] blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion

blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion URL: https://github.com/apache/cloudstack/pull/3540#issuecomment-519910156 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-240 This is an automated messa

[GitHub] [cloudstack] blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion

blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion URL: https://github.com/apache/cloudstack/pull/3540#issuecomment-519892199 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-239 This is an automated messa

[GitHub] [cloudstack] blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR

blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-519889374 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] [cloudstack] rhtyd commented on issue #3547: [WIP] 4.13/master stabilisation PR

rhtyd commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-519889206 @blueorangutan test This is an automated message from the Apache Git Service. To res

Jenkins build is back to normal : cloudstack-apidocs-master #3221

See

[GitHub] [cloudstack] blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR

blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-519882760 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-238 This is an automated me

[GitHub] [cloudstack] DaanHoogland commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception

DaanHoogland commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception URL: https://github.com/apache/cloudstack/pull/3546#issuecomment-519882278 ok, fair point but than at this point it can also not deal with CloudRuntimeException expecting not snapshot being created in the first

[GitHub] [cloudstack] blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR

blueorangutan commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-519877291 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] rhtyd commented on issue #3547: [WIP] 4.13/master stabilisation PR

rhtyd commented on issue #3547: [WIP] 4.13/master stabilisation PR URL: https://github.com/apache/cloudstack/pull/3547#issuecomment-519877071 With the NPE fixes, it should fix the following: test_02_list_snapshots_with_removed_data_store | Error | 1.18 | test_snapshots.py test_01_volu

[GitHub] [cloudstack] rhtyd merged pull request #3271: VMware: Allow configuring appliances on the VM instance wizard when OVF properties are available

rhtyd merged pull request #3271: VMware: Allow configuring appliances on the VM instance wizard when OVF properties are available URL: https://github.com/apache/cloudstack/pull/3271 This is an automated message from the Apac

[cloudstack] branch master updated: vmware: allow configuring appliances on the VM instance wizard when OVF properties are available (#3271)

This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 3c2af55 vmware: allow configuring appliances

[GitHub] [cloudstack] onitake commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception

onitake commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception URL: https://github.com/apache/cloudstack/pull/3546#issuecomment-519870665 There are other sources for exceptions, not just the part that did `return null`, so I would assume that there is already some sort of error

[GitHub] [cloudstack] blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion

blueorangutan commented on issue #3540: [WIP DO NOT MERGE] VM ingestion URL: https://github.com/apache/cloudstack/pull/3540#issuecomment-519867121 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-237 This is an automated messa

[GitHub] [cloudstack] DaanHoogland closed pull request #3522: New API endpoint: UpdateVlanIpRange

DaanHoogland closed pull request #3522: New API endpoint: UpdateVlanIpRange URL: https://github.com/apache/cloudstack/pull/3522 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [cloudstack] kioie opened a new pull request #3522: New API endpoint: UpdateVlanIpRange

kioie opened a new pull request #3522: New API endpoint: UpdateVlanIpRange URL: https://github.com/apache/cloudstack/pull/3522 ## Description Following #3111 A new endpoint that will allow the shrinking and extending of public IP range ## Types of

[GitHub] [cloudstack] kioie opened a new pull request #3532: Add new "Add volume" button, to add volume to VM instance directly

kioie opened a new pull request #3532: Add new "Add volume" button, to add volume to VM instance directly URL: https://github.com/apache/cloudstack/pull/3532 ## Description Volume button to allow user to quickly add volume to a VM instance. Also slight edit of createVolumeCmd to a

[GitHub] [cloudstack] DaanHoogland closed pull request #3532: Add new "Add volume" button, to add volume to VM instance directly

DaanHoogland closed pull request #3532: Add new "Add volume" button, to add volume to VM instance directly URL: https://github.com/apache/cloudstack/pull/3532 This is an automated message from the Apache Git Service. To resp

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly

DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly URL: https://github.com/apache/cloudstack/pull/3532#discussion_r312402426 ## File path: server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly

DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly URL: https://github.com/apache/cloudstack/pull/3532#discussion_r312402235 ## File path: server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly

DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly URL: https://github.com/apache/cloudstack/pull/3532#discussion_r312398181 ## File path: server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly

DaanHoogland commented on a change in pull request #3532: Add new "Add volume" button, to add volume to VM instance directly URL: https://github.com/apache/cloudstack/pull/3532#discussion_r312402532 ## File path: server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java

[GitHub] [cloudstack] DaanHoogland commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception

DaanHoogland commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception URL: https://github.com/apache/cloudstack/pull/3546#issuecomment-519848656 @onitake can you add setting the snapshot state before throwing the exception?

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3546: [FIX] Rethrow takeVMSnapshot() exception

DaanHoogland commented on a change in pull request #3546: [FIX] Rethrow takeVMSnapshot() exception URL: https://github.com/apache/cloudstack/pull/3546#discussion_r312396732 ## File path: server/src/main/java/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java ## @@ -515,7 +5

[GitHub] [cloudstack] DaanHoogland commented on issue #3518: Unhandled NPE in VMSnapShotManagerImpl.orchestrateCreateVMSnapshot produces snapshots that cannot be deleted

DaanHoogland commented on issue #3518: Unhandled NPE in VMSnapShotManagerImpl.orchestrateCreateVMSnapshot produces snapshots that cannot be deleted URL: https://github.com/apache/cloudstack/issues/3518#issuecomment-519846062 @onitake I think it is best to remove the stray snapshot entry fr

[GitHub] [cloudstack] rhtyd closed issue #3507: VR smoketests failures around VR reboot and shutdown+start

rhtyd closed issue #3507: VR smoketests failures around VR reboot and shutdown+start URL: https://github.com/apache/cloudstack/issues/3507 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] onitake edited a comment on issue #3546: [FIX] Rethrow takeVMSnapshot() exception

onitake edited a comment on issue #3546: [FIX] Rethrow takeVMSnapshot() exception URL: https://github.com/apache/cloudstack/pull/3546#issuecomment-519829463 The problem here is that this fairly difficult, as I don't have a reliable way to trigger throwing an exception, in a real life scena

[GitHub] [cloudstack] onitake commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception

onitake commented on issue #3546: [FIX] Rethrow takeVMSnapshot() exception URL: https://github.com/apache/cloudstack/pull/3546#issuecomment-519829463 The problem here is that this fairly difficult, as I don't have a reliable way to trigger throwing an exception, in a real life scenario.

[GitHub] [cloudstack] rhtyd commented on issue #3548: When adding a new KVM zone, the template_store_ref may get wrong install_path for systemvmtemplate

rhtyd commented on issue #3548: When adding a new KVM zone, the template_store_ref may get wrong install_path for systemvmtemplate URL: https://github.com/apache/cloudstack/issues/3548#issuecomment-519822397 Looks like the systemvm install script by default hard codes the account folder to

[GitHub] [cloudstack] rhtyd opened a new issue #3548: When adding a new KVM zone, the template_store_ref may get wrong install_path for systemvmtemplate

rhtyd opened a new issue #3548: When adding a new KVM zone, the template_store_ref may get wrong install_path for systemvmtemplate URL: https://github.com/apache/cloudstack/issues/3548 With an existing zone with a system template, add a new zone and scp/copy the system template directory t