[GitHub] cloudstack pull request: CID-1338387: handle unknown storage endpo...

[miguelaferreira]

Github user miguelaferreira commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155703549
  
I don't think so @DaanHoogland. But if I've miss interpreted what you said, 
I'll be happy to have you explain to me what that is.


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

[GitHub] cloudstack pull request: Cwe 190

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-155710538
  
I was just re-checking the code, what about creating a test case for 
"netMaskFromCidr" method?


---
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: Cwe 190

[wilderrodrigues]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44515608
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -869,31 +878,44 @@ public static boolean isNetworkAWithinNetworkB(final 
String cidrA, final String
 
 public static Long[] cidrToLong(final String cidr) {
 if (cidr == null || cidr.isEmpty()) {
-return null;
+throw new CloudRuntimeException("empty cidr can not be 
converted to longs");
 }
 final String[] cidrPair = cidr.split("\\/");
 if (cidrPair.length != 2) {
-return null;
+throw new CloudRuntimeException("cidr is not formatted 
correctly: "+ cidr);
 }
 final String cidrAddress = cidrPair[0];
 final String cidrSize = cidrPair[1];
 if (!isValidIp(cidrAddress)) {
-return null;
-}
-int cidrSizeNum = -1;
-
-try {
-cidrSizeNum = Integer.parseInt(cidrSize);
-} catch (final Exception e) {
-return null;
+throw new CloudRuntimeException("cidr is not bvalid in ip 
space" + cidr);
 }
-final long numericNetmask = 0x >> MAX_CIDR - cidrSizeNum 
<< MAX_CIDR - cidrSizeNum;
+long cidrSizeNum = getCidrSizeFromString(cidrSize);
+final long numericNetmask = netMaskFromCidr(cidrSizeNum);
 final long ipAddr = ip2Long(cidrAddress);
 final Long[] cidrlong = {ipAddr & numericNetmask, 
(long)cidrSizeNum};
 return cidrlong;
 
 }
 
+/**
+ * @param cidrSize
+ * @return
+ * @throws CloudRuntimeException
+ */
+static long getCidrSizeFromString(final String cidrSize) throws 
CloudRuntimeException {
+long cidrSizeNum = -1;
+
+try {
+cidrSizeNum = Integer.parseInt(cidrSize);
+} catch (final NumberFormatException e) {
+throw new CloudRuntimeException("cidrsize is not a valid int: 
" + cidrSize, e);
+}
+if(cidrSizeNum > 32 || cidrSizeNum < 0) {// assuming IPv4
+throw new CloudRuntimeException("cidr size out of range: " + 
cidrSizeNum);
+}
+return cidrSizeNum;
+}
+
--- End diff --

Same goes for this one. package (no modifier) vs private.


---
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: Cwe 190

[wilderrodrigues]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44515580
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -775,11 +776,19 @@ public static String getSubNet(final String ip, final 
String netmask) {
 }
 
 public static String getCidrSubNet(final String ip, final long 
cidrSize) {
-final long numericNetmask = 0x >> MAX_CIDR - cidrSize << 
MAX_CIDR - cidrSize;
+final long numericNetmask = netMaskFromCidr(cidrSize);
 final String netmask = NetUtils.long2Ip(numericNetmask);
 return getSubNet(ip, netmask);
 }
 
+/**
+ * @param cidrSize
+ * @return
+ */
+static long netMaskFromCidr(final long cidrSize) {
+return ((long)0x) >> MAX_CIDR - cidrSize << MAX_CIDR - 
cidrSize;
+}
+
--- End diff --

Why are tho static methods using "package" modifier? Are classes in the 
same package going to access it? If the answer is no, they can be made 
"private".


---
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: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155703357
  
@miguelaferreira you are assigning claims to me that I did not make.


---
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-9054 use of google-optional as...

[DaanHoogland]

GitHub user DaanHoogland opened a pull request:

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

CLOUDSTACK-9054 use of google-optional as PoC

this is using guave optional as suggested by @miguelaferreira in #1056. I 
am not convinced it is appropriate for that case (as opposed to throwing an 
exception) but we are going to use Optionals in someway or another. I familiar 
with the concept (except for in rust) and want to see if the 1.8 is 
different/better/worse as well.

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

$ git pull https://github.com/DaanHoogland/cloudstack CLOUDSTACK-9054

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

https://github.com/apache/cloudstack/pull/1060.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1060


commit c4dbcb261811c6ce0578f93e3cbd6a3ac4521175
Author: Daan Hoogland 
Date:   2015-11-11T11:14:09Z

CLOUDSTACK-9054 use of google-optional as PoC




---
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-9055: fix NPE in updating Redu...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

CLOUDSTACK-9055: fix NPE in updating Redundant State of VPC networks



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

$ git pull https://github.com/ustcweizhou/cloudstack CLOUDSTACK-9055

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

https://github.com/apache/cloudstack/pull/1059.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1059


commit ac4173d4cca62a6f50c8de674417fc9bfb101775
Author: Wei Zhou 
Date:   2015-11-11T11:18:54Z

CLOUDSTACK-9055: fix NPE in updating Redundant State of VPC networks




---
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-9055: fix NPE in updating Redu...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1059#issuecomment-155740109
  
@ustcweizhou 

Please, steps to reproduce/test your PR. Also, write a bit about how you 
got into this issue in the PR description. In addition, which integration tests 
did you execute?

I don't think it would that hard to elaborate a bit on a PR and give some 
hints to people that will end up testing it.

Cheers,
Wilder


---
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: Cwe 190

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-155708663
  
@rafaelweingartner sure, they're not needed but I'll add them for clarity.


---
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: Shuffling the password to avoid having a ...

[wido]

Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1058#issuecomment-155702537
  
LGTM to me. Simple fix to make passwords more secure


---
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: CID-1338387: handle unknown storage endpo...

[miguelaferreira]

Github user miguelaferreira commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155702506
  
@DaanHoogland both Optional and Exception will give the opportunity to 
handle the "null case", the difference is in the semantics. Optional means that 
you expect to either have or not have a value. Exception means that no value is 
an error state/case. So you should use one or the other depending on what does 
it mean semantically to not have a value. Newbies or otherwise will understand 
this.

Regarding your point about different solutions in different places, I would 
say that it argues for stagnation. You present two options: either apply a 
better solution to all places, or keep replication the current solution. I 
propose a different approach. And approach that will introduce a better 
solution incrementally, one (or a few) places at a time. I don't see any 
problem with this, since we actually keep history everyone can see that one 
change happened in 2010 and the other in 2015. Then people can ask about it, or 
better yet we can tell them upfront.

We just have to be willing to do the best we can at all times.


---
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: Cwe 190

[wilderrodrigues]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44519815
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -869,31 +878,44 @@ public static boolean isNetworkAWithinNetworkB(final 
String cidrA, final String
 
 public static Long[] cidrToLong(final String cidr) {
 if (cidr == null || cidr.isEmpty()) {
-return null;
+throw new CloudRuntimeException("empty cidr can not be 
converted to longs");
 }
 final String[] cidrPair = cidr.split("\\/");
 if (cidrPair.length != 2) {
-return null;
+throw new CloudRuntimeException("cidr is not formatted 
correctly: "+ cidr);
 }
 final String cidrAddress = cidrPair[0];
 final String cidrSize = cidrPair[1];
 if (!isValidIp(cidrAddress)) {
-return null;
-}
-int cidrSizeNum = -1;
-
-try {
-cidrSizeNum = Integer.parseInt(cidrSize);
-} catch (final Exception e) {
-return null;
+throw new CloudRuntimeException("cidr is not bvalid in ip 
space" + cidr);
 }
-final long numericNetmask = 0x >> MAX_CIDR - cidrSizeNum 
<< MAX_CIDR - cidrSizeNum;
+long cidrSizeNum = getCidrSizeFromString(cidrSize);
+final long numericNetmask = netMaskFromCidr(cidrSizeNum);
 final long ipAddr = ip2Long(cidrAddress);
 final Long[] cidrlong = {ipAddr & numericNetmask, 
(long)cidrSizeNum};
 return cidrlong;
 
 }
 
+/**
+ * @param cidrSize
+ * @return
+ * @throws CloudRuntimeException
+ */
+static long getCidrSizeFromString(final String cidrSize) throws 
CloudRuntimeException {
+long cidrSizeNum = -1;
+
+try {
+cidrSizeNum = Integer.parseInt(cidrSize);
+} catch (final NumberFormatException e) {
+throw new CloudRuntimeException("cidrsize is not a valid int: 
" + cidrSize, e);
+}
+if(cidrSizeNum > 32 || cidrSizeNum < 0) {// assuming IPv4
+throw new CloudRuntimeException("cidr size out of range: " + 
cidrSizeNum);
+}
+return cidrSizeNum;
+}
+
--- End diff --

But those methods are also used internally, right? I mean, if you call 
getCidrSubNet() it will call the other static method. So, you can make them 
private and manipulate the input via the getCidrSubNet() method.


---
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: Shuffling the password to avoid having a ...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1058#issuecomment-155739176
  
LGTM
nice!


---
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-9055: fix NPE in updating Redu...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1059#issuecomment-155741147
  
LGTM by code inspection. The issue is obvious and the solution as well. I 
do agree with @wilderrodrigues that permanent validation of the issue is 
needed. In this case a ref to a unit test may be enough if it exists. otherwise 
a marvin/integration test would be fine.


---
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: Cwe 190

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-155710144
  
That is it.
Everything looks good to me now
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.
---

Re: [ANNOUNCE] New committer: Boris Schrijver

[Boris Schrijver]

Thanks all! Now the real work begins... ;)

-- 

Met vriendelijke groet / Kind regards,

Boris Schrijver

PCextreme B.V.

http://www.pcextreme.nl/contact
Tel direct: +31 6 33784542

> On November 10, 2015 at 6:09 AM Jayapal Reddy Uradi
>  wrote:
> 
> 
> Congrats Boris !
> 
> -Jayapal
> 
> > On 09-Nov-2015, at 12:01 pm, Rajani Karuturi  wrote:
> > 
> > The Project Management Committee (PMC) for Apache CloudStack
> > has asked Boris Schrijver to become a committer and we are pleased to
> > announce that he has accepted.
> > 
> > Boris is an active code contributor, tester.
> > Helps resolving issues, reviewing PRs and testing them.
> > 
> > Being a committer allows many contributors to contribute more
> > autonomously. For developers, it makes it easier to submit changes and
> > eliminates the need to have contributions reviewed via the patch
> > submission process. Whether contributions are development-related or
> > otherwise, it is a recognition of a contributor's participation in the
> > project and commitment to the project and the Apache Way.
> > 
> > Please join me in congratulating Boris
> > 
> > on behalf of the CloudStack PMC,
> > 
> > ~Rajani
>

[GitHub] cloudstack pull request: Cwe 190

[DaanHoogland]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44515734
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -869,31 +878,44 @@ public static boolean isNetworkAWithinNetworkB(final 
String cidrA, final String
 
 public static Long[] cidrToLong(final String cidr) {
 if (cidr == null || cidr.isEmpty()) {
-return null;
+throw new CloudRuntimeException("empty cidr can not be 
converted to longs");
 }
 final String[] cidrPair = cidr.split("\\/");
 if (cidrPair.length != 2) {
-return null;
+throw new CloudRuntimeException("cidr is not formatted 
correctly: "+ cidr);
 }
 final String cidrAddress = cidrPair[0];
 final String cidrSize = cidrPair[1];
 if (!isValidIp(cidrAddress)) {
-return null;
-}
-int cidrSizeNum = -1;
-
-try {
-cidrSizeNum = Integer.parseInt(cidrSize);
-} catch (final Exception e) {
-return null;
+throw new CloudRuntimeException("cidr is not bvalid in ip 
space" + cidr);
 }
-final long numericNetmask = 0x >> MAX_CIDR - cidrSizeNum 
<< MAX_CIDR - cidrSizeNum;
+long cidrSizeNum = getCidrSizeFromString(cidrSize);
+final long numericNetmask = netMaskFromCidr(cidrSizeNum);
 final long ipAddr = ip2Long(cidrAddress);
 final Long[] cidrlong = {ipAddr & numericNetmask, 
(long)cidrSizeNum};
 return cidrlong;
 
 }
 
+/**
+ * @param cidrSize
+ * @return
+ * @throws CloudRuntimeException
+ */
+static long getCidrSizeFromString(final String cidrSize) throws 
CloudRuntimeException {
+long cidrSizeNum = -1;
+
+try {
+cidrSizeNum = Integer.parseInt(cidrSize);
+} catch (final NumberFormatException e) {
+throw new CloudRuntimeException("cidrsize is not a valid int: 
" + cidrSize, e);
+}
+if(cidrSizeNum > 32 || cidrSizeNum < 0) {// assuming IPv4
+throw new CloudRuntimeException("cidr size out of range: " + 
cidrSizeNum);
+}
+return cidrSizeNum;
+}
+
--- End diff --

for testing purpose


---
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: Cwe 190

[DaanHoogland]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44515725
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -775,11 +776,19 @@ public static String getSubNet(final String ip, final 
String netmask) {
 }
 
 public static String getCidrSubNet(final String ip, final long 
cidrSize) {
-final long numericNetmask = 0x >> MAX_CIDR - cidrSize << 
MAX_CIDR - cidrSize;
+final long numericNetmask = netMaskFromCidr(cidrSize);
 final String netmask = NetUtils.long2Ip(numericNetmask);
 return getSubNet(ip, netmask);
 }
 
+/**
+ * @param cidrSize
+ * @return
+ */
+static long netMaskFromCidr(final long cidrSize) {
+return ((long)0x) >> MAX_CIDR - cidrSize << MAX_CIDR - 
cidrSize;
+}
+
--- End diff --

for testing purpose


---
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-5822: keep user-added sshkeys ...

[NuxRo]

Github user NuxRo commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-155735899
  
On a second thought - and something worth pondering on - this could have 
some security implications.

Imagine you have a private cloud, a developer/employee leaves and you want 
to remove his key from the instances because "security". People used to the old 
behaviour might think they're safe when they are in fact not.
Thoughts?

Now, multi-key support, that'd be terrific. :-)


---
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-5822: keep user-added sshkeys ...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-155737852
  
@NuxRo valid point but isn't this unexpected behavior instead of expected? 
The key was not added by the UI (or API) but will be removed by it. If we need 
this a seperate API, resetAllSshKeysInVm should be made.

An angry employee having keys on a vm (out of band) is a real and present 
danger, indeed.


---
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: Shuffling the password to avoid having a ...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1058#issuecomment-155739860
  
@wido @ustcweizhou I see two "looks good to me" without validation test 
explenation. Did you guys test this and how?


---
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: Cwe 190

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1057#issuecomment-155710622
  
@rafaelweingartner yes of course. you caught me pants down ;)


---
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: Cwe 190

[wilderrodrigues]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44520446
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -869,31 +878,44 @@ public static boolean isNetworkAWithinNetworkB(final 
String cidrA, final String
 
 public static Long[] cidrToLong(final String cidr) {
 if (cidr == null || cidr.isEmpty()) {
-return null;
+throw new CloudRuntimeException("empty cidr can not be 
converted to longs");
 }
 final String[] cidrPair = cidr.split("\\/");
 if (cidrPair.length != 2) {
-return null;
+throw new CloudRuntimeException("cidr is not formatted 
correctly: "+ cidr);
 }
 final String cidrAddress = cidrPair[0];
 final String cidrSize = cidrPair[1];
 if (!isValidIp(cidrAddress)) {
-return null;
-}
-int cidrSizeNum = -1;
-
-try {
-cidrSizeNum = Integer.parseInt(cidrSize);
-} catch (final Exception e) {
-return null;
+throw new CloudRuntimeException("cidr is not bvalid in ip 
space" + cidr);
 }
-final long numericNetmask = 0x >> MAX_CIDR - cidrSizeNum 
<< MAX_CIDR - cidrSizeNum;
+long cidrSizeNum = getCidrSizeFromString(cidrSize);
+final long numericNetmask = netMaskFromCidr(cidrSizeNum);
 final long ipAddr = ip2Long(cidrAddress);
 final Long[] cidrlong = {ipAddr & numericNetmask, 
(long)cidrSizeNum};
 return cidrlong;
 
 }
 
+/**
+ * @param cidrSize
+ * @return
+ * @throws CloudRuntimeException
+ */
+static long getCidrSizeFromString(final String cidrSize) throws 
CloudRuntimeException {
+long cidrSizeNum = -1;
+
+try {
+cidrSizeNum = Integer.parseInt(cidrSize);
+} catch (final NumberFormatException e) {
+throw new CloudRuntimeException("cidrsize is not a valid int: 
" + cidrSize, e);
+}
+if(cidrSizeNum > 32 || cidrSizeNum < 0) {// assuming IPv4
+throw new CloudRuntimeException("cidr size out of range: " + 
cidrSizeNum);
+}
+return cidrSizeNum;
+}
+
--- End diff --

I'm saying you don't need to. The test will have the same effect and 
encapsulation will be use properly. I know that Mockito could be better if we 
wouldn't have to kill encapsulation, but if there is a method that already can 
access the "private" method, then no need to open it.

Perhaps after 4.6 we can put some effort in place and move towards JMockit.


---
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: Cwe 190

[rafaelweingartner]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44524995
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -775,11 +776,19 @@ public static String getSubNet(final String ip, final 
String netmask) {
 }
 
 public static String getCidrSubNet(final String ip, final long 
cidrSize) {
-final long numericNetmask = 0x >> MAX_CIDR - cidrSize << 
MAX_CIDR - cidrSize;
+final long numericNetmask = netMaskFromCidr(cidrSize);
 final String netmask = NetUtils.long2Ip(numericNetmask);
 return getSubNet(ip, netmask);
 }
 
+/**
+ * @param cidrSize
+ * @return
+ */
+static long netMaskFromCidr(final long cidrSize) {
+return ((long)0x) >> MAX_CIDR - cidrSize << MAX_CIDR - 
cidrSize;
+}
+
--- End diff --

I think that it is fine letting the method with default modifier. 
The point of a test case is that we can/should write tests for a single 
method. Therefore, if we want to test the method “getCidrSizeFromString”, 
we should write a test case for that specific method, and not for another one 
that uses it. 

After we make sure that a method works fine, when we test another method 
(let’s say “cidrToLong”)  that uses the first one already tested, we can 
use EasyMock/Mockito or another test suite to mock the method that we know 
works, this way we facilitate the writing of test cases. 



---
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: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Nux!]

-1

I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and have 
hit 2 issues.

1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1 
complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of the 
problem, perhaps there's a better way to obsolete this package

2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5 
systemvm is missing - wtf?
opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this with more 
info

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Remi Bergsma" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 10 November, 2015 15:03:03
> Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)

> Hi all,
> 
> I've created a 4.6.0 release candidate, with the following artifacts up for a
> vote:
> 
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
> 
> Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
> 
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
> 
> PGP release keys (signed using A47DDC4F):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> 
> Vote will be open for at least 72 hours.
> 
> For sanity in tallying the vote, can PMC members please be sure to indicate
> "(binding)" with their vote?
> 
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Wilder Rodrigues]

Hi Nux,

Concerning your second comment:

2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5 
systemvm is missing - wtf?

We explained how the upgrade is done in the issue ==> 
https://issues.apache.org/jira/browse/CLOUDSTACK-9046

The current way ACS does the upgrade requires one to follow all the path. So, 
going from 4.4.x to 4.6.x requires an upgrade to 4.5.x first. In order to avoid 
that, you have to register a SystemVM template 4.5.x before as well. That’s how 
I did that and how I have tested. All in the issue above. So, the second point 
doesn’t really block the RC.

Now, about your first point, since you already mentioned how to get it working, 
I wouldn’t say that’s a blocker, right?

Given the current state of the upgrade path, or how it is implemented, perhaps 
the issue you created should be marked either “won’t fix” or be changed into an 
improvement.

Cheers,
Wilder


On 11 Nov 2015, at 13:11, Nux! > wrote:

-1

I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and have 
hit 2 issues.

1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1 
complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of the 
problem, perhaps there's a better way to obsolete this package

2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5 
systemvm is missing - wtf?
opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this with more 
info

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
From: "Remi Bergsma" 
To: dev@cloudstack.apache.org
Sent: Tuesday, 10 November, 2015 15:03:03
Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)

Hi all,

I've created a 4.6.0 release candidate, with the following artifacts up for a
vote:

Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545

Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d

Source release (checksums and signatures are available at the same
location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/

PGP release keys (signed using A47DDC4F):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for at least 72 hours.

For sanity in tallying the vote, can PMC members please be sure to indicate
"(binding)" with their vote?

[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)

[GitHub] cloudstack pull request: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155843339
  
@rafaelweingartner now you are editting XenServerGuru instead of 
Ovm3HypervisorGuru. Of coure that can be improved as well but now the scope of 
this PR has not only changed but also moved ;) I'll close this one.


---
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: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

Github user DaanHoogland closed the pull request at:

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


---
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: CID-1338387: handle unknown storage endpo...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155846413
  
Sorry for that.
Bear in mind that the same occurs to “Ovm3HypervisorGuru.java”. 
However, the change is a little different, in that class you can remove lines 
from 103 – 123.



---
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: DevCloud deployment

[Илья Толстихин]

Yes, this is Xen.

2015-11-11 22:00 GMT+06:00 Daan Hoogland :

> this is a xen host, right?
>
> On Wed, Nov 11, 2015 at 3:43 PM, Илья Толстихин 
> wrote:
>
> > On host ssh is up with 22 port only.
> >
> > 2015-11-11 20:03 GMT+06:00 Илья Толстихин :
> >
> > > Cloudstack 4.6.0
> > > Yes, I can ssh to host
> > >
> > > 2015-11-11 19:41 GMT+06:00 Daan Hoogland :
> > >
> > >> what version of ACS?
> > >> It sounds like a network issue between ms and host? can you ssh from
> ms
> > to
> > >> host on the right port? (i think it was 3922, might be 3022)
> > >>
> > >> On Wed, Nov 11, 2015 at 1:42 PM, Илья Толстихин 
> > >> wrote:
> > >>
> > >> > Hi all,
> > >> > Could you please help with the DevCloud deployment.
> > >> >
> > >> > I imported devcloud into virtualbox, in settings *enabled PAE* and
> > >> > created *host-only
> > >> > network 192.168.56.1/24 *
> > >> > Devcloud is pinged now.
> > >> >
> > >> > I built and run cloudstack on my local machine with commands
> > >> > *mvn -Pdeveloper,systemvm clean install*
> > >> > *mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
> > >> > *mvn -pl :cloud-client-ui jetty:run*
> > >> >
> > >> >
> > >> > Then I installed marvin and tried to deploy datacenter with command
> > >> > *mvn -P developer -pl tools/devcloud -Ddeploysvr*
> > >> > and got exception
> > >> >
> > >> > * Deploy DC Started *
> > >> > *Exception Occurred :['Traceback (most recent call last):\n', '
> File
> > >> > "../marvin/marvin/deployDataCenter.py", line 136, in addHosts\n
> > ret =
> > >> > self.__apiClient.addHost(hostcmd)\n', '  File
> > >> >
> > >> >
> > >>
> >
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
> > >> > line 1572, in addHost\nresponse =
> > >> > self.connection.marvinRequest(command, response_type=response,
> > >> > method=method)\n', '  File
> > >> >
> > >> >
> > >>
> >
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
> > >> > line 379, in marvinRequest\nraise e\n', 'CloudstackAPIException:
> > >> > Execute cmd: addhost failed, due to: errorCode: 530,
> errorText:Cannot
> > >> > transit agent status with event AgentDisconnected for host 1,
> > mangement
> > >> > server id is 18413734744879,Unable to transition to a new state from
> > >> > Creating via AgentDisconnected\n']*
> > >> >
> > >> > *===deploy dc failed, so cleaning the created entries===*
> > >> >
> > >> > *DeployDC: CleanUp Started*
> > >> >
> > >> > *Clean Up Entries=== {'Network':
> > >> > [u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
> > >> > [u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
> > >> > [u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
> > >> > [u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
> > >> > [u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster',
> 'Pod',
> > >> > 'Network', 'PhysicalNetwork', 'Zone']}*
> > >> >
> > >> > *===Removing DataCenter Failed===*
> > >> >
> > >> > As a result in cloudstack were created zone, cluster, pod, and a
> host
> > >> with
> > >> > state "Alert".
> > >> >
> > >> > Also I run
> > >> > *python tools/marvin/marvin/deployDataCenter.py -i
> > >> > tools/devcloud/devcloud.cfg*
> > >> > that gave same result.
> > >> >
> > >> > What am I doing wrong?
> > >> >
> > >>
> > >>
> > >>
> > >> --
> > >> Daan
> > >>
> > >
> > >
> >
>
>
>
> --
> Daan
>

[GitHub] cloudstack pull request: CID-1338387: remove logicless execution c...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155861504
  
running the regression tests on this but these are not in ovm(yet) will 
look into 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 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: CID-1338387: handle unknown storage endpo...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155838061
  
Actually, 

My bad, I think the best thing to do is to change the method 
“com.cloud.hypervisor.XenServerGuru.getCommandHostDelegation(long, 
Command)” to something like this:
```
HostVO host = hostDao.findById(hostId);
hostDao.loadDetails(host);
boolean snapshotHotFix = 
Boolean.parseBoolean(host.getDetail(HostInfo.XS620_SNAPSHOT_HOTFIX));
if(snapshotHotFix){
return new Pair(Boolean.TRUE, new Long(hostId));
}
return super.getCommandHostDelegation(hostId, cmd);;
```


---
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: [Proposal] Template for CloudStack API Reference Pages

[Yiping Zhang]

As a user who uses API a lot,  I would like to see following improvements in 
api reference pages:

1) In brief description for Title section, please specify if the referenced API 
is async or not.  Currently, this info is available only on the API listing 
pages with “(A)” after the api name, but not available or obvious anywhere on 
the api reference page itself. 

2)  For each parameter, in addition to , ,  
attributes, it would be great to also provide following:
 := integer | string | array | enumerate | boolean etc
 := true | false | null | 0 etc

A Notes subsection for parameters: IMHO, there are several reasons that 
such a section will be useful:
* A list of values which have special meaning to the api and what are 
their special meanings, if any.  For example, for listVirtualMachines api, 
projectid=-1 would return instances belonging to ALL projects.  Here value “-1” 
is special.
* combination of certain parameters are mutually exclusive, or are 
required.  Some of this info are currently present in the parameter’s 
description field. But they are usually too brief, hard to read and hard to 
understand.


3) Add a limitations section:
   This section describes scenarios where the referenced API does not apply to, 
or not implemented yet, or known to not work properly.  Many apis have 
limitations and the information is scattered all over places in documents, if 
exists at all. So most often users can only find out by trial and errors.
   
For example,  assignVirtualMachine api has following limitations: 1) does 
not work with VM instances belonging to a project, 2) not implemented for 
Advanced networking with security group enabled.

4) Add an Authorization section or just provide info on the page somewhere: 
describe who can make this api call:  root admin, domain admin, or regular 
users.  Currently, this info is  provided by listing available apis in 
different pages titled “Root admin API”, “domain admin api” and “User api”.  
Personally,  I prefer a separate section on each api’s reference page for this 
info so that it can’t be missed.
   
5)  Error response:  I really like the idea of adding this section to the 
reference page.  Please list both HTTP response code as well as CloudStack 
internal error code and error messages.




Finally, please get some one to proof-read all descriptions.  Some of current 
API document are really hard to understand!

BTW: which release is this proposal targeted for ?

Just my $0.02.

Yiping


On 11/10/15, 9:10 PM, "Daejuan Jacobs"  wrote:

>I assume by "Format" you mean data type.
>
>But I think this looks good. It's simple, yet it manages to nail all the
>points you need when developing on a software's API.
>
>On Tue, Nov 10, 2015 at 8:33 AM Rajsekhar K 
>wrote:
>
>> Hi, All,
>>
>> This is the proposal for a new template for CloudStack API reference
>> pages. This template is based on the reference page templates for REST APIs.
>>
>> Please find attached the following documents for your review:
>>
>>- Template for normal and asynchronous CloudStack API references.
>>- Sample API reference page using the template for a CloudStack API
>>(listZones).
>>
>>
>> Please review this template and let me know your thoughts on this.
>>
>> Thanks,
>> Rajsekhar
>>

[GitHub] cloudstack pull request: CID-1338387: handle unknown storage endpo...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155837263
  
Well, 
I would remover the method: 
“com.cloud.hypervisor.XenServerGuru.getCommandHostDelegation(long, 
Command)”, because it is already coded into its super class. However, I would 
worry about the following code: 
```
hostDao.loadDetails(host);
boolean snapshotHotFix = 
Boolean.parseBoolean(host.getDetail(HostInfo.XS620_SNAPSHOT_HOTFIX));
if (snapshotHotFix) {
return new Pair(Boolean.TRUE, new Long(ep.getId()));
}
```
It is pretty subtle, if the server has that hotfix, we set the first value 
of the pair as true.

Therefore, I would change the code from 
“com.cloud.hypervisor.HypervisorGuruBase.getCommandHostDelegation(long, 
Command)”
To something like this:
```
HostVO host = hostDao.findById(hostId);
hostDao.loadDetails(host);
boolean snapshotHotFix = 
Boolean.parseBoolean(host.getDetail(HostInfo.XS620_SNAPSHOT_HOTFIX));
if(snapshotHotFix){
return new Pair(Boolean.TRUE, new Long(hostId));
}
return new Pair(Boolean.FALSE, new Long(hostId));

```




---
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: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155855273
  
didn't do the xenserver change yet
removed the ovm3 code that seems to have no use at all


---
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: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

GitHub user DaanHoogland reopened a pull request:

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

CID-1338387: handle unknown storage endpoint

I have removed the paramoia null check and created a specific exception as 
i think we always should for any specific condition. This would require careful 
consideration each time on whether and how specific the condition is of course 
;)

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

$ git pull https://github.com/DaanHoogland/cloudstack CID-1338387

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

https://github.com/apache/cloudstack/pull/1056.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1056


commit 7b76cd2fb40d7b26bd786a4a0f8724d8c316a9d3
Author: Daan Hoogland 
Date:   2015-11-10T11:53:35Z

CID-1338387: paranoia null checking

commit a67fc10461b5d0b7482c4cd18267cc3a6bc020d2
Author: Daan Hoogland 
Date:   2015-11-10T16:52:50Z

CID-1338387 specific exception for specific condition

commit 78d35465d7b24b6c9f569dbe83529c3fed2e2071
Author: Daan Hoogland 
Date:   2015-11-10T17:02:55Z

forgotten details

throw on host is null
no iteration for retrieving only one element

commit b0bbba21d3bd1e1332dc72a1caeb4a4cd85d7f4b
Author: Daan Hoogland 
Date:   2015-11-11T17:28:02Z

CID-1338387 removal of unnecessary 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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: UI customization

[Илья Толстихин]

Many thanks.

2015-11-09 18:29 GMT+06:00 Vadim Kimlaychuk :

> From my opinion better way is to write a plugin:
> http://docs.cloudstack.apache.org/en/latest/plugins.html#how-to-write-a-plugin-overview
> and follow UI customization guide:
> http://support.citrix.com/servlet/KbServlet/download/38315-102-714564/CloudPlatform_4.5_User_Interface_Customization.pdf
>
> Doing direct changes to cloudStack.js that are not contributed back to
> community is guaranteed way to have problems after updates. You will have
> to merge your code with mainstream all the time.
>
> Another way is to implement your own interface based on CS REST API. Doing
> that for just 1 button is overkill for sure, but such changes will never
> end as soon as you start to customize. Think about it. The easiest way is
> not always the best one.
>
> Vadim.
>
> On 2015-11-09 13:58, Kshitij Kansal wrote:
>
> Hi,
>>
>> You can find the scripts at the location
>> "client/target/generated-webapp/scripts/". Make changes to cloudstack.js
>> here.
>>
>> *Note: *First remove the corresponding gz file(in your case
>> cloudstack.js.gz) otherwise the changes won't reflect.
>>
>> *The changes in generated-webapps are not saved and tracked in git. So
>> make
>> sure to copy your changes to actual scripts (in ui/scripts) before finally
>> rebuilding the project. *The changes made this way are directly reflected
>>
>> by refreshing the page.
>>
>> PS: Not sure if its the correct way to develop but it works :). Once
>> again,
>> make sure to copy your changes back to correct scripts before building the
>> project.
>>
>> Regards,
>> Kshitij
>>
>> On Mon, Nov 9, 2015 at 12:34 PM, Wei ZHOU  wrote:
>>
>> I guess you worked on old version before.
>> in 4.6, something changed. you might compress cloudStack.js to get
>> cloudStack.js.gz as well.
>>
>> 2015-11-08 15:05 GMT+01:00 Илья Толстихин :
>>
>> Hi all,
>>
>> Could please help with the process of UI customization:
>>
>> I have CloudStack 4.6.0 built from the source.
>> I want to add a custom button to the WebUI menu.
>> In file ui/scripts/cloudStack.js I added my button into sections array
>>
>> *sections: {*
>> * /***
>> * * Dashboard*
>> * */*
>> * dashboard: {},*
>> * instances: {},*
>> * affinityGroups: {},*
>> * storage: {},*
>> * network: {},*
>> * templates: {},*
>> * events: {},*
>> * projects: {},*
>> * accounts: {},*
>>
>> * domains: {}, //domain-admin and root-admin only*
>>
>> * regions: {}, //root-admin only*
>> * system: {}, //root-admin only*
>> * 'global-settings': {}, //root-admin only*
>> * configuration: {}, //root-admin only*
>> * plugins: {},*
>> * test: {*
>> * title: 'TestSection',*
>> * id: 'TestSection',*
>> * show: function(args){*
>> * var div = document.createElement('div');*
>> * div.innerHTML = "Test section";*
>> * return div;*
>> * }*
>> * }*
>> * }*
>>
>> and in sectionPreFilter array
>>
>> *sectionPreFilter: function(args) {*
>> * var sections = [];*
>>
>> * if (isAdmin()) {*
>> * sections = ["dashboard", "instances", "storage",
>> "network", "templates", "accounts", "domains", "events", "system",
>> "global-settings", "configuration", "projects", "regions",
>> "affinityGroups", "test"];*
>>
>> After building the project, the button was added to the menu, but when I
>> edit cloudStack.js and restart jetty UI doesn't change, any changes are
>> applied only after the full project rebuild with command mvn
>> -Pdeveloper,systemvm -DskipTests clean install.
>> Jetty version 6.1.26
>>
>> How to apply changes without rebuilding project?
>> And is it possible to apply it without restart Jetty?
>>
>

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Nux!]

Daan,

What do you mean remove the 4.5 upgrade code? What would be the consequences of 
that?

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Daan Hoogland" 
> To: "dev" 
> Sent: Wednesday, 11 November, 2015 14:18:02
> Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

> Wilder, Borg,
> 
> I would be fine with this as a known issue but I'll create a PR to remove
> the 4.5 upgrade code anyway. We can put it in 4.6.1 if nothing else pops up.
> 
> 
> On Wed, Nov 11, 2015 at 3:00 PM, Wilder Rodrigues <
> wrodrig...@schubergphilis.com> wrote:
> 
>> Nice to know, Lucian!
>>
>> Happy testing! :)
>>
>> Cheers,
>> Wilder
>>
>>
>> On 11 Nov 2015, at 14:46, Nux! >
>> wrote:
>>
>> Wilder,
>>
>> Thanks for that, adding systemvm-kvm-4.5 solved the problem.
>> I will do more testing, but the main issue is sorted.
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro
>>
>> - Original Message -
>> From: "Wilder Rodrigues"  wrodrig...@schubergphilis.com>>
>> To: dev@cloudstack.apache.org
>> Sent: Wednesday, 11 November, 2015 12:39:25
>> Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)
>>
>> Hi Nux,
>>
>> Concerning your second comment:
>>
>> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the
>> 4.5
>> systemvm is missing - wtf?
>>
>> We explained how the upgrade is done in the issue ==>
>> https://issues.apache.org/jira/browse/CLOUDSTACK-9046
>>
>> The current way ACS does the upgrade requires one to follow all the path.
>> So,
>> going from 4.4.x to 4.6.x requires an upgrade to 4.5.x first. In order to
>> avoid
>> that, you have to register a SystemVM template 4.5.x before as well.
>> That’s how
>> I did that and how I have tested. All in the issue above. So, the second
>> point
>> doesn’t really block the RC.
>>
>> Now, about your first point, since you already mentioned how to get it
>> working,
>> I wouldn’t say that’s a blocker, right?
>>
>> Given the current state of the upgrade path, or how it is implemented,
>> perhaps
>> the issue you created should be marked either “won’t fix” or be changed
>> into an
>> improvement.
>>
>> Cheers,
>> Wilder
>>
>>
>> On 11 Nov 2015, at 13:11, Nux! > >> wrote:
>>
>> -1
>>
>> I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and
>> have hit
>> 2 issues.
>>
>> 1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1
>> complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of
>> the
>> problem, perhaps there's a better way to obsolete this package
>>
>> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the
>> 4.5
>> systemvm is missing - wtf?
>> opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this
>> with more
>> info
>>
>> Lucian
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro>
>>
>> - Original Message -
>> From: "Remi Bergsma" 
>> To: dev@cloudstack.apache.org
>> Sent: Tuesday, 10 November, 2015 15:03:03
>> Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)
>>
>> Hi all,
>>
>> I've created a 4.6.0 release candidate, with the following artifacts up
>> for a
>> vote:
>>
>> Git Branch and Commit SH:
>>
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
>>
>> Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
>>
>> Source release (checksums and signatures are available at the same
>> location):
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
>>
>> PGP release keys (signed using A47DDC4F):
>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>>
>> Vote will be open for at least 72 hours.
>>
>> For sanity in tallying the vote, can PMC members please be sure to indicate
>> "(binding)" with their vote?
>>
>> [ ] +1  approve
>> [ ] +0  no opinion
>> [ ] -1  disapprove (and reason why)
>>
>>
> 
> 
> --
> Daan

[GitHub] cloudstack pull request: CLOUDSTACK-9057 remove old system vm upgr...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1061#issuecomment-155809389
  
LGTM
that is what we did for previous versions


---
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: distribution of Cloudstack API plugins

[Giebner, Sascha]

Hey Mike,

thanks for the hint. Would you mind sharing these instructions with me? ;)

Thx and best regards,
Sascha

From: Mike Tutkowski [mike.tutkow...@solidfire.com]
Sent: Wednesday, November 11, 2015 3:25 PM
To: dev@cloudstack.apache.org
Subject: Re: distribution of Cloudstack API plugins

What I have always done is build the JAR file myself and then provide to
the third party instructions on where it goes (and that to make use of it,
it does require a reboot of the management server).

On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
sascha.gieb...@bautzen-it.de> wrote:

> Hello everyone,
>
> I have a question: After successfully implementing a Cloudstack API plugin
> (no UI plugin), how would one go about sharing the results with a third
> party?
>
> Is the only way to add the changes and rebuild cloudstack?
>
> Or is there a possibility to somehow package a plugin and add it
> dynamically to a running cloudstack instance?
>
> Or in general what would be the easiest way to prepare a plugin for a
> simple end-user-installation?
>
> Thx a bunch,
> Sascha Giebner
>
> --
> This email was Virus checked by UTM 9. http://www.sophos.com
>



--
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

--
This email was Virus checked by UTM 9. http://www.sophos.com

Storage plugin docs?

[Nux!]

Hello,

If someone wanted to write a storage plugin for Cloudstack, is there any 
documentation and/or examples that I could point this person to?

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Nux!]

Ok, I'm changing my vote to:

+1 (binding)

The upgrade issue I had was a known one, easily fixed thanks to Wilder, should 
be mentioned in the upgrade docs though.

I tested KVM with CentOS 6 HVs and Adv+SG zone. Security groups work, basic 
functions works, restore from snapshot works, create volume/template from 
snapshot works etc.

Looks good!

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Nux!" 
> To: dev@cloudstack.apache.org
> Sent: Wednesday, 11 November, 2015 12:11:37
> Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

> -1
> 
> I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and have 
> hit
> 2 issues.
> 
> 1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1
> complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of the
> problem, perhaps there's a better way to obsolete this package
> 
> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5
> systemvm is missing - wtf?
> opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this with 
> more
> info
> 
> Lucian
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> - Original Message -
>> From: "Remi Bergsma" 
>> To: dev@cloudstack.apache.org
>> Sent: Tuesday, 10 November, 2015 15:03:03
>> Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)
> 
>> Hi all,
>> 
>> I've created a 4.6.0 release candidate, with the following artifacts up for a
>> vote:
>> 
>> Git Branch and Commit SH:
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
>> 
>> Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
>> 
>> Source release (checksums and signatures are available at the same
>> location):
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
>> 
>> PGP release keys (signed using A47DDC4F):
>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>> 
>> Vote will be open for at least 72 hours.
>> 
>> For sanity in tallying the vote, can PMC members please be sure to indicate
>> "(binding)" with their vote?
>> 
>> [ ] +1  approve
>> [ ] +0  no opinion
> > [ ] -1  disapprove (and reason why)

[GitHub] cloudstack pull request: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155831862
  
@rafaelweingartner nice analysis. meaning, all of my change is senseless. I 
think the conclusion of your research is that the side effects for CopyCommand 
are none and we can actually remove this bit of the code. @snuf can you concur?


---
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: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Daan Hoogland]

Lucian, are you testing RC2 or 1?

On Wed, Nov 11, 2015 at 1:11 PM, Nux!  wrote:

> -1
>
> I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and
> have hit 2 issues.
>
> 1 - minor packaging issue, upgrading to 4.6.0 makes
> cloudstack-awsapi-4.4.1 complain about missing deps; rpm -e --nodeps
> cloudstack-awsapi gets rid of the problem, perhaps there's a better way to
> obsolete this package
>
> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the
> 4.5 systemvm is missing - wtf?
> opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this
> with more info
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
> > From: "Remi Bergsma" 
> > To: dev@cloudstack.apache.org
> > Sent: Tuesday, 10 November, 2015 15:03:03
> > Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)
>
> > Hi all,
> >
> > I've created a 4.6.0 release candidate, with the following artifacts up
> for a
> > vote:
> >
> > Git Branch and Commit SH:
> >
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
> >
> > Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
> >
> > PGP release keys (signed using A47DDC4F):
> > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> >
> > Vote will be open for at least 72 hours.
> >
> > For sanity in tallying the vote, can PMC members please be sure to
> indicate
> > "(binding)" with their vote?
> >
> > [ ] +1  approve
> > [ ] +0  no opinion
> > [ ] -1  disapprove (and reason why)
>



-- 
Daan

Re: distribution of Cloudstack API plugins

[Mike Tutkowski]

What I have always done is build the JAR file myself and then provide to
the third party instructions on where it goes (and that to make use of it,
it does require a reboot of the management server).

On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
sascha.gieb...@bautzen-it.de> wrote:

> Hello everyone,
>
> I have a question: After successfully implementing a Cloudstack API plugin
> (no UI plugin), how would one go about sharing the results with a third
> party?
>
> Is the only way to add the changes and rebuild cloudstack?
>
> Or is there a possibility to somehow package a plugin and add it
> dynamically to a running cloudstack instance?
>
> Or in general what would be the easiest way to prepare a plugin for a
> simple end-user-installation?
>
> Thx a bunch,
> Sascha Giebner
>
> --
> This email was Virus checked by UTM 9. http://www.sophos.com
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: DevCloud deployment

[Илья Толстихин]

On host ssh is up with 22 port only.

2015-11-11 20:03 GMT+06:00 Илья Толстихин :

> Cloudstack 4.6.0
> Yes, I can ssh to host
>
> 2015-11-11 19:41 GMT+06:00 Daan Hoogland :
>
>> what version of ACS?
>> It sounds like a network issue between ms and host? can you ssh from ms to
>> host on the right port? (i think it was 3922, might be 3022)
>>
>> On Wed, Nov 11, 2015 at 1:42 PM, Илья Толстихин 
>> wrote:
>>
>> > Hi all,
>> > Could you please help with the DevCloud deployment.
>> >
>> > I imported devcloud into virtualbox, in settings *enabled PAE* and
>> > created *host-only
>> > network 192.168.56.1/24 *
>> > Devcloud is pinged now.
>> >
>> > I built and run cloudstack on my local machine with commands
>> > *mvn -Pdeveloper,systemvm clean install*
>> > *mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
>> > *mvn -pl :cloud-client-ui jetty:run*
>> >
>> >
>> > Then I installed marvin and tried to deploy datacenter with command
>> > *mvn -P developer -pl tools/devcloud -Ddeploysvr*
>> > and got exception
>> >
>> > * Deploy DC Started *
>> > *Exception Occurred :['Traceback (most recent call last):\n', '  File
>> > "../marvin/marvin/deployDataCenter.py", line 136, in addHosts\nret =
>> > self.__apiClient.addHost(hostcmd)\n', '  File
>> >
>> >
>> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
>> > line 1572, in addHost\nresponse =
>> > self.connection.marvinRequest(command, response_type=response,
>> > method=method)\n', '  File
>> >
>> >
>> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
>> > line 379, in marvinRequest\nraise e\n', 'CloudstackAPIException:
>> > Execute cmd: addhost failed, due to: errorCode: 530, errorText:Cannot
>> > transit agent status with event AgentDisconnected for host 1, mangement
>> > server id is 18413734744879,Unable to transition to a new state from
>> > Creating via AgentDisconnected\n']*
>> >
>> > *===deploy dc failed, so cleaning the created entries===*
>> >
>> > *DeployDC: CleanUp Started*
>> >
>> > *Clean Up Entries=== {'Network':
>> > [u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
>> > [u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
>> > [u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
>> > [u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
>> > [u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster', 'Pod',
>> > 'Network', 'PhysicalNetwork', 'Zone']}*
>> >
>> > *===Removing DataCenter Failed===*
>> >
>> > As a result in cloudstack were created zone, cluster, pod, and a host
>> with
>> > state "Alert".
>> >
>> > Also I run
>> > *python tools/marvin/marvin/deployDataCenter.py -i
>> > tools/devcloud/devcloud.cfg*
>> > that gave same result.
>> >
>> > What am I doing wrong?
>> >
>>
>>
>>
>> --
>> Daan
>>
>
>

Re: DevCloud deployment

[Daan Hoogland]

this is a xen host, right?

On Wed, Nov 11, 2015 at 3:43 PM, Илья Толстихин  wrote:

> On host ssh is up with 22 port only.
>
> 2015-11-11 20:03 GMT+06:00 Илья Толстихин :
>
> > Cloudstack 4.6.0
> > Yes, I can ssh to host
> >
> > 2015-11-11 19:41 GMT+06:00 Daan Hoogland :
> >
> >> what version of ACS?
> >> It sounds like a network issue between ms and host? can you ssh from ms
> to
> >> host on the right port? (i think it was 3922, might be 3022)
> >>
> >> On Wed, Nov 11, 2015 at 1:42 PM, Илья Толстихин 
> >> wrote:
> >>
> >> > Hi all,
> >> > Could you please help with the DevCloud deployment.
> >> >
> >> > I imported devcloud into virtualbox, in settings *enabled PAE* and
> >> > created *host-only
> >> > network 192.168.56.1/24 *
> >> > Devcloud is pinged now.
> >> >
> >> > I built and run cloudstack on my local machine with commands
> >> > *mvn -Pdeveloper,systemvm clean install*
> >> > *mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
> >> > *mvn -pl :cloud-client-ui jetty:run*
> >> >
> >> >
> >> > Then I installed marvin and tried to deploy datacenter with command
> >> > *mvn -P developer -pl tools/devcloud -Ddeploysvr*
> >> > and got exception
> >> >
> >> > * Deploy DC Started *
> >> > *Exception Occurred :['Traceback (most recent call last):\n', '  File
> >> > "../marvin/marvin/deployDataCenter.py", line 136, in addHosts\n
> ret =
> >> > self.__apiClient.addHost(hostcmd)\n', '  File
> >> >
> >> >
> >>
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
> >> > line 1572, in addHost\nresponse =
> >> > self.connection.marvinRequest(command, response_type=response,
> >> > method=method)\n', '  File
> >> >
> >> >
> >>
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
> >> > line 379, in marvinRequest\nraise e\n', 'CloudstackAPIException:
> >> > Execute cmd: addhost failed, due to: errorCode: 530, errorText:Cannot
> >> > transit agent status with event AgentDisconnected for host 1,
> mangement
> >> > server id is 18413734744879,Unable to transition to a new state from
> >> > Creating via AgentDisconnected\n']*
> >> >
> >> > *===deploy dc failed, so cleaning the created entries===*
> >> >
> >> > *DeployDC: CleanUp Started*
> >> >
> >> > *Clean Up Entries=== {'Network':
> >> > [u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
> >> > [u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
> >> > [u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
> >> > [u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
> >> > [u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster', 'Pod',
> >> > 'Network', 'PhysicalNetwork', 'Zone']}*
> >> >
> >> > *===Removing DataCenter Failed===*
> >> >
> >> > As a result in cloudstack were created zone, cluster, pod, and a host
> >> with
> >> > state "Alert".
> >> >
> >> > Also I run
> >> > *python tools/marvin/marvin/deployDataCenter.py -i
> >> > tools/devcloud/devcloud.cfg*
> >> > that gave same result.
> >> >
> >> > What am I doing wrong?
> >> >
> >>
> >>
> >>
> >> --
> >> Daan
> >>
> >
> >
>



-- 
Daan

[GitHub] cloudstack pull request: CID-1338387: handle unknown storage endpo...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155833511
  
That is it. We can remove the 
“org.apache.cloudstack.storage.endpoint.DefaultEndPointSelector.selectHypervisorHost”
  method


---
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: CID-1338387: handle unknown storage endpo...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155834298
  
we can remove the entire ```if (cmd instanceof CopyCommand) {}``` block


---
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: CID-1338387: handle unknown storage endpo...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155821415
  
Hi @DaanHoogland, 
About the method: “endPointSelector.selectHypervisorHost”.

I debugged the use of that method in my environment. I am using Xen; and 
that method is used when creating a template from a snapshot; the CS will 
execute the code: 
“com.cloud.hypervisor.XenServerGuru.getCommandHostDelegation(long, Command)”
ACS: 4.3.2
Xen (4.1) + XAPI (1.6), those components together are basically the same as 
a xenserver 6.0. The difference is that we use Debian as our OS and not CentOS.

Look at what happened in my test, the hostHd received in the 
“XenServerGuru.getCommandHostDelegation” method was (100).

“XenServerGuru.getCommandHostDelegation” loaded the host data from host 
table to retrieve the dataCenterID, and then executed 
“endPointSelector.selectHypervisorHost”, that ended up selecting a host 
that has the id (87) which is in a different cluster. Then it created the 
“EndPoint” object that does not do much; there is a configure method 
“org.apache.cloudstack.storage.RemoteHostEndPoint.configure(Host)” that is 
called, but it merely retrieves some data from the host object that was sent as 
a parameter. After that in line 161 of 
“XenServerGuru.getCommandHostDelegation” it is loaded the host data using 
the host id of the “EndPoint” object; then we check if the host (87) has a 
fix “snapshotHotFix”; in my case, the host did not have, so it ended up 
returning the original host id (100) at line 170.
Even if the host (87) had the “snapshotHotFix”, I do not understand why 
it is getting a host at random in the environment; If the host does not matter, 
why not use the host that the method already received as a parameter?

The name of the method 
“org.apache.cloudstack.storage.endpoint.DefaultEndPointSelector.selectHypervisorHost”
 does not say much, and its execution does not make much sense to me. I believe 
it can be removed without any harm.



---
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: Cwe 190

[DaanHoogland]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44519906
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -869,31 +878,44 @@ public static boolean isNetworkAWithinNetworkB(final 
String cidrA, final String
 
 public static Long[] cidrToLong(final String cidr) {
 if (cidr == null || cidr.isEmpty()) {
-return null;
+throw new CloudRuntimeException("empty cidr can not be 
converted to longs");
 }
 final String[] cidrPair = cidr.split("\\/");
 if (cidrPair.length != 2) {
-return null;
+throw new CloudRuntimeException("cidr is not formatted 
correctly: "+ cidr);
 }
 final String cidrAddress = cidrPair[0];
 final String cidrSize = cidrPair[1];
 if (!isValidIp(cidrAddress)) {
-return null;
-}
-int cidrSizeNum = -1;
-
-try {
-cidrSizeNum = Integer.parseInt(cidrSize);
-} catch (final Exception e) {
-return null;
+throw new CloudRuntimeException("cidr is not bvalid in ip 
space" + cidr);
 }
-final long numericNetmask = 0x >> MAX_CIDR - cidrSizeNum 
<< MAX_CIDR - cidrSizeNum;
+long cidrSizeNum = getCidrSizeFromString(cidrSize);
+final long numericNetmask = netMaskFromCidr(cidrSizeNum);
 final long ipAddr = ip2Long(cidrAddress);
 final Long[] cidrlong = {ipAddr & numericNetmask, 
(long)cidrSizeNum};
 return cidrlong;
 
 }
 
+/**
+ * @param cidrSize
+ * @return
+ * @throws CloudRuntimeException
+ */
+static long getCidrSizeFromString(final String cidrSize) throws 
CloudRuntimeException {
+long cidrSizeNum = -1;
+
+try {
+cidrSizeNum = Integer.parseInt(cidrSize);
+} catch (final NumberFormatException e) {
+throw new CloudRuntimeException("cidrsize is not a valid int: 
" + cidrSize, e);
+}
+if(cidrSizeNum > 32 || cidrSizeNum < 0) {// assuming IPv4
+throw new CloudRuntimeException("cidr size out of range: " + 
cidrSizeNum);
+}
+return cidrSizeNum;
+}
+
--- End diff --

I test them directly. Are you saying i shouldn't?


---
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: Cwe 190

[DaanHoogland]

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

https://github.com/apache/cloudstack/pull/1057#discussion_r44520633
  
--- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java ---
@@ -869,31 +878,44 @@ public static boolean isNetworkAWithinNetworkB(final 
String cidrA, final String
 
 public static Long[] cidrToLong(final String cidr) {
 if (cidr == null || cidr.isEmpty()) {
-return null;
+throw new CloudRuntimeException("empty cidr can not be 
converted to longs");
 }
 final String[] cidrPair = cidr.split("\\/");
 if (cidrPair.length != 2) {
-return null;
+throw new CloudRuntimeException("cidr is not formatted 
correctly: "+ cidr);
 }
 final String cidrAddress = cidrPair[0];
 final String cidrSize = cidrPair[1];
 if (!isValidIp(cidrAddress)) {
-return null;
-}
-int cidrSizeNum = -1;
-
-try {
-cidrSizeNum = Integer.parseInt(cidrSize);
-} catch (final Exception e) {
-return null;
+throw new CloudRuntimeException("cidr is not bvalid in ip 
space" + cidr);
 }
-final long numericNetmask = 0x >> MAX_CIDR - cidrSizeNum 
<< MAX_CIDR - cidrSizeNum;
+long cidrSizeNum = getCidrSizeFromString(cidrSize);
+final long numericNetmask = netMaskFromCidr(cidrSizeNum);
 final long ipAddr = ip2Long(cidrAddress);
 final Long[] cidrlong = {ipAddr & numericNetmask, 
(long)cidrSizeNum};
 return cidrlong;
 
 }
 
+/**
+ * @param cidrSize
+ * @return
+ * @throws CloudRuntimeException
+ */
+static long getCidrSizeFromString(final String cidrSize) throws 
CloudRuntimeException {
+long cidrSizeNum = -1;
+
+try {
+cidrSizeNum = Integer.parseInt(cidrSize);
+} catch (final NumberFormatException e) {
+throw new CloudRuntimeException("cidrsize is not a valid int: 
" + cidrSize, e);
+}
+if(cidrSizeNum > 32 || cidrSizeNum < 0) {// assuming IPv4
+throw new CloudRuntimeException("cidr size out of range: " + 
cidrSizeNum);
+}
+return cidrSizeNum;
+}
+
--- End diff --

I left it like this because the methods may have more use. They don't at 
the moment so closing them of could be done. For now this is fine by me. i 
haven't looked at implicit tests of these methods.


---
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-5822: keep user-added sshkeys ...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-155734540
  
@ustcweizhou 

How did you test it?

I think we should stick to a LGTM being given only if tests have been done 
and steps, on how to test, have been made clear. 

Cheers,
Wilder


---
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-5822: keep user-added sshkeys ...

[NuxRo]

Github user NuxRo commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-155740283
  
Good point as well with the "unexpected". I definitely see where Wei is 
coming from, but I think it could be misleading. Perhaps a better way to do 
this is mark the ACS key and only reset that one. i.e.
When we add the key append a "# added by Cloudstack" and when we issue a 
reset, just delete that one.

Am I overcomplicating this? I might be, especially as these scripts are 
being slowly phased out.


---
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-9055: fix NPE in updating Redu...

[DaanHoogland]

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

https://github.com/apache/cloudstack/pull/1059#discussion_r44523082
  
--- Diff: 
server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ---
@@ -1015,7 +1015,7 @@ protected void updateRoutersRedundantState(final 
List routers) {
 RedundantState state = RedundantState.UNKNOWN;
 if (answer != null && answer.getResult()) {
 state = answer.getState();
-} else {
+} else if (answer != null) {
 s_logger.info("Agent response doesn't seem to be 
correct ==> " + answer.getResult());
 }
--- End diff --

trivial comment: looks like
```
if (answer != null) {
if (answer.getResult()) {
state = answer.getState();
} else {
s_logger.info("Agent response doesn't seem to be correct ==> " + 
answer.getResult());
}
}
```


---
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-9055: fix NPE in updating Redu...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1059#issuecomment-155763566
  
@wilderrodrigues @DaanHoogland This happened when one of nodes is down. I 
think it will happen when the cloudstack-agent is stopped or killed.




---
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: DevCloud deployment

[Daan Hoogland]

what version of ACS?
It sounds like a network issue between ms and host? can you ssh from ms to
host on the right port? (i think it was 3922, might be 3022)

On Wed, Nov 11, 2015 at 1:42 PM, Илья Толстихин  wrote:

> Hi all,
> Could you please help with the DevCloud deployment.
>
> I imported devcloud into virtualbox, in settings *enabled PAE* and
> created *host-only
> network 192.168.56.1/24 *
> Devcloud is pinged now.
>
> I built and run cloudstack on my local machine with commands
> *mvn -Pdeveloper,systemvm clean install*
> *mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
> *mvn -pl :cloud-client-ui jetty:run*
>
>
> Then I installed marvin and tried to deploy datacenter with command
> *mvn -P developer -pl tools/devcloud -Ddeploysvr*
> and got exception
>
> * Deploy DC Started *
> *Exception Occurred :['Traceback (most recent call last):\n', '  File
> "../marvin/marvin/deployDataCenter.py", line 136, in addHosts\nret =
> self.__apiClient.addHost(hostcmd)\n', '  File
>
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
> line 1572, in addHost\nresponse =
> self.connection.marvinRequest(command, response_type=response,
> method=method)\n', '  File
>
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
> line 379, in marvinRequest\nraise e\n', 'CloudstackAPIException:
> Execute cmd: addhost failed, due to: errorCode: 530, errorText:Cannot
> transit agent status with event AgentDisconnected for host 1, mangement
> server id is 18413734744879,Unable to transition to a new state from
> Creating via AgentDisconnected\n']*
>
> *===deploy dc failed, so cleaning the created entries===*
>
> *DeployDC: CleanUp Started*
>
> *Clean Up Entries=== {'Network':
> [u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
> [u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
> [u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
> [u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
> [u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster', 'Pod',
> 'Network', 'PhysicalNetwork', 'Zone']}*
>
> *===Removing DataCenter Failed===*
>
> As a result in cloudstack were created zone, cluster, pod, and a host with
> state "Alert".
>
> Also I run
> *python tools/marvin/marvin/deployDataCenter.py -i
> tools/devcloud/devcloud.cfg*
> that gave same result.
>
> What am I doing wrong?
>



-- 
Daan

distribution of Cloudstack API plugins

[Giebner, Sascha]

Hello everyone,

I have a question: After successfully implementing a Cloudstack API plugin (no 
UI plugin), how would one go about sharing the results with a third party?

Is the only way to add the changes and rebuild cloudstack?

Or is there a possibility to somehow package a plugin and add it dynamically to 
a running cloudstack instance?

Or in general what would be the easiest way to prepare a plugin for a simple 
end-user-installation?

Thx a bunch,
Sascha Giebner

-- 
This email was Virus checked by UTM 9. http://www.sophos.com

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Wilder Rodrigues]

Nice to know, Lucian!

Happy testing! :)

Cheers,
Wilder


On 11 Nov 2015, at 14:46, Nux! > wrote:

Wilder,

Thanks for that, adding systemvm-kvm-4.5 solved the problem.
I will do more testing, but the main issue is sorted.

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
From: "Wilder Rodrigues" 
>
To: dev@cloudstack.apache.org
Sent: Wednesday, 11 November, 2015 12:39:25
Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

Hi Nux,

Concerning your second comment:

2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5
systemvm is missing - wtf?

We explained how the upgrade is done in the issue ==>
https://issues.apache.org/jira/browse/CLOUDSTACK-9046

The current way ACS does the upgrade requires one to follow all the path. So,
going from 4.4.x to 4.6.x requires an upgrade to 4.5.x first. In order to avoid
that, you have to register a SystemVM template 4.5.x before as well. That’s how
I did that and how I have tested. All in the issue above. So, the second point
doesn’t really block the RC.

Now, about your first point, since you already mentioned how to get it working,
I wouldn’t say that’s a blocker, right?

Given the current state of the upgrade path, or how it is implemented, perhaps
the issue you created should be marked either “won’t fix” or be changed into an
improvement.

Cheers,
Wilder


On 11 Nov 2015, at 13:11, Nux! 
> wrote:

-1

I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and have hit
2 issues.

1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1
complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of the
problem, perhaps there's a better way to obsolete this package

2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5
systemvm is missing - wtf?
opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this with more
info

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro>

- Original Message -
From: "Remi Bergsma" 
To: dev@cloudstack.apache.org
Sent: Tuesday, 10 November, 2015 15:03:03
Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)

Hi all,

I've created a 4.6.0 release candidate, with the following artifacts up for a
vote:

Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545

Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d

Source release (checksums and signatures are available at the same
location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/

PGP release keys (signed using A47DDC4F):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for at least 72 hours.

For sanity in tallying the vote, can PMC members please be sure to indicate
"(binding)" with their vote?

[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)

[GitHub] cloudstack pull request: CLOUDSTACK-5822: keep user-added sshkeys ...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-155748424
  
@NuxRo @ustcweizhou is using "cloudst...@apache.org$" for this.


---
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: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Wilder Rodrigues]

Hi all,

The XenServer 6.2 are done. Based on the tests I have execute against 2 
different hypervisors, with a single host and a clustered host, my vote for the 
4.6.0 RC2 is..

==> +1 (binding)

Test report below.

Cheers,
Wilder


:: Environment 1 ::

* Hardware required: TRUE
* Management Server + MySQL on CentOS 7.1
* Two XenServer 6.2 hosts


:: Tests Suites Executed ::

nosetests --with-marvin 
--marvin-config=/data/shared/marvin/mct-zone1-xen1-ISOLATED.cfg -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 
component/test_password_server.py component/test_router_dhcphosts.py 
smoke/test_loadbalance.py smoke/test_ssvm.py smoke/test_network.py


:: Environment 2 ::

* Hardware required: FALSE
* Management Server + MySQL on CentOS 7.1
* Two XenServer 6.2 hosts


:: Tests Suites Executed ::

nosetests --with-marvin 
--marvin-config=/data/shared/marvin/mct-zone1-xen1-ISOLATED.cfg -s -a 
tags=advanced,required_hardware=false smoke/test_routers.py 
smoke/test_reset_vm_on_reboot.py smoke/test_vm_life_cycle.py 
component/test_vpc_routers.py smoke/test_service_offerings.py 
component/test_vpc_offerings.py smoke/test_network_acl.py 
smoke/test_privategw_acl.py smoke/test_network.py


:: Summary ::

* Tests executes: 71
* Successfull tests: 61
* Skipped tests: 10(*)
* Failed tests: 0

(*) Due to host credentials. It will works if we have if the DC has only 1 
host. All those tests were executed successfully on KVM, when I had 1 host 
only. See previous email.

:: Test results for Environment 1 ::

Create a redundant VPC with two networks with two VMs in each network ... SKIP: 
Marvin configuration has no host credentials to 
   chec
k router services
Create a redundant VPC with two networks with two VMs in each network and check 
default routes ... SKIP: Marvin configuration has no host credentials to
 check router services
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
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 the password file in the Router VM ... === TestName: 
test_isolate_network_password_server | 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 SSVM Internals ... SKIP: Marvin configuration has no host  
  credentials to check router services
Test CPVM Internals ... SKIP: Marvin configuration has no host  
  credentials to check router services
Test stop SSVM ... SKIP: Marvin configuration has no host   
 credentials to check router services
Test stop CPVM ... SKIP: Marvin configuration has no host   
 credentials to check router services
Test reboot SSVM ... SKIP: Marvin configuration has no host 
   credentials to check router services
Test reboot CPVM ... SKIP: Marvin configuration has no host 
   credentials to check router services
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... SKIP: Marvin configuration has no host
credentials to check router services
Test for port forwarding on source NAT ... === TestName: 
test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: 
test_02_port_fwd_on_non_src_nat | Status : 

DevCloud deployment

[Илья Толстихин]

Hi all,
Could you please help with the DevCloud deployment.

I imported devcloud into virtualbox, in settings *enabled PAE* and
created *host-only
network 192.168.56.1/24 *
Devcloud is pinged now.

I built and run cloudstack on my local machine with commands
*mvn -Pdeveloper,systemvm clean install*
*mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
*mvn -pl :cloud-client-ui jetty:run*


Then I installed marvin and tried to deploy datacenter with command
*mvn -P developer -pl tools/devcloud -Ddeploysvr*
and got exception

* Deploy DC Started *
*Exception Occurred :['Traceback (most recent call last):\n', '  File
"../marvin/marvin/deployDataCenter.py", line 136, in addHosts\nret =
self.__apiClient.addHost(hostcmd)\n', '  File
"/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
line 1572, in addHost\nresponse =
self.connection.marvinRequest(command, response_type=response,
method=method)\n', '  File
"/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
line 379, in marvinRequest\nraise e\n', 'CloudstackAPIException:
Execute cmd: addhost failed, due to: errorCode: 530, errorText:Cannot
transit agent status with event AgentDisconnected for host 1, mangement
server id is 18413734744879,Unable to transition to a new state from
Creating via AgentDisconnected\n']*

*===deploy dc failed, so cleaning the created entries===*

*DeployDC: CleanUp Started*

*Clean Up Entries=== {'Network':
[u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
[u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
[u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
[u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
[u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster', 'Pod',
'Network', 'PhysicalNetwork', 'Zone']}*

*===Removing DataCenter Failed===*

As a result in cloudstack were created zone, cluster, pod, and a host with
state "Alert".

Also I run
*python tools/marvin/marvin/deployDataCenter.py -i
tools/devcloud/devcloud.cfg*
that gave same result.

What am I doing wrong?

[GitHub] cloudstack pull request: CLOUDSTACK-5822: keep user-added sshkeys ...

[NuxRo]

Github user NuxRo commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-155751828
  
Ah, right, checking the code is important. :-D
I'll go back to my corner.


---
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: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Nux!]

Wilder,

Thanks for that, adding systemvm-kvm-4.5 solved the problem.
I will do more testing, but the main issue is sorted.

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Wilder Rodrigues" 
> To: dev@cloudstack.apache.org
> Sent: Wednesday, 11 November, 2015 12:39:25
> Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

> Hi Nux,
> 
> Concerning your second comment:
> 
> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5
> systemvm is missing - wtf?
> 
> We explained how the upgrade is done in the issue ==>
> https://issues.apache.org/jira/browse/CLOUDSTACK-9046
> 
> The current way ACS does the upgrade requires one to follow all the path. So,
> going from 4.4.x to 4.6.x requires an upgrade to 4.5.x first. In order to 
> avoid
> that, you have to register a SystemVM template 4.5.x before as well. That’s 
> how
> I did that and how I have tested. All in the issue above. So, the second point
> doesn’t really block the RC.
> 
> Now, about your first point, since you already mentioned how to get it 
> working,
> I wouldn’t say that’s a blocker, right?
> 
> Given the current state of the upgrade path, or how it is implemented, perhaps
> the issue you created should be marked either “won’t fix” or be changed into 
> an
> improvement.
> 
> Cheers,
> Wilder
> 
> 
> On 11 Nov 2015, at 13:11, Nux! > wrote:
> 
> -1
> 
> I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and have 
> hit
> 2 issues.
> 
> 1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1
> complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of the
> problem, perhaps there's a better way to obsolete this package
> 
> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the 4.5
> systemvm is missing - wtf?
> opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this with 
> more
> info
> 
> Lucian
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> - Original Message -
> From: "Remi Bergsma" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 10 November, 2015 15:03:03
> Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)
> 
> Hi all,
> 
> I've created a 4.6.0 release candidate, with the following artifacts up for a
> vote:
> 
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
> 
> Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
> 
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
> 
> PGP release keys (signed using A47DDC4F):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> 
> Vote will be open for at least 72 hours.
> 
> For sanity in tallying the vote, can PMC members please be sure to indicate
> "(binding)" with their vote?
> 
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Nux!]

Round 2 RPMs from 
http://jenkins.buildacloud.org/view/parameterized/job/cloudstack-rpm-packages-with-branch-parameter/22/

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Daan Hoogland" 
> To: "dev" 
> Sent: Wednesday, 11 November, 2015 12:13:59
> Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

> Lucian, are you testing RC2 or 1?
> 
> On Wed, Nov 11, 2015 at 1:11 PM, Nux!  wrote:
> 
>> -1
>>
>> I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and
>> have hit 2 issues.
>>
>> 1 - minor packaging issue, upgrading to 4.6.0 makes
>> cloudstack-awsapi-4.4.1 complain about missing deps; rpm -e --nodeps
>> cloudstack-awsapi gets rid of the problem, perhaps there's a better way to
>> obsolete this package
>>
>> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the
>> 4.5 systemvm is missing - wtf?
>> opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this
>> with more info
>>
>> Lucian
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro
>>
>> - Original Message -
>> > From: "Remi Bergsma" 
>> > To: dev@cloudstack.apache.org
>> > Sent: Tuesday, 10 November, 2015 15:03:03
>> > Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)
>>
>> > Hi all,
>> >
>> > I've created a 4.6.0 release candidate, with the following artifacts up
>> for a
>> > vote:
>> >
>> > Git Branch and Commit SH:
>> >
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
>> >
>> > Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
>> >
>> > Source release (checksums and signatures are available at the same
>> > location):
>> > https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
>> >
>> > PGP release keys (signed using A47DDC4F):
>> > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>> >
>> > Vote will be open for at least 72 hours.
>> >
>> > For sanity in tallying the vote, can PMC members please be sure to
>> indicate
>> > "(binding)" with their vote?
>> >
>> > [ ] +1  approve
>> > [ ] +0  no opinion
>> > [ ] -1  disapprove (and reason why)
>>
> 
> 
> 
> --
> Daan

[GitHub] cloudstack pull request: CLOUDSTACK-9057 remove old system vm upgr...

[DaanHoogland]

GitHub user DaanHoogland opened a pull request:

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

CLOUDSTACK-9057 remove old system vm upgrade code

the upgrade step to 4.5 system templates is removed for 4.6 in this. 
validation would be to install any 4.4.2 or lower version and upgrade to 4.6. 
It would only require the templates for 4.6 and not the ones for 4.5. as it 
would before this patch. only unittests performed hence for all practical 
purposes untested.

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

$ git pull https://github.com/DaanHoogland/cloudstack CLOUDSTACK-9057

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

https://github.com/apache/cloudstack/pull/1061.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1061


commit d8e7e7d99355dcb79aec1e16ac94ae546a8bed41
Author: Daan Hoogland 
Date:   2015-11-11T14:02:28Z

CLOUDSTACK-9057 remove old system vm upgrade 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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: DevCloud deployment

[Илья Толстихин]

Cloudstack 4.6.0
Yes, I can ssh to host

2015-11-11 19:41 GMT+06:00 Daan Hoogland :

> what version of ACS?
> It sounds like a network issue between ms and host? can you ssh from ms to
> host on the right port? (i think it was 3922, might be 3022)
>
> On Wed, Nov 11, 2015 at 1:42 PM, Илья Толстихин 
> wrote:
>
> > Hi all,
> > Could you please help with the DevCloud deployment.
> >
> > I imported devcloud into virtualbox, in settings *enabled PAE* and
> > created *host-only
> > network 192.168.56.1/24 *
> > Devcloud is pinged now.
> >
> > I built and run cloudstack on my local machine with commands
> > *mvn -Pdeveloper,systemvm clean install*
> > *mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
> > *mvn -pl :cloud-client-ui jetty:run*
> >
> >
> > Then I installed marvin and tried to deploy datacenter with command
> > *mvn -P developer -pl tools/devcloud -Ddeploysvr*
> > and got exception
> >
> > * Deploy DC Started *
> > *Exception Occurred :['Traceback (most recent call last):\n', '  File
> > "../marvin/marvin/deployDataCenter.py", line 136, in addHosts\nret =
> > self.__apiClient.addHost(hostcmd)\n', '  File
> >
> >
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
> > line 1572, in addHost\nresponse =
> > self.connection.marvinRequest(command, response_type=response,
> > method=method)\n', '  File
> >
> >
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
> > line 379, in marvinRequest\nraise e\n', 'CloudstackAPIException:
> > Execute cmd: addhost failed, due to: errorCode: 530, errorText:Cannot
> > transit agent status with event AgentDisconnected for host 1, mangement
> > server id is 18413734744879,Unable to transition to a new state from
> > Creating via AgentDisconnected\n']*
> >
> > *===deploy dc failed, so cleaning the created entries===*
> >
> > *DeployDC: CleanUp Started*
> >
> > *Clean Up Entries=== {'Network':
> > [u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
> > [u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
> > [u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
> > [u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
> > [u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster', 'Pod',
> > 'Network', 'PhysicalNetwork', 'Zone']}*
> >
> > *===Removing DataCenter Failed===*
> >
> > As a result in cloudstack were created zone, cluster, pod, and a host
> with
> > state "Alert".
> >
> > Also I run
> > *python tools/marvin/marvin/deployDataCenter.py -i
> > tools/devcloud/devcloud.cfg*
> > that gave same result.
> >
> > What am I doing wrong?
> >
>
>
>
> --
> Daan
>

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

[Daan Hoogland]

Wilder, Borg,

I would be fine with this as a known issue but I'll create a PR to remove
the 4.5 upgrade code anyway. We can put it in 4.6.1 if nothing else pops up.


On Wed, Nov 11, 2015 at 3:00 PM, Wilder Rodrigues <
wrodrig...@schubergphilis.com> wrote:

> Nice to know, Lucian!
>
> Happy testing! :)
>
> Cheers,
> Wilder
>
>
> On 11 Nov 2015, at 14:46, Nux! >
> wrote:
>
> Wilder,
>
> Thanks for that, adding systemvm-kvm-4.5 solved the problem.
> I will do more testing, but the main issue is sorted.
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
> From: "Wilder Rodrigues" >
> To: dev@cloudstack.apache.org
> Sent: Wednesday, 11 November, 2015 12:39:25
> Subject: Re: [VOTE] Apache CloudStack 4.6.0 (round 2)
>
> Hi Nux,
>
> Concerning your second comment:
>
> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the
> 4.5
> systemvm is missing - wtf?
>
> We explained how the upgrade is done in the issue ==>
> https://issues.apache.org/jira/browse/CLOUDSTACK-9046
>
> The current way ACS does the upgrade requires one to follow all the path.
> So,
> going from 4.4.x to 4.6.x requires an upgrade to 4.5.x first. In order to
> avoid
> that, you have to register a SystemVM template 4.5.x before as well.
> That’s how
> I did that and how I have tested. All in the issue above. So, the second
> point
> doesn’t really block the RC.
>
> Now, about your first point, since you already mentioned how to get it
> working,
> I wouldn’t say that’s a blocker, right?
>
> Given the current state of the upgrade path, or how it is implemented,
> perhaps
> the issue you created should be marked either “won’t fix” or be changed
> into an
> improvement.
>
> Cheers,
> Wilder
>
>
> On 11 Nov 2015, at 13:11, Nux!  >> wrote:
>
> -1
>
> I'm testing upgrade from 4.4.1 (what we run in production) to 4.6.0 and
> have hit
> 2 issues.
>
> 1 - minor packaging issue, upgrading to 4.6.0 makes cloudstack-awsapi-4.4.1
> complain about missing deps; rpm -e --nodeps cloudstack-awsapi gets rid of
> the
> problem, perhaps there's a better way to obsolete this package
>
> 2 - after upgrading the packages to 4.6.0, the mgmt server complains the
> 4.5
> systemvm is missing - wtf?
> opened https://issues.apache.org/jira/browse/CLOUDSTACK-9056 for this
> with more
> info
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro>
>
> - Original Message -
> From: "Remi Bergsma" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 10 November, 2015 15:03:03
> Subject: [VOTE] Apache CloudStack 4.6.0 (round 2)
>
> Hi all,
>
> I've created a 4.6.0 release candidate, with the following artifacts up
> for a
> vote:
>
> Git Branch and Commit SH:
>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=4.6.0-RC20151110T1545
>
> Commit: e31ade03c66368c64f0cd66cb7b0b754cddfb79d
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.0/
>
> PGP release keys (signed using A47DDC4F):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for at least 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to indicate
> "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>


-- 
Daan

Re: [RFC] Metrics views for CloudStack UI

[ilya]

I think we need a new set of APIs in the future to retrieve the content
much faster, current API calls in environment with thousands of VMs are
really slow.



On 11/6/15 11:15 AM, Rohit Yadav wrote:
> Hi Wido,
> 
> Yes, we can do that but the issue here is we'll have to make a listVMs call 
> for each zone/host row in respective metric views, such changes would cause 
> several API requests and UI to block for longer times. The other option could 
> be to implement new sets of APIs that aggregate data at the backend so that 
> the client/UI won't need to make several API calls.
> 
> Regards.
> 
> 
> From: Wido den Hollander 
> Sent: Friday, November 6, 2015 1:45 PM
> To: dev@cloudstack.apache.org
> Subject: Re: [RFC] Metrics views for CloudStack UI
> 
> On 11/05/2015 03:09 PM, Rohit Yadav wrote:
>> Hi all,
>>
>> The present CloudStack UI hides most of the metrics data such as cpu, 
>> memory, disk, network usage in inner detail views. Such information is 
>> critical to find issues in one’s cloud, for example finding clusters where 
>> hosts are failing, or finding storage pools where disk space has depleted 
>> beyond configured global or cluster thresholds.
>>
>> The metrics views for CloudStack UI is an attempt to solve those problems 
>> that brings in several UI enhancements such as sortable tables, new status 
>> icons, methods to control breadcrumb navigation, making UI’s global list* 
>> API pagesize dynamic, a new table widget based on listView widget that is 
>> both horizontally and vertically scrollable, supports cell/threshold 
>> coloring, collapsible columns along with navigation from one view to another 
>> and quick-view actions. For example, currently support navigation are: Zone 
>> to Cluster to Host to Instance to Volumes, and Storage Pool to Volumes.
>>
>> The current version implements six resource views for zone, cluster, host, 
>> instance, volume and storage pool (primary storage). The metrics framework 
>> (based on listView widget) would allow developers to write more such view 
>> where information can be densely packed.
>>
>> Please checkout the FS (with some screenshots) and the PR;
>>
>> FS: https://issues.apache.org/jira/browse/CLOUDSTACK-9020
>> JIRA: https://issues.apache.org/jira/browse/CLOUDSTACK-9020
>> PR: https://github.com/apache/cloudstack/pull/1038
>>
>> Comments and suggestions?
>>
> 
> Overall it looks very good, but I personally would like to see the
> amount of Instances per Zone/Host in a Quick overview, that's what
> lacking currently imho.
> 
> Would that be easy to do?
> 
> Wido
> 
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>>
>>
>> [cid:image003.png@01D104EF.CE276C40]
>>
>>
>> M. +91 88 262 30892 | 
>> rohit.ya...@shapeblue.com
>> Blog: bhaisaab.org | Twitter: @_bhaisaab
>> ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
>>
>> Find out more about ShapeBlue and our range of CloudStack related services
>>
>> IaaS Cloud Design & Build
>> CSForge – rapid IaaS deployment framework
>> CloudStack Consulting
>> CloudStack Software 
>> Engineering
>> CloudStack Infrastructure 
>> Support
>> CloudStack Bootcamp Training 
>> Courses
>>
>> 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, you must neither take any action based 
>> upon its contents, nor copy or show it to anyone. Please contact the sender 
>> if you believe you have received this email in error. 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.
>>
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software 
> Engineering
> CloudStack 

Re: UI customization

[ilya]

As per Vadim, plugins is the way to go.

Regards
-ilya


On 11/11/15 6:45 AM, Илья Толстихин wrote:
> Many thanks.
> 
> 2015-11-09 18:29 GMT+06:00 Vadim Kimlaychuk :
> 
>> From my opinion better way is to write a plugin:
>> http://docs.cloudstack.apache.org/en/latest/plugins.html#how-to-write-a-plugin-overview
>> and follow UI customization guide:
>> http://support.citrix.com/servlet/KbServlet/download/38315-102-714564/CloudPlatform_4.5_User_Interface_Customization.pdf
>>
>> Doing direct changes to cloudStack.js that are not contributed back to
>> community is guaranteed way to have problems after updates. You will have
>> to merge your code with mainstream all the time.
>>
>> Another way is to implement your own interface based on CS REST API. Doing
>> that for just 1 button is overkill for sure, but such changes will never
>> end as soon as you start to customize. Think about it. The easiest way is
>> not always the best one.
>>
>> Vadim.
>>
>> On 2015-11-09 13:58, Kshitij Kansal wrote:
>>
>> Hi,
>>>
>>> You can find the scripts at the location
>>> "client/target/generated-webapp/scripts/". Make changes to cloudstack.js
>>> here.
>>>
>>> *Note: *First remove the corresponding gz file(in your case
>>> cloudstack.js.gz) otherwise the changes won't reflect.
>>>
>>> *The changes in generated-webapps are not saved and tracked in git. So
>>> make
>>> sure to copy your changes to actual scripts (in ui/scripts) before finally
>>> rebuilding the project. *The changes made this way are directly reflected
>>>
>>> by refreshing the page.
>>>
>>> PS: Not sure if its the correct way to develop but it works :). Once
>>> again,
>>> make sure to copy your changes back to correct scripts before building the
>>> project.
>>>
>>> Regards,
>>> Kshitij
>>>
>>> On Mon, Nov 9, 2015 at 12:34 PM, Wei ZHOU  wrote:
>>>
>>> I guess you worked on old version before.
>>> in 4.6, something changed. you might compress cloudStack.js to get
>>> cloudStack.js.gz as well.
>>>
>>> 2015-11-08 15:05 GMT+01:00 Илья Толстихин :
>>>
>>> Hi all,
>>>
>>> Could please help with the process of UI customization:
>>>
>>> I have CloudStack 4.6.0 built from the source.
>>> I want to add a custom button to the WebUI menu.
>>> In file ui/scripts/cloudStack.js I added my button into sections array
>>>
>>> *sections: {*
>>> * /***
>>> * * Dashboard*
>>> * */*
>>> * dashboard: {},*
>>> * instances: {},*
>>> * affinityGroups: {},*
>>> * storage: {},*
>>> * network: {},*
>>> * templates: {},*
>>> * events: {},*
>>> * projects: {},*
>>> * accounts: {},*
>>>
>>> * domains: {}, //domain-admin and root-admin only*
>>>
>>> * regions: {}, //root-admin only*
>>> * system: {}, //root-admin only*
>>> * 'global-settings': {}, //root-admin only*
>>> * configuration: {}, //root-admin only*
>>> * plugins: {},*
>>> * test: {*
>>> * title: 'TestSection',*
>>> * id: 'TestSection',*
>>> * show: function(args){*
>>> * var div = document.createElement('div');*
>>> * div.innerHTML = "Test section";*
>>> * return div;*
>>> * }*
>>> * }*
>>> * }*
>>>
>>> and in sectionPreFilter array
>>>
>>> *sectionPreFilter: function(args) {*
>>> * var sections = [];*
>>>
>>> * if (isAdmin()) {*
>>> * sections = ["dashboard", "instances", "storage",
>>> "network", "templates", "accounts", "domains", "events", "system",
>>> "global-settings", "configuration", "projects", "regions",
>>> "affinityGroups", "test"];*
>>>
>>> After building the project, the button was added to the menu, but when I
>>> edit cloudStack.js and restart jetty UI doesn't change, any changes are
>>> applied only after the full project rebuild with command mvn
>>> -Pdeveloper,systemvm -DskipTests clean install.
>>> Jetty version 6.1.26
>>>
>>> How to apply changes without rebuilding project?
>>> And is it possible to apply it without restart Jetty?
>>>
>>
> 

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

[jharshman]

Github user jharshman commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-155946568
  
@DaanHoogland 
Looks like we are voting on RC2 at the moment.  Just wanted to check in and 
make sure that this PR is still slated to be merged after 4.6 releases.


---
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: CID-1338387: remove logicless execution c...

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155882720
  
Cool, 
It seems that the JVM crashed during Jenkins execution.



---
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: DevCloud deployment

[Daan Hoogland]

Ilya, Can you share the log of your management server? There must be some
more info in there. And maybe you can check SRLog on the host To see if
anything happened there at all.

On Wed, Nov 11, 2015 at 6:43 PM, Илья Толстихин  wrote:

> Yes, this is Xen.
>
> 2015-11-11 22:00 GMT+06:00 Daan Hoogland :
>
> > this is a xen host, right?
> >
> > On Wed, Nov 11, 2015 at 3:43 PM, Илья Толстихин 
> > wrote:
> >
> > > On host ssh is up with 22 port only.
> > >
> > > 2015-11-11 20:03 GMT+06:00 Илья Толстихин :
> > >
> > > > Cloudstack 4.6.0
> > > > Yes, I can ssh to host
> > > >
> > > > 2015-11-11 19:41 GMT+06:00 Daan Hoogland :
> > > >
> > > >> what version of ACS?
> > > >> It sounds like a network issue between ms and host? can you ssh from
> > ms
> > > to
> > > >> host on the right port? (i think it was 3922, might be 3022)
> > > >>
> > > >> On Wed, Nov 11, 2015 at 1:42 PM, Илья Толстихин  >
> > > >> wrote:
> > > >>
> > > >> > Hi all,
> > > >> > Could you please help with the DevCloud deployment.
> > > >> >
> > > >> > I imported devcloud into virtualbox, in settings *enabled PAE* and
> > > >> > created *host-only
> > > >> > network 192.168.56.1/24 *
> > > >> > Devcloud is pinged now.
> > > >> >
> > > >> > I built and run cloudstack on my local machine with commands
> > > >> > *mvn -Pdeveloper,systemvm clean install*
> > > >> > *mvn -Pdeveloper -pl developer,tools/devcloud -Ddeploydb*
> > > >> > *mvn -pl :cloud-client-ui jetty:run*
> > > >> >
> > > >> >
> > > >> > Then I installed marvin and tried to deploy datacenter with
> command
> > > >> > *mvn -P developer -pl tools/devcloud -Ddeploysvr*
> > > >> > and got exception
> > > >> >
> > > >> > * Deploy DC Started *
> > > >> > *Exception Occurred :['Traceback (most recent call last):\n', '
> > File
> > > >> > "../marvin/marvin/deployDataCenter.py", line 136, in addHosts\n
> > > ret =
> > > >> > self.__apiClient.addHost(hostcmd)\n', '  File
> > > >> >
> > > >> >
> > > >>
> > >
> >
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackAPI/cloudstackAPIClient.py",
> > > >> > line 1572, in addHost\nresponse =
> > > >> > self.connection.marvinRequest(command, response_type=response,
> > > >> > method=method)\n', '  File
> > > >> >
> > > >> >
> > > >>
> > >
> >
> "/usr/local/lib/python2.7/dist-packages/Marvin-4.6.0_SNAPSHOT-py2.7.egg/marvin/cloudstackConnection.py",
> > > >> > line 379, in marvinRequest\nraise e\n',
> 'CloudstackAPIException:
> > > >> > Execute cmd: addhost failed, due to: errorCode: 530,
> > errorText:Cannot
> > > >> > transit agent status with event AgentDisconnected for host 1,
> > > mangement
> > > >> > server id is 18413734744879,Unable to transition to a new state
> from
> > > >> > Creating via AgentDisconnected\n']*
> > > >> >
> > > >> > *===deploy dc failed, so cleaning the created entries===*
> > > >> >
> > > >> > *DeployDC: CleanUp Started*
> > > >> >
> > > >> > *Clean Up Entries=== {'Network':
> > > >> > [u'5720be8d-f8eb-4dee-af4f-9551409c8722'], 'Zone':
> > > >> > [u'd05cb102-2766-4d59-a94b-0998842b3717'], 'PhysicalNetwork':
> > > >> > [u'd641f7e3-0996-4064-a6ce-54620c0524e2'], 'Cluster':
> > > >> > [u'eb213a95-4b5b-4a7d-9b4d-be1754a3b932'], 'Pod':
> > > >> > [u'659bc864-4d10-48c9-83ef-30ee40f1ce8d'], 'order': ['Cluster',
> > 'Pod',
> > > >> > 'Network', 'PhysicalNetwork', 'Zone']}*
> > > >> >
> > > >> > *===Removing DataCenter Failed===*
> > > >> >
> > > >> > As a result in cloudstack were created zone, cluster, pod, and a
> > host
> > > >> with
> > > >> > state "Alert".
> > > >> >
> > > >> > Also I run
> > > >> > *python tools/marvin/marvin/deployDataCenter.py -i
> > > >> > tools/devcloud/devcloud.cfg*
> > > >> > that gave same result.
> > > >> >
> > > >> > What am I doing wrong?
> > > >> >
> > > >>
> > > >>
> > > >>
> > > >> --
> > > >> Daan
> > > >>
> > > >
> > > >
> > >
> >
> >
> >
> > --
> > Daan
> >
>



-- 
Daan

Re: distribution of Cloudstack API plugins

[Mike Tutkowski]

Good point, Marcus - thanks!

On Wed, Nov 11, 2015 at 12:30 PM, Marcus  wrote:

> Note you don't have to edit commands.properties if your API command has the
> APICommand annotation:
>
> @APICommand(name = "getTimeOfDay",
>
> description = "get time of day",
>
> responseView = ResponseView.Restricted,
>
> responseObject = GetTimeOfDayResponse.class,
>
> authorized = {RoleType.Admin})
>
> On Wed, Nov 11, 2015 at 11:26 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
> > It sounds like you've already developed your API plug-in, so perhaps this
> > video won't be super useful to you, but - just in case - here's my
> > presentation on this topic from the CloudStack Collaboration Conference
> in
> > Dublin, which took place last month:
> >
> >
> >
> https://www.youtube.com/watch?v=J6N1sf43psw=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA=8
> >
> > On Wed, Nov 11, 2015 at 12:23 PM, Mike Tutkowski <
> > mike.tutkow...@solidfire.com> wrote:
> >
> > > No problem. There's actually very little for the customer to do:
> > >
> > > *Add to commands.properties (located in
> > > /usr/share/cloudstack-management/webapps/client/WEB-INF/classes). (The
> > > vendor can provide these lines (API command = permission level) to
> you.)*
> > >
> > > *Place the new JAR file in
> > > /usr/share/cloudstack-management/webapps/client/WEB-INF/lib.*
> > >
> > > *Reboot the management server.*
> > >
> > > On Wed, Nov 11, 2015 at 8:56 AM, Giebner, Sascha <
> > > sascha.gieb...@bautzen-it.de> wrote:
> > >
> > >> Hey Mike,
> > >>
> > >> thanks for the hint. Would you mind sharing these instructions with
> me?
> > ;)
> > >>
> > >> Thx and best regards,
> > >> Sascha
> > >> 
> > >> From: Mike Tutkowski [mike.tutkow...@solidfire.com]
> > >> Sent: Wednesday, November 11, 2015 3:25 PM
> > >> To: dev@cloudstack.apache.org
> > >> Subject: Re: distribution of Cloudstack API plugins
> > >>
> > >> What I have always done is build the JAR file myself and then provide
> to
> > >> the third party instructions on where it goes (and that to make use of
> > it,
> > >> it does require a reboot of the management server).
> > >>
> > >> On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
> > >> sascha.gieb...@bautzen-it.de> wrote:
> > >>
> > >> > Hello everyone,
> > >> >
> > >> > I have a question: After successfully implementing a Cloudstack API
> > >> plugin
> > >> > (no UI plugin), how would one go about sharing the results with a
> > third
> > >> > party?
> > >> >
> > >> > Is the only way to add the changes and rebuild cloudstack?
> > >> >
> > >> > Or is there a possibility to somehow package a plugin and add it
> > >> > dynamically to a running cloudstack instance?
> > >> >
> > >> > Or in general what would be the easiest way to prepare a plugin for
> a
> > >> > simple end-user-installation?
> > >> >
> > >> > Thx a bunch,
> > >> > Sascha Giebner
> > >> >
> > >> > --
> > >> > This email was Virus checked by UTM 9. http://www.sophos.com
> > >> >
> > >>
> > >>
> > >>
> > >> --
> > >> *Mike Tutkowski*
> > >> *Senior CloudStack Developer, SolidFire Inc.*
> > >> e: mike.tutkow...@solidfire.com
> > >> o: 303.746.7302
> > >> Advancing the way the world uses the cloud
> > >> *™*
> > >>
> > >> --
> > >> This email was Virus checked by UTM 9. http://www.sophos.com
> > >>
> > >
> > >
> > >
> > > --
> > > *Mike Tutkowski*
> > > *Senior CloudStack Developer, SolidFire Inc.*
> > > e: mike.tutkow...@solidfire.com
> > > o: 303.746.7302
> > > Advancing the way the world uses the cloud
> > > *™*
> > >
> >
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the cloud
> > *™*
> >
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

[GitHub] cloudstack pull request: CID-1338387: remove logicless execution c...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1056#issuecomment-155887354
  
see INFRA-10703 at the apache jira :) and cast your vote for it


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

Videos from CCCEU 2015 in Dublin

[Mike Tutkowski]

Hi everyone,

I noticed today that the videos from CCCEU 2015 in Dublin are now available:

https://www.youtube.com/playlist?list=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA

It was a really fun conference.

Enjoy!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: bringing jenkins.bac.o down

[Daan Hoogland]

done for now. Please let any problems know at this list

On Wed, Nov 11, 2015 at 8:55 PM, Daan Hoogland 
wrote:

> jenkins restarting again
>
> On Wed, Nov 11, 2015 at 8:45 PM, Daan Hoogland 
> wrote:
>
>> And we're back. I'm not done fiddling though.
>>
>> On Wed, Nov 11, 2015 at 8:34 PM, Daan Hoogland 
>> wrote:
>>
>>> I am bringing down the ci server imminently to update it with the latest
>>> sec patched release.
>>>
>>> please stop me within minutes or pray.
>>>
>>> --
>>> Daan
>>>
>>
>>
>>
>> --
>> Daan
>>
>
>
>
> --
> Daan
>



-- 
Daan

Re: distribution of Cloudstack API plugins

[Mike Tutkowski]

No problem. There's actually very little for the customer to do:

*Add to commands.properties (located in
/usr/share/cloudstack-management/webapps/client/WEB-INF/classes). (The
vendor can provide these lines (API command = permission level) to you.)*

*Place the new JAR file in
/usr/share/cloudstack-management/webapps/client/WEB-INF/lib.*

*Reboot the management server.*

On Wed, Nov 11, 2015 at 8:56 AM, Giebner, Sascha <
sascha.gieb...@bautzen-it.de> wrote:

> Hey Mike,
>
> thanks for the hint. Would you mind sharing these instructions with me? ;)
>
> Thx and best regards,
> Sascha
> 
> From: Mike Tutkowski [mike.tutkow...@solidfire.com]
> Sent: Wednesday, November 11, 2015 3:25 PM
> To: dev@cloudstack.apache.org
> Subject: Re: distribution of Cloudstack API plugins
>
> What I have always done is build the JAR file myself and then provide to
> the third party instructions on where it goes (and that to make use of it,
> it does require a reboot of the management server).
>
> On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
> sascha.gieb...@bautzen-it.de> wrote:
>
> > Hello everyone,
> >
> > I have a question: After successfully implementing a Cloudstack API
> plugin
> > (no UI plugin), how would one go about sharing the results with a third
> > party?
> >
> > Is the only way to add the changes and rebuild cloudstack?
> >
> > Or is there a possibility to somehow package a plugin and add it
> > dynamically to a running cloudstack instance?
> >
> > Or in general what would be the easiest way to prepare a plugin for a
> > simple end-user-installation?
> >
> > Thx a bunch,
> > Sascha Giebner
> >
> > --
> > This email was Virus checked by UTM 9. http://www.sophos.com
> >
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>
> --
> This email was Virus checked by UTM 9. http://www.sophos.com
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: distribution of Cloudstack API plugins

[Marcus]

Note you don't have to edit commands.properties if your API command has the
APICommand annotation:

@APICommand(name = "getTimeOfDay",

description = "get time of day",

responseView = ResponseView.Restricted,

responseObject = GetTimeOfDayResponse.class,

authorized = {RoleType.Admin})

On Wed, Nov 11, 2015 at 11:26 AM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> It sounds like you've already developed your API plug-in, so perhaps this
> video won't be super useful to you, but - just in case - here's my
> presentation on this topic from the CloudStack Collaboration Conference in
> Dublin, which took place last month:
>
>
> https://www.youtube.com/watch?v=J6N1sf43psw=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA=8
>
> On Wed, Nov 11, 2015 at 12:23 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
> > No problem. There's actually very little for the customer to do:
> >
> > *Add to commands.properties (located in
> > /usr/share/cloudstack-management/webapps/client/WEB-INF/classes). (The
> > vendor can provide these lines (API command = permission level) to you.)*
> >
> > *Place the new JAR file in
> > /usr/share/cloudstack-management/webapps/client/WEB-INF/lib.*
> >
> > *Reboot the management server.*
> >
> > On Wed, Nov 11, 2015 at 8:56 AM, Giebner, Sascha <
> > sascha.gieb...@bautzen-it.de> wrote:
> >
> >> Hey Mike,
> >>
> >> thanks for the hint. Would you mind sharing these instructions with me?
> ;)
> >>
> >> Thx and best regards,
> >> Sascha
> >> 
> >> From: Mike Tutkowski [mike.tutkow...@solidfire.com]
> >> Sent: Wednesday, November 11, 2015 3:25 PM
> >> To: dev@cloudstack.apache.org
> >> Subject: Re: distribution of Cloudstack API plugins
> >>
> >> What I have always done is build the JAR file myself and then provide to
> >> the third party instructions on where it goes (and that to make use of
> it,
> >> it does require a reboot of the management server).
> >>
> >> On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
> >> sascha.gieb...@bautzen-it.de> wrote:
> >>
> >> > Hello everyone,
> >> >
> >> > I have a question: After successfully implementing a Cloudstack API
> >> plugin
> >> > (no UI plugin), how would one go about sharing the results with a
> third
> >> > party?
> >> >
> >> > Is the only way to add the changes and rebuild cloudstack?
> >> >
> >> > Or is there a possibility to somehow package a plugin and add it
> >> > dynamically to a running cloudstack instance?
> >> >
> >> > Or in general what would be the easiest way to prepare a plugin for a
> >> > simple end-user-installation?
> >> >
> >> > Thx a bunch,
> >> > Sascha Giebner
> >> >
> >> > --
> >> > This email was Virus checked by UTM 9. http://www.sophos.com
> >> >
> >>
> >>
> >>
> >> --
> >> *Mike Tutkowski*
> >> *Senior CloudStack Developer, SolidFire Inc.*
> >> e: mike.tutkow...@solidfire.com
> >> o: 303.746.7302
> >> Advancing the way the world uses the cloud
> >> *™*
> >>
> >> --
> >> This email was Virus checked by UTM 9. http://www.sophos.com
> >>
> >
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the cloud
> > *™*
> >
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>

Re: distribution of Cloudstack API plugins

[Mike Tutkowski]

The fact that that annotation can be used instead of commands.properties is
not included in my video, as I had learned about that after that
presentation, so it's great you pointed it out as it does make the work of
installing an API plug-in easier. :)

On Wed, Nov 11, 2015 at 1:15 PM, Marcus  wrote:

> I have found managing commands.properties pretty painful, so figuring that
> out was a huge relief.
>
> On Wed, Nov 11, 2015 at 11:32 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
> > Good point, Marcus - thanks!
> >
> > On Wed, Nov 11, 2015 at 12:30 PM, Marcus  wrote:
> >
> > > Note you don't have to edit commands.properties if your API command has
> > the
> > > APICommand annotation:
> > >
> > > @APICommand(name = "getTimeOfDay",
> > >
> > > description = "get time of day",
> > >
> > > responseView = ResponseView.Restricted,
> > >
> > > responseObject = GetTimeOfDayResponse.class,
> > >
> > > authorized = {RoleType.Admin})
> > >
> > > On Wed, Nov 11, 2015 at 11:26 AM, Mike Tutkowski <
> > > mike.tutkow...@solidfire.com> wrote:
> > >
> > > > It sounds like you've already developed your API plug-in, so perhaps
> > this
> > > > video won't be super useful to you, but - just in case - here's my
> > > > presentation on this topic from the CloudStack Collaboration
> Conference
> > > in
> > > > Dublin, which took place last month:
> > > >
> > > >
> > > >
> > >
> >
> https://www.youtube.com/watch?v=J6N1sf43psw=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA=8
> > > >
> > > > On Wed, Nov 11, 2015 at 12:23 PM, Mike Tutkowski <
> > > > mike.tutkow...@solidfire.com> wrote:
> > > >
> > > > > No problem. There's actually very little for the customer to do:
> > > > >
> > > > > *Add to commands.properties (located in
> > > > > /usr/share/cloudstack-management/webapps/client/WEB-INF/classes).
> > (The
> > > > > vendor can provide these lines (API command = permission level) to
> > > you.)*
> > > > >
> > > > > *Place the new JAR file in
> > > > > /usr/share/cloudstack-management/webapps/client/WEB-INF/lib.*
> > > > >
> > > > > *Reboot the management server.*
> > > > >
> > > > > On Wed, Nov 11, 2015 at 8:56 AM, Giebner, Sascha <
> > > > > sascha.gieb...@bautzen-it.de> wrote:
> > > > >
> > > > >> Hey Mike,
> > > > >>
> > > > >> thanks for the hint. Would you mind sharing these instructions
> with
> > > me?
> > > > ;)
> > > > >>
> > > > >> Thx and best regards,
> > > > >> Sascha
> > > > >> 
> > > > >> From: Mike Tutkowski [mike.tutkow...@solidfire.com]
> > > > >> Sent: Wednesday, November 11, 2015 3:25 PM
> > > > >> To: dev@cloudstack.apache.org
> > > > >> Subject: Re: distribution of Cloudstack API plugins
> > > > >>
> > > > >> What I have always done is build the JAR file myself and then
> > provide
> > > to
> > > > >> the third party instructions on where it goes (and that to make
> use
> > of
> > > > it,
> > > > >> it does require a reboot of the management server).
> > > > >>
> > > > >> On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
> > > > >> sascha.gieb...@bautzen-it.de> wrote:
> > > > >>
> > > > >> > Hello everyone,
> > > > >> >
> > > > >> > I have a question: After successfully implementing a Cloudstack
> > API
> > > > >> plugin
> > > > >> > (no UI plugin), how would one go about sharing the results with
> a
> > > > third
> > > > >> > party?
> > > > >> >
> > > > >> > Is the only way to add the changes and rebuild cloudstack?
> > > > >> >
> > > > >> > Or is there a possibility to somehow package a plugin and add it
> > > > >> > dynamically to a running cloudstack instance?
> > > > >> >
> > > > >> > Or in general what would be the easiest way to prepare a plugin
> > for
> > > a
> > > > >> > simple end-user-installation?
> > > > >> >
> > > > >> > Thx a bunch,
> > > > >> > Sascha Giebner
> > > > >> >
> > > > >> > --
> > > > >> > This email was Virus checked by UTM 9. http://www.sophos.com
> > > > >> >
> > > > >>
> > > > >>
> > > > >>
> > > > >> --
> > > > >> *Mike Tutkowski*
> > > > >> *Senior CloudStack Developer, SolidFire Inc.*
> > > > >> e: mike.tutkow...@solidfire.com
> > > > >> o: 303.746.7302
> > > > >> Advancing the way the world uses the cloud
> > > > >> *™*
> > > > >>
> > > > >> --
> > > > >> This email was Virus checked by UTM 9. http://www.sophos.com
> > > > >>
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > *Mike Tutkowski*
> > > > > *Senior CloudStack Developer, SolidFire Inc.*
> > > > > e: mike.tutkow...@solidfire.com
> > > > > o: 303.746.7302
> > > > > Advancing the way the world uses the cloud
> > > > > *™*
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > *Mike Tutkowski*
> > > > *Senior CloudStack Developer, SolidFire Inc.*
> > > > e: mike.tutkow...@solidfire.com
> > > > o: 303.746.7302
> > > > Advancing the way the world uses the cloud
> > > > 

bringing jenkins.bac.o down

[Daan Hoogland]

I am bringing down the ci server imminently to update it with the latest
sec patched release.

please stop me within minutes or pray.

-- 
Daan

Re: Storage plugin docs?

[Mike Tutkowski]

Feel free to use the SolidFire storage plug-ins as examples.

There are two:

1) Called "SolidFire" - makes use of managed storage (1:1 mapping between a
backend volume and a virtual disk) and is zone wide.

2) Called "SolidFireShared" - makes use of standard (non-managed storage)
(1:M mapping between a backend volume and virtual disks) and is cluster
scoped.

Talk to you later!
Mike

On Wed, Nov 11, 2015 at 9:03 AM, Nux!  wrote:

> Hello,
>
> If someone wanted to write a storage plugin for Cloudstack, is there any
> documentation and/or examples that I could point this person to?
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: distribution of Cloudstack API plugins

[Marcus]

I have found managing commands.properties pretty painful, so figuring that
out was a huge relief.

On Wed, Nov 11, 2015 at 11:32 AM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Good point, Marcus - thanks!
>
> On Wed, Nov 11, 2015 at 12:30 PM, Marcus  wrote:
>
> > Note you don't have to edit commands.properties if your API command has
> the
> > APICommand annotation:
> >
> > @APICommand(name = "getTimeOfDay",
> >
> > description = "get time of day",
> >
> > responseView = ResponseView.Restricted,
> >
> > responseObject = GetTimeOfDayResponse.class,
> >
> > authorized = {RoleType.Admin})
> >
> > On Wed, Nov 11, 2015 at 11:26 AM, Mike Tutkowski <
> > mike.tutkow...@solidfire.com> wrote:
> >
> > > It sounds like you've already developed your API plug-in, so perhaps
> this
> > > video won't be super useful to you, but - just in case - here's my
> > > presentation on this topic from the CloudStack Collaboration Conference
> > in
> > > Dublin, which took place last month:
> > >
> > >
> > >
> >
> https://www.youtube.com/watch?v=J6N1sf43psw=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA=8
> > >
> > > On Wed, Nov 11, 2015 at 12:23 PM, Mike Tutkowski <
> > > mike.tutkow...@solidfire.com> wrote:
> > >
> > > > No problem. There's actually very little for the customer to do:
> > > >
> > > > *Add to commands.properties (located in
> > > > /usr/share/cloudstack-management/webapps/client/WEB-INF/classes).
> (The
> > > > vendor can provide these lines (API command = permission level) to
> > you.)*
> > > >
> > > > *Place the new JAR file in
> > > > /usr/share/cloudstack-management/webapps/client/WEB-INF/lib.*
> > > >
> > > > *Reboot the management server.*
> > > >
> > > > On Wed, Nov 11, 2015 at 8:56 AM, Giebner, Sascha <
> > > > sascha.gieb...@bautzen-it.de> wrote:
> > > >
> > > >> Hey Mike,
> > > >>
> > > >> thanks for the hint. Would you mind sharing these instructions with
> > me?
> > > ;)
> > > >>
> > > >> Thx and best regards,
> > > >> Sascha
> > > >> 
> > > >> From: Mike Tutkowski [mike.tutkow...@solidfire.com]
> > > >> Sent: Wednesday, November 11, 2015 3:25 PM
> > > >> To: dev@cloudstack.apache.org
> > > >> Subject: Re: distribution of Cloudstack API plugins
> > > >>
> > > >> What I have always done is build the JAR file myself and then
> provide
> > to
> > > >> the third party instructions on where it goes (and that to make use
> of
> > > it,
> > > >> it does require a reboot of the management server).
> > > >>
> > > >> On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
> > > >> sascha.gieb...@bautzen-it.de> wrote:
> > > >>
> > > >> > Hello everyone,
> > > >> >
> > > >> > I have a question: After successfully implementing a Cloudstack
> API
> > > >> plugin
> > > >> > (no UI plugin), how would one go about sharing the results with a
> > > third
> > > >> > party?
> > > >> >
> > > >> > Is the only way to add the changes and rebuild cloudstack?
> > > >> >
> > > >> > Or is there a possibility to somehow package a plugin and add it
> > > >> > dynamically to a running cloudstack instance?
> > > >> >
> > > >> > Or in general what would be the easiest way to prepare a plugin
> for
> > a
> > > >> > simple end-user-installation?
> > > >> >
> > > >> > Thx a bunch,
> > > >> > Sascha Giebner
> > > >> >
> > > >> > --
> > > >> > This email was Virus checked by UTM 9. http://www.sophos.com
> > > >> >
> > > >>
> > > >>
> > > >>
> > > >> --
> > > >> *Mike Tutkowski*
> > > >> *Senior CloudStack Developer, SolidFire Inc.*
> > > >> e: mike.tutkow...@solidfire.com
> > > >> o: 303.746.7302
> > > >> Advancing the way the world uses the cloud
> > > >> *™*
> > > >>
> > > >> --
> > > >> This email was Virus checked by UTM 9. http://www.sophos.com
> > > >>
> > > >
> > > >
> > > >
> > > > --
> > > > *Mike Tutkowski*
> > > > *Senior CloudStack Developer, SolidFire Inc.*
> > > > e: mike.tutkow...@solidfire.com
> > > > o: 303.746.7302
> > > > Advancing the way the world uses the cloud
> > > > *™*
> > > >
> > >
> > >
> > >
> > > --
> > > *Mike Tutkowski*
> > > *Senior CloudStack Developer, SolidFire Inc.*
> > > e: mike.tutkow...@solidfire.com
> > > o: 303.746.7302
> > > Advancing the way the world uses the cloud
> > > *™*
> > >
> >
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>

Re: Storage plugin docs?

[Mike Tutkowski]

The SolidFireShared (1:M) plug-in allows you to pass in info to the
createStoragePool API (via the URL field and use of the applicable provider
(i.e. plug-in)) so that the storage plug-in creates a volume on its SAN
(SolidFire, of course, for me), then tells the hypervisor in question (for
this plug-in, XenServer or VMware) to consume the new volume with an SR
(XenServer) or a datastore (VMware).

In the past, you would have needed to manually create a volume on your SAN,
then manually tell XenServer or VMware to make use of it. At that point,
you'd be able to issue a createStoragePool command to CloudStack (and
leverage the default CloudStack storage plug-in).

This SolidFireShared plug-in also allows you to control IOPS for the
backend volume, as well. If you decide you need more performance to the
volume, you can issue that change request via a CloudStack API command and
the storage plug-in will make the necessary changes on the SolidFire SAN
for the volume in question.

The data path protocol to the SolidFire volume is iSCSI.

On Wed, Nov 11, 2015 at 1:48 PM, Nux!  wrote:

> Thanks a lot, Mike. If this ever happens I'll point him to your plug-in.
>
> As a curiosity, is the 1:M a way to share a single solidfire volume
> between multiple VMs/users? If yes, what protocol do you expose the volume
> through?
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
> > From: "Mike Tutkowski" 
> > To: dev@cloudstack.apache.org
> > Sent: Wednesday, 11 November, 2015 20:09:43
> > Subject: Re: Storage plugin docs?
>
> > Feel free to use the SolidFire storage plug-ins as examples.
> >
> > There are two:
> >
> > 1) Called "SolidFire" - makes use of managed storage (1:1 mapping
> between a
> > backend volume and a virtual disk) and is zone wide.
> >
> > 2) Called "SolidFireShared" - makes use of standard (non-managed storage)
> > (1:M mapping between a backend volume and virtual disks) and is cluster
> > scoped.
> >
> > Talk to you later!
> > Mike
> >
> > On Wed, Nov 11, 2015 at 9:03 AM, Nux!  wrote:
> >
> >> Hello,
> >>
> >> If someone wanted to write a storage plugin for Cloudstack, is there any
> >> documentation and/or examples that I could point this person to?
> >>
> >> Lucian
> >>
> >> --
> >> Sent from the Delta quadrant using Borg technology!
> >>
> >> Nux!
> >> www.nux.ro
> >>
> >
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the cloud
> > *™*
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: [DISCUSS] upgrade of system templates

[Daan Hoogland]

Rafael, check the tools/appliance dir or the systemvm template builds in
jenkins. you'll find the prereqs in there.

On Wed, Nov 11, 2015 at 9:54 PM, Rafael Weingärtner <
rafaelweingart...@gmail.com> wrote:

> Hi Daan,
> That is something I would like to use. However, there is somethings that I
> do not fully understand when we create/customize the system VMs.
>
> What does an Operating System need to work as a system VM? I mean, does it
> need an agent to be installed into the VM? Does it need some version of
> some software to be installed (Java, Python, nfs-commons and others)? Does
> it need to be password enabled?
>
> On Wed, Nov 11, 2015 at 6:48 PM, Daan Hoogland 
> wrote:
>
> > Devs,
> >
> > This has bitten us in indecent places time after time and after some
> > contemplating the solution seems quite simple. An api call implemented in
> > the TemplateService that lets you MarkTemplateAsSystemVmTemplate(optional
> > uuid, optional name). No other params are needed as the template should
> > allready contain most information. The actual marking code is implemented
> > several upgrade versions and only needs to simplified from that
> > implementation.
> >
> > Do I miss something?
> >
> > --
> > Daan
> >
>
>
>
> --
> Rafael Weingärtner
>



-- 
Daan

Re: [DISCUSS] upgrade of system templates

[Daan Hoogland]

sure, but I never read docs. here is the job script:
```
whoami
export PATH=/home/jenkins/.rvm/bin:$PATH
export rvm_path=/home/jenkins/.rvm
export HOME=/home/jenkins/

cd tools/appliance
if [ -d iso ]; then
rm -fvr iso
fi
if [ -d dist ]; then
rm -fvr dist
fi
if [ -d box ]; then
rm -fvr box
fi
if [ -d /home/jenkins/iso ]; then
cp -rv /home/jenkins/iso .
fi

if [ ! -d iso ]; then
mkdir iso
ln -s $WORKSPACE/*.iso iso/
fi


export clean_vbox=1
export BUILD_NUMBER=
export version=4.6.0
export branch=master
chmod +x build.sh
./build.sh systemvm64template
```

On Wed, Nov 11, 2015 at 10:05 PM, Rafael Weingärtner <
rafaelweingart...@gmail.com> wrote:

> nice thanks.
> Do not we have a doc page for that?
>
> On Wed, Nov 11, 2015 at 7:03 PM, Daan Hoogland 
> wrote:
>
> > Rafael, check the tools/appliance dir or the systemvm template builds in
> > jenkins. you'll find the prereqs in there.
> >
> > On Wed, Nov 11, 2015 at 9:54 PM, Rafael Weingärtner <
> > rafaelweingart...@gmail.com> wrote:
> >
> > > Hi Daan,
> > > That is something I would like to use. However, there is somethings
> that
> > I
> > > do not fully understand when we create/customize the system VMs.
> > >
> > > What does an Operating System need to work as a system VM? I mean, does
> > it
> > > need an agent to be installed into the VM? Does it need some version of
> > > some software to be installed (Java, Python, nfs-commons and others)?
> > Does
> > > it need to be password enabled?
> > >
> > > On Wed, Nov 11, 2015 at 6:48 PM, Daan Hoogland <
> daan.hoogl...@gmail.com>
> > > wrote:
> > >
> > > > Devs,
> > > >
> > > > This has bitten us in indecent places time after time and after some
> > > > contemplating the solution seems quite simple. An api call
> implemented
> > in
> > > > the TemplateService that lets you
> > MarkTemplateAsSystemVmTemplate(optional
> > > > uuid, optional name). No other params are needed as the template
> should
> > > > allready contain most information. The actual marking code is
> > implemented
> > > > several upgrade versions and only needs to simplified from that
> > > > implementation.
> > > >
> > > > Do I miss something?
> > > >
> > > > --
> > > > Daan
> > > >
> > >
> > >
> > >
> > > --
> > > Rafael Weingärtner
> > >
> >
> >
> >
> > --
> > Daan
> >
>
>
>
> --
> Rafael Weingärtner
>



-- 
Daan

Re: distribution of Cloudstack API plugins

[Mike Tutkowski]

It sounds like you've already developed your API plug-in, so perhaps this
video won't be super useful to you, but - just in case - here's my
presentation on this topic from the CloudStack Collaboration Conference in
Dublin, which took place last month:

https://www.youtube.com/watch?v=J6N1sf43psw=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA=8

On Wed, Nov 11, 2015 at 12:23 PM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> No problem. There's actually very little for the customer to do:
>
> *Add to commands.properties (located in
> /usr/share/cloudstack-management/webapps/client/WEB-INF/classes). (The
> vendor can provide these lines (API command = permission level) to you.)*
>
> *Place the new JAR file in
> /usr/share/cloudstack-management/webapps/client/WEB-INF/lib.*
>
> *Reboot the management server.*
>
> On Wed, Nov 11, 2015 at 8:56 AM, Giebner, Sascha <
> sascha.gieb...@bautzen-it.de> wrote:
>
>> Hey Mike,
>>
>> thanks for the hint. Would you mind sharing these instructions with me? ;)
>>
>> Thx and best regards,
>> Sascha
>> 
>> From: Mike Tutkowski [mike.tutkow...@solidfire.com]
>> Sent: Wednesday, November 11, 2015 3:25 PM
>> To: dev@cloudstack.apache.org
>> Subject: Re: distribution of Cloudstack API plugins
>>
>> What I have always done is build the JAR file myself and then provide to
>> the third party instructions on where it goes (and that to make use of it,
>> it does require a reboot of the management server).
>>
>> On Wed, Nov 11, 2015 at 6:54 AM, Giebner, Sascha <
>> sascha.gieb...@bautzen-it.de> wrote:
>>
>> > Hello everyone,
>> >
>> > I have a question: After successfully implementing a Cloudstack API
>> plugin
>> > (no UI plugin), how would one go about sharing the results with a third
>> > party?
>> >
>> > Is the only way to add the changes and rebuild cloudstack?
>> >
>> > Or is there a possibility to somehow package a plugin and add it
>> > dynamically to a running cloudstack instance?
>> >
>> > Or in general what would be the easiest way to prepare a plugin for a
>> > simple end-user-installation?
>> >
>> > Thx a bunch,
>> > Sascha Giebner
>> >
>> > --
>> > This email was Virus checked by UTM 9. http://www.sophos.com
>> >
>>
>>
>>
>> --
>> *Mike Tutkowski*
>> *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkow...@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the cloud
>> *™*
>>
>> --
>> This email was Virus checked by UTM 9. http://www.sophos.com
>>
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: bringing jenkins.bac.o down

[Daan Hoogland]

And we're back. I'm not done fiddling though.

On Wed, Nov 11, 2015 at 8:34 PM, Daan Hoogland 
wrote:

> I am bringing down the ci server imminently to update it with the latest
> sec patched release.
>
> please stop me within minutes or pray.
>
> --
> Daan
>



-- 
Daan

Re: bringing jenkins.bac.o down

[Daan Hoogland]

jenkins restarting again

On Wed, Nov 11, 2015 at 8:45 PM, Daan Hoogland 
wrote:

> And we're back. I'm not done fiddling though.
>
> On Wed, Nov 11, 2015 at 8:34 PM, Daan Hoogland 
> wrote:
>
>> I am bringing down the ci server imminently to update it with the latest
>> sec patched release.
>>
>> please stop me within minutes or pray.
>>
>> --
>> Daan
>>
>
>
>
> --
> Daan
>



-- 
Daan

[DISCUSS] upgrade of system templates

[Daan Hoogland]

Devs,

This has bitten us in indecent places time after time and after some
contemplating the solution seems quite simple. An api call implemented in
the TemplateService that lets you MarkTemplateAsSystemVmTemplate(optional
uuid, optional name). No other params are needed as the template should
allready contain most information. The actual marking code is implemented
several upgrade versions and only needs to simplified from that
implementation.

Do I miss something?

-- 
Daan

Re: Storage plugin docs?

[Nux!]

Thanks a lot, Mike. If this ever happens I'll point him to your plug-in.

As a curiosity, is the 1:M a way to share a single solidfire volume between 
multiple VMs/users? If yes, what protocol do you expose the volume through? 

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Mike Tutkowski" 
> To: dev@cloudstack.apache.org
> Sent: Wednesday, 11 November, 2015 20:09:43
> Subject: Re: Storage plugin docs?

> Feel free to use the SolidFire storage plug-ins as examples.
> 
> There are two:
> 
> 1) Called "SolidFire" - makes use of managed storage (1:1 mapping between a
> backend volume and a virtual disk) and is zone wide.
> 
> 2) Called "SolidFireShared" - makes use of standard (non-managed storage)
> (1:M mapping between a backend volume and virtual disks) and is cluster
> scoped.
> 
> Talk to you later!
> Mike
> 
> On Wed, Nov 11, 2015 at 9:03 AM, Nux!  wrote:
> 
>> Hello,
>>
>> If someone wanted to write a storage plugin for Cloudstack, is there any
>> documentation and/or examples that I could point this person to?
>>
>> Lucian
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro
>>
> 
> 
> 
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*

Re: Videos from CCCEU 2015 in Dublin

[Mike Tutkowski]

It's not just fun and games at these conferences, though. :) To see how
dangerous is can be to present at a CloudStack Collaboration Conference,
please check out 7:44 in this video of mine:

https://www.youtube.com/watch?v=7I4Oa4KcawE=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA=18

On Wed, Nov 11, 2015 at 12:59 PM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Hi everyone,
>
> I noticed today that the videos from CCCEU 2015 in Dublin are now
> available:
>
> https://www.youtube.com/playlist?list=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA
>
> It was a really fun conference.
>
> Enjoy!
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*

Re: [DISCUSS] upgrade of system templates

[Rafael Weingärtner]

Hi Daan,
That is something I would like to use. However, there is somethings that I
do not fully understand when we create/customize the system VMs.

What does an Operating System need to work as a system VM? I mean, does it
need an agent to be installed into the VM? Does it need some version of
some software to be installed (Java, Python, nfs-commons and others)? Does
it need to be password enabled?

On Wed, Nov 11, 2015 at 6:48 PM, Daan Hoogland 
wrote:

> Devs,
>
> This has bitten us in indecent places time after time and after some
> contemplating the solution seems quite simple. An api call implemented in
> the TemplateService that lets you MarkTemplateAsSystemVmTemplate(optional
> uuid, optional name). No other params are needed as the template should
> allready contain most information. The actual marking code is implemented
> several upgrade versions and only needs to simplified from that
> implementation.
>
> Do I miss something?
>
> --
> Daan
>



-- 
Rafael Weingärtner

Re: [DISCUSS] upgrade of system templates

[Rafael Weingärtner]

nice thanks.
Do not we have a doc page for that?

On Wed, Nov 11, 2015 at 7:03 PM, Daan Hoogland 
wrote:

> Rafael, check the tools/appliance dir or the systemvm template builds in
> jenkins. you'll find the prereqs in there.
>
> On Wed, Nov 11, 2015 at 9:54 PM, Rafael Weingärtner <
> rafaelweingart...@gmail.com> wrote:
>
> > Hi Daan,
> > That is something I would like to use. However, there is somethings that
> I
> > do not fully understand when we create/customize the system VMs.
> >
> > What does an Operating System need to work as a system VM? I mean, does
> it
> > need an agent to be installed into the VM? Does it need some version of
> > some software to be installed (Java, Python, nfs-commons and others)?
> Does
> > it need to be password enabled?
> >
> > On Wed, Nov 11, 2015 at 6:48 PM, Daan Hoogland 
> > wrote:
> >
> > > Devs,
> > >
> > > This has bitten us in indecent places time after time and after some
> > > contemplating the solution seems quite simple. An api call implemented
> in
> > > the TemplateService that lets you
> MarkTemplateAsSystemVmTemplate(optional
> > > uuid, optional name). No other params are needed as the template should
> > > allready contain most information. The actual marking code is
> implemented
> > > several upgrade versions and only needs to simplified from that
> > > implementation.
> > >
> > > Do I miss something?
> > >
> > > --
> > > Daan
> > >
> >
> >
> >
> > --
> > Rafael Weingärtner
> >
>
>
>
> --
> Daan
>



-- 
Rafael Weingärtner

Re: "Unable to add the host" error, CS host XS

[B Prakash]

Thanks Mike, Remi.  Cluster was up.  I followed your instructions and got it 
working!!  three changes 1. changing network configuration to bridge and 
2.increasing the xenserver heatbeat timeout 3.global settings for 
secstorage.allowed.internal.sites and system.vm.use.local.storage... got the 
setup working. 
Thanks againg!  I really appreciate your patience and help.
- Prakash.

From: Remi Bergsma 
Sent: Wednesday, November 11, 2015 12:38 AM
To: dev@cloudstack.apache.org
Subject: Re: "Unable to add the host" error, CS host XS

Make sure management server can talk to the xenserver (xapi) over port 443 
(firewalls?). Right now it says it cannot due to a timeout:

Failed to read server's response: connect timed out

@mike From the logs this seems the first host to be added to this zone.

Sent from my iPhone

> On 11 Nov 2015, at 04:36, B Prakash  wrote:
>
> There are two more exceptions printed in vmops.log as follows;
>
> 2015-11-10 19:12:55,075 DEBUG [c.c.a.ApiServlet] 
> (1034892751@qtp-1967400458-7:ctx-23ec1109) ===START===  0:0:0:0:0:0:0:1 -- 
> POST  command=addHost=json
> 2015-11-10 19:12:55,124 WARN  [c.c.a.d.ParamGenericValidationWorker] 
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) Received unknown 
> parameters for command addHost. Unknown parameters : clustertype
> 2015-11-10 19:12:55,160 INFO  [c.c.r.ResourceManagerImpl] 
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) Trying to add a new 
> host at http://192.168.0.19 in data center 1
> 2015-11-10 19:12:56,041 DEBUG [c.c.c.ConsoleProxyManagerImpl] 
> (consoleproxy-1:ctx-620ccc56) Skip capacity scan as there is no Primary 
> Storage in 'Up' state
> 2015-11-10 19:12:56,436 DEBUG [o.a.c.s.SecondaryStorageManagerImpl] 
> (secstorage-1:ctx-8341054e) Zone 1 is not ready to launch secondary storage 
> VM yet
> 2015-11-10 19:12:57,366 DEBUG [c.c.h.d.HostDaoImpl] (ClusteredAgentManager 
> Timer:ctx-c19ba5d9) Resetting hosts suitable for reconnect
> 2015-11-10 19:12:57,370 DEBUG [c.c.h.d.HostDaoImpl] (ClusteredAgentManager 
> Timer:ctx-c19ba5d9) Completed resetting hosts suitable for reconnect
> 2015-11-10 19:12:57,370 DEBUG [c.c.h.d.HostDaoImpl] (ClusteredAgentManager 
> Timer:ctx-c19ba5d9) Acquiring hosts for clusters already owned by this 
> management server
> 2015-11-10 19:12:57,372 DEBUG [c.c.h.d.HostDaoImpl] (ClusteredAgentManager 
> Timer:ctx-c19ba5d9) Completed acquiring hosts for clusters already owned by 
> this management server
> 2015-11-10 19:12:57,372 DEBUG [c.c.h.d.HostDaoImpl] (ClusteredAgentManager 
> Timer:ctx-c19ba5d9) Acquiring hosts for clusters not owned by any management 
> server
> 2015-11-10 19:12:57,373 DEBUG [c.c.h.d.HostDaoImpl] (ClusteredAgentManager 
> Timer:ctx-c19ba5d9) Completed acquiring hosts for clusters not owned by any 
> management server
> 2015-11-10 19:13:00,405 DEBUG [c.c.h.x.r.XenServerConnectionPool] 
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) Unable to create 
> master connection to host(192.168.0.19) , due to 
> org.apache.xmlrpc.XmlRpcException: Failed to read server's response: connect 
> timed out
> 2015-11-10 19:13:00,406 DEBUG [c.c.h.x.d.XcpServerDiscoverer] 
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) other exceptions: 
> com.cloud.utils.exception.CloudRuntimeException: Unable to create master 
> connection to host(192.168.0.19) , due to org.apache.xmlrpc.XmlRpcException: 
> Failed to read server's response: connect timed out
> com.cloud.utils.exception.CloudRuntimeException: Unable to create master 
> connection to host(192.168.0.19) , due to org.apache.xmlrpc.XmlRpcException: 
> Failed to read server's response: connect timed out
>at 
> com.cloud.hypervisor.xenserver.resource.XenServerConnectionPool.getConnect(XenServerConnectionPool.java:168)
>at 
> com.cloud.hypervisor.xenserver.discoverer.XcpServerDiscoverer.find(XcpServerDiscoverer.java:217)
>at 
> com.cloud.resource.ResourceManagerImpl.discoverHostsFull(ResourceManagerImpl.java:748)
>at 
> com.cloud.resource.ResourceManagerImpl.discoverHosts(ResourceManagerImpl.java:599)
>at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
>at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>at java.lang.reflect.Method.invoke(Method.java:497)
>at 
> org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
>at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
>at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
>at 
> org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
>at 
> 

Re: "Unable to add the host" error, CS host XS

[Mike Tutkowski]

Glad to hear it's working now. :)

On Wednesday, November 11, 2015, B Prakash  wrote:

> Thanks Mike, Remi.  Cluster was up.  I followed your instructions and got
> it working!!  three changes 1. changing network configuration to bridge and
> 2.increasing the xenserver heatbeat timeout 3.global settings for
> secstorage.allowed.internal.sites and system.vm.use.local.storage... got
> the setup working.
> Thanks againg!  I really appreciate your patience and help.
> - Prakash.
> 
> From: Remi Bergsma >
> Sent: Wednesday, November 11, 2015 12:38 AM
> To: dev@cloudstack.apache.org 
> Subject: Re: "Unable to add the host" error, CS host XS
>
> Make sure management server can talk to the xenserver (xapi) over port 443
> (firewalls?). Right now it says it cannot due to a timeout:
>
> Failed to read server's response: connect timed out
>
> @mike From the logs this seems the first host to be added to this zone.
>
> Sent from my iPhone
>
> > On 11 Nov 2015, at 04:36, B Prakash >
> wrote:
> >
> > There are two more exceptions printed in vmops.log as follows;
> >
> > 2015-11-10 19:12:55,075 DEBUG [c.c.a.ApiServlet]
> (1034892751@qtp-1967400458-7:ctx-23ec1109) ===START===  0:0:0:0:0:0:0:1
> -- POST  command=addHost=json
> > 2015-11-10 19:12:55,124 WARN  [c.c.a.d.ParamGenericValidationWorker]
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) Received unknown
> parameters for command addHost. Unknown parameters : clustertype
> > 2015-11-10 19:12:55,160 INFO  [c.c.r.ResourceManagerImpl]
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) Trying to add a
> new host at http://192.168.0.19 in data center 1
> > 2015-11-10 19:12:56,041 DEBUG [c.c.c.ConsoleProxyManagerImpl]
> (consoleproxy-1:ctx-620ccc56) Skip capacity scan as there is no Primary
> Storage in 'Up' state
> > 2015-11-10 19:12:56,436 DEBUG [o.a.c.s.SecondaryStorageManagerImpl]
> (secstorage-1:ctx-8341054e) Zone 1 is not ready to launch secondary storage
> VM yet
> > 2015-11-10 19:12:57,366 DEBUG [c.c.h.d.HostDaoImpl]
> (ClusteredAgentManager Timer:ctx-c19ba5d9) Resetting hosts suitable for
> reconnect
> > 2015-11-10 19:12:57,370 DEBUG [c.c.h.d.HostDaoImpl]
> (ClusteredAgentManager Timer:ctx-c19ba5d9) Completed resetting hosts
> suitable for reconnect
> > 2015-11-10 19:12:57,370 DEBUG [c.c.h.d.HostDaoImpl]
> (ClusteredAgentManager Timer:ctx-c19ba5d9) Acquiring hosts for clusters
> already owned by this management server
> > 2015-11-10 19:12:57,372 DEBUG [c.c.h.d.HostDaoImpl]
> (ClusteredAgentManager Timer:ctx-c19ba5d9) Completed acquiring hosts for
> clusters already owned by this management server
> > 2015-11-10 19:12:57,372 DEBUG [c.c.h.d.HostDaoImpl]
> (ClusteredAgentManager Timer:ctx-c19ba5d9) Acquiring hosts for clusters not
> owned by any management server
> > 2015-11-10 19:12:57,373 DEBUG [c.c.h.d.HostDaoImpl]
> (ClusteredAgentManager Timer:ctx-c19ba5d9) Completed acquiring hosts for
> clusters not owned by any management server
> > 2015-11-10 19:13:00,405 DEBUG [c.c.h.x.r.XenServerConnectionPool]
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) Unable to create
> master connection to host(192.168.0.19) , due to
> org.apache.xmlrpc.XmlRpcException: Failed to read server's response:
> connect timed out
> > 2015-11-10 19:13:00,406 DEBUG [c.c.h.x.d.XcpServerDiscoverer]
> (1034892751@qtp-1967400458-7:ctx-23ec1109 ctx-beece821) other exceptions:
> com.cloud.utils.exception.CloudRuntimeException: Unable to create master
> connection to host(192.168.0.19) , due to
> org.apache.xmlrpc.XmlRpcException: Failed to read server's response:
> connect timed out
> > com.cloud.utils.exception.CloudRuntimeException: Unable to create master
> connection to host(192.168.0.19) , due to
> org.apache.xmlrpc.XmlRpcException: Failed to read server's response:
> connect timed out
> >at
> com.cloud.hypervisor.xenserver.resource.XenServerConnectionPool.getConnect(XenServerConnectionPool.java:168)
> >at
> com.cloud.hypervisor.xenserver.discoverer.XcpServerDiscoverer.find(XcpServerDiscoverer.java:217)
> >at
> com.cloud.resource.ResourceManagerImpl.discoverHostsFull(ResourceManagerImpl.java:748)
> >at
> com.cloud.resource.ResourceManagerImpl.discoverHosts(ResourceManagerImpl.java:599)
> >at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> >at
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> >at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> >at java.lang.reflect.Method.invoke(Method.java:497)
> >at
> org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
> >at
> org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
> >at
>