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

2015-11-11 Thread 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 e

[GitHub] cloudstack pull request: Shuffling the password to avoid having a ...

2015-11-11 Thread 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

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

2015-11-11 Thread 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 appe

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

2015-11-11 Thread 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 se

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 repl

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 G

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 thi

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 G

Re: [ANNOUNCE] New committer: Boris Schrijver

2015-11-11 Thread 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-N

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 S

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 St

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 Strin

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 Stri

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 St

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 Strin

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 St

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

2015-11-11 Thread 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, ha

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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 Strin

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

2015-11-11 Thread 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 le

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

2015-11-11 Thread 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

[GitHub] cloudstack pull request: Shuffling the password to avoid having a ...

2015-11-11 Thread 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 thi

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-11-11 Thread 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/clouds

[GitHub] cloudstack pull request: CLOUDSTACK-9054 use of google-optional as...

2015-11-11 Thread 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 thro

[GitHub] cloudstack pull request: Shuffling the password to avoid having a ...

2015-11-11 Thread 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

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-11-11 Thread 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

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

2015-11-11 Thread 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 A

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-11-11 Thread 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 updateRoutersRedundan

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-11-11 Thread 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 t

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

2015-11-11 Thread 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] cloudstack pull request: CLOUDSTACK-5822: keep user-added sshkeys ...

2015-11-11 Thread 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 ap

[GitHub] cloudstack pull request: Cwe 190

2015-11-11 Thread 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

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

2015-11-11 Thread 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 obso

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

2015-11-11 Thread 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

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

2015-11-11 Thread 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, 201

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-11-11 Thread 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 you

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

2015-11-11 Thread 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 requ

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

2015-11-11 Thread 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 Serve

DevCloud deployment

2015-11-11 Thread Илья Толстихин
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

Re: DevCloud deployment

2015-11-11 Thread 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 devcl

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

2015-11-11 Thread 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: Wedn

distribution of Cloudstack API plugins

2015-11-11 Thread 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 d

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

2015-11-11 Thread Wilder Rodrigues
Nice to know, Lucian! Happy testing! :) Cheers, Wilder On 11 Nov 2015, at 14:46, Nux! mailto:n...@li.nux.ro>> 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 technolog

Re: DevCloud deployment

2015-11-11 Thread Илья Толстихин
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, Илья Толстихин

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

2015-11-11 Thread 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

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

2015-11-11 Thread 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 testin

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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 e

Re: DevCloud deployment

2015-11-11 Thread Илья Толстихин
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

Re: UI customization

2015-11-11 Thread Илья Толстихин
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

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

2015-11-11 Thread 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: [VO

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

2015-11-11 Thread 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 wel

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

2015-11-11 Thread 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 vol

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

2015-11-11 Thread 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 usi

RE: distribution of Cloudstack API plugins

2015-11-11 Thread 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: dist

Re: DevCloud deployment

2015-11-11 Thread 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 A

Storage plugin docs?

2015-11-11 Thread 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

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

2015-11-11 Thread 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

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

2015-11-11 Thread 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 s

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

2015-11-11 Thread 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 a

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

2015-11-11 Thread 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

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

2015-11-11 Thread 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, Comm

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

2015-11-11 Thread 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 feat

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

2015-11-11 Thread 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

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

2015-11-11 Thread 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 re

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

2015-11-11 Thread 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 re

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

2015-11-11 Thread 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

Re: DevCloud deployment

2015-11-11 Thread Илья Толстихин
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

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

2015-11-11 Thread 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

Re: [Proposal] Template for CloudStack API Reference Pages

2015-11-11 Thread 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

Re: DevCloud deployment

2015-11-11 Thread 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 Hooglan

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

2015-11-11 Thread 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

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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/cloudstac

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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&l

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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,

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

2015-11-11 Thread 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

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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

bringing jenkins.bac.o down

2015-11-11 Thread 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: bringing jenkins.bac.o down

2015-11-11 Thread 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

2015-11-11 Thread 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. >> >>

Videos from CCCEU 2015 in Dublin

2015-11-11 Thread 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...@solidfi

Re: Storage plugin docs?

2015-11-11 Thread 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 betw

Re: bringing jenkins.bac.o down

2015-11-11 Thread 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 Hoo

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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 t

Re: distribution of Cloudstack API plugins

2015-11-11 Thread 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 wrot

Re: Videos from CCCEU 2015 in Dublin

2015-11-11 Thread 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&list=PLGeM09tlguZSeNyOyQKJHNX4pxgK-yoTA&index=18 On Wed, Nov 11

[DISCUSS] upgrade of system templates

2015-11-11 Thread 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 all

Re: Storage plugin docs?

2015-11-11 Thread 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! w

Re: [DISCUSS] upgrade of system templates

2015-11-11 Thread 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 so

Re: Storage plugin docs?

2015-11-11 Thread 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 plu

Re: [DISCUSS] upgrade of system templates

2015-11-11 Thread 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 > d

Re: [DISCUSS] upgrade of system templates

2015-11-11 Thread 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 < > rafaelweingar

Re: [DISCUSS] upgrade of system templates

2015-11-11 Thread 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

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

2015-11-11 Thread 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 se

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

2015-11-11 Thread 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 set

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

2015-11-11 Thread 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 releas

Re: UI customization

2015-11-11 Thread 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-plu

Re: [RFC] Metrics views for CloudStack UI

2015-11-11 Thread 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