Re: 4.8.2.0/4.9.1.0/4.10.0.0 RC Status

2016-11-17 Thread Özhan Rüzgar Karaman
Hi John & Rohit;
If we have time could we add PR 1710 to the upcoming RC for 4.9.1.0. Its a
small fix but its waiting for log time...

Thanks again,
Özhan

On Thu, Nov 17, 2016 at 9:23 AM, John Burwell 
wrote:

> All,
>
> I apologize for being relatively radio silent.  We have mage good progress
> towards getting RCs out for 4.8.2.0, 4.9.1.0, and 4.10.0.0.  On 31 October
> 2016, we 17 outstanding PRs to be merged.  As of today (17 Nov 2016), we
> have 9 PRs to merge with pending one potential blocker/critical defect fix:
>
> * 4.10.0.0 (+ all 4.8.2.0 and 4.9.1.0 PRs)
> * 1542 (regression tests running)
>   * 1545 (code review comments; regression tests pending)
>   * 1577 (Jenkins failures; regression tests pending)
>   * 1579 (needs rebase + commit squash; regression tests pending)
>   * 1580 (needs rebase + commit squash; regression tests pending)
> * 4.9.1.0 (+ all 4.8.2.0 PRs)
>   * 1681 (test failures to resolve)
>   * 1684 (working out the proper fix to an upgrade issue)
> * 4.8.2.0
> * 1745 (test failures to resolve)
> * 1765 (squash commits; code review feedback; regression tests
> pending)
>
> Once all PRs are merged, we will re-execute the tests on each of the
> release branches.  Rohit has opened tracking PRs for 4.8 [1], 4.9 [2], and
> master [3] to understand the state of the release branches as we continue
> to merge PRs.
>
> Thank you to everyone who has reviewed, tested, and merged PRs.  I am
> hopeful that we are very close to cutting these RCs.
>
> Thanks again,
> -John
>
> [1]: https://github.com/apache/cloudstack/pull/1752
> [2]: https://github.com/apache/cloudstack/pull/1753
> [3]: https://github.com/apache/cloudstack/pull/1754
>
> john.burw...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK
> @shapeblue
>
>
>
>


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

2016-11-17 Thread ozhanrk
Github user ozhanrk commented on the issue:

https://github.com/apache/cloudstack/pull/1710
  
Hi @rhtyd @jburwell, 
With the comments of @ustcweizhou, could we start required tests and if its 
all ok, could we merge this pr to upcoming 4.9.1.0 release please?


---
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 pull request #1761: CLOUDSTACK-9592 Empty responses from site to ...

2016-11-17 Thread yvsubhash
Github user yvsubhash commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1761#discussion_r88429916
  
--- Diff: core/src/com/cloud/agent/api/CheckS2SVpnConnectionsAnswer.java ---
@@ -76,4 +76,14 @@ public String getDetail(String ip) {
 }
 return null;
 }
