[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-07 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
Thanks @abhinandanprateek @syed 


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-07 Thread abhinandanprateek 
Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed please do a check of various snapshot functions on a upgraded setup. 

@rhtyd this PR can be closed.


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-07 Thread syed 
Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@abhinandanprateek I will check this. I will start with a non-upgraded 
cloudstack, create snapshots, upgrade cloudstack to have #1600 and then try to 
delete older snapshots. From the code it looks like it should delete 
irrespective of weather the snapshot has the `vhd` extension or not (see [here] 
(https://github.com/apache/cloudstack/blob/master/services/secondary-storage/server/src/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java#L1546)
 ) because it does a wildcard delete of the directory before.

Are you sure that delete will break? Can you point out where this might 
happen?


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-07 Thread abhinandanprateek 
Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed @swill @rhtyd @serg38 I think i missed some updates. This PR will 
only break the original PR. The patch suggested by @serg38 looks good. 
@syed in a upgraded setup, I think removal of a older snapshot will fail 
for one, as these donot have a .vhd extension.


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-07 Thread abhinandanprateek 
Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed @swill the change is good. What needs to be considered is what 
happens with old snapshots, from a older version upgraded to this version, that 
are without the .vhd extension. If it can be checked that in a upgraded version 
where the older snapshots are without .vhd extension and new ones with .vhd 
extension, that cloudstack operations will work fine. Otherwise during upgrade 
we need to add .vhd to older snapshots for Xen.
Another thing is that the marvin tests still assume that there is no .vhd 
extension and adds it. This needs to be fixed. We were seeing some marvin test 
fail on Xenserver due to a additonal .vhd extension. Look at the 
test_01_snapshot_root_disk  failure for Xen in this PR for example: 
https://github.com/apache/cloudstack/pull/1754 and the cause that @serg38  
pointed out.


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread swill 
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@rhtyd If you merge this you will break #1600.  We may want to do some more 
testing on the upgrade path, but I believe @syed has done some testing of the 
upgrade path.  Maybe we can put together a list of items to test for the 
upgrade path so we can confirm that this revert is not needed.


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@serg38 I think we can do that, but will be also need some tooling to 
rename the actual files on the SRs? /cc @abhinandanprateek 


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed Thanks, so just to confirm... with this PR, is your work with #1600 
works and is not broken?


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread serg38 
Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed @rhtyd @abhinandanprateek Why cant we simply rename all install paths 
in DB for existing snapshots instead. It can be as simple as adding this query 
to schema upgrade path
UPDATE cloud.snapshots s
JOIN
cloud.snapshot_store_ref ssr ON ssr.snapshot_id = s.id 
SET 
ssr.install_path = CONCAT(ssr.install_path, '.vhd')
WHERE
s.hypervisor_type = 'XenServer'
AND RIGHT(ssr.install_path, 4) != '.vhd';


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread syed 
Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@abhinandanprateek the snapshot path is something which is internal to 
Cloudstack, as an end user, I would not be able to tell the difference before 
and after the upgrade, unless I missed something. 


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-06 Thread syed 
Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@rhtyd I understand that we want to avoid breaking upgrades. But I have 
upgraded my setup and I did not find any problems. I am trying to understand 
what will break if upgrades happen, given the fact that this change will 
definitely break the newer functionality that we have added.


---
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.
---

Re: [GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread Rohit Yadav 
Syed,


The aim of this PR #1807 is to avoid breaking upgrades for users.


With introduction of your PR (#1600), an extension was seen to be added to 
snapshot files. The PR I've shared removes '.vhd' extension from the snapshot 
file names, while regression smoke tests against XenServer65sp1 seem to be 
passing -- we're not sure if this will have any side-effects in your code (i.e. 
changes from your PR#1600).


Can you validate and review the PR and run tests on your side (i.e. test master 
branch including PR#1807)?

If we fail to do that this week, otherwise to avoid further delaying the 
release work we'll have to revert your previous PR merge (#1600), request you 
to submit it again.


Regards.


From: Syed Ahmed <sah...@cloudops.com>
Sent: 06 December 2016 10:20:45
To: dev@cloudstack.apache.org
Subject: Re: [GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition 
of `vhd` extention ...

I see. I was more interested in the functionality that is broken on
Cloudstack. I have verified that older operations are still working
correctly. Did I miss anything?
On Mon, Dec 5, 2016 at 23:44 Abhinandan Prateek <
abhinandan.prat...@shapeblue.com> wrote:

> Syed,
>
>   Any test that is calling is_snapshot_on_nfs from utils.py will fail for
> Xen as it ends up appending an additional “.vhd” to the install_path. There
> are several of them both in smoke and component folder. The reason this is
> under scrutiny is die to marvin test failures.
>
>
>
> On 06/12/16, 9:33 AM, "Syed Ahmed" <sah...@cloudops.com> wrote:
>
> >Yes this will break managed snapshot archives. I want to understand what
> >this revert is trying to fix and what tests were failing.
> >On Mon, Dec 5, 2016 at 22:38 abhinandanprateek <g...@git.apache.org>
> wrote:
> >
> >> Github user abhinandanprateek commented on the issue:
> >>
> >> https://github.com/apache/cloudstack/pull/1807
> >>
> >> @syed can you look at this 'revert' i guess this is going to be an
> >> issue with the managed storage change that has been put in. cc @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 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.
> >> ---
> >>
>
> abhinandan.prat...@shapeblue.com
> www.shapeblue.com<http://www.shapeblue.com>
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
>
>
>
>

rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@abhinandanprateek @syed I'm seeing known intermittent failures with vpc/vr 
related tests, can I get LGTM on this? 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread abhinandanprateek 
Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed yes I assume that PR in itself is good. You need to consider the 
broken test and broken upgrade ,where upgrading users will have path without 
the extension.


---
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.
---

Re: [GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread Syed Ahmed 
I see. I was more interested in the functionality that is broken on
Cloudstack. I have verified that older operations are still working
correctly. Did I miss anything?
On Mon, Dec 5, 2016 at 23:44 Abhinandan Prateek <
abhinandan.prat...@shapeblue.com> wrote:

> Syed,
>
>   Any test that is calling is_snapshot_on_nfs from utils.py will fail for
> Xen as it ends up appending an additional “.vhd” to the install_path. There
> are several of them both in smoke and component folder. The reason this is
> under scrutiny is die to marvin test failures.
>
>
>
> On 06/12/16, 9:33 AM, "Syed Ahmed"  wrote:
>
> >Yes this will break managed snapshot archives. I want to understand what
> >this revert is trying to fix and what tests were failing.
> >On Mon, Dec 5, 2016 at 22:38 abhinandanprateek 
> wrote:
> >
> >> Github user abhinandanprateek commented on the issue:
> >>
> >> https://github.com/apache/cloudstack/pull/1807
> >>
> >> @syed can you look at this 'revert' i guess this is going to be an
> >> issue with the managed storage change that has been put in. cc @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 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.
> >> ---
> >>
>
> abhinandan.prat...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
>
>
>
>

Re: [GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread Abhinandan Prateek 
Syed,

  Any test that is calling is_snapshot_on_nfs from utils.py will fail for Xen 
as it ends up appending an additional “.vhd” to the install_path. There are 
several of them both in smoke and component folder. The reason this is under 
scrutiny is die to marvin test failures.



On 06/12/16, 9:33 AM, "Syed Ahmed"  wrote:

>Yes this will break managed snapshot archives. I want to understand what
>this revert is trying to fix and what tests were failing.
>On Mon, Dec 5, 2016 at 22:38 abhinandanprateek  wrote:
>
>> Github user abhinandanprateek commented on the issue:
>>
>> https://github.com/apache/cloudstack/pull/1807
>>
>> @syed can you look at this 'revert' i guess this is going to be an
>> issue with the managed storage change that has been put in. cc @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 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.
>> ---
>>

abhinandan.prat...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue

Re: [GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread Syed Ahmed 
Yes this will break managed snapshot archives. I want to understand what
this revert is trying to fix and what tests were failing.
On Mon, Dec 5, 2016 at 22:38 abhinandanprateek  wrote:

> Github user abhinandanprateek commented on the issue:
>
> https://github.com/apache/cloudstack/pull/1807
>
> @syed can you look at this 'revert' i guess this is going to be an
> issue with the managed storage change that has been put in. cc @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 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.
> ---
>

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread abhinandanprateek 
Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@syed can you look at this 'revert' i guess this is going to be an issue 
with the managed storage change that has been put in. cc @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 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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
Trillian test result (tid-578)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37415 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1807-t578-xenserver-65sp1.zip
Test completed. 48 look ok, 1 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_05_rvpc_multi_tiers | `Failure` | 516.08 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Failure` | 1356.90 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 615.09 
| test_vpc_redundant.py
test_01_vpc_site2site_vpn | Success | 361.88 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 176.89 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 673.53 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 341.19 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 760.63 | test_vpc_router_nics.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 891.92 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 1102.85 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 20.94 | test_volumes.py
test_08_resize_volume | Success | 111.07 | test_volumes.py
test_07_resize_fail | Success | 116.09 | test_volumes.py
test_06_download_detached_volume | Success | 20.32 | test_volumes.py
test_05_detach_volume | Success | 100.27 | test_volumes.py
test_04_delete_attached_volume | Success | 10.20 | test_volumes.py
test_03_download_attached_volume | Success | 15.28 | test_volumes.py
test_02_attach_volume | Success | 16.47 | test_volumes.py
test_01_create_volume | Success | 393.70 | test_volumes.py
test_03_delete_vm_snapshots | Success | 280.20 | test_vm_snapshots.py
test_02_revert_vm_snapshots | Success | 197.71 | test_vm_snapshots.py
test_01_create_vm_snapshots | Success | 141.88 | test_vm_snapshots.py
test_deploy_vm_multiple | Success | 289.05 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.04 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 46.82 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.18 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 136.56 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.11 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 15.20 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 20.25 | test_vm_life_cycle.py
test_02_start_vm | Success | 25.27 | test_vm_life_cycle.py
test_01_stop_vm | Success | 30.29 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 166.28 | test_templates.py
test_08_list_system_templates | Success | 0.04 | test_templates.py
test_07_list_public_templates | Success | 0.04 | test_templates.py
test_05_template_permissions | Success | 0.11 | test_templates.py
test_04_extract_template | Success | 5.16 | test_templates.py
test_03_delete_template | Success | 5.12 | test_templates.py
test_02_edit_template | Success | 90.10 | test_templates.py
test_01_create_template | Success | 65.63 | test_templates.py
test_10_destroy_cpvm | Success | 261.86 | test_ssvm.py
test_09_destroy_ssvm | Success | 229.85 | test_ssvm.py
test_08_reboot_cpvm | Success | 157.00 | test_ssvm.py
test_07_reboot_ssvm | Success | 144.08 | test_ssvm.py
test_06_stop_cpvm | Success | 172.18 | test_ssvm.py
test_05_stop_ssvm | Success | 139.45 | test_ssvm.py
test_04_cpvm_internals | Success | 1.12 | test_ssvm.py
test_03_ssvm_internals | Success | 3.50 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.13 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.14 | test_ssvm.py
test_01_snapshot_root_disk | Success | 21.32 | test_snapshots.py
test_04_change_offering_small | Success | 129.16 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.06 | test_service_offerings.py
test_01_create_service_offering | Success | 0.08 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.13 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.19 | test_secondary_storage.py
test_01_scale_vm | Success | 5.21 | test_scale_vm.py
test_09_reboot_router | Success | 75.60 | test_routers.py
test_08_start_router | Success | 60.96 | test_routers.py
test_07_stop_router | Success | 15.22 | test_routers.py
test_06_router_advanced | Success | 0.06 | test_routers.py
test_05_router_basic | Success | 0.04 | test_routers.py
test_04_restart_network_wo_cleanup | Success | 5.57 |

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread serg38 
Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has 
been kicked to run smoke tests


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-05 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@blueorangutan test centos7 xenserver-65sp1



---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-02 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has 
been kicked to run smoke tests


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-02 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@blueorangutan test centos7 xenserver-65sp1


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-02 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-311


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-01 Thread blueorangutan 
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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.
---

[GitHub] cloudstack issue #1807: CLOUDSTACK-9633: Revert addition of `vhd` extention ...

2016-12-01 Thread rhtyd 
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1807
  
Pinging for review -- @jburwell @abhinandanprateek @borisstoyanov 
@blueorangutan package


---
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.
---