Re: [Proposal] Concerning open PRs

[sebgoa]

> On Feb 18, 2016, at 8:38 AM, Wilder Rodrigues  
> wrote:
> 
> Hi all,
> 
> 
> It’s been a long time, but the Wolverine is not dead yet. ;)
> 
> Currently we have 175 opened PRs, which we all agree to be a lot, given the 
> fact that few people, if any, are testing/merging them. I have been a bit off 
> the radar, but from next week I will start helping to get some of those PRs 
> tested and, hopefully, merged.
> 
> In order to get the community working as an unit, I would like to propose the 
> following:
> 
> * One should only create a PR after testing an existing PR.
>  - By testing I mean… testing. Not just looking into it and saying “LGTM”. 
> Manual tests should also count, with screenshots attached to the PR.
> 
> That will make those with test environment pitch in and help, and in addition 
> might also decrease the frenzy for creating PRs which occasionally won’t be 
> tested within a month time - or longer.
> 
> For others not creating PRs that often, like me, we should help testing at 
> least 1 PR per week.
> 
> Being a bit more blunt now, if a PR is created but the person does not 
> contribute with testing an existing one, the new PR should be closed.
> 
> What do you think?
> 

That sounds too extreme to my taste.

Bottom line, folks who submit PR need to stay on top of them and address 
comments.

But then we need to have that CI in place

-sebastien

> Cheers,
> Wilder

[GitHub] cloudstack pull request: CLOUDSTACK-9091: Update testpath for para...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1130#issuecomment-185610104
  
Testcase result after modifications:

Test Volume (root) Snapshot ... === TestName: test_01_volume_snapshot | 
Status : SUCCESS ===
ok

--
Ran 1 test in 1012.564s

OK



---
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: SystemVM cleanups

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1414#issuecomment-185628849
  
thanks for the explanation.
Your PR LGTM



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-9128: Testcase to verify physi...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-9091: Update testpath for para...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-9287 - Fix unique mac address ...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1413#issuecomment-185635109
  
LGTM, based on the tests below. This runs in prod at SBP.

Test results:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
component/test_routers_iptables_default_policy.py \
component/test_routers_network_ops.py \
component/test_vpc_router_nics.py \
smoke/test_loadbalance.py \
smoke/test_internal_lb.py \
smoke/test_ssvm.py \
smoke/test_network.py

```

Result:

```
Check the password file in the Router VM ... === TestName: 
test_isolate_network_password_server | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : 
SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and 
check default routes ... === TestName: test_02_redundant_VPC_default_routes | 
Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: 
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : 
SUCCESS ===
ok
Create a redundant VPC with 1 Tier, 1 VM, 1 ACL, 1 PF and test Network GC 
Nics ... === TestName: test_04_rvpc_network_garbage_collector_nics | Status : 
SUCCESS ===
ok
Create a redundant VPC with 1 Tier, 1 VM, 1 ACL, 1 PF and test Network GC 
Nics ... === TestName: test_05_rvpc_multi_tiers | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: 
test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === 
TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_01_isolate_network_FW_PF_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_02_isolate_network_FW_PF_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_03_RVR_Network_check_router_state | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics 
after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : 
SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default 
routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === 
TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: 
test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: 
test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: 
test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic 
to 3 vm's in a Single VPC ... === TestName: 
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic 
to 3 vm's in a Redundant VPC ... === TestName: 
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === 
TestName: test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Status : 
SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === 
TestName: test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Status : 
SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : 
SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : 
SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS 
===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS 
===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS 
===
ok
Test destroy CPVM ... === TestName: test_10_destroy_

Re: [update] ACS management unable to connect to xenserver hosts after reboot

[Stephan Seitz]

Paul,

thank you for your hint! That was the root cause of our problems:

https://bugs.launchpad.net/ubuntu/+source/ifenslave/+bug/1288196

We simply just didn't know that the msid is derived from the MAC.

Our services tend to be manageable again ;)

Thanks again guys!

cheers,

- Stephan


Am Mittwoch, den 17.02.2016, 19:16 + schrieb Paul Angus: 
> The msid is generated from the MAC address of the host when the service 
> starts, the two IDs are subtly different do you have some bonding in place 
> that is maybe miss-configured, which is generating the 2nd MAC?
> 
> 
> 
> Paul Angus
> VP Technology   ,   ShapeBlue