+
+public boolean isIPPresent(String ip) {
+if (this.getResult()) {
+Boolean status = ipToConnected.get(ip);
+if (status != null) {
--- End diff --

Yes The check to see whether ip is there in the response. So it should be 
true even when status is false


---
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 pull request #1761: CLOUDSTACK-9592 Empty responses from site to ...

2016-11-17 Thread yvsubhash
Github user yvsubhash commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1761#discussion_r88431269
  
--- Diff: core/src/com/cloud/agent/api/CheckS2SVpnConnectionsAnswer.java ---
@@ -76,4 +76,14 @@ public String getDetail(String ip) {
 }
 return null;
 }
+
+public boolean isIPPresent(String ip) {
+if (this.getResult()) {
+Boolean status = ipToConnected.get(ip);
+if (status != null) {
--- End diff --

@jburwell changed the base branch to 4.9


---
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 pull request #1761: CLOUDSTACK-9592 Empty responses from site to ...

2016-11-17 Thread yvsubhash
Github user yvsubhash commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1761#discussion_r88432900
  
--- Diff: core/src/com/cloud/agent/api/CheckS2SVpnConnectionsAnswer.java ---
@@ -76,4 +76,14 @@ public String getDetail(String ip) {
 }
 return null;
 }
+
+public boolean isIPPresent(String ip) {
+if (this.getResult()) {
+Boolean status = ipToConnected.get(ip);
+if (status != null) {
--- End diff --

@jburwell  VirtualNetworkApplianceManagerImplTest.java has all tests 
commented out. are there any other place having tests covering this?


---
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 #1759: CLOUDSTACK-9589 vmName entries from host_details tab...

2016-11-17 Thread yvsubhash
Github user yvsubhash commented on the issue:

https://github.com/apache/cloudstack/pull/1759
  
@jburwell 
Change is moved to schema-481to4820.sql and base branch is changed to 4.8


---
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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

2016-11-17 Thread yvsubhash
Github user yvsubhash commented on the issue:

https://github.com/apache/cloudstack/pull/1586
  
@syed @koushik-das can one of you please merge this


---
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 #1760: CLOUDSTACK-9593: userdata: enforce data is a multipl...

2016-11-17 Thread marcaurele
Github user marcaurele commented on the issue:

https://github.com/apache/cloudstack/pull/1760
  
I changed the code to make it resilient to incorrect data that are already 
in the database. Moreover, instead of throwing an error when the padding is not 
present, it appends it. I think it's a better solution. I'll squash it when 
it's an accepted.


---
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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
Trillian test result (tid-343)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36589 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1542-t343-vmware-55u3.zip
Test completed. 42 look ok, 7 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_04_rvpc_privategw_static_routes | `Failure` | 117.54 | 
test_privategw_acl.py
test_02_vpc_privategw_static_routes | `Failure` | 375.38 | 
test_privategw_acl.py
test_01_vpc_site2site_vpn | `Error` | 557.58 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | `Error` | 753.40 | test_vpc_vpn.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
`Error` | 622.29 | test_vpc_redundant.py
test_01_test_vm_volume_snapshot | `Error` | 322.22 | test_vm_snapshots.py
test_CreateTemplateWithDuplicateName | `Error` | 60.47 | test_templates.py
ContextSuite context=TestTemplates>:setup | `Error` | 351.48 | 
test_templates.py
test_reboot_router | `Error` | 560.78 | test_network.py
test_nested_virtualization_vmware | `Error` | 353.13 | 
test_nested_virtualization.py
test_01_vpc_remote_access_vpn | Success | 166.92 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 365.60 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 944.50 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 744.25 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1560.08 | 
test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 684.86 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1378.05 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 25.85 | test_volumes.py
test_06_download_detached_volume | Success | 60.58 | 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 | 20.39 | test_volumes.py
test_02_attach_volume | Success | 58.79 | test_volumes.py
test_01_create_volume | Success | 514.80 | test_volumes.py
test_03_delete_vm_snapshots | Success | 275.24 | test_vm_snapshots.py
test_02_revert_vm_snapshots | Success | 235.41 | test_vm_snapshots.py
test_01_create_vm_snapshots | Success | 161.75 | test_vm_snapshots.py
test_deploy_vm_multiple | Success | 288.70 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 142.72 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 185.34 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 81.18 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.10 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 10.15 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 5.14 | test_vm_life_cycle.py
test_02_start_vm | Success | 25.26 | test_vm_life_cycle.py
test_01_stop_vm | Success | 10.15 | test_vm_life_cycle.py
test_01_create_template | Success | 141.09 | test_templates.py
test_10_destroy_cpvm | Success | 241.92 | test_ssvm.py
test_09_destroy_ssvm | Success | 238.88 | test_ssvm.py
test_08_reboot_cpvm | Success | 156.60 | test_ssvm.py
test_07_reboot_ssvm | Success | 158.72 | test_ssvm.py
test_06_stop_cpvm | Success | 206.97 | test_ssvm.py
test_05_stop_ssvm | Success | 208.84 | test_ssvm.py
test_04_cpvm_internals | Success | 1.20 | test_ssvm.py
test_03_ssvm_internals | Success | 3.51 | 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 | 66.50 | test_snapshots.py
test_04_change_offering_small | Success | 92.32 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.08 | test_service_offerings.py
test_01_create_service_offering | Success | 0.16 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.14 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.19 | test_secondary_storage.py
test_09_reboot_router | Success | 151.01 | test_routers.py
test_08_start_router | Success | 140.95 | test_routers.py
test_07_stop_router | Success | 25.24 | 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.65 | test_routers.py
test_03_restart_network_cleanup | Success | 181.54 | test_routers.py
test_02_router_internal_adv | Succ

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

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1710
  
@ustcweizhou @ozhanrk this feels like a relatively narrow case (RBD on 
KVM), and we are extremely late for 4.9.1.0.  We have numerous "small" fixes 
that could be included that would further delay significant fixes that affect 
nearly users.  I am sorry, but this fix will have to wait until 
4.9.2.0/4.10.1.0/4.11.0.0.


---
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: 4.8.2.0/4.9.1.0/4.10.0.0 RC Status

2016-11-17 Thread John Burwell
Ozhan,

As I said on the PR, #1710 addresses a fairly narrow case (RBD on KVM), and 
4.9.1.0 is extremely late.  There are a lot “small” fixes that could be 
included.  However, it would further delay delivery of significant dies that 
nearly all users.  At some point, we must draw a line, and this fix will have 
to wait until 4.9.2.0/4.10.1.0/4.11.0.0.

Thanks,
-John


john.burw...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London VA WC2N 4HSUK
@shapeblue
  
 

> On Nov 17, 2016, at 5:22 AM, Özhan Rüzgar Karaman  
> wrote:
> 
> Hi John & Rohit;
> If we have time could we add PR 1710 to the upcoming RC for 4.9.1.0. Its a
> small fix but its waiting for log time...
> 
> Thanks again,
> Özhan
> 
> On Thu, Nov 17, 2016 at 9:23 AM, John Burwell 
> wrote:
> 
>> All,
>> 
>> I apologize for being relatively radio silent.  We have mage good progress
>> towards getting RCs out for 4.8.2.0, 4.9.1.0, and 4.10.0.0.  On 31 October
>> 2016, we 17 outstanding PRs to be merged.  As of today (17 Nov 2016), we
>> have 9 PRs to merge with pending one potential blocker/critical defect fix:
>> 
>> * 4.10.0.0 (+ all 4.8.2.0 and 4.9.1.0 PRs)
>> * 1542 (regression tests running)
>>  * 1545 (code review comments; regression tests pending)
>>  * 1577 (Jenkins failures; regression tests pending)
>>  * 1579 (needs rebase + commit squash; regression tests pending)
>>  * 1580 (needs rebase + commit squash; regression tests pending)
>> * 4.9.1.0 (+ all 4.8.2.0 PRs)
>>  * 1681 (test failures to resolve)
>>  * 1684 (working out the proper fix to an upgrade issue)
>> * 4.8.2.0
>>* 1745 (test failures to resolve)
>>* 1765 (squash commits; code review feedback; regression tests
>> pending)
>> 
>> Once all PRs are merged, we will re-execute the tests on each of the
>> release branches.  Rohit has opened tracking PRs for 4.8 [1], 4.9 [2], and
>> master [3] to understand the state of the release branches as we continue
>> to merge PRs.
>> 
>> Thank you to everyone who has reviewed, tested, and merged PRs.  I am
>> hopeful that we are very close to cutting these RCs.
>> 
>> Thanks again,
>> -John
>> 
>> [1]: https://github.com/apache/cloudstack/pull/1752
>> [2]: https://github.com/apache/cloudstack/pull/1753
>> [3]: https://github.com/apache/cloudstack/pull/1754
>> 
>> john.burw...@shapeblue.com
>> www.shapeblue.com
>> 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK
>> @shapeblue
>> 
>> 
>> 
>> 



Re: 4.8.2.0/4.9.1.0/4.10.0.0 RC Status

2016-11-17 Thread Özhan Rüzgar Karaman
Ok John, i will ping you and Rohit again when we start to work for 4.9.2
release. Thanks for your time.

Özhan

On Thu, Nov 17, 2016 at 5:32 PM, John Burwell 
wrote:

> Ozhan,
>
> As I said on the PR, #1710 addresses a fairly narrow case (RBD on KVM),
> and 4.9.1.0 is extremely late.  There are a lot “small” fixes that could be
> included.  However, it would further delay delivery of significant dies
> that nearly all users.  At some point, we must draw a line, and this fix
> will have to wait until 4.9.2.0/4.10.1.0/4.11.0.0.
>
> Thanks,
> -John
>
>
> john.burw...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK
> @shapeblue
>
>
>
> > On Nov 17, 2016, at 5:22 AM, Özhan Rüzgar Karaman <
> oruzgarkara...@gmail.com> wrote:
> >
> > Hi John & Rohit;
> > If we have time could we add PR 1710 to the upcoming RC for 4.9.1.0. Its
> a
> > small fix but its waiting for log time...
> >
> > Thanks again,
> > Özhan
> >
> > On Thu, Nov 17, 2016 at 9:23 AM, John Burwell <
> john.burw...@shapeblue.com>
> > wrote:
> >
> >> All,
> >>
> >> I apologize for being relatively radio silent.  We have mage good
> progress
> >> towards getting RCs out for 4.8.2.0, 4.9.1.0, and 4.10.0.0.  On 31
> October
> >> 2016, we 17 outstanding PRs to be merged.  As of today (17 Nov 2016), we
> >> have 9 PRs to merge with pending one potential blocker/critical defect
> fix:
> >>
> >> * 4.10.0.0 (+ all 4.8.2.0 and 4.9.1.0 PRs)
> >> * 1542 (regression tests running)
> >>  * 1545 (code review comments; regression tests pending)
> >>  * 1577 (Jenkins failures; regression tests pending)
> >>  * 1579 (needs rebase + commit squash; regression tests pending)
> >>  * 1580 (needs rebase + commit squash; regression tests pending)
> >> * 4.9.1.0 (+ all 4.8.2.0 PRs)
> >>  * 1681 (test failures to resolve)
> >>  * 1684 (working out the proper fix to an upgrade issue)
> >> * 4.8.2.0
> >>* 1745 (test failures to resolve)
> >>* 1765 (squash commits; code review feedback; regression tests
> >> pending)
> >>
> >> Once all PRs are merged, we will re-execute the tests on each of the
> >> release branches.  Rohit has opened tracking PRs for 4.8 [1], 4.9 [2],
> and
> >> master [3] to understand the state of the release branches as we
> continue
> >> to merge PRs.
> >>
> >> Thank you to everyone who has reviewed, tested, and merged PRs.  I am
> >> hopeful that we are very close to cutting these RCs.
> >>
> >> Thanks again,
> >> -John
> >>
> >> [1]: https://github.com/apache/cloudstack/pull/1752
> >> [2]: https://github.com/apache/cloudstack/pull/1753
> >> [3]: https://github.com/apache/cloudstack/pull/1754
> >>
> >> john.burw...@shapeblue.com
> >> www.shapeblue.com
> >> 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK
> >> @shapeblue
> >>
> >>
> >>
> >>
>
>


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

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1710
  
@ozhanrk @ustcweizhou I asked a [functionality 
question](https://github.com/apache/cloudstack/pull/1710#issuecomment-257697031)
 regarding this functionality to which I cannot find an answer.  Namely, that 
this patch is deleting snapshots in a circumstance where not all users want 
them deleted.


---
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 #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-17 Thread ozhanrk
Github user ozhanrk commented on the issue:

https://github.com/apache/cloudstack/pull/1710
  
hi @jburwell this pr does not delete any kind of snapshots, this patch only 
prevents,  solves a kind of timeout issue on code thats all, nothing more. This 
is a 4 lines of small fix where delete snapshot code could not find already 
deleted snapshots ids so it waits till timeout reached, with this patch it 
understands they are already deleted so it did not wait and continue the 
process.


---
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 #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-17 Thread prashanthvarma
Github user prashanthvarma commented on the issue:

https://github.com/apache/cloudstack/pull/1579
  
@jburwell @rhtyd We want to get this PR merged in after merging in PR #1580 
as there are some feature interactions and dependencies. Thus, we will rebase 
this PR on top of the PR #1580 (2 dev + 2 test commits) and let you know.

Let me know, if this is fine with you.


---
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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread prashanthvarma
Github user prashanthvarma commented on the issue:

https://github.com/apache/cloudstack/pull/1577
  
@jburwell @rhtyd The previous Jenkins failure was due to missing solidfire 
plugin dependency. 

I have re-based this PR with latest master, and we now only have two 
commits (1 dev + 1 test).

Jenkins and Travis are green now, you can now kick off regression tests on 
this PR via blueorangutan.

Meanwhile, we are re-running our regression tests on the latest build. 

Let me know, if you need anything from our side.


---
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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread prashanthvarma
Github user prashanthvarma commented on the issue:

https://github.com/apache/cloudstack/pull/1580
  
@jburwell @rhtyd I have re-based this PR with latest master, and we now 
only have two commits (1 dev + 1 test).

Jenkins and Travis are green now, you can now kick off regression tests on 
this PR via blueorangutan.

Meanwhile, we are re-running our local regression tests on the latest 
build. 

Let me know, if you need anything from our side.


---
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 pull request #1766: CLOUDSTACK-9598: wrong defaut gateway for the...

2016-11-17 Thread murali-reddy
GitHub user murali-reddy opened a pull request:

https://github.com/apache/cloudstack/pull/1766

 CLOUDSTACK-9598: wrong defaut gateway for the nic in non-default network

 when guest VM has nic's in more than one guest network set the tag for 
each host in /etc/dhcphosts.txt, and use the tag to add exception in 
/etc/dhcpopts.txt to prevent sending default route, dns server in case if the 
nic is in non-default network

this was the behaviour with edithosts.sh prior to 4.6

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/murali-reddy/cloudstack 
vr-default-network-gateway

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1766.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1766


commit 17be6d806378cbacb21e8256e02e6d92188ed183
Author: Murali Reddy 
Date:   2016-11-17T18:00:25Z

 CLOUDSTACK-9598: wrong defaut gateway for the nic in non-default network
 when guest VM has nic's in more than one guest network

 set the tag for each host in /etc/dhcphosts.txt, and use the tag to add 
exception
 in /etc/dhcpopts.txt to prevent sending default route, dns server in case 
if
 the nic is in non-default network




---
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 #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread murali-reddy
Github user murali-reddy commented on the issue:

https://github.com/apache/cloudstack/pull/1766
  
@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.
---


[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1766
  
@murali-reddy 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 #1711: XenServer 7 Support

2016-11-17 Thread syed
Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1711
  
Sounds good @jburwell 


---
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 #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


---
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 #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread murali-reddy
Github user murali-reddy commented on the issue:

https://github.com/apache/cloudstack/pull/1766
  
@blueorangutan test


---
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 #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1766
  
@murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) 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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-11-17 Thread serg38
Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
@jburwell @rhtyd Looks like blueorangutain vmware side has issues on 
hypervisor. Can you take a look and then re-kick the tests? Some errors were 
due to that.

530, errortext : u'Unable to add NIC to VM[User|i-39-54-VM]: 
com.cloud.exception.AgentUnavailableException: Resource [Host:1] is 
unreachable: Host 1: Unable to start instance due to Unable to start  
VM:f22d93f6-e197-46e4-8e0c-0a926fece518 due to error in finalizeStart, not 
retrying'}, 



---
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 #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1766
  
I suggest to use default_entry in the script.
see 
https://issues.apache.org/jira/browse/CLOUDSTACK-9017?focusedCommentId=15653794&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15653794


---
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 #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1766
  
Trillian test result (tid-344)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 23535 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1766-t344-kvm-centos7.zip
Test completed. 41 look ok, 2 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_redundant_VPC_default_routes | `Failure` | 858.96 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 358.43 
| test_vpc_redundant.py
test_router_dhcphosts | `Failure` | 88.74 | test_router_dhcphosts.py
ContextSuite context=TestRouterDHCPHosts>:teardown | `Error` | 134.10 | 
test_router_dhcphosts.py
test_01_vpc_site2site_vpn | Success | 140.86 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 55.78 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 235.72 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 279.36 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 482.65 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 502.48 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1455.91 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 538.26 | test_vpc_redundant.py
test_09_delete_detached_volume | Success | 15.33 | test_volumes.py
test_08_resize_volume | Success | 15.34 | test_volumes.py
test_07_resize_fail | Success | 20.49 | test_volumes.py
test_06_download_detached_volume | Success | 15.33 | test_volumes.py
test_05_detach_volume | Success | 100.21 | test_volumes.py
test_04_delete_attached_volume | Success | 10.20 | test_volumes.py
test_03_download_attached_volume | Success | 15.22 | test_volumes.py
test_02_attach_volume | Success | 43.60 | test_volumes.py
test_01_create_volume | Success | 711.21 | test_volumes.py
test_deploy_vm_multiple | Success | 268.02 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.52 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.11 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 35.66 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.09 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 130.75 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.77 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.11 | test_vm_life_cycle.py
test_01_stop_vm | Success | 40.26 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 61.48 | test_templates.py
test_08_list_system_templates | Success | 0.02 | test_templates.py
test_07_list_public_templates | Success | 0.03 | test_templates.py
test_05_template_permissions | Success | 0.05 | test_templates.py
test_04_extract_template | Success | 5.10 | test_templates.py
test_03_delete_template | Success | 5.09 | test_templates.py
test_02_edit_template | Success | 90.13 | test_templates.py
test_01_create_template | Success | 55.50 | test_templates.py
test_10_destroy_cpvm | Success | 161.54 | test_ssvm.py
test_09_destroy_ssvm | Success | 133.15 | test_ssvm.py
test_08_reboot_cpvm | Success | 101.41 | test_ssvm.py
test_07_reboot_ssvm | Success | 102.94 | test_ssvm.py
test_06_stop_cpvm | Success | 131.65 | test_ssvm.py
test_05_stop_ssvm | Success | 133.10 | test_ssvm.py
test_04_cpvm_internals | Success | 1.22 | test_ssvm.py
test_03_ssvm_internals | Success | 2.90 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.08 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.08 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.04 | test_snapshots.py
test_04_change_offering_small | Success | 209.38 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.03 | test_service_offerings.py
test_01_create_service_offering | Success | 0.08 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.09 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.13 | test_secondary_storage.py
test_09_reboot_router | Success | 35.41 | test_routers.py
test_08_start_router | Success | 30.29 | test_routers.py
test_07_stop_router | Success | 10.21 | 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.65 | test_routers.py
test_03_restart_network_cleanup | Success | 60.47 | test_routers.py
test_02_router_internal_adv | Success 

[GitHub] cloudstack pull request #1767: CLOUDSTACK-9457: Allow retrieval and modifica...

2016-11-17 Thread nvazquez
GitHub user nvazquez opened a pull request:

https://github.com/apache/cloudstack/pull/1767

CLOUDSTACK-9457: Allow retrieval and modification of VM and template 
details via API and UI

JIRA TICKET: https://issues.apache.org/jira/browse/CLOUDSTACK-9457

### Goal
This PR proposes list/add/update/delete user vm and vm template details via 
API and UI.

### New API methods

| Method name| Desc | Params|
| - |:-:|:-:|
| `listTemplateDetails` / `listVirtualMachineDetails` | List all details | 
`templateid` / `virtualmachineid` (required) |
|   | | detailid (optional) |

| Method name| Desc | Params|
| - |:-:|:-:|
| `addTemplateDetail` / `addVirtualMachineDetail` | Add detail | 
`templateid` / `virtualmachineid` (required) |
|  |  | `detailname` (required) |
|  |  | `detailvalue` (required) |

| Method name| Desc | Params|
| - |:-:|:-:|
| `updateTemplateDetail` / `updateVirtualMachineDetail` | Update detail | 
`templateid` / `virtualmachineid` (required) |
|  |  | `detailid` (required) |
|  |  | `detailname` (required) |
|  |  | `detailvalue` (required) |

| Method name| Desc | Params|
| - |:-:|:-:|
| `deleteTemplateDetail` / `deleteVirtualMachineDetail` | Delete detail | 
`templateid` / `virtualmachineid` (required) |
|  |  | detailid (required) |


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/nvazquez/cloudstack userVmAndTemplatesDetails

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1767.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1767


commit 04bd4a49cd71da4096f9fe05d2cb17b903cb72cb
Author: nvazquez 
Date:   2016-11-18T02:18:16Z

CLOUDSTACK-9457: Allow retrieval and modification of VM and template 
details via API and UI




---
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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1580
  
@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.
---


[GitHub] cloudstack issue #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1577
  
@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.
---


[GitHub] cloudstack issue #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1580
  
@jburwell 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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1577
  
@jburwell 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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
@serg38 these errors look similar to those we have seen on other PRs.  We 
are currently investigating them with the assumption that they are most likely 
environmental.  Once we have them resolved, we will re-kick the tests.

/cc @PaulAngus @murali-reddy @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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


---
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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


---
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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1577
  
@blueorangutan test


---
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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1577
  
@jburwell a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) 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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1580
  
@prashanthvarma please squash all commits down into one logical commit for 
the enhancement.  Multiple commits for the same feature muddy the long term 
history.


---
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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1580
  
@blueorangutan test


---
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 #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1579
  
@prashanthvarma please squash all commits down into one logical commit for 
the enhancement.  Multiple commits for the same feature muddy the long term 
history.


---
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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-17 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1580
  
@jburwell a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) 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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1577
  
@prashanthvarma please squash all commits down into one logical commit for 
the enhancement.  Multiple commits for the same feature muddy the long term 
history.


---
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 #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1579
  
@blueorangutan test


---
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 #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-17 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1579
  
@blueorganutan 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.
---