[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread K0zka
Github user K0zka commented on the issue: https://github.com/apache/cloudstack/pull/1858 🥇 👍 :) --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-30 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-152597661 I do not know if a new web UI is needed, but tomcat 6 is a walking dead. @miguelaferreira yes correct, the session can not be created once the headers are out

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-28 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-151989108 thank you for the response, then that prehistoric tomcat may not be able to compile the jsp. Could you try if it works well with

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-27 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-151635108 @ustcweizhou * could you try to clear the tomcat work directory? * could you check the service with ```curl -v -o /dev/null``` --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-26 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-151234767 @ustcweizhou thats exciting... can you share any log, errormessage, configuration, etc? What http status do you get? --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-26 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-151102999 @ustcweizhou could you share some details about the problem? Are you not able to login? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-22 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-150164133 Hi Daan, I have verified the result with curl -v, therefore manual verification. It would be nice to have automatic test to verify that unauthenticated users

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-21 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/961 CLOUDSTACK-8977: remove session creation from index.jsp Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by running: $ git pull https://github.com/K0zka

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-26 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-143480210 LGTM :+1: --- 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

[GitHub] cloudstack pull request: debian: allow tomcat7 as optional depende...

2015-05-13 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/247#issuecomment-101647560 This pull request looks good :+1: Could be also tomcat8? (ubuntu 15.04 has tomcat8 as well) --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-12 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/238#discussion_r30176487 --- Diff: ui/index.jsp --- @@ -1761,73 +1761,73

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-12 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/238#discussion_r30169643 --- Diff: ui/index.jsp --- @@ -1761,73 +1761,73

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-12 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/238#discussion_r30168912 --- Diff: ui/index.jsp --- @@ -1761,73 +1761,73

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-12 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/238#issuecomment-101387790 Rafael, the gzip compression *alone* indeed does not make the tomcat serve it compressed * tomcat 8.0 allows [configuring the default servlet](https

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-12 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/238#discussion_r30166628 --- Diff: client/pom.xml --- @@ -384,23 +408,6 @@ -com.googlecode.todomap -maven-jettygzip

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-12 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/238#discussion_r30165536 --- Diff: ui/index.jsp --- @@ -1761,73 +1761,73

[GitHub] cloudstack pull request: UI localization - hu

2015-05-07 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/224 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: 4.5 hu

2015-05-07 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/237 4.5 hu Hungarian l10n You can merge this pull request into a Git repository by running: $ git pull https://github.com/K0zka/cloudstack 4.5_hu Alternatively you can review and apply these

[GitHub] cloudstack pull request: 4.5 hu

2015-05-07 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/236 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: 4.5 hu

2015-05-07 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/236#issuecomment-99973511 oh, not against master... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request: 4.5 hu

2015-05-07 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/236 4.5 hu Hungarian l10n You can merge this pull request into a Git repository by running: $ git pull https://github.com/K0zka/cloudstack 4.5_hu Alternatively you can review and apply these

[GitHub] cloudstack pull request: Refactor/libvirt resource

2015-05-07 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/233#issuecomment-99825704 looks nice :+1: --- 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

[GitHub] cloudstack pull request: Refactor/libvirt resource

2015-05-07 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/233#discussion_r29843511 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtRequestWrapper.java --- @@ -0,0 +1,184 @@ +// +// Licensed

[GitHub] cloudstack pull request: added favicon

2015-05-04 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/225#issuecomment-98706871 I only looked at it in firefox, it was OK. If you have an IE, could you make a screenshot? Just curious... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: added favicon

2015-05-04 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/225#issuecomment-98690734 I was affraid that the monkey will not look good in 16x16, but it works :+1: Thank you @bhaisaab ! --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: UI localization - hu

2015-05-03 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/224#issuecomment-98595716 Hmm, the repo cleanup should have been merged https://github.com/apache/cloudstack/pull/221 --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: added favicon

2015-05-03 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/225#issuecomment-98448616 Hi @bhaisaab , It is in the same commit. This is what it looks like. ![icon](https://raw.githubusercontent.com/K0zka/cloudstack/favicon_master/ui/images

[GitHub] cloudstack pull request: added favicon

2015-05-02 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/225 added favicon Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by running: $ git pull https://github.com/K0zka/cloudstack favicon_master Alternatively

[GitHub] cloudstack pull request: UI localization - hu

2015-05-01 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/224 UI localization - hu Added hungarian to the list of languages - resource file can be pulled from transifex You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed ceph repository

2015-05-01 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/221#issuecomment-98148819 Thank you @bhaisaab ! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed ceph repository

2015-05-01 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/221 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed ceph repository

2015-05-01 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/221 CLOUDSTACK-8335: removed ceph repository Backport to 4.5 Signed-off-by: Laszlo Hornyak Signed-off-by: Rajani Karuturi You can merge this pull request into a Git repository by

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed ceph repository

2015-05-01 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/217#issuecomment-98060060 Thank you @karuturi ! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed ceph repository

2015-05-01 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/217 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed ceph repository

2015-04-30 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/217 CLOUDSTACK-8335: removed ceph repository artifact resolved from maven central https://repo1.maven.org/maven2/com/ceph/rados/0.1.4/ Signed-off-by: Laszlo Hornyak You can merge this

[GitHub] cloudstack pull request: CLOUDSTACK-8304: Modify tags accordingly ...

2015-04-27 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/193#issuecomment-96743926 Just a sidenote: CLOUDSTACK-8304 ("maven repositories are checked for snapshots") is not the relevant issue. more likely CLOUDSTACK-8394 --- If your project

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed libvirt.org repo...

2015-04-24 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/180#issuecomment-95947645 ping? :) --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-18 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/172 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8335: removed libvirt.org repo...

2015-04-18 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/180 CLOUDSTACK-8335: removed libvirt.org repository The only artifact resolved from libvirt.org was org.libvirt:libvirt:0.5.1 this artifact is now available from maven's default central repos

[GitHub] cloudstack pull request: use eu.ceph.com as ceph.com is down

2015-04-16 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/175#issuecomment-93740908 LGTM [CLOUDSTACK-8335](https://issues.apache.org/jira/browse/CLOUDSTACK-8335) should solve issues like this for the future, it just takes time to go through the

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-15 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/172#issuecomment-93639076 Backport was requested by @wilderrodrigues It is not really meant as build *performance* improvement, but rather a build stability patch. --- If your project is

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-15 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/172 CLOUDSTACK-8304: disable snapshots from non-snapshot repositories Backport from master to 4.4 Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-15 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/163#issuecomment-93350242 Hi @wilderrodrigues Patch 9a38b2f7de79888cdbbb2dca2305d6ec10f81ee0 should be a solution for *most* of the problem. The objective here is to completely get rid

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-15 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/163 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-15 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/163#discussion_r28400164 --- Diff: plugins/hypervisors/kvm/pom.xml --- @@ -45,7 +38,7 @@ ${project.version} - org.libvirt

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-13 Thread K0zka
Github user K0zka commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/163#discussion_r28268011 --- Diff: plugins/hypervisors/kvm/pom.xml --- @@ -45,7 +38,7 @@ ${project.version} - org.libvirt

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

2015-04-13 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/163 CLOUDSTACK-8335: use libvirt from central repository Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by running: $ git pull https://github.com/K0zka

[GitHub] cloudstack pull request: findbugs: Boolen function should not retu...

2015-03-25 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/131#issuecomment-86220564 I agree with the change (not tested yet) Once it does not return null, the Boolean could be boolean? --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: 4.4 for centos7

2015-03-24 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/125#issuecomment-85372139 Looks good, I can only test it in the evening, anyone if you get there first pls go ahead and merge! --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: 4.4 for centos7

2015-03-23 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/125#issuecomment-85189126 pom.xml specifies 1.9.0 as dependency --- 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

[GitHub] cloudstack pull request: 4.4 for centos7

2015-03-23 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/125#issuecomment-85180862 Hi Daan, I installed a clean centos 7 and I got this error while testing the build: ``` + install -D client/target/pythonlibs/jasypt-1.9.2.jar

[GitHub] cloudstack pull request: fix yum install require java => 1.7.0 whe...

2015-03-10 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/108#issuecomment-78122073 looks ok --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-10 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-78041502 @karuturi Correct, if you do not have the libvirt jar and libvirt.org goes down, the build will still fail. It would be nice if we could use only the central

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-10 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-78022061 @karuturi So the problem that it adresses is this: We have SNAPSHOT dependencies, since snapshots are not disabled on those repositories, they will be checked. Now

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-06 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/100 CLOUDSTACK-8304: disable snapshots from non-snapshot repositories Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] cloudstack pull request: kvm: Check the return value of the clouds...