> 
> 
> t:  @cloudyangus
> 
> e:  paul.an...@shapeblue.com|
>   w:  www.shapeblue.com
> 
> 
> 
> 
> 
> -Original Message-
> From: Simon Weller [mailto:swel...@ena.com]
> Sent: Wednesday, February 17, 2016 6:11 PM
> To: dev@cloudstack.apache.org
> Cc: Glenn Wagner 
> Subject: Re: [update] ACS management unable to connect to xenserver hosts 
> after reboot
> 
> Stephan,
> 
> When you restart the management process, do you see any logs indicating it's 
> trying to peer with another management server?
> 
> - Si
> 
> 
> From: Stephan Seitz 
> Sent: Wednesday, February 17, 2016 9:28 AM
> To: dev@cloudstack.apache.org
> Cc: Glenn Wagner
> Subject: Re: [update] ACS management unable to connect to xenserver hosts 
> after reboot
> 
> Glenn,
> 
> thanks for your reply. Unfortunately the SSVM has been destroyed.
> 
> We don't have any firewall in between. ACS and XenServers are located in the 
> same /22. I've double checked every connection and there's no iptables or 
> similar in the way.
> Instead of the SSVM, I've just successfully checked if the consoleproxy VM is 
> able to connect to Port 8250.
> 
> To me it looks, like there's some strange "identity" problem.
> 
> mysql> select * from mshost;
> +++---+--+---+-++--+-+-+-+
> | id | msid | runid | name | state |
> version | service_ip | service_port | last_update | removed |
> alert_count |
> +++---+--+---+-++--+-+-+-+
> | 1 | 57177340185274 | 1455209855143 | acs-management-1 | Up | 4.7.1
> | 10.97.13.1 | 9090 | 2016-02-12 16:55:56 | NULL |
> 0 |
> | 3 | 57177340185273 | 1455639355379 | acs-management-1 | Up | 4.7.1
> | 10.97.13.1 | 9090 | 2016-02-17 11:31:50 | NULL |
> 0 |
> +++---+--+---+-++--+-+-+-+
> 2 rows in set (0.00 sec)
> 
> Indeed, there is (and always has been) only one management host in this 
> infrastructure.
> 
> With sqldumps at hand, we removed the second row and purged all the related 
> jobs to that id, but after restarting cloudstack-management, this entry wasi 
> created again.
> 
> Maybe, I'm completely wrong, but is it possible that our management host 
> "thinks" there's another management host responsible for our cluster?
> 
> Since we're fiddling at least two days without any success here, I'm willing 
> to get a few consulting hours thrown on that.
> 
> cheers,
> 
> - Stephan
> 
> btw. sorry, if this is a double post, but I think the list ate my last mail...
> 
> 
> Am Dienstag, den 16.02.2016, 20:39 + schrieb Glenn Wagner:
> > Hi Stephan,
> >
> > Check that you can telnet port 8250 on the management server from SSVM
> > , check that iptables has been setup correctly Looks like it’s a
> > firewall issue on the ACS Management server
> >
> > Thanks
> > Glenn
> >
> >
> >
> >
> >
> > ShapeBlue
> > Glenn Wagner
> > Senior
> > Consultant
> > ,
> > ShapeBlue
> > d:
> > | s: +27 21 527 0091
> > |
> > m:
> > +27 73 917 4111
> > e:
> > glenn.wag...@shapeblue.com | t:
> > |
> > w:
> > www.shapeblue.com
> > a:
> > 2nd Floor, Oudehuis Centre, 122 Main Rd, Somerset West Cape Town 7130
> > South Africa
> >
> > Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> > Services India LLP is a company incorporated in India and is operated
> > under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
> > is a company incorporated in Brasil and is operated under license from
> > Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The
> > Republic of South Africa and is traded under license from Shape Blue
> > Ltd. ShapeBlue is a registered trademark.
> > This email and any attachments to it may be confidential and are
> > intended solely for the use of the individual to whom it is addressed.
> > Any views or opinions expressed are solely those of the author and do
> > not necessarily represent those of Shape Blue Ltd or related
> > companies. If you are not the intended recipient of this email, 

Re: [Proposal] Concerning open PRs

[Jeff Hair]

I think that requiring everyone to test a pull request before opening a new
one will just grind development to a halt. Although it has to be done,
nobody likes testing. Automated testing relieves a lot of this, and many of
the pull requests are involved enough to require extensive
manual/integration testing by themselves.

