[GitHub] cloudstack pull request: CLOUDSTACK-9198: Virtual router gets depl...

[bvbharatk]

Github user bvbharatk commented on the pull request:

https://github.com/apache/cloudstack/pull/1278#issuecomment-202714007
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 142
 Hypervisor xenserver
 NetworkType Advanced
 Passed=99
 Failed=3
 Skipped=4

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


**Failed tests:**
* integration.smoke.test_privategw_acl.TestPrivateGwACL

 * test_01_vpc_privategw_acl Failed

* integration.smoke.test_reset_vm_on_reboot.TestResetVmOnReboot

 * test_01_reset_vm_on_reboot Failed

* integration.smoke.test_volumes.TestCreateVolume

 * test_06_download_detached_volume Failing since 2 runs


**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
test_06_copy_template
test_06_copy_iso

**Passed test suits:**
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData

integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_scale_vm.TestScaleVm
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_loadbalance.TestLoadBalance
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_snapshots.TestSnapshotRootDisk

integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_disk_offerings.TestCreateDiskOffering


---
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-cloudmonkey pull request: Build a CI environment in ord...

[imduffy15]

Github user imduffy15 closed the pull request at:

https://github.com/apache/cloudstack-cloudmonkey/pull/2


---
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: CLOUDSTACK-9199: Fixed deployVirtualMachi...

[bvbharatk]

Github user bvbharatk commented on the pull request:

https://github.com/apache/cloudstack/pull/1280#issuecomment-202622942
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 141
 Hypervisor xenserver
 NetworkType Advanced
 Passed=101
 Failed=5
 Skipped=4

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


**Failed tests:**
* integration.smoke.test_loadbalance.TestLoadBalance

 * test_01_create_lb_rule_src_nat Failed

 * test_02_create_lb_rule_non_nat Failed

 * test_assign_and_removal_lb Failed

* integration.smoke.test_volumes.TestCreateVolume

 * test_01_create_volume Failed

 * test_06_download_detached_volume Failed


**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
test_06_copy_template
test_06_copy_iso

**Passed test suits:**
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData

integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_guest_vlan_range.TestDedicateGuestVlanRange
integration.smoke.test_scale_vm.TestScaleVm
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_reset_vm_on_reboot.TestResetVmOnReboot
integration.smoke.test_snapshots.TestSnapshotRootDisk

integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_vm_life_cycle.TestDeployVM
integration.smoke.test_disk_offerings.TestCreateDiskOffering


---
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: Introduction

[ilya]

Hi Boris

Welcome!

On 3/28/16 5:21 AM, Boris Stoyanov wrote:
> Hi CloudStack, 
> 
> My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m 
> based in Sofia, Bulgaria. I will be taking the role of Software Engineer in 
> Test, and as you may have guessed I’ll mostly focus on testing CloudStack. I 
> have about 10 years of experience in testing, which I’ve mostly spend in 
> doing test automation frameworks and deployment automation. I’m new to the 
> CloudStack business and I have a lot to learn, but I hope I’ll get up to 
> speed in short time. Looking forward to working with you! 
> 
> Best Regards,
> Bobby.
> Regards,
> 
> Boris Stoyanov
> 
> boris.stoya...@shapeblue.com 
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
> 

[SSL CERTS] Importing ROOT and INTERMEDIATE certs for SSVM

[ilya]

I'm having difficulty getting ROOT and INTERMEDIATE certificates to show
up in SSVM java keystore.


I've followed the procedure on
http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/systemvm.html?highlight=pkcs

and

https://cwiki.apache.org/confluence/display/CLOUDSTACK/Procedure+to+Replace+realhostip.com+with+Your+Own+Domain+Name

But after restart of SSVM and MS - the keystore still has default Go
Daddy certs.

Would any know how to troubleshoot it?

Also, one thing to note, i'm not uploading the actual wild card cert -
is its against security policy. It will be impossible for me to get a
wildcard cert.

Regards
ilya

Re: Introduction

[Ahmad Emneina]

Welcome Bobby!

Ahmad E

> On Mar 28, 2016, at 5:21 AM, Boris Stoyanov  
> wrote:
> 
> Hi CloudStack, 
> 
> My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m 
> based in Sofia, Bulgaria. I will be taking the role of Software Engineer in 
> Test, and as you may have guessed I’ll mostly focus on testing CloudStack. I 
> have about 10 years of experience in testing, which I’ve mostly spend in 
> doing test automation frameworks and deployment automation. I’m new to the 
> CloudStack business and I have a lot to learn, but I hope I’ll get up to 
> speed in short time. Looking forward to working with you! 
> 
> Best Regards,
> Bobby.
> Regards,
> 
> Boris Stoyanov
> 
> boris.stoya...@shapeblue.com 
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue

[GitHub] cloudstack pull request: Fix Sync of template.properties in Swift

[kollyma]