2015-03-03 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/93 kvm: Check the return value of the cloudstack-setup-agent command Bug-Url: https://issues.apache.org/jira/browse/CLOUDSTACK-8016 Signed-off-by: Laszlo Hornyak You can merge this pull request

[GitHub] cloudstack pull request: replace cache workaround parameter, cache...

2015-03-03 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/60#issuecomment-77015510 Yes I messed this up when rebasing my working branches, will send another one... --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: replace cache workaround parameter, cache...

2015-03-03 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/60 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8294: fix for java 1.8 build

2015-03-01 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/92 CLOUDSTACK-8294: fix for java 1.8 build The test testFindSecurityProfileByUuid built an assumptio on the order of the arguments after transforming through a HashMap. Signed-off-by: Laszlo

[GitHub] cloudstack pull request: CLOUDSTACK-8292: fix for java 1.8 build

2015-02-28 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/91 CLOUDSTACK-8292: fix for java 1.8 build Refactored to use the XPath expressions to check the generated domain xml rathern than string comparison. Signed-off-by: Laszlo Hornyak You can

[GitHub] cloudstack pull request: Add distribution fedora21 in packaging.sh...

2015-02-27 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/88#issuecomment-76479323 Looks good. Thank you @ghxandsky ! --- 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

[GitHub] cloudstack pull request: Cloudstack 8279