On Thu, Feb 18, 2016 at 7:38 AM, Wilder Rodrigues <
wrodrig...@schubergphilis.com> wrote:

> Hi all,
>
>
> It’s been a long time, but the Wolverine is not dead yet. ;)
>
> Currently we have 175 opened PRs, which we all agree to be a lot, given
> the fact that few people, if any, are testing/merging them. I have been a
> bit off the radar, but from next week I will start helping to get some of
> those PRs tested and, hopefully, merged.
>
> In order to get the community working as an unit, I would like to propose
> the following:
>
> * One should only create a PR after testing an existing PR.
>   - By testing I mean… testing. Not just looking into it and saying
> “LGTM”. Manual tests should also count, with screenshots attached to the PR.
>
> That will make those with test environment pitch in and help, and in
> addition might also decrease the frenzy for creating PRs which occasionally
> won’t be tested within a month time - or longer.
>
> For others not creating PRs that often, like me, we should help testing at
> least 1 PR per week.
>
> Being a bit more blunt now, if a PR is created but the person does not
> contribute with testing an existing one, the new PR should be closed.
>
> What do you think?
>
> Cheers,
> Wilder




-- 
*Jeff Hair*
Technical Lead and Software Developer

Tel: (+354) 415 0200
j...@greenqloud.com
www.greenqloud.com

[GitHub] cloudstack pull request: Refactor system VM default network creati...

[ProjectMoon]

Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/1360#issuecomment-185645919
  
PR rebased against latest 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 pull request: Followup fix for #1162: Add support for n...

[ProjectMoon]

Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/1330#issuecomment-185646057
  
PR updated to latest 4.6.


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

Controlling Load Order of Modules

[Jeff Hair]

Hi,

We've run across an interesting case where we have the need to load a
module before other modules. Normally this kind of thing is handled by
putting the module up higher in the hierarchy. Our problem is that this
module is part of core, and to make it load first we'd have to put it above
core.

Is there any established method for controlling the load order of
sub-modules within a module? Currently they are sorted by a TreeMap using
natural sorting. We've implemented a comparator to force our module to the
top of the list, but this seems like a hack (though it's much better than
giving our module the name "aaa-ourmodule".)

Jeff

Re: template available on http://dl.openvm.eu/cloudstack/macchinina/x86_64/

[Sanjeev N]

Hi Nux,

Can you please convert this template to .ova format , so that tests can run
on vmware.

http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.vmdk.bz2

Thanks,
Sanjeev

On Mon, Jan 4, 2016 at 5:42 PM, Raja Pullela 
wrote:

> Nux, I will provide you the files and if you can upload them that will be
> great.  Hope that is possible,
>
> Raja
>
> -Original Message-
> From: Nux! [mailto:n...@li.nux.ro]
> Sent: Saturday, January 2, 2016 4:08 PM
> To: dev@cloudstack.apache.org
> Subject: Re: template available on
> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>
> Raja,
>
> Do you have access to the sysvm tmpl generation? What commands are used to
> create OVA format?
> If you use the uncompressed RAW file, does that help you? It's already
> there http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina.img
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
> > From: "Raja Pullela" 
> > To: dev@cloudstack.apache.org
> > Sent: Saturday, 2 January, 2016 05:33:41
> > Subject: RE: template available on
> > http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>
> > Nux, you are right... it is working for the system but not for the test
> suite.
> > I am going to create a bug to track it.
> > However, it will be great if you could post a BZ2 templates for HyperV
> > and for VMWare a OVA format - that will help!
> >
> > Thanks,
> > Raja
> >
> > -Original Message-
> > From: Nux! [mailto:n...@li.nux.ro]
> > Sent: Friday, January 1, 2016 5:36 PM
> > To: dev@cloudstack.apache.org
> > Subject: Re: template available on
> > http://dl.openvm.eu/cloudstack/macchinina/x86_64/
> >
> > Raja,
> >
> > This is confusing me. I am looking here and the VHD is ZIP:
> > http://jenkins.buildacloud.org/view/master/job/build-systemvm64-master
> > /
> >
> > I don't mind bz2-ing the VHD, but it's not clear what ACS/HyperV expects.
> >
> > --
> > Sent from the Delta quadrant using Borg technology!
> >
> > Nux!
> > www.nux.ro
> >
> > - Original Message -
> >> From: "Raja Pullela" 
> >> To: dev@cloudstack.apache.org
> >> Sent: Friday, 1 January, 2016 11:12:12
> >> Subject: RE: template available on
> >> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
> >
> >> Hi Nux,
> >>
> >> these templates available are not working...
> >> I see the following System Templates available for -
> >> - VMWare is in OVA format
> >> - HyperV is in VZD with BZ2 format
> >> - Xen is in VHD with BZ2 compression
> >> - KVM is in qCow2 with BZ2 compression
> >>
> >> If you can give me access to this VM/webserver, I can upload the files.
> >> Having the correct files will help pass few additional tests on my
> setups.
> >> Can you please let me know,
> >>
> >> Raja
> >>
> >> -Original Message-
> >> From: Nux! [mailto:n...@li.nux.ro]
> >> Sent: Tuesday, December 22, 2015 2:19 PM
> >> To: dev@cloudstack.apache.org
> >> Subject: Re: template available on
> >> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
> >>
> >> BTW, Cloudstack ships systemvm templates for HyperV also in ZIP,
> >> that's the reason I also pack it like that.
> >> AFAIK Windows can't deal with any other compression.
> >>
> >> --
> >> Sent from the Delta quadrant using Borg technology!
> >>
> >> Nux!
> >> www.nux.ro
> >>
> >> - Original Message -
> >>> From: "Nux!" 
> >>> To: dev@cloudstack.apache.org
> >>> Sent: Tuesday, 22 December, 2015 08:44:50
> >>> Subject: Re: template available on
> >>> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
> >>
> >>> Hi,
> >>>
> >>> I do not know what you are running there, but:
> >>>
> >>> unzip macchinina-hyperv.vhd.zip
> >>> Archive:  macchinina-hyperv.vhd.zip
> >>>  inflating: macchinina-hyperv.vhd
> >>>
> >>>
> >>> qemu-img info macchinina-hyperv.vhd
> >>> image: macchinina-hyperv.vhd
> >>> file format: vpc
> >>> virtual size: 50M (52432896 bytes)
> >>> disk size: 50M
> >>> cluster_size: 2097152
> >>>
> >>>
> >>> Also, VHD is completely untested, let me know if it boots for you.
> >>> :)
> >>>
> >>> --
> >>> Sent from the Delta quadrant using Borg technology!
> >>>
> >>> Nux!
> >>> www.nux.ro
> >>>
> >>> - Original Message -
>  From: "Raja Pullela" 
>  To: dev@cloudstack.apache.org
>  Sent: Tuesday, 22 December, 2015 04:45:29
>  Subject: RE: template available on
>  http://dl.openvm.eu/cloudstack/macchinina/x86_64/
> >>>
>  Hello Nux,
> 
>  Three test scripts are failing with the error messages as below -
>  File "/root/cloudstack/test/integration/smoke/test_internal_lb.py",
>  line 285, in  setUpClass
> cls.template.download(cls.apiclient)
>   File "/usr/local/lib/python2.7/dist-packages/marvin/lib/base.py",
>  line 1261, in  download
> template.status)
>  'Error in downloading template: status - Failed post download script:
>  Unable to install due to invalid file format\n
> 
>  Hope that helps,
>  Raja
> 
>  -Original Message--

[GitHub] cloudstack pull request: CLOUDSTACK-9026: Modifying testpath for a...

[sanju1010]

Github user sanju1010 commented on the pull request:

https://github.com/apache/cloudstack/pull/1031#issuecomment-185662747
  
LGTM  based on test results!!

Test migrate Volume (root and data disk) ... === TestName: 
test_01_migrate_root_and_data_disk_nonlive | Status : SUCCESS ===
ok

--
Ran 1 test in 1778.920s

OK



---
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-9026: Modifying testpath for a...

[nitt10prashant]

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

https://github.com/apache/cloudstack/pull/1031#discussion_r53300709
  
--- Diff: test/integration/testpaths/testpath_storage_migration.py ---
@@ -248,6 +248,11 @@ def setUpClass(cls):
 DomainName=cls.account.domain
 )
 # Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"]
+)
+
--- End diff --

please add cls._cleanup.append(cls.service_offering)  


---
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: Add lsb-release dependency to mgmt server...

[ProjectMoon]

Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/1412#issuecomment-185670520
  
Apparently the build of this pull request sucked Jenkins into a black hole. 
The build returns 404.


---
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: Followup fix for #1162: Add support for n...

[ProjectMoon]

Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/1330#issuecomment-185671218
  