Github user kollyma commented on the pull request:

https://github.com/apache/cloudstack/pull/1331#issuecomment-202547429
  
@syed: thanks for the feedback. Here a more detailed description, based on 
your comments. 
Setup: CS4.8, nfs secondary storage, kvm with local storage
```
# cat template.properties 
uniquename=225-2-c7401a18-2163-3c22-9799-3cc9b79cb771
filename=eabd7703-4203-49c5-a3a3-f8234f2d1c23.qcow2
size=1753481216c22-9799-3cc9b79cb771
qcow2.filename=eabd7703-4203-49c5-a3a3-f8234f2d1c23.qcow2
qcow2.virtualsize=21474836480
public=true
id=1
```
Entry on the management server: 
```
 select * from vm_template where name = "myname2" \G;
*** 1. row ***
  id: 225
 unique_name: 225-2-c7401a18-2163-3c22-9799-3cc9b79cb771
name: myname2
uuid: cec0787b-5362-4e29-9ca6-63697ff8d872
  public: 1
featured: 0
type: USER
 hvm: 1
bits: 64
 url: NULL
  format: QCOW2
 created: 2016-03-28 18:36:00
 removed: NULL
  account_id: 2
checksum: NULL
display_text: myname
 enable_password: 0
   enable_sshkey: 0
 guest_os_id: 221
bootable: 1
 prepopulate: 0
 cross_zones: 0
 extractable: 0
 hypervisor_type: KVM
  source_template_id: 215
template_tag: NULL
sort_key: 0
size: 21474836480
   state: Active
update_count: 0
 updated: NULL
dynamically_scalable: 0
```
Logs after management server restart:
```
2016-03-28 21:06:33,660 INFO  [o.a.c.s.i.TemplateServiceImpl] 
(AgentConnectTaskPool-3:ctx-3eb6adca) (logid:9e020fbf) Template Sync did not 
find 225-2-c7401a18-2163-3c22-9799-3cc9b79cb771 on image store 1, may request 
download based on available hypervisor types
2016-03-28 21:06:33,661 INFO  [o.a.c.s.i.TemplateServiceImpl] 
(AgentConnectTaskPool-3:ctx-3eb6adca) (logid:9e020fbf) Removing leftover 
template 225-2-c7401a18-2163-3c22-9799-3cc9b79cb771 entry from template store 
table
2016-03-28 21:06:33,668 INFO  [o.a.c.s.i.TemplateServiceImpl] 
(AgentConnectTaskPool-3:ctx-3eb6adca) (logid:9e020fbf) Skip downloading 
template 225-2-c7401a18-2163-3c22-9799-3cc9b79cb771 since no url is specified.
```
You are referring to the path set incorrectly in template properties. The 
cloudstack logs says "Template Sync did not find..." Do we have the same issue 
? the template file is present on the nfs:

template/tmpl/2/225# ls
eabd7703-4203-49c5-a3a3-f8234f2d1c23.qcow2  template.properties




---
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: Travis build and tests

[Rohit Yadav]

> 
Regards,

Rohit Yadav

rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
On 26-Mar-2016, at 9:56 PM, Sebastien Goasguen  wrote:
> 
> Rohit,
> 
> Thank you, thank you so very very much for this !

:) Thanks Sebastien, we all need this

> 
> -Sebastien
> 
>> On 26 Mar 2016, at 17:49, Rohit Yadav  wrote:
>> 
>> While Travis builds are based on ACS-simulator and a green Travis build does 
>> not represent that PR is “good enough”, it gives us a way to fail fast while 
>> performing basic checks such as maven build checks, checkstyle checks, unit 
>> tests, and a basic integration test against simulator against mysql.
>> 
>> Since last couple of months, our Travis build has been unreliable. Even 
>> though it may be green, the result parsing scripts had an issue which caused 
>> unreliable. I just want to share that I’ve fixed them in this PR along with 
>> changes from security hackathon (from Daan et al) and requesting here for 
>> reviewing it and merging it soon:
>> https://github.com/apache/cloudstack/pull/1397
>> 
>> Travis Changes:
>> - Travis machine image changed to trusty (Ubuntu 14.04), it would now now 
>> build PRs on GCE VMs instead of previous precise based environment. It 
>> guarantees a clean environment (separate VMs on every run) with 2Cores and 
>> upto 7.5GB RAM
>> - Uses local pip/python, by avoiding sudo usage and maybe in future use 
>> Travis’s container based testing service that offers few other feature such 
>> as caching, parallel executions etc
>> - Fixes marvin test result parser, that now brings in a “Time” column which 
>> can be used to optimize tests
>> - Optimizations around tests and JVM memory to run these tests under an hour
>> 
>> For example, you may see this Travis run jobs and see the improved build 
>> output and result parsing:
>> https://travis-ci.org/apache/cloudstack/builds/118692017
>> 
>> On local machine I was able to run all smoke tests and some component tests 
>> against a simulator based marvin cfg, one can simply provide a 
>> kvm/xen/vmware based marvin cfg and use required_hardware=true to run tests 
>> against real hardware:
>> https://gist.github.com/bhaisaab/e970264da5005c76c611
>> 
>> Regards.
>> 
>> 
>> Regards,
>> 
>> Rohit Yadav
>> 
>> rohit.ya...@shapeblue.com 
>> www.shapeblue.com
>> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
>> @shapeblue