2015-02-27 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/90 Cloudstack 8279 Fixes for [CLOUDSTACK-8279](https://issues.apache.org/jira/browse/CLOUDSTACK-8279) and two minor cleanups You can merge this pull request into a Git repository by running

[GitHub] cloudstack pull request: Cloudstack 8210

2015-02-23 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/85#issuecomment-75564612 Hi Daan, Yes it is OK to merge, at least it is something for fedora users to start playing with, but let's keep the jira ticket open since the packaging o

[GitHub] cloudstack pull request: Cloudstack 8210

2015-02-21 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/85 Cloudstack 8210 Basic packaging for Fedora 20 and Fedora 21 The agent works, the management is not yet functional You can merge this pull request into a Git repository by running: $ git

[GitHub] cloudstack pull request: fixed NPE

2015-02-05 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/70 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: fixed NPE

2015-02-04 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/70 fixed NPE You can merge this pull request into a Git repository by running: $ git pull https://github.com/K0zka/cloudstack CLOUDSTACK-8210 Alternatively you can review and apply these

[GitHub] cloudstack pull request: replace cache workaround parameter, cache...

2014-12-17 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/60 replace cache workaround parameter, cache-workaround only for IE Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] cloudstack pull request: upgrade to jetty 9

2014-12-17 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/59 upgrade to jetty 9 upgraded jetty from 6.1 to 9.2. The 6.1 branch was announced EOL a few weeks ago. Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository

[GitHub] cloudstack pull request: noncritical fixes on packaging script

2014-12-04 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/51 noncritical fixes on packaging script fixed: - a syntax issue, which was tolerated by bash - removed an option from usage text You can merge this pull request into a Git repository by