Also, forgot I was making a test case for this. Will update my agenda to 
get this done.


---
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: template available on http://dl.openvm.eu/cloudstack/macchinina/x86_64/

[Nux!]

Done

http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.ova

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Sanjeev N" 
> To: dev@cloudstack.apache.org
> Sent: Thursday, 18 February, 2016 11:07:32
> Subject: Re: template available on 
> http://dl.openvm.eu/cloudstack/macchinina/x86_64/

> Hi Nux,
> 
> Can you please convert this template to .ova format , so that tests can run
> on vmware.
> 
> http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.vmdk.bz2
> 
> Thanks,
> Sanjeev
> 
> On Mon, Jan 4, 2016 at 5:42 PM, Raja Pullela 
> wrote:
> 
>> Nux, I will provide you the files and if you can upload them that will be
>> great.  Hope that is possible,
>>
>> Raja
>>
>> -Original Message-
>> From: Nux! [mailto:n...@li.nux.ro]
>> Sent: Saturday, January 2, 2016 4:08 PM
>> To: dev@cloudstack.apache.org
>> Subject: Re: template available on
>> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>>
>> Raja,
>>
>> Do you have access to the sysvm tmpl generation? What commands are used to
>> create OVA format?
>> If you use the uncompressed RAW file, does that help you? It's already
>> there http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina.img
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro
>>
>> - Original Message -
>> > From: "Raja Pullela" 
>> > To: dev@cloudstack.apache.org
>> > Sent: Saturday, 2 January, 2016 05:33:41
>> > Subject: RE: template available on
>> > http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>>
>> > Nux, you are right... it is working for the system but not for the test
>> suite.
>> > I am going to create a bug to track it.
>> > However, it will be great if you could post a BZ2 templates for HyperV
>> > and for VMWare a OVA format - that will help!
>> >
>> > Thanks,
>> > Raja
>> >
>> > -Original Message-
>> > From: Nux! [mailto:n...@li.nux.ro]
>> > Sent: Friday, January 1, 2016 5:36 PM
>> > To: dev@cloudstack.apache.org
>> > Subject: Re: template available on
>> > http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>> >
>> > Raja,
>> >
>> > This is confusing me. I am looking here and the VHD is ZIP:
>> > http://jenkins.buildacloud.org/view/master/job/build-systemvm64-master
>> > /
>> >
>> > I don't mind bz2-ing the VHD, but it's not clear what ACS/HyperV expects.
>> >
>> > --
>> > Sent from the Delta quadrant using Borg technology!
>> >
>> > Nux!
>> > www.nux.ro
>> >
>> > - Original Message -
>> >> From: "Raja Pullela" 
>> >> To: dev@cloudstack.apache.org
>> >> Sent: Friday, 1 January, 2016 11:12:12
>> >> Subject: RE: template available on
>> >> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>> >
>> >> Hi Nux,
>> >>
>> >> these templates available are not working...
>> >> I see the following System Templates available for -
>> >> - VMWare is in OVA format
>> >> - HyperV is in VZD with BZ2 format
>> >> - Xen is in VHD with BZ2 compression
>> >> - KVM is in qCow2 with BZ2 compression
>> >>
>> >> If you can give me access to this VM/webserver, I can upload the files.
>> >> Having the correct files will help pass few additional tests on my
>> setups.
>> >> Can you please let me know,
>> >>
>> >> Raja
>> >>
>> >> -Original Message-
>> >> From: Nux! [mailto:n...@li.nux.ro]
>> >> Sent: Tuesday, December 22, 2015 2:19 PM
>> >> To: dev@cloudstack.apache.org
>> >> Subject: Re: template available on
>> >> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>> >>
>> >> BTW, Cloudstack ships systemvm templates for HyperV also in ZIP,
>> >> that's the reason I also pack it like that.
>> >> AFAIK Windows can't deal with any other compression.
>> >>
>> >> --
>> >> Sent from the Delta quadrant using Borg technology!
>> >>
>> >> Nux!
>> >> www.nux.ro
>> >>
>> >> - Original Message -
>> >>> From: "Nux!" 
>> >>> To: dev@cloudstack.apache.org
>> >>> Sent: Tuesday, 22 December, 2015 08:44:50
>> >>> Subject: Re: template available on
>> >>> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>> >>
>> >>> Hi,
>> >>>
>> >>> I do not know what you are running there, but:
>> >>>
>> >>> unzip macchinina-hyperv.vhd.zip
>> >>> Archive:  macchinina-hyperv.vhd.zip
>> >>>  inflating: macchinina-hyperv.vhd
>> >>>
>> >>>
>> >>> qemu-img info macchinina-hyperv.vhd
>> >>> image: macchinina-hyperv.vhd
>> >>> file format: vpc
>> >>> virtual size: 50M (52432896 bytes)
>> >>> disk size: 50M
>> >>> cluster_size: 2097152
>> >>>
>> >>>
>> >>> Also, VHD is completely untested, let me know if it boots for you.
>> >>> :)
>> >>>
>> >>> --
>> >>> Sent from the Delta quadrant using Borg technology!
>> >>>
>> >>> Nux!
>> >>> www.nux.ro
>> >>>
>> >>> - Original Message -
>>  From: "Raja Pullela" 
>>  To: dev@cloudstack.apache.org
>>  Sent: Tuesday, 22 December, 2015 04:45:29
>>  Subject: RE: template available on
>>  http://dl.openvm.eu/cloudstack/macchinina/x86_64/
>> >>>
>>  Hello Nux,
>> 
>>  Three test scripts are 

