Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1493#discussion_r59671416
--- Diff: utils/src/main/java/com/cloud/utils/nio/Link.java ---
@@ -453,115 +449,150 @@ public static SSLContext initSSLContext(boolean
isClient) throws
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1493#issuecomment-209809491
@jburwell fixed all issues
---
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
Mike, did the iso copy process not complete as expected. Sound like they
are a remanence of some task ending in an exception. Probably a silently
ignored one ;|
On Thu, Apr 14, 2016 at 2:49 AM, Tutkowski, Mike
wrote:
> Just an FYI, but when I kicked off my first VM in this cloud, the VR
> happen
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1460#issuecomment-209822016
so it was a timeout:
```
[INFO] Apache CloudStack Plugin - Quota Service ... SUCCESS [
44.634 s]
[INFO] Apache CloudStack Framework - Spring
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/1491#issuecomment-209830019
@ProjectMoon On latest master during VM deployment no VOLUME.CREATE is
getting generated for ROOT volume. Also during destroy there is no
VOLUME.DELETE event. O
GitHub user nlivens opened a pull request:
https://github.com/apache/cloudstack/pull/1494
Master nuage
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nlivens/cloudstack master_nuage
Alternatively you can review and apply these
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-209856283
@ProjectMoon we'll need more information on why you're doing this, why we
should have it, what is fixes and will it guarantee backward resource-name
compatibility
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1489#issuecomment-209856388
thanks @DaanHoogland
---
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
Github user ProjectMoon commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-209875708
> @ProjectMoon correct resource naming is critical to the proper operation
of the management server. We have had a significant bugs and production issues
caused
Yesterday, we submitted this pull request:
https://github.com/apache/cloudstack/pull/1492
This originally grew out of making the VirtualMachineName class non-static
(original PR is mentioned in the above link). We're presenting this as a
refactoring of the existing code to enable more extensibilit
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-209893539
@nlivens Please add a description because this confuses me ;-) You guys go
the non-oss way? :-(
---
If your project is set up for it, you can reply to this em
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1493#issuecomment-209896417
- Tested against KVM, mgmt server - KVM links and clustered management
server
- NioTest modified to have multiple clients against a server instance with
just o
Github user nlivens commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-209912550
@remibergsma, I've added description :)
---
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 nnesic commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1491#discussion_r59707938
--- Diff: server/src/com/cloud/user/AccountManagerImpl.java ---
@@ -761,6 +774,17 @@ protected boolean cleanupAccount(AccountVO account,
long callerUserId
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1251#issuecomment-209922105
### ACS CI BVT Run
**Sumarry:**
Build Number 175
Hypervisor xenserver
NetworkType Advanced
Passed=34
Failed=21
Skipped=2
Github user nnesic commented on the pull request:
https://github.com/apache/cloudstack/pull/1491#issuecomment-209922376
@koushik-das We are mostly working with 4.7 version, where other volume
usage events get emitted correctly. We haven't investigated the behavior on
master yet.
--
Hi All
I think that we can all agree that CloudStack logs are very difficult to read,
especially for operational staff.
I believe that the primary reason for this is that a large amount of what
should be INFO level events are categorised as DEBUG. The logging therefore
must be set at DEBUG fo
I largely agree with just one sneer to the operators amongst us: the level
can be set on a per package and even class basis. So you can work through
disabling anoying logging step by step. Makes sense to keep looking at the
source in the meanwhile to make sure sensible diagnostics remains available
sounds usefull for companies that for instance want to enforce uuid in name
or include some user string in it, same would be true for networks. look
forward to your design.
On Thu, Apr 14, 2016 at 1:40 PM, Jeff Hair wrote:
> Yesterday, we submitted this pull request:
> https://github.com/apache/
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1492#discussion_r59717124
--- Diff:
plugins/naming-policies/src/com/cloud/naming/DefaultConsoleProxyNamingPolicy.java
---
@@ -0,0 +1,58 @@
+package com.cloud.naming;
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-209945535
@ProjectMoon I will run through the code after I read the detailed design.
one remark so far. This is a nice feature to have. please add integration tests
for
Is there an easy way to do that Daan, or is it a tedious task you just have
to power through?
I agree this initiative would be helpful.
On Apr 14, 2016 9:04 AM, "Daan Hoogland" wrote:
> I largely agree with just one sneer to the operators amongst us: the level
> can be set on a per package and e
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1492#discussion_r59717381
--- Diff: engine/schema/src/com/cloud/naming/ConsoleProxyNamingPolicy.java
---
@@ -0,0 +1,9 @@
+package com.cloud.naming;
--- End diff --
On Thu, Apr 14, 2016 at 3:41 PM, Will Stevens
wrote:
> Is there an easy way to do that Daan, or is it a tedious task you just have
> to power through?
>
It is hard work, mostly tedious, sometimes hilarious and most definitely
devops! It certainly wouldn't classify as development.
--
Da
Do you have to recompile in order to turn off the logging for a specific
package or class? If yes, that is a show stopper for almost everyone...
If it only requires a management server restart, that is more realistic.
*Will STEVENS*
Lead Developer
*CloudOps* *| *Cloud Solutions Experts
420 rue G
> Op 14 april 2016 om 14:49 schreef Paul Angus :
>
>
> Hi All
>
> I think that we can all agree that CloudStack logs are very difficult to read,
> especially for operational staff.
>
> I believe that the primary reason for this is that a large amount of what
> should be INFO level events are c
Grabbing a couple of examples - these should be debug - I can't 'do' anything
with this information.
INFO [o.a.c.f.j.i.AsyncJobManagerImpl] (AsyncJobMgr-Heartbeat-1:ctx-4119d5bc)
Begin cleanup expired async-jobs
INFO [o.a.c.f.j.i.AsyncJobManagerImpl] (AsyncJobMgr-Heartbeat-1:ctx-4119d5bc)
End
Github user ProjectMoon commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-209963503
@DaanHoogland Do you have any kind of integration tests in mind?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
I'm all for this as well.
Training our systems folks on how to work through the logs is a pain. Focusing
in on the most important items is more useful and also makes it easier to
automate log fetching and categorization.
- Si
From: Paul Angus
Sent: Th
Really agree with this one. With a toolchain like ELK, it isn't as bad
because you can filter out the things you don't want, but generally
speaking, the log levels need to be reevaluated. I would also add that many
log statements would benefit from additional context, but that is a whole
different
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59730081
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59730334
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Based on the namespace log level can be set. It needs to be configured in the
log4j.
Check client/tomcatconf/log4j-cloud.xml.in in the source code and you will get
an idea.
-Koushik
From: williamstev...@gmail.com on behalf of Will
Stevens
Sent: Thursd
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59730927
--- Diff: api/src/org/apache/cloudstack/acl/RoleService.java ---
@@ -0,0 +1,43 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+//
GitHub user sverrirab opened a pull request:
https://github.com/apache/cloudstack/pull/1495
Add perl-modules as install dependency for cloudstack-agent
Required to run perl scripts that configure networking for VMs.
You can merge this pull request into a Git repository by running:
I am confused about the configuration file there two files one is named
Log4j-cloud.xml The other is Log4j-cloud.xml.in
What is the difference between these two files ? I hope someone can help me
clear this confusion
Original message
From: Koushik Das
Date: 14/04/2016 6:49 P
The file in a running installation you want is:
/etc/cloudstack/management/ log4j-cloud.xml
It is generally dynamic - you don't have to restart mgmt. server, it just takes
a few seconds to kick in.
If you're looking at the source code, various components use logging -
SystemVMs, KVM agents, Clo
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210009483
This does not compile:
```
[ERROR] COMPILATION ERROR :
[INFO] -
[ERROR]
/data/git/cs1/clou
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210012980
Looking closer at this code. It look like `primaryStoreDriver` could
easily be `null` in this case, so that return statement could potentially cause
a null pointer e
Github user eriweb commented on the pull request:
https://github.com/apache/cloudstack/pull/1495#issuecomment-210016535
Quite horribly that we need those modules for a simple perl script of 58
lines (including license header and empty lines). Not related to your PR
though, but a good
Github user jburwell commented on the pull request:
https://github.com/apache/cloudstack/pull/1495#issuecomment-210018290
@sverrirab how much effort would it be to port to Python? That seems far
preferable for two reasons. First, as @eriweb rightly points out, it is a
large dependen
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1331#issuecomment-210018309
@syed, I totally understand that you do not have much experience with Java
development and that you were following the practices already put in place. The
Are these the names that are sent to HV? For e.g. VM name in the format
i-xx-yy-VM. Currently there are functionalities that rely on the internal
naming convention. All such functionalities needs to be handled appropriately.
-Koushik
From: Jeff Hair
Sen
With this feature, it is possible to change the names that get sent to the
hypervisor, yes. In 4.2 we actually had to fix an issue with the security
groups because they weren't parsing properly. That isn't in the commit yet.
We will put it in.
It would be useful to have a list of all the places th
On Thu, Apr 14, 2016 at 4:14 PM, ProjectMoon wrote:
> Github user ProjectMoon commented on the pull request:
>
> https://github.com/apache/cloudstack/pull/1492#issuecomment-209963503
>
> @DaanHoogland Do you have any kind of integration tests in mind?
>
I could imagine a policy that keep
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-210052430
sorry for not answerring on github
I could imagine a policy that keeps a static counter and name a resource
bla- and then checks if the next created
Github user jburwell commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-210055791
@DaanHoogland -1 to counters -- the require coordination across all nodes
in the cluster. In multi-region configurations, they would require
coordination across m
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-210056555
@jburwell we are talking about integration test code here! not about a
production policy.
---
If your project is set up for it, you can reply to this email an
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1331#discussion_r59756105
--- Diff:
services/secondary-storage/server/test/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResourceTest.java
---
@@ -88,6 +93,19 @@ pub
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210066399
@swill Weird, I've compiled ee53705 with the following command on two
separate machines and I ran regression tests against that SHA for a few hours
yesterday
Github user jburwell commented on the pull request:
https://github.com/apache/cloudstack/pull/1331#issuecomment-210066343
@rafaelweingartner all statics should ``final`` unless there is explicit
handling for their value changing due to thread safety issues. For loggers,
they should al
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210072035
@swill This is pretty strange. The code that is failing to compile for you
is actually not part of the codebase for ee53705:
https://github.com/mike-
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210072697
This might be a better link (as it includes the SHA):
https://github.com/apache/cloudstack/tree/ee5370536ac3f87457d5f74adbbb8c6af88da449/plugins/api
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1331#discussion_r59762357
--- Diff:
services/secondary-storage/server/test/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResourceTest.java
---
@@ -88,6 +93,
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1331#issuecomment-210078579
@jburwell I agree with you that all static can be final. I disagree that
loggers should be static, but that is a matter of taste. I understand the cache
m
Github user jburwell commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-210078861
@DaanHoogland I apologize -- I missed the context. Yes, for an integration
test, a counter would be perfectly acceptable, and likely, the best choice to
ensure de
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1492#issuecomment-210081172
np, imagined that
---
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
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210081474
@swill I just pushed a new commit to rename the SolidFire integration
testing project (which you saw as failing).
While that should not have anything
Github user jburwell commented on the pull request:
https://github.com/apache/cloudstack/pull/1331#issuecomment-210081688
@rafaelweingartner The issue is not one of style but performance and
availability. The garbage collector skips reachability evaluations of ``final
static`` fields
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210084608
@swill I think I might know what happened.
There used to be a ApiSolidFireServiceImpl.java, but I renamed it in 940d5b.
It seems the file is
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210085510
i will blow away my git repo and start from scratch and we will go from
there. :) thx...
---
If your project is set up for it, you can reply to this email and have
Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/1403#issuecomment-210087782
No problem. :)
I had forgotten that I had renamed that file at one point. So, when I saw
that you had a file by that name, I was really confused how
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1331#issuecomment-210086309
I see your point in regards to the garbage collector; but, that class is a
singleton, it is not going to be allocated and deallocated.
About the p
Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59773107
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1493#issuecomment-210103368
I've created two commits to show: (1) test to prove denial of service
behavior due to blocking main IO loop, (2) the fix (as mentioned earlier long
term fix would
Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59773313
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59777518
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59785020
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59786533
--- Diff: api/src/org/apache/cloudstack/acl/Rule.java ---
@@ -0,0 +1,42 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59787000
--- Diff: api/src/org/apache/cloudstack/acl/Rule.java ---
@@ -0,0 +1,42 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59787197
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java
---
@@ -196,5 +204,8 @@ private void validateParams() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59788370
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/acl/CreateRoleCmd.java ---
@@ -0,0 +1,104 @@
+// Licensed to the Apache Software Foundatio
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59788517
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/acl/DeleteRoleCmd.java ---
@@ -0,0 +1,84 @@
+// Licensed to the Apache Software Foundation
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59788566
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/acl/DeleteRolePermissionCmd.java
---
@@ -0,0 +1,84 @@
+// Licensed to the Apache Software
Github user GabrielBrascher commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1493#discussion_r59790778
--- Diff: utils/src/test/java/com/cloud/utils/testcase/NioTest.java ---
@@ -19,146 +19,198 @@
package com.cloud.utils.testcase;
Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1493#discussion_r59791280
--- Diff: utils/src/test/java/com/cloud/utils/testcase/NioTest.java ---
@@ -19,146 +19,198 @@
package com.cloud.utils.testcase;
-imp
Github user swill commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1493#discussion_r59792529
--- Diff: utils/src/test/java/com/cloud/utils/testcase/NioTest.java ---
@@ -19,146 +19,198 @@
package com.cloud.utils.testcase;
-import
Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1493#discussion_r59795416
--- Diff: utils/src/test/java/com/cloud/utils/testcase/NioTest.java ---
@@ -19,146 +19,198 @@
package com.cloud.utils.testcase;
-imp
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59797686
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/acl/ListRolePermissionsCmd.java
---
@@ -0,0 +1,104 @@
+// Licensed to the Apache Software
Github user swill commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1493#discussion_r59799743
--- Diff: utils/src/test/java/com/cloud/utils/testcase/NioTest.java ---
@@ -19,146 +19,198 @@
package com.cloud.utils.testcase;
-import
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59800096
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ---
@@ -117,6 +123,12 @@ public long getEntityOwnerId() {
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59800354
--- Diff: api/test/org/apache/cloudstack/acl/RoleTypeTest.java ---
@@ -0,0 +1,92 @@
+// 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/1489#discussion_r59800553
--- Diff: api/test/org/apache/cloudstack/acl/RoleTypeTest.java ---
@@ -0,0 +1,92 @@
+// 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/1489#discussion_r59800725
--- Diff: api/test/org/apache/cloudstack/acl/RoleTypeTest.java ---
@@ -0,0 +1,92 @@
+// 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/1489#discussion_r59800993
--- Diff: api/test/org/apache/cloudstack/acl/RuleTest.java ---
@@ -0,0 +1,39 @@
+package org.apache.cloudstack.acl;
+
+import com.cloud.excep
Awesome and long awaited
On 4/14/16 4:40 AM, Jeff Hair wrote:
> Yesterday, we submitted this pull request:
> https://github.com/apache/cloudstack/pull/1492
>
> This originally grew out of making the VirtualMachineName class non-static
> (original PR is mentioned in the above link). We're pr
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59801226
--- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade481to490.java ---
@@ -53,6 +62,139 @@ public boolean supportsRollingUpgrade() {
@
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59801354
--- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade481to490.java ---
@@ -53,6 +62,139 @@ public boolean supportsRollingUpgrade() {
@
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59801445
--- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade481to490.java ---
@@ -53,6 +62,139 @@ public boolean supportsRollingUpgrade() {
@
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59801500
--- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade481to490.java ---
@@ -53,6 +62,139 @@ public boolean supportsRollingUpgrade() {
@
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59801544
--- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade481to490.java ---
@@ -53,6 +62,139 @@ public boolean supportsRollingUpgrade() {
@
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59812425
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl;
Hi,
I noticed an issue in Marvin the other day and I tracked it to this commit:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=bbe0fc4be9527d51820b067a602886003991db4d
The problem is that it assumes the "ispublic" parameter will be provided. If it
is not, then an excepti
I'm not sure, Daan.
I plan to keep an eye on this behavior for a while when creating new clouds.
From: Daan Hoogland
Sent: Thursday, April 14, 2016 2:12 AM
To: dev
Subject: Re: Strange XenServer SR behavior when deploying system VMs in Basic
Zone on 4.9
Mike, what type of storage are you using?
> On 15-Apr-2016, at 9:49 AM, Tutkowski, Mike wrote:
>
> I'm not sure, Daan.
>
> I plan to keep an eye on this behavior for a while when creating new clouds.
>
>
> From: Daan Hoogland
> Sent: Thursday, April 14,
95 matches
Mail list logo