Regards.

Re: Introduction

[Tutkowski, Mike]

Welcome aboard, Boris!

> On Mar 28, 2016, at 6:21 AM, Boris Stoyanov  
> wrote:
> 
> Hi CloudStack, 
> 
> My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m 
> based in Sofia, Bulgaria. I will be taking the role of Software Engineer in 
> Test, and as you may have guessed I’ll mostly focus on testing CloudStack. I 
> have about 10 years of experience in testing, which I’ve mostly spend in 
> doing test automation frameworks and deployment automation. I’m new to the 
> CloudStack business and I have a lot to learn, but I hope I’ll get up to 
> speed in short time. Looking forward to working with you! 
> 
> Best Regards,
> Bobby.
> Regards,
> 
> Boris Stoyanov
> 
> boris.stoya...@shapeblue.com 
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1425#issuecomment-202498197
  
Thanks @rafaelweingartner!
@pedro-martins could you review the last changes?


---
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: CLOUDSTACK-9200: Fixed failed to delete s...

[bvbharatk]

Github user bvbharatk commented on the pull request:

https://github.com/apache/cloudstack/pull/1282#issuecomment-202495572
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 140
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 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:**
* integration.smoke.test_volumes.TestCreateVolume

 * test_02_attach_volume Failing since 2 runs


**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
test_06_copy_template
test_06_copy_iso

**Passed test suits:**
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData

integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_scale_vm.TestScaleVm
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_loadbalance.TestLoadBalance
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_reset_vm_on_reboot.TestResetVmOnReboot
integration.smoke.test_snapshots.TestSnapshotRootDisk

integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_vm_life_cycle.TestDeployVM
integration.smoke.test_disk_offerings.TestCreateDiskOffering


---
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: OSPF: adding dynamically routing capabili...

[agneya2001]

Github user agneya2001 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1371#discussion_r57577607
  
