We also noticed this issue.
2016-08-09 19:21 GMT+02:00 Jeff Hair :
> Hi,
>
> I'm looking at the documentation and then the code for stopVirtualMachine.
> The forced parameter is passed down into the VM manager, where it seems to
> be ignored. This means that cleanupEvenIfFailed during VM stop w
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@rhtyd Yes, we could not add the port to SSVMs right now and thus skip the
Agent inside the SSVM.
That way only Instances will get the Socket and we can use it.
We can add it to th
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1541
I pushed a new version of the commits, could @rhtyd and @vincentbernat take
a look?
Yes, Ubuntu 12.04 is dropped, but I did that earlier. See my commits. It
will go EOL in April next year,
Github user vincentbernat commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1541#discussion_r74198886
--- Diff: debian/rules ---
@@ -136,5 +156,8 @@ override_dh_auto_install:
override_dh_installinit:
dh_installinit -pcloudstack-managemen
Github user vincentbernat commented on the issue:
https://github.com/apache/cloudstack/pull/1541
Except the extra `--with systemd`, 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 f
Github user wido commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1541#discussion_r74199703
--- Diff: debian/rules ---
@@ -136,5 +156,8 @@ override_dh_auto_install:
override_dh_installinit:
dh_installinit -pcloudstack-management -pcloud
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1633
CLOUDSTACK-9452: use optparse instead of argparse in patchviasocket
Use optparse that is supported by python 2.3+, instead of argparse that is
supported by python 2.7+. On KVM hosts which don'
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1633
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1633
@karuturi @jburwell @wido @sverrirab please review, it's a blocker for kvm
centos6 hosts without argparse/python2.7 installed.
I could run an existing test: (gave same results before and a
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1633
Do we seriously want to write code that still works on CentOS 6 or systems
without Python 2.7?
Looking at the Python website:
https://docs.python.org/2/library/optparse.html
Depre
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1615
@jburwell the test pipeline had issues, I'll rekick it once I can get the
issues sorted. For now, only package works without any issues. We also
found/fixed a blocker that failed all of our kvm en
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1628
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 feat
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1615
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1615
@blueorangutan package
@jburwell @nvazquez since a new code commit was made, we need to call
package before we could kick in tests.
---
If your project is set up for it, you can reply to
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@wido thanks, @PaulAngus found an issue with the kvm patchviasocket script
and I've sent a fix #1533
I was not sure what was causing the failure and saw this PR which looked
promising. Let's
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1633
@wido I'm sorry we will need to support CentOS6 kvm hosts, just because you
don't use it does not mean we should not support for people. It is a valid
`blocker` as the KVM CentOS6 hosts have stopp
Github user sverrirab commented on the issue:
https://github.com/apache/cloudstack/pull/1633
The updated script that now depends on python 2.7 (or python 2.6 + argparse
I assume) is replacing the perl installation requirement that was there before.
Would it not be the correct solutio
Github user sverrirab commented on the issue:
https://github.com/apache/cloudstack/pull/1633
The reason for my suggestion is that I don't think anyone should be running
python 2.6 in production. The last update for it was in 2008 so I would
consider it risky at this point.
---
If
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1633
CentOS6 default/epel repos don't have python 2.7 :) There are large
installations running old code, old hardware, old kernels/OSs; they will get
upgraded over time, but they need to be supported f
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1633
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1633
Job ID-73
---
If your project is set up for it, you can reply to this email an
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1633
@wido @sverrirab will adding python-argparse as a dependency be an
acceptable solution to the problem? this package is available on el6, but
python 2.7 is not available (without using one of the 3
Github user sverrirab commented on the issue:
https://github.com/apache/cloudstack/pull/1633
adding the python-argparse dependency is fine by me. I have not tested
that combination though.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1615
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1615
Job ID-74
---
If your project is set up for it, you can reply to this email an
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1633
@rhtyd I'm fine with adding it as a dependency.
Perception is different here I think. My personal opinion is that nobody
should be running CentOS 6 with CloudStack in production just as muc
Looks like Nathan has already opened a PR for this:
https://github.com/apache/cloudstack/pull/1632
*Jeff Hair*
Technical Lead and Software Developer
Tel: (+354) 415 0200
j...@greenqloud.com
www.greenqloud.com
On Wed, Aug 10, 2016 at 7:01 AM, Wei ZHOU wrote:
> We also noticed this issue.
>
>
>
Github user nlivens closed the pull request at:
https://github.com/apache/cloudstack/pull/1580
---
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 i
GitHub user nlivens reopened a pull request:
https://github.com/apache/cloudstack/pull/1580
CLOUDSTACK-9402 : Support for underlay features (Source & Static NAT to
underlay) in Nuage VSP plugin
Support for underlay features (Source & Static NAT to underlay) with Nuage
VSP SDN Plugi
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1633
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1633
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1634
[blocker] CLOUDSTACK-9452: add python-argparse dependency on el6,7 rpms
The patchviasocket script was rewritten in Python from PR #1533 and made
assumptions that Python 2.7 would be available.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1634
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user sverrirab commented on the issue:
https://github.com/apache/cloudstack/pull/1634
+1 from me (for what it's worth)
---
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
enab
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1633
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1633
Job ID-75
---
If your project is set up for it, you can reply to this email an
There is a problem in the virtual router in 4.7 where the time to execute
router configuration commands linearly increases. This is because the
router continually executes command updates for instances it no longer
needs to care about, such as ones that were destroyed. The main culprits
are SetStat
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1634
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1634
Job ID-77
---
If your project is set up for it, you can reply to this email an
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1541
@wido sure I'll spend sometime testing them as well. Since this is
packaging only changes, should we get this on 4.9 for the lts /cc @jburwell
@karuturi ?
---
If your project is set up for it, y
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1541
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1541
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1578
Marvin test code PEP8 & PyFlakes compliance:
CloudStack$
CloudStack$ pep8 test/integration/plugins/nuagevsp/*.py
CloudStack$
CloudStack$ pyflakes test/integration/plugins/nu
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1541
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1541
Job ID-78
---
If your project is set up for it, you can reply to this email an
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1541
Build system did not have dh-systemd pkg installed, rekicked build job.
---
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 proj
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1541
@wido can you apply the PR on 4.9/master and fix build? I got following
failures. It would be great if this PR can opened for 4.9. Thanks.
Error from Jenkins build:
dh_install
d
Jeff,
do you mean this PR ? it is already in 4.7 branch.
https://github.com/apache/cloudstack/pull/1356
-Wei
2016-08-10 11:55 GMT+02:00 Jeff Hair :
> There is a problem in the virtual router in 4.7 where the time to execute
> router configuration commands linearly increases. This is because the
Hi,
Most of us have been running marvin tests in virtual environments and it
takes days to run all the tests.
I have filed https://issues.apache.org/jira/browse/CLOUDSTACK-9453 to work on
few enhancements that have the potential to substantially speedup the tests.
I think there are others wil
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1612
Just to note here for future reference: this PR was backported to 4.5 and
4.6 branches as well.
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1580
Marvin test code PEP8 & PyFlakes compliance:
CloudStack$
CloudStack$ pep8 test/integration/plugins/nuagevsp/.py
CloudStack$
CloudStack$ pyflakes test/integration/plugins/nua
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1577
Marvin test code PEP8 & PyFlakes compliance:
CloudStack$
CloudStack$ pep8 test/integration/plugins/nuagevsp/*.py
CloudStack$
CloudStack$ pyflakes test/integration/plugins/nu
Hi,
I did discover that PR while looking, but it's not that one. Our
production system has that PR in it as well, yet we still see the issue
with the virtual routers. Perhaps I am mis-remembering and I need to open a
new bug report?
Jeff
*Jeff Hair*
Technical Lead and Software Developer
Tel: (
Hey guys,
I am new to Cloudstack and Apache (from a dev perspective) so forgive me
if this is not the proper place for this... and let me know where I
should be reporting issues like this.
I am getting Remote Access VPN configured in my test environment and in
my debugging I noticed that if
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74247733
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
---
@@ -768,6 +770,12 @@ public boolean configure
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74248311
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
---
@@ -1978,11 +1986,16 @@ So if getMinSpeed() r
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74248622
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
---
@@ -244,6 +245,7 @@
protected long _
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74248818
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
---
@@ -171,6 +174,25 @@ public boolean parseDomain
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74249185
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
---
@@ -171,6 +174,25 @@ public boolean parseDomain
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74249341
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
---
@@ -171,6 +174,25 @@ public boolean parseDomain
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74249850
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
---
@@ -234,6 +256,10 @@ public Integer getVncPort(
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250104
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250269
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250260
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250288
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250378
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250473
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74250773
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74251286
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1545#discussion_r74251739
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
---
@@ -1209,25 +1209,95 @@ public String toString() {
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1579
Marvin test code PEP8 & PyFlakes compliance:
CloudStack$
CloudStack$ pep8 test/integration/plugins/nuagevsp/*.py
CloudStack$
CloudStack$ pyflakes test/integration/plugins/nu
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@wido @rhtyd given the flakiness of patchviasocket, I think we should
consider including this fix in all supported branches (4.8, 4.9, and master).
The issues I have seen move this bug from a
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1602#discussion_r74271023
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageSubsystemCommandHandler.java
---
@@ -66,19 +68,28 @@ public
VmwareSto
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1634
LGTM for code
---
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, o
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1541
Done @rhtyd , it should build now
---
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
Github user nvazquez commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1602#discussion_r74301008
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageSubsystemCommandHandler.java
---
@@ -66,19 +68,28 @@ public
VmwareSto
+1 (binding)
I have run tests on this using a Basic Zone with both VMware 5.5 (two hosts)
and XenServer 6.5 (two hosts).
The zone was created using the RC with XenServer. All system VMs were running
on local storage.
I added primary storages to the zone that make use of different SolidFire
pl
Thanks Mike. :)
*Will STEVENS*
Lead Developer
*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_
On Wed, Aug 10, 2016 at 2:24 PM, Tutkowski, Mike
wrote:
> +1 (binding)
>
> I have run tests on this using a Basic Zone with bot
Github user swill commented on the issue:
https://github.com/apache/cloudstack/pull/1630
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 fea
+1
We're running this in production with a few cherry picked upstream patches.
Environment: Centos 7, KVM with advanced networking (VPCs) and RBD.
From: williamstev...@gmail.com on behalf of Will
Stevens
Sent: Wednesday, August 10, 2016 1:49 PM
To: dev@cloud
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1541
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74313420
--- Diff: server/test/com/cloud/user/AccountManagerImplTest.java ---
@@ -231,6 +253,73 @@ public void cleanup() {
CallContext.unregister();
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74313401
--- Diff: server/test/com/cloud/user/AccountManagerImplTest.java ---
@@ -231,6 +253,73 @@ public void cleanup() {
CallContext.unregister();
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1541
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74313885
--- Diff: server/test/com/cloud/user/AccountManagerImplTest.java ---
@@ -231,6 +253,73 @@ public void cleanup() {
CallContext.unregister();
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74313796
--- Diff: server/test/com/cloud/user/AccountManagerImplTest.java ---
@@ -231,6 +253,73 @@ public void cleanup() {
CallContext.unregister();
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74313918
--- Diff: server/test/com/cloud/user/AccountManagerImplTest.java ---
@@ -231,6 +253,73 @@ public void cleanup() {
CallContext.unregister();
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314062
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314074
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314037
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314114
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314104
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314127
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314149
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314166
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314190
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314175
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314220
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314203
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314244
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314253
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314233
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314270
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314285
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1624#discussion_r74314305
--- Diff: server/test/com/cloud/user/MockUsageEventDao.java ---
@@ -0,0 +1,356 @@
+// Licensed to the Apache Software Foundation (ASF) under one
1 - 100 of 137 matches
Mail list logo