[GitHub] cloudstack pull request: CLOUDSTACK-9026: Modifying testpath for a...

[pritisarap12]

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

https://github.com/apache/cloudstack/pull/1031#discussion_r53309123
  
--- Diff: test/integration/testpaths/testpath_storage_migration.py ---
@@ -248,6 +248,11 @@ def setUpClass(cls):
 DomainName=cls.account.domain
 )
 # Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"]
+)
+
--- End diff --

Added service_offering in cleanup.


---
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: Controlling Load Order of Modules

[Ron Wheeler]

Does this not depend on the classloader rather than the package being 
loaded?

http://www.javaworld.com/article/2077260/learn-java/learn-java-the-basics-of-java-class-loaders.html

It seems that any solution that take advantage of the "current" 
classloader will be fragile and subject to sudden failure if the JVM 
updates the classloader to one with a different strategy.


Can you not write your own classloader that forces the order that you want?

Never tried any of this so take these suggestions as being worth no more 
than they cost!


Ron

On 18/02/2016 5:47 AM, Jeff Hair wrote:

Hi,

We've run across an interesting case where we have the need to load a
module before other modules. Normally this kind of thing is handled by
putting the module up higher in the hierarchy. Our problem is that this
module is part of core, and to make it load first we'd have to put it above
core.

Is there any established method for controlling the load order of
sub-modules within a module? Currently they are sorted by a TreeMap using
natural sorting. We've implemented a comparator to force our module to the
top of the list, but this seems like a hack (though it's much better than
giving our module the name "aaa-ourmodule".)

Jeff




--
Ron Wheeler
President
Artifact Software Inc
email: rwhee...@artifact-software.com
skype: ronaldmwheeler
phone: 866-970-2435, ext 102

Re: Controlling Load Order of Modules

[Jeff Hair]

This is above the classloader level. It's at CloudStack's module level,
which is an abstraction built on top of Spring's dependency injection. The
relevant stuff can be found
in 
framework/spring/module/src/main/java/org/apache/cloudstack/spring/module/model/impl/.

On Thu, Feb 18, 2016 at 2:01 PM, Ron Wheeler  wrote:

> Does this not depend on the classloader rather than the package being
> loaded?
>
> http://www.javaworld.com/article/2077260/learn-java/learn-java-the-basics-of-java-class-loaders.html
>
> It seems that any solution that take advantage of the "current"
> classloader will be fragile and subject to sudden failure if the JVM
> updates the classloader to one with a different strategy.
>
> Can you not write your own classloader that forces the order that you want?
>
> Never tried any of this so take these suggestions as being worth no more
> than they cost!
>
> Ron
>
>
> On 18/02/2016 5:47 AM, Jeff Hair wrote:
>
>> Hi,
>>
>> We've run across an interesting case where we have the need to load a
>> module before other modules. Normally this kind of thing is handled by
>> putting the module up higher in the hierarchy. Our problem is that this
>> module is part of core, and to make it load first we'd have to put it
>> above
>> core.
>>
>> Is there any established method for controlling the load order of
>> sub-modules within a module? Currently they are sorted by a TreeMap using
>> natural sorting. We've implemented a comparator to force our module to the
>> top of the list, but this seems like a hack (though it's much better than
>> giving our module the name "aaa-ourmodule".)
>>
>> Jeff
>>
>>
>
> --
> Ron Wheeler
> President
> Artifact Software Inc
> email: rwhee...@artifact-software.com
> skype: ronaldmwheeler
> phone: 866-970-2435, ext 102
>
>