--- Diff: core/src/com/cloud/agent/api/storage/CopyVolumeCommand.java ---
@@ -75,4 +76,12 @@ public boolean toSecondaryStorage() {
 public String getVmName() {
 return vmName;
 }
+
+public String getNfsVersion() {
+return nfsVersion;
+}
+
+public void setNfsVersion(String nfsVersion) {
+this.nfsVersion = nfsVersion;
+}
--- End diff --

@jburwell Can you click on the OSPF commit for review, somehow during 
rebase non ospf commits also got included. I will try to clean them up.


---
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: OSPF: adding dynamically routing capabili...

[jburwell]

Github user jburwell commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1371#discussion_r57573475
  
--- Diff: core/src/com/cloud/agent/api/storage/CopyVolumeCommand.java ---
@@ -75,4 +76,12 @@ public boolean toSecondaryStorage() {
 public String getVmName() {
 return vmName;
 }
+
+public String getNfsVersion() {
+return nfsVersion;
+}
+
+public void setNfsVersion(String nfsVersion) {
+this.nfsVersion = nfsVersion;
+}
--- End diff --

This looks like a useful change to the storage API, but why is it necessary 
to implement OSPF?  Seems like it should be in a separate PR.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your 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: OSPF: adding dynamically routing capabili...

[jburwell]

Github user jburwell commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1371#discussion_r57573431
  
--- Diff: core/src/com/cloud/agent/api/storage/ListTemplateCommand.java ---
@@ -43,6 +49,10 @@ public DataStoreTO getDataStore() {
 return store;
 }
 
+public String getNfsVersion() {
+return nfsVersion;
+}
+
--- End diff --

This looks like a useful change to the storage API, but why is it necessary 
to implement OSPF?  Seems like it should be in a separate PR.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your 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: OSPF: adding dynamically routing capabili...

[jburwell]

Github user jburwell commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1371#discussion_r57573444
  
--- Diff: 
core/src/com/cloud/agent/api/storage/PrimaryStorageDownloadCommand.java ---
@@ -87,4 +88,12 @@ public String getPrimaryStorageUrl() {
 public boolean executeInSequence() {
 return true;
 }
+
+public String getNfsVersion() {
+return nfsVersion;
+}
+
+public void setNfsVersion(String nfsVersion) {
+this.nfsVersion = nfsVersion;
+}
--- End diff --

This looks like a useful change to the storage API, but why is it necessary 
to implement OSPF?  Seems like it should be in a separate PR.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your 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: Introduction

[Erik Weber]

Welcome :-)

Den mandag 28. mars 2016 skrev Boris Stoyanov 
følgende:

> Hi CloudStack,
>
> My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue.
> I’m based in Sofia, Bulgaria. I will be taking the role of Software
> Engineer in Test, and as you may have guessed I’ll mostly focus on testing
> CloudStack. I have about 10 years of experience in testing, which I’ve
> mostly spend in doing test automation frameworks and deployment automation.
> I’m new to the CloudStack business and I have a lot to learn, but I hope
> I’ll get up to speed in short time. Looking forward to working with you!
>
> Best Regards,
> Bobby.
> Regards,
>
> Boris Stoyanov
>
> boris.stoya...@shapeblue.com 
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
>

Re: Introduction

[Simon Weller]

Welcome Boris.

- Si


From: Boris Stoyanov 
Sent: Monday, March 28, 2016 7:21 AM
To: dev@cloudstack.apache.org
Subject: Introduction

Hi CloudStack,

My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m 
based in Sofia, Bulgaria. I will be taking the role of Software Engineer in 
Test, and as you may have guessed I’ll mostly focus on testing CloudStack. I 
have about 10 years of experience in testing, which I’ve mostly spend in doing 
test automation frameworks and deployment automation. I’m new to the CloudStack 
business and I have a lot to learn, but I hope I’ll get up to speed in short 
time. Looking forward to working with you!

Best Regards,
Bobby.
Regards,

Boris Stoyanov

boris.stoya...@shapeblue.com
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

[runseb]

Github user runseb commented on the pull request:

https://github.com/apache/cloudstack/pull/1397#issuecomment-202369219
  
+1 to all the Travis changes based on code review and verification of 
travis tests. I would have put those changes in a separate PR however.


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

Introduction

[Boris Stoyanov]

Hi CloudStack, 

My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m 
based in Sofia, Bulgaria. I will be taking the role of Software Engineer in 
Test, and as you may have guessed I’ll mostly focus on testing CloudStack. I 
have about 10 years of experience in testing, which I’ve mostly spend in doing 
test automation frameworks and deployment automation. I’m new to the CloudStack 
business and I have a lot to learn, but I hope I’ll get up to speed in short 
time. Looking forward to working with you! 

Best Regards,
Bobby.
Regards,

Boris Stoyanov

boris.stoya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

[bvbharatk]

Github user bvbharatk commented on the pull request:

https://github.com/apache/cloudstack/pull/1285#issuecomment-202347507
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 139
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 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:**
* integration.smoke.test_volumes.TestCreateVolume

 * test_02_attach_volume Failed


**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
test_06_copy_template
test_06_copy_iso

**Passed test suits:**
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData

integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_scale_vm.TestScaleVm
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_loadbalance.TestLoadBalance
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_reset_vm_on_reboot.TestResetVmOnReboot
integration.smoke.test_snapshots.TestSnapshotRootDisk

integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_vm_life_cycle.TestDeployVM
integration.smoke.test_disk_offerings.TestCreateDiskOffering


---
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: [CLOUDSTACK-9328]: Fix vlan issues from t...

[sanju1010]

Github user sanju1010 commented on the pull request:

https://github.com/apache/cloudstack/pull/1455#issuecomment-202342204
  
Made changes as per Shwetaag's comments.


---
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: [CLOUDSTACK-9328]: Fix vlan issues from t...

[shwetaag]

Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1455#discussion_r57562054
  
--- Diff: test/integration/smoke/test_privategw_acl.py ---
@@ -287,6 +298,16 @@ def performVPCTests(self, vpc_off, 
restart_with_cleanup = False):
 if not physical_networks:
 self.fail("No Physical Networks found!")
 
+qresultset = self.dbclient.execute(
+"select vnet from op_dc_vnet_alloc where physical_network_id=\
+(select id from physical_network where uuid='%s' ) and taken 
is NULL;" % physical_networks[0].id
+)
+self.assertEqual(validateList(qresultset)[0],
+ PASS,
+ "Invalid sql query response"
+)
+#vlans = physical_networks[0].vlan.split('-')
+vlans = qresultset
 vlans = physical_networks[0].vlan.split('-')
--- End diff --

this split is not needed for getting correct vlan entry


---
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: [CLOUDSTACK-9328]: Fix vlan issues from t...

[sanju1010]

GitHub user sanju1010 opened a pull request:

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

[CLOUDSTACK-9328]: Fix vlan issues from test suite test_privategw_acl.py in 
BVT

Please refer to CLOUDSTACK-9328 for the details.

Test Results:
==
test_01_vpc_privategw_acl 
(integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: 
test_01_vpc_privategw_acl | Status : SUCCESS ===
ok

--
Ran 1 test in 108.273s

OK


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

$ git pull https://github.com/sanju1010/cloudstack vlan

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

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


commit fe73673d115b948930adab77669b1ef9236add0a
Author: sanjeev 
Date:   2016-03-28T10:36:57Z

[CLOUDSTACK-9328]: Fix vlan issues from test suite test_privategw_acl.py
in BVT
Bug-Id: CLOUDSTACK-9328




---
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: [DISCUSS] Request for comments: Dynamic Role Based API Access Checker for CloudStack

[Koushik Das]

Thanks Rohit, for the replies. Added some more comments based on the replies.

-Koushik


From: Rohit Yadav 
Sent: Friday, March 25, 2016 6:42 PM
To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
Subject: RE: [DISCUSS] Request for comments: Dynamic Role Based API Access 
Checker for CloudStack

Hi Koushik,

Thanks for the comments on the FS, I've replied to all of them and updated the 
FS (for example, default param in APIs).

Let me know on this ML thread or on the FS if you've further 
questions/comments. Thanks.

Regards.

Regards,

Rohit Yadav

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

-Original Message-
From: Koushik Das [mailto:koushik@accelerite.com]
Sent: Friday, March 25, 2016 1:34 PM
To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
Subject: Re: [DISCUSS] Request for comments: Dynamic Role Based API Access 
Checker for CloudStack

The idea looks good. I have provided some questions/comments on the FS itself.

Thanks,
Koushik


From: Rohit Yadav 
Sent: Wednesday, March 23, 2016 10:34 PM
To: dev@cloudstack.apache.org
Cc: us...@cloudstack.apache.org
Subject: [DISCUSS] Request for comments: Dynamic Role Based API Access Checker 
for CloudStack

Hi all,

I want to propose a new feature for CloudStack, dynamic role-based API access 
checker. This feature will allow us to migrate rules define in 
commands.properties file to database, while role management (such as 
creating/editing roles, adding/removing rules) won't require restarting 
management server(s).

Please find more details in the FS here:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dynamic+Role+Based+API+Access+Checker+for+CloudStack

I look forward to your comments, suggestions and questions. Thanks.

Regards,
Rohit Yadav

Regards,

Rohit Yadav

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



DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.



DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.

RE: Migrating CloudStack content from download.cloud.com

[Paul Angus]

If someone can figure out a community/ASF acceptable way of the ACS community 
owning a suitable vendor-neutral domain name, within the community we have 
enough resources to physically host it (including the space donated by BT via 
ShapeBlue).



Regards,

Paul Angus

paul.an...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue

-Original Message-
From: Erik Weber [mailto:terbol...@gmail.com] 
Sent: Saturday, March 26, 2016 4:19 PM
To: dev@cloudstack.apache.org
Subject: Re: Migrating CloudStack content from download.cloud.com

One of the issues as far as I know is that as Apache CloudStack we have to be 
strict with what we distribute. Meaning, we won't be able to push noredist 
stuff, which is a pity.

Erik

Den lørdag 26. mars 2016 skrev Ian Rae  følgende:

> Does Apache provide such hosting services? It seems that the Apache 
> infrastructure is very restrictive, perhaps we should ask whether they 
> can provide reliable hosting for templates and other downloads.
>
> If Apache is ruled out as an option I recommend we have the community 
> contribute the hosting (happy to volunteer) but the URL should be a 
> community URL and not specific to a commercial vendor. It should 
> probably be mirrored geographically as well.
>
> Ian
>
> On Saturday, 26 March 2016, Ian Duffy >
> wrote:
>
> > I could be completely wrong here, but wasn't there some specific 
> > closed source citrix magic added to the templates at 
> > downloads.cloud.com that
> was
> > Cloud Platform specific?
> >
> > Ideally, this stuff should be hosted on an Apache Cloudstack 
> > infrastructure or atleast the main community source ( 
> > cloudstack.apt-get.eu, shapeblue repo, etc.).
> >
> > On 25 March 2016 at 15:00, Sebastien Goasguen  
> > > wrote:
> >
> > >
> > > > On Mar 25, 2016, at 10:24 AM, Raja Pullela <
> > raja.pull...@accelerite.com  >
> > > wrote:
> > > >
> > > > @Sebastien,  thanks for the feedback.  please note that the goal 
> > > > of
> > this
> > > exercise was not to impact any users during this migration and 
> > > hence
> the
> > > efforts to place the content at a location where we could move it to.
> > > Since Wido is ok with hosting the content on 
> > > 'cloudstack.apt-get.eu',
> it
> > > is even better.  I will work with Wido on the next steps.
> > > >
> > >
> > > Ok cool.
> > >
> > > We had other threads about avoiding company specific URL in docs. 
> > > So
> > let’s
> > > definitely avoid that.
> > >
> > >
> > > > @Ilya,  'cloud.com' is not getting transferred.
> > > >
> > > > Best,
> > > > Raja
> > > > Senior Manager, Product Development Accelerite, 
> > > > www.accelerite.com
> > > >
> > > > -Original Message-
> > > > From: Sebastien Goasguen [mailto:run...@gmail.com 
> ]
> > > > Sent: Friday, March 25, 2016 1:57 PM
> > > > To: dev@cloudstack.apache.org  
> > > > Subject: Re: Migrating CloudStack content from 
> > > > download.cloud.com
> > > >
> > > 
> > >  Citrix has been hosting   "download.cloud.com"  for  quite  some
> > > time  now
> > >  and  it holds  the  System Templates for all the releases and
> some
> > > tools.
> > >  Going forward,  this  content  needs  to  be  moved  from
> > >  "download.cloud.com".So, we will be moving  this content  to
> > >  "cloudstack.accelerite.com".I  will also be  updating  the
> > > links in the
> > >  documentation  to reflect
> > > >
> > > > Raja, I am going to give this a strong -1
> > > >
> > > > We talked about these sorts of things before and it is not
> appropriate.
> > > >
> > > >
> > > >
> > >  these  changes and will provide an update  once the  content  
> > >  move is complete.
> > > 
> > >  @Wido, if you could also copy this content to 
> > >  "cloudstack.apt-get.eu"  that will be great.  I can provide 
> > >  you
> the
> > > details in a separate email.
> > > >>>
> > > >>> Super! If you have a rsync source I will set it up.
> > > >>>
> > > >
> > > > @Wido, I think we need to host the sysVM on apt-get.eu and let 
> > > > that
> be
> > > the primary source and links in the docs.
> > > >
> > > >
> > > >>> Wido
> > > >>>
> > > 
> > >  Best,
> > >  Raja
> > >  Senior Manager, Product Development, Accelerite, 
> > >  www.accelerite.com
> > > 
> > > 
> > > 
> > > 
> > >  DISCLAIMER
> > >  ==
> > >  This e-mail may contain privileged and confidential 
> > >  information which is the property of Accelerite, a Persistent 
> > >  Systems
> business.
> > >  It is intended only for the use of the individual or entity 
> > >  to
> which
> > >  it is addressed. If you are not the intended recipient, you 
> > >  are
> not
> > >  authorized to read, retain, 

Re: [DISCUSS] PR testing process

[Remi Bergsma]

Indeed. If a merge fails then you'll not be able to merge it to master or 
another release branch later on anyway so the author must rebase against the 
base branch first. No need spending test cycles on that. 

Most likely github already reports that PR as 'unstable' so you could also 
check for that. 

Regards, Remi 

Sent from my iPhone

> On 28 Mar 2016, at 09:28, Bharat Kumar  wrote:
> 
> Hi Sanjeev,
> 
> Thanks for bringing this up.
> 
> This is happening because the PR code and the master have deviated since the 
> pr has been created(assuming that PR was rebased with master at the time of 
> creation)  and we start testing it. Ideally we should start testing as soon 
> as the pr is created but due to hardware limitation we have to queue the PRs 
> for testing.
> 
> One way of fixing this would be to merge the PR with master before testing, 
> If the merge fails we post a comment on the PR and skip testing it, until it 
> can me merged.
> 
> We need to make sure that all the PR we create can be merged to master 
> without conflicts.
> 
> Thanks,
> Bharat.
> 
> 
> On 28-Mar-2016, at 12:35 PM, Sanjeev Neelarapu 
> > 
> wrote:
> 
> Hi,
> 
> Currently CI is picking the code from the PR branch, which may or may not be 
> rebased with latest master. This is causing test failures even though they 
> were fixed in latest master.
> e.g.: test_vpc_site2site_vpn.
> Error Message
> local variable 'vm1' referenced before assignment.
> 
> There were few issues with this test suite which were fixed in master. 
> However, we don’t see these changes in some of the PR branches.
> 
> Any thoughts on how to overcome this?
> 
> 
> Best Regards,
> Sanjeev N
> Chief Product Engineer, Accelerite
> Off: +91 40 6722 9368 | EMail: 
> sanjeev.neelar...@accelerite.com
> 
> 
> DISCLAIMER == This e-mail may contain privileged and confidential 
> information which is the property of Accelerite, a Persistent Systems 
> business. It is intended only for the use of the individual or entity to 
> which it is addressed. If you are not the intended recipient, you are not 
> authorized to read, retain, copy, print, distribute or use this message. If 
> you have received this communication in error, please notify the sender and 
> delete all copies of this message. Accelerite, a Persistent Systems business 
> does not accept any liability for virus infected mails.
> 
> 
> 
> 
> DISCLAIMER
> ==
> This e-mail may contain privileged and confidential information which is the 
> property of Accelerite, a Persistent Systems business. It is intended only 
> for the use of the individual or entity to which it is addressed. If you are 
> not the intended recipient, you are not authorized to read, retain, copy, 
> print, distribute or use this message. If you have received this 
> communication in error, please notify the sender and delete all copies of 
> this message. Accelerite, a Persistent Systems business does not accept any 
> liability for virus infected mails.

Re: [DISCUSS] PR testing process

[Remi Bergsma]

To be more exact: Merge to the base branch the PR was created against. Could be 
master, could be 4.7 and such. The code snipped I linked to also does that. 

Sent from my iPhone

> On 28 Mar 2016, at 09:13, Remi Bergsma  wrote:
> 
> Hi Sanjeev,
> 
> I’d propose taking the latest master, ensure it is up2date, make a temp 
> branch then merge the PR to be tested on that. This ensures you test the 
> result of the merge, like it will happen om master.
> 
> For the merging, use the same script we use when we merge for real:
> https://github.com/apache/cloudstack/blob/master/tools/git/git-pr
> 
> When I tested loads of PRs I used a similar approach. You can probably 
> copy/paste these lines:
> https://github.com/MissionCriticalCloud/bubble-toolkit/blob/master/helper_scripts/cloudstack/check-pr.sh#L80#L96
> 
> Regards,
> Remi
> 
> 
> From: Sanjeev Neelarapu 
> >
> Reply-To: "dev@cloudstack.apache.org" 
> >
> Date: Monday 28 March 2016 at 09:05
> To: "dev@cloudstack.apache.org" 
> >
> Subject: [DISCUSS] PR testing process
> 
> Hi,
> 
> Currently CI is picking the code from the PR branch, which may or may not be 
> rebased with latest master. This is causing test failures even though they 
> were fixed in latest master.
> e.g.: test_vpc_site2site_vpn.
> Error Message
> local variable 'vm1' referenced before assignment.
> 
> There were few issues with this test suite which were fixed in master. 
> However, we don’t see these changes in some of the PR branches.
> 
> Any thoughts on how to overcome this?
> 
> 
> Best Regards,
> Sanjeev N
> Chief Product Engineer, Accelerite
> Off: +91 40 6722 9368 | EMail: 
> sanjeev.neelar...@accelerite.com
> [cid:image001.png@01D18361.DE542D60]
> 
> 
> DISCLAIMER == This e-mail may contain privileged and confidential 
> information which is the property of Accelerite, a Persistent Systems 
> business. It is intended only for the use of the individual or entity to 
> which it is addressed. If you are not the intended recipient, you are not 
> authorized to read, retain, copy, print, distribute or use this message. If 
> you have received this communication in error, please notify the sender and 
> delete all copies of this message. Accelerite, a Persistent Systems business 
> does not accept any liability for virus infected mails.

Re: [DISCUSS] PR testing process

[Bharat Kumar]

Hi Sanjeev,

Thanks for bringing this up.

This is happening because the PR code and the master have deviated since the pr 
has been created(assuming that PR was rebased with master at the time of 
creation)  and we start testing it. Ideally we should start testing as soon as 
the pr is created but due to hardware limitation we have to queue the PRs for 
testing.

One way of fixing this would be to merge the PR with master before testing, If 
the merge fails we post a comment on the PR and skip testing it, until it can 
me merged.

We need to make sure that all the PR we create can be merged to master without 
conflicts.

Thanks,
Bharat.


On 28-Mar-2016, at 12:35 PM, Sanjeev Neelarapu 
> 
wrote:

Hi,

Currently CI is picking the code from the PR branch, which may or may not be 
rebased with latest master. This is causing test failures even though they were 
fixed in latest master.
e.g.: test_vpc_site2site_vpn.
Error Message
local variable 'vm1' referenced before assignment.

There were few issues with this test suite which were fixed in master. However, 
we don’t see these changes in some of the PR branches.

Any thoughts on how to overcome this?


Best Regards,
Sanjeev N
Chief Product Engineer, Accelerite
Off: +91 40 6722 9368 | EMail: 
sanjeev.neelar...@accelerite.com


DISCLAIMER == This e-mail may contain privileged and confidential 
information which is the property of Accelerite, a Persistent Systems business. 
It is intended only for the use of the individual or entity to which it is 
addressed. If you are not the intended recipient, you are not authorized to 
read, retain, copy, print, distribute or use this message. If you have received 
this communication in error, please notify the sender and delete all copies of 
this message. Accelerite, a Persistent Systems business does not accept any 
liability for virus infected mails.




DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.

Re: [DISCUSS] PR testing process

[Remi Bergsma]

Hi Sanjeev,

I’d propose taking the latest master, ensure it is up2date, make a temp branch 
then merge the PR to be tested on that. This ensures you test the result of the 
merge, like it will happen om master.

For the merging, use the same script we use when we merge for real:
https://github.com/apache/cloudstack/blob/master/tools/git/git-pr

When I tested loads of PRs I used a similar approach. You can probably 
copy/paste these lines:
https://github.com/MissionCriticalCloud/bubble-toolkit/blob/master/helper_scripts/cloudstack/check-pr.sh#L80#L96

Regards,
Remi


From: Sanjeev Neelarapu 
>
Reply-To: "dev@cloudstack.apache.org" 
>
Date: Monday 28 March 2016 at 09:05
To: "dev@cloudstack.apache.org" 
>
Subject: [DISCUSS] PR testing process

Hi,

Currently CI is picking the code from the PR branch, which may or may not be 
rebased with latest master. This is causing test failures even though they were 
fixed in latest master.
e.g.: test_vpc_site2site_vpn.
Error Message
local variable 'vm1' referenced before assignment.

There were few issues with this test suite which were fixed in master. However, 
we don’t see these changes in some of the PR branches.

Any thoughts on how to overcome this?


Best Regards,
Sanjeev N
Chief Product Engineer, Accelerite
Off: +91 40 6722 9368 | EMail: 
sanjeev.neelar...@accelerite.com
[cid:image001.png@01D18361.DE542D60]


DISCLAIMER == This e-mail may contain privileged and confidential 
information which is the property of Accelerite, a Persistent Systems business. 
It is intended only for the use of the individual or entity to which it is 
addressed. If you are not the intended recipient, you are not authorized to 
read, retain, copy, print, distribute or use this message. If you have received 
this communication in error, please notify the sender and delete all copies of 
this message. Accelerite, a Persistent Systems business does not accept any 
liability for virus infected mails.

[DISCUSS] PR testing process

[Sanjeev Neelarapu]

Hi,

Currently CI is picking the code from the PR branch, which may or may not be 
rebased with latest master. This is causing test failures even though they were 
fixed in latest master.
e.g.: test_vpc_site2site_vpn.
Error Message
local variable 'vm1' referenced before assignment.

There were few issues with this test suite which were fixed in master. However, 
we don't see these changes in some of the PR branches.

Any thoughts on how to overcome this?


Best Regards,
Sanjeev N
Chief Product Engineer, Accelerite
Off: +91 40 6722 9368 | EMail: 
sanjeev.neelar...@accelerite.com
[cid:image001.png@01D18361.DE542D60]




DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

[abhinandanprateek]

Github user abhinandanprateek commented on the pull request:

https://github.com/apache/cloudstack/pull/1454#issuecomment-202256571
  
Marvin test output:

root@ccp:~/cloudstack(host-maint)# ./host_maint.sh 
++ date
+ echo Mon Mar 28 11:47:45 IST 2016
Mon Mar 28 11:47:45 IST 2016
+ TMP=/tmp
+ CLOUDDIR=/root/cloudstack
+ mkdir -p /tmp/simulator/smoke/misc
+ nosetests --with-xunit --xunit-file=/tmp/quagga/test_quagga.xml 
--with-marvin --marvin-config=/root/cloudstack/advanced.cfg 
/root/cloudstack/test/integration/component/test_host_maintenance.py -s -a 
tags=advanced,required_hardware=false --zone=Bootcamp --hypervisor=XenServer

 Marvin Init Started 

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

 Log Folder Path: /tmp//MarvinLogs//Mar_28_2016_11_47_46_LB5B4I. All 
logs will be available here 

=== Marvin Init Logging Successful===

 Marvin Init Successful 
test_01_cancel_host_maintenace 
(integration.component.test_host_maintenance.TestHostMaintenance)
Hypervisor = 34777316-62ae-4590-868a-71daa23dad3d
Hypervisor = 2e872579-efb9-4bb2-ae00-dfb21a994f08
Create VMs as there are not enough vms to check host maintenance
Creating vms = 5
Using template 7e374f74-b471-46bd-bbbe-f457d5376acc 
Using service offering 94589b7c-ef32-484b-8ef7-59ad1ba6b69b 
Using template 7e374f74-b471-46bd-bbbe-f457d5376acc 
Using service offering 9c95b1bf-0ca3-4bba-a00b-1fe61eb078be 
VM create = a0901dc0-a568-4d78-8d46-ddf976f6af95
VM create = ebba3c2b-d950-4969-ae34-2cc785023ef9
VM create = 0a9635be-5599-4259-a89b-ad9017e2d4e7
VM create = 9a885588-db3f-4aca-9885-c931fbd6ea71
Host with id 34777316-62ae-4590-868a-71daa23dad3d is in 
prepareHostForMaintenance
Host with id 34777316-62ae-4590-868a-71daa23dad3d is in 
cancelHostMaintenance
Host with id 2e872579-efb9-4bb2-ae00-dfb21a994f08 is in 
prepareHostForMaintenance
Host with id 2e872579-efb9-4bb2-ae00-dfb21a994f08 is in 
cancelHostMaintenance
=== TestName: test_01_cancel_host_maintenace | Status : SUCCESS ===

===final results are now copied to: 
/tmp//MarvinLogs/test_host_maintenance_CXM1EL===
++ date
+ echo Mon Mar 28 11:58:36 IST 2016
Mon Mar 28 11:58:36 IST 2016



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