[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1659
  
@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 #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

[murali-reddy]

Github user murali-reddy commented on the issue:

https://github.com/apache/cloudstack/pull/1659
  
@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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

[karuturi]

Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
@mike-tutkowski  and @jburwell I think nobody has the test configurations 
requested. There is no point waiting indefinitely on a configuration which 
nobody has or wants to test. Let wait for 2 more days and if no one responds, 
lets merge. 


---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

[Tutkowski, Mike]

Thanks, Sergey!

On 9/21/16, 5:37 PM, "Sergey Levitskiy"  wrote:

@mike-tutkowski @nvazquez We can probably run it this week



On 9/21/16, 12:11 PM, "mike-tutkowski"  wrote:

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-9144 (but perhaps someone is 
using an Advanced Zone and doesn't see the problem described in 9144).


---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

[Sergey Levitskiy]

@mike-tutkowski @nvazquez We can probably run it this week



On 9/21/16, 12:11 PM, "mike-tutkowski"  wrote:

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-9144 (but perhaps someone is 
using an Advanced Zone and doesn't see the problem described in 9144).


---
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 #1683: fix ping tests to properly recognise successful ping...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1683
  
LGTM for testing

Before 
sshClient: DEBUG: {Cmd: ping -c 3 8.8.8.8 via Host: 10.140.12.103} 
{returns: [u'PING 8.8.8.8 (8.8.8.8) 56(84) bytes of data.', u'64 bytes from 
8.8.8.8: icmp_seq=1 ttl=54 time=4.
72 ms', u'64 bytes from 8.8.8.8: icmp_seq=2 ttl=54 time=4.66 ms', u'64 
bytes from 8.8.8.8: icmp_seq=3 ttl=54 time=4.61 ms', u'', u'--- 8.8.8.8 ping 
statistics ---', u'3 packets transmitted, 3 received, 0% packet loss, time 
2001ms', u'rtt min/avg/max/mdev = 4.611/4.666/4.723/0.091 ms']}
TestRouterDHCPHosts: DEBUG: SSH result: [u'PING 8.8.8.8 (8.8.8.8) 56(84) 
bytes of data.', u'64 bytes from 8.8.8.8: icmp_seq=1 ttl=54 time=4.72 ms', u'64 
bytes from 8.8.8.8: icmp_seq=2 ttl=54 time=4.66 ms', u'64 bytes from 8.8.8.8: 
icmp_seq=3 ttl=54 time=4.61 ms', u'', u'--- 8.8.8.8 ping statistics ---', u'3 
packets transmitted, 3 received, 0% packet loss, time 2001ms', u'rtt 
min/avg/max/mdev = 4.611/4.666/4.723/0.091 ms']; COUNT is ==> 0
test_router_dhcphosts 
(integration.smoke.test_router_dhcphosts.TestRouterDHCPHosts): CRITICAL: 
FAILED: test_router_dhcphosts: ['Traceback (most recent call last):\n', ' File 
"/usr/local/lib/python2.7/unittest/case.py", line 331, in run\n 
testMethod()\n', ' File 
"/home/t_vazqnadmin/cloudstack/test/integration/smoke/test_router_dhcphosts.py",
 line 356, in test_router_dhcphosts\n self.test_ssh_command(self.vm_1, 
nat_rule1, "natrule1")\n', ' File 
"/home/t_vazqnadmin/cloudstack/test/integration/smoke/test_router_dhcphosts.py",
 line 201, in test_ssh_command\n "Ping to outside world from VM should be 
successful"\n', ' File "/usr/local/lib/python2.7/unittest/case.py", line 515, 
in assertEqual\n assertion_func(first, second, msg=msg)\n', ' File 
"/usr/local/lib/python2.7/unittest/case.py", line 508, in _baseAssertEqual\n 
raise self.failureException(msg)\n', 'AssertionError: ## Ping to outside world 
from VM should be successful\n']

After
sshClient: DEBUG: Trying SSH Connection: Host:10.144.31.41 User:root 
Port:22 RetryCnt:60===
paramiko.transport: DEBUG: starting thread (client mode): 0x4206310L
paramiko.transport: INFO: Connected (version 2.0, client OpenSSH_5.3)
paramiko.transport: DEBUG: kex 
algos:[u'diffie-hellman-group-exchange-sha256', 
u'diffie-hellman-group-exchange-sha1', u'diffie-hellman-group14-sha1', 
u'diffie-hellman-group1-sha1'] server key:[u'ssh-rsa', u'ssh-dss'] client 
encrypt:[u'aes128-ctr', u'aes192-ctr', u'aes256-ctr', u'arcfour256', 
u'arcfour128', u'aes128-cbc', u'3des-cbc', u'blowfish-cbc', u'cast128-cbc', 
u'aes192-cbc', u'aes256-cbc', u'arcfour', u'rijndael-...@lysator.liu.se'] 
server encrypt:[u'aes128-ctr', u'aes192-ctr', u'aes256-ctr', u'arcfour256', 
u'arcfour128', u'aes128-cbc', u'3des-cbc', u'blowfish-cbc', u'cast128-cbc', 
u'aes192-cbc', u'aes256-cbc', u'arcfour', u'rijndael-...@lysator.liu.se'] 
client mac:[u'hmac-md5', u'hmac-sha1', u'umac...@openssh.com', 
u'hmac-sha2-256', u'hmac-sha2-512', u'hmac-ripemd160', 
u'hmac-ripemd...@openssh.com', u'hmac-sha1-96', u'hmac-md5-96'] server 
mac:[u'hmac-md5', u'hmac-sha1', u'umac...@openssh.com', u'hmac-sha2-256', 
u'hmac-sha2-512', u'hmac-ripemd160', u'hmac-ripemd160@openssh.
 com', u'hmac-sha1-96', u'hmac-md5-96'] client compress:[u'none', 
u'z...@openssh.com'] server compress:[u'none', u'z...@openssh.com'] client 
lang:[u''] server lang:[u''] kex follows?False
paramiko.transport: DEBUG: Ciphers agreed: local=aes128-ctr, 
remote=aes128-ctr
paramiko.transport: DEBUG: using kex diffie-hellman-group14-sha1; server 
key type ssh-rsa; cipher: local aes128-ctr, remote aes128-ctr; mac: local 
hmac-sha1, remote hmac-sha1; compression: local none, remote none
paramiko.transport: DEBUG: Switch to new keys ...
paramiko.transport: DEBUG: Adding ssh-rsa host key for 10.144.31.41: 
f662631595339bcea0d93948261bf655
paramiko.transport: DEBUG: Trying SSH agent key 
9523f82115a1b2102b669b21e8a1084f
paramiko.transport: DEBUG: userauth is OK
paramiko.transport: INFO: Authentication (publickey) failed.
paramiko.transport: DEBUG: userauth is OK
paramiko.transport: INFO: Authentication (password) successful!
sshClient: DEBUG: ===SSH to Host 10.144.31.41 port : 22 SUCCESSFUL===


---
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 #1683: fix ping tests to properly recognise successf...

[PaulAngus]

GitHub user PaulAngus opened a pull request:

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

fix ping tests to properly recognise successful ping test

fix ping tests to properly recognise successful ping test

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

$ git pull https://github.com/shapeblue/cloudstack pingfixes

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

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


commit 2b240e8aa6e66e6078a8b95a57c3850d139d7d77
Author: Paul Angus 
Date:   2016-09-20T19:09:08Z

fix ping tests to properly recognise successful ping 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.
---

Re: State of IPv6 implementation

[Nux!]

Good luck with that, hopefully an experienced coder can step up and help.
Lack of Ipv6 is really the big elephant in the room when it comes to Cloudstack.

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Thomas Schneider" 
> To: dev@cloudstack.apache.org
> Sent: Wednesday, 21 September, 2016 17:30:07
> Subject: Re: State of IPv6 implementation

> Hi,
> 
> Basicaly I have a plan to implement IPv6 in advanced zone with ospf or
> bgp routing based on this doc with some improvement:
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+VPC+Router
> 
> The first thing I will do, is to test my setup by running script manualy
> on the virtual router, and organizing the data about ipv6.
> Once that will be OK, I wloud try to impment the fonctions in the code.
> 
> I have skills in coding, but if somone can help me to start with the
> cloudstack code it will be cool.
> 
> *Thomas Schneider*
> Euskill SAS
> 
> Le 21/09/2016 à 15:30, Nux! a écrit :
>> You guys should get in touch with Exoscale, I think they made progress with 
>> IPv6
>> in SG zones.
>> 
>> Lucian
>> 
>> --
>> Sent from the Delta quadrant using Borg technology!
>> 
>> Nux!
>> www.nux.ro
>> 
>> - Original Message -
>>> From: "Wido den Hollander" 
>>> To: "Thomas Schneider" , 
>>> dev@cloudstack.apache.org
>>> Sent: Wednesday, 21 September, 2016 10:52:03
>>> Subject: Re: State of IPv6 implementation
>> 
 Op 20 september 2016 om 23:10 schreef Thomas Schneider
 :


 Hi Everybody

 I would like to know what is the state of IPv6 implementation in 
 Cloudstack.

>>>
>>> Stuff didn't really advance yet. Are you looking into Basic or Advanced
>>> Networking?
>>>
 I have good skills in IPv6 and routing and I would like to contribute to
 the project.

 If somebody can tell me how to start ?
>>>
>>> The core of CloudStack is still very, very much IPv4 minded and it will be 
>>> a lot
>>> of work to get this in there.
>>>
>>> You might want to read this:
>>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking
>>>
>>> Wido
>>>
 Regards

 --
 *Thomas Schneider*
> >>> Euskill SAS

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

[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-9144 
(but perhaps someone is using an Advanced Zone and doesn't see the problem 
described in 9144).


---
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 #871: CLOUDSTACK-8893: Fixing script as per the latest func...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/871
  
@jburwell @rhtyd @karuturi @nvazquez I assume the current code doesn’t 
allow volume snapshots on the top of vmsnapshots on purpose. In that case is it 
best to revert PR871 or create a new PR to address the Marvin script?

From: Nicolas Vazquez 
Reply-To: apache/cloudstack 
Date: Wednesday, September 21, 2016 at 11:15 AM
To: apache/cloudstack 
Cc: Sergey Levitskiy , Mention 

Subject: Re: [apache/cloudstack] CLOUDSTACK-8893: Fixing script as per the 
latest functionality (#871)


Hi @sanju1010,

@serg38 and I are working on this PR 
#1677 which solves vm snapshot 
creation issue. After running test_vm_snapshots.py we're getting exception on 
test_01_test_vm_volume_snapshot as it was expected before your changes in this 
PR.

We're getting: 'CloudstackAPIException: Execute cmd: createsnapshot failed, 
due to: errorCode: 431, errorText:Volume snapshot is not allowed, please detach 
it from VM with VM Snapshots\n']

This is consistent with VolumeApiServiceImpl.allocSnapshot method, in which 
it is:

if (volume.getInstanceId() != null) {

// Check that Vm to which this volume is attached does not have 
VM Snapshots

if (_vmSnapshotDao.findByVm(volume.getInstanceId()).size() > 0) 
{

throw new InvalidParameterValueException("Volume snapshot 
is not allowed, please detach it from VM with VM Snapshots");

}

}

Do you want us to adapt test_vm_snapshots.py in mentioned PR to support 
this or would you like to address it yourself?

Thanks

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on 
GitHub, 
or mute the 
thread.



---
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 #871: CLOUDSTACK-8893: Fixing script as per the latest func...

[nvazquez]

Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/871
  
Hi @sanju1010,

@serg38 and I are working on this PR #1677 which solves vm snapshot 
creation issue. After running `test_vm_snapshots.py` we're getting exception on 
`test_01_test_vm_volume_snapshot` as it was expected before your changes in 
this PR.

We're getting: `'CloudstackAPIException: Execute cmd: createsnapshot 
failed, due to: errorCode: 431, errorText:Volume snapshot is not allowed, 
please detach it from VM with VM Snapshots\n']`

This is consistent with `VolumeApiServiceImpl.allocSnapshot` method, in 
which it is:
```
if (volume.getInstanceId() != null) {
// Check that Vm to which this volume is attached does not have 
VM Snapshots
if (_vmSnapshotDao.findByVm(volume.getInstanceId()).size() > 0) 
{
throw new InvalidParameterValueException("Volume snapshot 
is not allowed, please detach it from VM with VM Snapshots");
}
}
```

Do you want us to adapt `test_vm_snapshots.py` in mentioned PR to support 
this or would you like to address it yourself?

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 #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

[nvazquez]

Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/1677
  
@jburwell @rhtyd could you please review this? As problem is that vCenter 
sends recents tasks with null name, I simply added `StringUtils.isNotBlank` 
check before comparing task's name, do you agree with this solution?


---
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: State of IPv6 implementation

[Thomas Schneider]

Hi,

Basicaly I have a plan to implement IPv6 in advanced zone with ospf or
bgp routing based on this doc with some improvement:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+VPC+Router

The first thing I will do, is to test my setup by running script manualy
on the virtual router, and organizing the data about ipv6.
Once that will be OK, I wloud try to impment the fonctions in the code.

I have skills in coding, but if somone can help me to start with the
cloudstack code it will be cool.

*Thomas Schneider*
Euskill SAS

Le 21/09/2016 à 15:30, Nux! a écrit :
> You guys should get in touch with Exoscale, I think they made progress with 
> IPv6 in SG zones.
> 
> Lucian
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> - Original Message -
>> From: "Wido den Hollander" 
>> To: "Thomas Schneider" , 
>> dev@cloudstack.apache.org
>> Sent: Wednesday, 21 September, 2016 10:52:03
>> Subject: Re: State of IPv6 implementation
> 
>>> Op 20 september 2016 om 23:10 schreef Thomas Schneider
>>> :
>>>
>>>
>>> Hi Everybody
>>>
>>> I would like to know what is the state of IPv6 implementation in Cloudstack.
>>>
>>
>> Stuff didn't really advance yet. Are you looking into Basic or Advanced
>> Networking?
>>
>>> I have good skills in IPv6 and routing and I would like to contribute to
>>> the project.
>>>
>>> If somebody can tell me how to start ?
>>
>> The core of CloudStack is still very, very much IPv4 minded and it will be a 
>> lot
>> of work to get this in there.
>>
>> You might want to read this:
>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking
>>
>> Wido
>>
>>> Regards
>>>
>>> --
>>> *Thomas Schneider*
>>> Euskill SAS

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

[nvazquez]

Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
Solved, thanks @rafaelweingartner 


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

[rafaelweingartner]

Github user rafaelweingartner commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
@serg38 with your explanations, I am ok with the current state of this PR. 
No further changes are needed.
There is only one problem now. There are some merge conflicts that have to 
be resolved.



---
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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1615
  
@jburwell This PR is waiting for your review


---
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 #1593: CLOUDSTACK-9417: Usage module refactoring

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1593
  
@jburwell @rhtyd What are your ideas on where in DB to store it?


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

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
@rhtyd @jburwell @rafaelweingartner @koushik-das Can we agree if this PR is 
OK as it is ?


---
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: State of IPv6 implementation

[Nux!]

You guys should get in touch with Exoscale, I think they made progress with 
IPv6 in SG zones.

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Wido den Hollander" 
> To: "Thomas Schneider" , 
> dev@cloudstack.apache.org
> Sent: Wednesday, 21 September, 2016 10:52:03
> Subject: Re: State of IPv6 implementation

>> Op 20 september 2016 om 23:10 schreef Thomas Schneider
>> :
>> 
>> 
>> Hi Everybody
>> 
>> I would like to know what is the state of IPv6 implementation in Cloudstack.
>> 
> 
> Stuff didn't really advance yet. Are you looking into Basic or Advanced
> Networking?
> 
>> I have good skills in IPv6 and routing and I would like to contribute to
>> the project.
>> 
>> If somebody can tell me how to start ?
> 
> The core of CloudStack is still very, very much IPv4 minded and it will be a 
> lot
> of work to get this in there.
> 
> You might want to read this:
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking
> 
> Wido
> 
>> Regards
>> 
>> --
>> *Thomas Schneider*
> > Euskill SAS

[GitHub] cloudstack issue #1602: CLOUDSTACK-9422: Granular 'vmware.create.full.clone'...

[nvazquez]

Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/1602
  
Great, 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 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 #1624: Fixes regarding VOLUME_DELETE events resulting from ...

[cloudmonger]

Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1624
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 102
 Hypervisor xenserver
 NetworkType Advanced
 Passed=99
 Failed=2
 Skipped=4

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_service_offerings.py

 * ContextSuite context=TestServiceOfferings>:setup Failing since 14 runs

* test_volumes.py

 * test_01_create_volume Failed


**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_snapshots.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_ssvm.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


---
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 #1651: Marvin Tests: fix expected return string for success...

[PaulAngus]

Github user PaulAngus commented on the issue:

https://github.com/apache/cloudstack/pull/1651
  
My git foo abandoned me and left a mess ☹
.  I’ll be sending a clean pr later today…


Kind regards,

Paul Angus



VP Technology
paul.an...@shapeblue.com
www.shapeblue.com
 
 

From: serg38 [mailto:notificati...@github.com]
Sent: 21 September 2016 12:40
To: apache/cloudstack 
Cc: Paul Angus ; Mention 

Subject: Re: [apache/cloudstack] Marvin Tests: fix expected return string 
for successful pings. (#1651)


@PaulAngus 
@jburwell @rhtyd This PR 
was closed without merging. Are we still on track to include the fix to 4.9.1 ? 
I can ask @nvazquez to pick this one up if needed.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on 
GitHub, 
or mute the 
thread.



---
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 #1651: Marvin Tests: fix expected return string for success...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1651
  
@PaulAngus @jburwell @rhtyd This PR was closed without merging. Are we 
still on track to include the fix to 4.9.1 ?  I can ask @nvazquez to pick this 
one up if 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 #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1681
  
@murali-reddy This issue might be related to PR1660. Do you mind testing 
against latest master to see if you it is addressed already?


---
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: State of IPv6 implementation

[Wido den Hollander]

> Op 20 september 2016 om 23:10 schreef Thomas Schneider 
> :
> 
> 
> Hi Everybody
> 
> I would like to know what is the state of IPv6 implementation in Cloudstack.
> 

Stuff didn't really advance yet. Are you looking into Basic or Advanced 
Networking?

> I have good skills in IPv6 and routing and I would like to contribute to 
> the project.
> 
> If somebody can tell me how to start ?

The core of CloudStack is still very, very much IPv4 minded and it will be a 
lot of work to get this in there.

You might want to read this: 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking

Wido

> Regards
> 
> --
> *Thomas Schneider*
> Euskill SAS

[GitHub] cloudstack issue #1678: CLOUDSTACK-9503: Increased the VR script timeout. Mo...

[abhinandanprateek]

Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/1678
  
@rhtyd thanks for sharing nordist repo. I have sanitized the vmware plugin 
too.


---
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 #1624: Fixes regarding VOLUME_DELETE events resulting from ...

[karuturi]

Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1624
  
I started BVT run(xenserver, advanced zone) on this PR. @cloudmonger will 
post the results here in approx. 4 hrs.


---
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 #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress r...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

[karuturi]

Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
I already posted the results for that configuration(a comment form 
@cloudmonger with test results) its all good there.


---
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 #1682: IGNORE this PR

[blueorangutan]

Github user blueorangutan commented on the issue:

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


---
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 #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

[murali-reddy]

Github user murali-reddy commented on the issue:

https://github.com/apache/cloudstack/pull/1666
  
@cloudmonger thanks

test_extendPhysicalNetworkVlan failed due to 'physicalnetwork 200 has 
allocated vnets in the range 734-735'. Possibly due to stale network that was 
not cleaned by one of the tests.

Ran test manually, it passed.


---
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 #1643: CLOUDSTACK-9460: For long running transaction...

[abhinandanprateek]

Github user abhinandanprateek closed the pull request at:

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


---
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 #1682: IGNORE this PR

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1682
  
@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 #1682: IGNORE this PR

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1682
  
@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 pull request #1682: IGNORE this PR

[rhtyd]

GitHub user rhtyd opened a pull request:

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

IGNORE this PR

Ignore this PR/commit, this is for testing Trilllian, @BlueOrangutan

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

$ git pull https://github.com/shapeblue/cloudstack bo-testing

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

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


commit 19342ef97f3161ca6ff2b25f1032c0893f7fe88f
Author: Rohit Yadav 
Date:   2016-09-20T07:47:34Z

INSTALL: fix java jdk version

Ignore this PR/commit, this is for testing Trilllian, @BlueOrangutan

Signed-off-by: Rohit Yadav 




---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[blueorangutan]

Github user blueorangutan commented on the issue:

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


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[jburwell]

Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@rhtyd I didn't realize that we needed a system VM template for this one.  
It is too late in the release process for 4.9.1.0 and 4.10.0.0 to cut new 
system VM template.

For LTS, yes, we can cut a new system VM template.  Any objection to 
holding this patch over to 4.9.2.0 and 4.10.1.0?  I will make it top priority 
to merge once the release branches are unfrozen.  


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@wido can you change the base back to master for now, @jburwell to use this 
we'll need to build/publish new systemvmtemplate. Is that feasible for the LTS 
release? If so, we'll need to make changes in the upgrade paths. This is why 
I'm not in favour of getting this on 4.8 branch, we can discuss whether it's 
solid enough to include in 4.9 for lts release. In case we accept this in 4.9, 
then we've to build a new systemvm template in which case I would also want to 
include the strongswan PR (so we dont' have to rebuild another systemvmtemplate 
within next month/weeks for 4.10/4.11+)?


---
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 #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

[cloudmonger]

Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1666
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 101
 Hypervisor xenserver
 NetworkType Advanced
 Passed=102
 Failed=1
 Skipped=4

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_non_contigiousvlan.py

 * test_extendPhysicalNetworkVlan Failing since 2 runs


**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_snapshots.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_ssvm.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[wido]

Github user wido commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@rhtyd Do you want to backport this to 4.9 then? I've changed the base to 
4.9 now, take a look about what you think.


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
I think we can change base to 4.9 as the amount of conflicts will be 
minimal. At this point, it would add a lot of overhead to support 4.8, 4.9, 
master (4.10) simultaneous. As per our agreed guideline, we have been 
maintaining only two trees at a given times -- the master and the latest/last 
release branch (n, n+1, i.e. 4.9 and master).


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[wido]

Github user wido commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@jburwell Ok, thanks. Why against 4.8? I'm probably missing a point here, 
but master is where new features go, right?

If I change the base to 4.8 the amount of commits which come in are huge.


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[jburwell]

Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@jburwell a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been 
kicked to run smoke tests against packages at 
http://packages.shapeblue.com/cloudstack/pr/1545


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[blueorangutan]

Github user blueorangutan commented on the issue:

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


---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

[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 Unmanaged Storage for System VMs


---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

[karuturi]

Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
@mike-tutkowski sorry mike. 2.b(xen+unmanaged) is what I ran. I dont have 
others.


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[jburwell]

Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@wido I will re-review the code later today.  Could you change the base 
branch to 4.8 (these 
[instructions](https://github.com/blog/2224-change-the-base-branch-of-a-pull-request)
 explain how to change the base branch without opening a new PR)?

We also need a test LGTM.  I am going to kick blueorangutan to package and 
test it.  Are there any additional component tests that should be run?


---
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 #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

[jburwell]

Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1545
  
@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 pull request #1681: CLOUDSTACK-9491: incorrect parsing of device ...

[murali-reddy]

GitHub user murali-reddy opened a pull request:

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

CLOUDSTACK-9491: incorrect parsing of device list to find ethernet index of 
plugged NIC


In VmwareResource, findRouterEthDeviceIndex() method find ethernet 
interface index given
the mac address. This method is used, once a nic is plugged to determine 
ethernet interface.
"/proc/sys/net/ipv4/conf" from the VR and looped through the devices to 
find the right
ethernet interface. Howver current logic read it once, and loops through 
the device list.
Its observerd device may not show up '/proc/sys/net/ipv4/conf' immediatly 
once NIC is plugged
in the VM from vCenter.

Fix ensured, while waiting for 15 sec in the loop, read the latest content 
from /proc/sys/net/ipv4/conf
, so that right device list is processed.

Manual tested VPC scenarios of adding new tiers which uses 
findRouterEthDeviceIndex, to find the guest/public network ethernet index.

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

$ git pull https://github.com/murali-reddy/cloudstack 
router_eth_device_index

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

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


commit be7ade3153d3b29c18d904d5327c1af139e1c037
Author: Murali Reddy 
Date:   2016-09-21T06:08:47Z

CLOUDSTACK-9491: incorrect parsing of device list to find ethener index of 
plugged nic

In VmwareResource, findRouterEthDeviceIndex() method find ethernet 
interface index given
the mac address. This method is used, once a nic is plugged to determine 
ethernet interface.
"/proc/sys/net/ipv4/conf" from the VR and looped through the devices to 
find the right
ethernet interface. Howver current logic read it once, and loops through 
the device list.
Its observerd device may not show up '/proc/sys/net/ipv4/conf' immediatly 
once NIC is plugged
in the VM from vCenter.

Fix ensured, while waiting for 15 sec in the loop, read the latest content 
from /proc/sys/net/ipv4/conf
, so that right device list is processed.




---
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 #1642: System VMs on Managed Storage

[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
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 #1642: System VMs on Managed Storage

[jburwell]

Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
@mike-tutkowski the background makes sense.  I would like to see that 
comment amended to commit message.  I have the following outstanding 
questions/items:

1. Is there a JIRA ticket for this enhancement?  If not, please create one. 
 Also, please reference it in the commit message.
1. Test results for the following test scenarios:
* [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) + the new test integration test suite added for this PR against 
XenServer using Managed Storage for System VMs
* [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) against XenServer using Unmanaged Storage for System VMs
* [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) against VMware using Unmanaged Storage for System VMs
* [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) against KVM using Unmanaged Storage for System VMs

As I mentioned previously, these tests can be run across separate labs.  We 
simply need to collect the results for each of these test scenarios to verify 
the patch functionality and no regressions were introduced.


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