-- 
*Jeff Hair*
Technical Lead and Software Developer

Tel: (+354) 415 0200
j...@greenqloud.com
www.greenqloud.com

[GitHub] cloudstack pull request: Add lsb-release dependency to mgmt server...

[ProjectMoon]

Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/1412#issuecomment-185756432
  
I've updated this pull request @bhaisaab. The dependency needs to be added 
to cloudstack-agent as well due to a similar problem with the 
cloudstack-setup-agent script (they both depend on the same Python libraries).


---
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: Usage event fixes for deleted accounts

[nnesic]

Github user nnesic closed the pull request at:

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


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

Build failed in Jenkins: build-master-slowbuild #3267

[jenkins]

See 

--
[...truncated 824 lines...]
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
2016-02-18 23:13:33,181 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c /not/existing/scripts/1455837213181 
2016-02-18 23:13:33,184 DEBUG [utils.script.Script] (main:) Exit value is 127
2016-02-18 23:13:33,184 DEBUG [utils.script.Script] (main:) /bin/bash: 
/not/existing/scripts/1455837213181: No such file or directory
2016-02-18 23:13:33,185 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c echo 'hello world!' 
2016-02-18 23:13:33,188 DEBUG [utils.script.Script] (main:) Execution is 
successful.
2016-02-18 23:13:33,189 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c echo 'hello world!' 
2016-02-18 23:13:33,192 DEBUG [utils.script.Script] (main:) Execution is 
successful.
Tests run: 10, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.048 sec - in 
com.cloud.utils.ScriptTest
Running com.cloud.utils.TestProfiler
Configure log4j with default properties
2016-02-18 23:13:33,219 INFO  [cloud.utils.TestProfiler] (main:) testProfiler() 
started
2016-02-18 23:13:34,230 INFO  [cloud.utils.TestProfiler] (main:) Duration in 
Millis: 1010
2016-02-18 23:13:34,230 INFO  [cloud.utils.TestProfiler] (main:) testProfiler() 
stopped
Configure log4j with default properties
Configure log4j with default properties
2016-02-18 23:13:35,262 INFO  [cloud.utils.TestProfiler] (main:) Duration in 
Nano: 1010154999
Configure log4j with default properties
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.068 sec - in 
com.cloud.utils.TestProfiler
Running com.cloud.utils.net.NetUtilsTest
2016-02-18 23:13:35,306 INFO  [utils.net.NetUtils] (main:) Invalid value of 
cidr 10.3.6.5/50
2016-02-18 23:13:35,317 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::2
2016-02-18 23:13:35,318 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::2
2016-02-18 23:13:35,320 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::1
2016-02-18 23:13:35,322 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::2
2016-02-18 23:13:35,323 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::2
2016-02-18 23:13:35,325 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::9684:cabf:5c18:bfcd
2016-02-18 23:13:35,327 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::aa6c:7be6:ac41:434a
2016-02-18 23:13:35,329 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::8d90:562b:3c4c:1536
2016-02-18 23:13:35,330 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::78b:652d:ddac:a23b
2016-02-18 23:13:35,332 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::77d8:a1c8:7dce:4ded
2016-02-18 23:13:35,333 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::4ff1:f380:a7dd:64d9
2016-02-18 23:13:35,335 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::29cc:90b:39f3:97c
2016-02-18 23:13:35,336 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::9754:735e:c6da:3313
2016-02-18 23:13:35,338 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::dcfe:de35:4c4b:3b58
2016-02-18 23:13:35,340 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::29ce:831b:212d:daae
2016-02-18 23:13:35,357 ERROR [utils.net.NetUtils] (main:) empty cidr can not 
be converted to longs
com.cloud.utils.exception.CloudRuntimeException: empty cidr can not be 
converted to longs
at com.cloud.utils.net.NetUtils.cidrToLong(NetUtils.java:887)
at com.cloud.utils.net.NetUtils.isNetworksOverlap(NetUtils.java:1174)
at 
com.cloud.utils.net.NetUtilsTest.testIsNetworksOverlapWithEmptyValues(NetUtilsTest.java:509)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
at org.junit.runners.ParentRunner.runChildren(ParentRunn