[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-212267285 @dmabry Thanks for the fix. I ran it on simulator and it worked as expected. So +1 on the changes. Although it existed for XenServer and Vmware, I am trying to

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60352254 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,144 @@ def test_01_volume_attach_detach(self): "Check

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-212265582 @nvazquez Can you add integration tests for all these APIs? Checkout test_deploy_vm_multiple test in test/integration/smoke/test_vm_life_cycle.py --- If your

[GitHub] cloudstack pull request: CLOUDSTACK-9165 unable to use reserved IP...

Github user SudharmaJain commented on the pull request: https://github.com/apache/cloudstack/pull/1246#issuecomment-212258625 @alexandrelimassantana I have made changes with respect to ur last comment. --- If your project is set up for it, you can reply to this email and have your

Re: Apache Jira rights for the ACS project

you should be able to assign issues now. ~Rajani On Wed, Apr 20, 2016 at 2:19 AM, Simon Weller wrote: > > > >From: Erik Weber > >Sent: Tuesday, April 19, 2016 3:44 PM > >To: dev > >Subject: Re: Apache Jira rights

Re: [DISCUSS] Network offerings for Isolated Networks / VPCs

Another way to look at it would be to make isolated network a special case of VPC (having a single tier). -Koushik From: Nick LIVENS Sent: Tuesday, April 19, 2016 2:46 PM To: dev@cloudstack.apache.org Subject: [DISCUSS]

退订

退订 wangyuyang1...@163.com

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1423 [CLOUDSTACK-9296] Start ipsec for client VPN This fix starts the IPSEC daemon when enabling client side vpn You can merge this pull request into a Git repository by running: $ git pull

Fwd: Advice for community participation to lower tension

This was recently sent to ComDev (d...@community.apache.org) and I feel it is worth sharing. Cheers, Erik -- Forwarded message -- From: Niclas Hedhman Date: Sat, Apr 9, 2016 at 3:50 AM Subject: Advice for community participation to lower tension To:

[DISCUSS] Emeritus Committer Status

All, As I am sure many have observed, we have a large number of committers [1] who are no longer active. Currently, we no status that continues to recognize someone’s committer merit while allowing them to declare inactive in the project. To address this gap, I am proposed amending our

Re: [DISCUSS] ACS 4.9 systemvmtemplate

Thank for sharing Erik. Regards. From: Erik Weber Sent: 19 April 2016 23:15 To: dev@cloudstack.apache.org Subject: Re: [DISCUSS] ACS 4.9 systemvmtemplate Regarding packer, I believe cosmic has done most of that. See

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212129730 @borisstoyanov awesome, thanks. Are you able to push the test results to the PR? I have built a tool (https://github.com/cloudops/upr) to hopefully simplify the

Re: anybody doing load testing?

Before my current employer/job I worked at an A/B testing startup where I scaled their geo-distributed data-acquisition/cdn servers to capture/serve each up to 50k requests/second -- for internal network testing, we used wrk, ab, JMeter and we also wrote and ran a custom tool, and for external

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user borisstoyanov commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212127186 @bhaisaab @swill Hey Guys, I've successfully checkout and build the PR, will continue with testing tomorrow. I'd like to take the opportunity to tell you

Re: CloudStack Logging

Paul, Are you wanting for focus on logs on releases later than a particular version (e.g. 4.6)? - Si From: Paul Angus Sent: Tuesday, April 19, 2016 10:55 AM To: us...@cloudstack.apache.org; dev@cloudstack.apache.org Subject:

Re: Apache Jira rights for the ACS project

On Tue, Apr 19, 2016 at 10:38 PM, Simon Weller wrote: > All, > > > What is the correct way to request some additional rights on the Apache > Jira instance? > > I'd like to be able to assign some issues to myself (and other members of > my team). > > I believe the general way is

Apache Jira rights for the ACS project

All, What is the correct way to request some additional rights on the Apache Jira instance? I'd like to be able to assign some issues to myself (and other members of my team). Thanks, - Si

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212106389 The tests are now being run, so this time everything built alright. I will have results for you later tonight or in the morning. :) --- If your project is set up

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-212097103 @Nv, that is great. The code now LGTM. There is a problem with Jenkins, maybe a push –f can solve it? Your work as always is very good ;) I

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user dmabry commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-212085924 Ok, I have made the requested changes. Please review and let me know if there is anything else I need to change. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-212074504 @bhaisaab thank you, I appreciate the effort. 👍 I am actively trying to get more people involved in code review, so we will have to lead by example for

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212071544 @bhaisaab my environment builds it twice. Once with packaging and once without. If I have the problem again, I will send you the command that was run that had the

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212065979 @borisstoyanov would like to share our QA results, thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212065155 @swill just completed local build here: (also pushed -f, let's see if Travis reports any failure) [INFO]

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212064683 I was confused by this as well. I will kick off another CI run against this PR and see if I have the problem again. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-212063724 @swill no problem, I'll deal with this during weekends and see how much we can safely get into master. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-212063057 @swill I looked at it, looks like an environment issue? I was able to build it without issue (also did not find 'root_admin' key in the code. I've rebased master

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1423#issuecomment-212062046 @syed looks like marvin plugin is missing; if you've both python 2.6/2.7 (looks like nose is using python 2.6) make sure to install it for the one you use (the

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-212060526 @swill I've explained my intent above, and it was not intended to be hard on @rafaelweingartner either While he's indeed putting good effort on PRs, I'm seeing

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1420#discussion_r60284224 --- Diff: packaging/centos7/cloud-management.service --- @@ -24,6 +24,7 @@ Description=CloudStack Management Server After=syslog.target

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-212043816 Cool, I added both 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

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-212040090 @bhaisaab I think you are being too hard on @rafaelweingartner. He simply asked a question because the format was presented differently in two different files in

Re: [DISCUSS] ACS 4.9 systemvmtemplate

Regarding packer, I believe cosmic has done most of that. See https://github.com/MissionCriticalCloud/systemvm-packer/blob/master/README.md Erik Den tirsdag 19. april 2016 skrev Rohit Yadav følgende: > Hi all, > > Will and I had some discussions around systemvm

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-212034630 @swill we may ignore them, those are non-issues unless @rafaelweingartner has strong opinions @rafaelweingartner things you can do to improve your

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1420#discussion_r60275113 --- Diff: packaging/centos7/cloud-management.service --- @@ -24,6 +24,7 @@ Description=CloudStack Management Server After=syslog.target

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1420#discussion_r60272259 --- Diff: packaging/centos7/cloud-management.service --- @@ -24,6 +24,7 @@ Description=CloudStack Management Server After=syslog.target

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-212009376 That is great. Now reading your documentation, I noticed another question. And if both are null? Or if ID is null and IDs is empty? The way it is

Re: anybody doing load testing?

We load test with real production work load :) I know thats not the answer you want to hear. By the way, dont hit 8250 without a recent non-blocking SSL Handshake patch - it will disconnect all other agents. Or perhaps - try without patch and with patch - to see if you can find a breaking point.

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-212008157 I included last changes and squashed all commits into one --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[QUESTION] ACS 4.8 Centos 7 ARM / Raspberry PI B 2 with Intel KVM hypervisor

I built ACS 4.8 on Centos 7 ARM on a Raspberry PI B 2. With ACS 4.8 built on a Centos 7 x86_68 for the cloudstack_agent code. And I am planning on managing a cluster of Intel Centos 7 kvm hypervisors. I was wonder if anyone has tried a mixed platform configuration before. I will report back

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-212001526 No problem, I appreciate your comments and think they are really helpful ;) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211999079 Sorry if I am very picky sometimes; but, I believe that the more clear we are about a method behavior and the more tests to enforce that behavior we

Re: CloudStack Collab - Montreal - June 1-3

I'm looking forward to visiting you guys in Montreal again. Thanks for organizing the conference! > On Apr 19, 2016, at 7:37 AM, Will Stevens wrote: > > Hey All, > The planning of the CCC event in Montreal is in full swing. A lot of > people have submitted talks, so

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1423 --- 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-9296] Start ipsec for client ...

Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1423#issuecomment-211996541 @bhaisaab Do you have an idea of why Marvin is failing on my dev setup? I've followed the guide mentioned

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211996504 Ok, I would modify javadoc and add a test for that particular case --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Fix Sync of template.properties in Swift

Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-211995051 @rafaelweingartner I've added the test mocking the `BufferedWriter`. When I tried to run the tests I found that the test required `agent.properties` and `log4j.xml`

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211994713 The method name is ok. I just asked to confirm how you see the method workings. I only feel that the Javadoc should say that, whenever the ID variable is

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211993545 @rafaelweingartner Actually the idea was that they were mutually exclusive, that only of them should be provided, emulating `listVirtualMachines` API method

RE: CloudStack Logging

No problem. Good question - Any and all logs. Often, it's the mundane stuff which obscures the messages that you really need to see! Kind regards, Paul Angus Regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-211991515 I have one LGTM. I need one more and this PR is ready to merge. Thanks guys... --- If your project is set up for it, you can reply to this email and have your

Re: CloudStack Logging

Thanks Paul. Are you looking for only logs with errors or are you just looking for just any management server logs? *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Tue, Apr 19, 2016 at

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-211989957 Thanks @swill! --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211988788 @nvazquez I have a question about the method "getIdsListFromCmd". If we inform an ID and a list of IDs (not empty) at the same time, what should be

RE: CloudStack Logging

Thanks Will, As far as I know anything sensitive is removed or obfuscated (vcenter password displayed as h**). IP addresses and hostnames are shown. Users may want to use a tool like SED to replace IP addresses or internal domain names

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1497#discussion_r60256696 --- Diff: server/test/com/cloud/api/query/MutualExclusiveIdsManagerBaseTest.java --- @@ -0,0 +1,84 @@ +// +// Licensed to the Apache

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1497#discussion_r60255794 --- Diff: server/test/com/cloud/api/query/MutualExclusiveIdsManagerBaseTest.java --- @@ -0,0 +1,84 @@ +// +// Licensed to the Apache

Re: CloudStack Logging

I like this initiative Paul. I just wanted to check with you. Do you know if there is any confidential information like credentials or the like that is logged in the management server log? I thought we had done a push at one point to make sure no sensitive data was logged, but I don't remember

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1497#discussion_r60255522 --- Diff: server/test/com/cloud/api/query/MutualExclusiveIdsManagerBaseTest.java --- @@ -0,0 +1,84 @@ +// +// Licensed to the Apache

CloudStack Logging

Hi All, I'm running an initiative to improve CloudStack logging. I'm going to be analysing logs and re-categorising the DEBUG, INFO, WARN and ERROR where appropriate such that we can reduce the default logging level to INFO without losing important troubleshooting information, while at the

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211965259 That's cool! Thanks @swill! --- 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-9351: Add ids parameter to res...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211962282 @nvazquez I have added this to my CI queue. I have a pretty serious backlog, so it may take me a few days to get to it though. In the mean time, lets see if we can

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1394#issuecomment-211958527 @koushik-das oh, i understand your question better now. i would like to do it on every commit. It makes the commit history clearer and it mitigates conflict

[GitHub] cloudstack pull request: CLOUDSTACK-9351: Add ids parameter to res...

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1497#issuecomment-211957155 @rafaelweingartner I pushed new changes, would integration tests be needed for this pr? I have tested api calls with new ids parameters and succeeded --- If

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1394#issuecomment-211954088 @swill I wanted to understand if it is ok to do these fwd merges once every week/fortnight (based on number of commits that went into the release branch) or do

[GitHub] cloudstack-www pull request: Update events from the landing pages

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-www/pull/26 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1394#issuecomment-211947137 Every commit that goes into 4.7 gets fwd-merged into 4.8 and then gets fwd-merged into master. We have tools in `cloudstack/tools/git/` which make this process MUCH

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1394#issuecomment-211943539 @swill I didn't fwd merge into 4.8 and master. Are we fwd merging for every individual commit in order to avoid conflicts? --- If your project is set up for

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-211936017 ## CI RESULTS ### 84/85 TESTS PASSED The test that failed is a test that commonly fails in my environment and has been

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-211932110 @bhaisaab thank you sir. sorry this is proving to be a bit complicated. i appreciate the support. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-211930508 @bhaisaab I will let you address @rafaelweingartner's comment. If you choose to make a change, it will not impact the 'readiness' of this PR. I am not sure which

Re: [DISCUSS] ACS 4.9 systemvmtemplate

Thanks for starting this discussion Rohit. :) *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Tue, Apr 19, 2016 at 7:22 AM, Rohit Yadav wrote: > Hi all, > >

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1394#issuecomment-211926700 @koushik-das I am not seeing the commit in master. Did you merge into 4.7 and then forward merge into 4.8 and then forward merge into master??? --- If your project

CloudStack Collab - Montreal - June 1-3

Hey All, The planning of the CCC event in Montreal is in full swing. A lot of people have submitted talks, so this is going to be an awesome event. We currently have a venue booked, but based on the number of talks submitted, this event may be bigger than we were initially expecting, so we may

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-211923768 @swill okay I'll check this out and see if I can reproduce the failure. I'll keep you posted. --- If your project is set up for it, you can reply to this email

anybody doing load testing?

I will go to a talk this evening about http://gatling.io/ Is there anybody busy testing cloudstack this way? What tools are you using? ​nice to hear,​ -- Daan

[GitHub] cloudstack pull request: L10n update 4.8 20160127

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1375#issuecomment-211918463 This one looks like it will have merge conflicts with #1376. @milamberspace should I first merge #1376 and then have you rebase this PR and fix the conflicts before

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-211918504 The command “”umask” at “scripts/vm/systemvm/injectkeys.sh” is using “022” as a parameter, while the one at

[GitHub] cloudstack pull request: L10n update master 20160127

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-211917846 Thanks koushik. I tend to agree with you on this one. I will add it to my merge list. Thanks... --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1420#discussion_r60226618 --- Diff: packaging/centos7/cloud-management.service --- @@ -24,6 +24,7 @@ Description=CloudStack Management Server After=syslog.target

[GitHub] cloudstack pull request: Check the existence of 'forceencap' param...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1402#issuecomment-211916588 Thanks. Looks good now... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user dmabry commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60225742 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self): "Check the

[GitHub] cloudstack pull request: systemvm: preserve file permissions, set ...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1420#issuecomment-211915697 @mlsorensen did you do a code review of this or are you just excited about the change. :) Ideally I would get another LGTM before merging. I am pretty

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-211911369 This failed to compile. Can you have a look? ``` [INFO] --- exec-maven-plugin:1.2.1:exec (compile) @ cloud-apidoc --- log4j:WARN No appenders could

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user dmabry commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60222679 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self): "Check the

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user dmabry commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60222132 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self): "Check the

[GitHub] cloudstack pull request: CLOUDSTACK-9228: Network update with mist...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1333#issuecomment-21190 @nitin-maharana Looks like there is some problem. In confirmation dialog when I click "yes" nothing happens and on clicking "no" the offering is changed.

Re: [DISCUSS] ACS 4.9 systemvmtemplate

> Op 19 april 2016 om 13:22 schreef Rohit Yadav : > > > Hi all, > > Will and I had some discussions around systemvm template on one of the PRs, I > wanted to bring the discussion here on dev@. > > Provided we've some features merged before the freeze date in May, we

[GitHub] cloudstack pull request: CLOUDSTACK-9261: Query to traffic sentine...

Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/1418#issuecomment-211873904 @GabrielBrascher These adjustments can be done, but I don't see any point of introducing the function calls. Anyhow this code is not used anywhere else in the whole

[DISCUSS] ACS 4.9 systemvmtemplate

Hi all, Will and I had some discussions around systemvm template on one of the PRs, I wanted to bring the discussion here on dev@. Provided we've some features merged before the freeze date in May, we will need to build a new systemvm-template with likely following new packages (assuming the

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1394 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9268: Display VM in Load balan...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1394#issuecomment-211864757 Merging as UI change and 2 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

[GitHub] cloudstack pull request: Update L10N resource files with 4.7 strin...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1374#issuecomment-211860607 @swill Again UI changes related to l10n. I think these can be merged. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: L10n update 4.8 20160127

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1375#issuecomment-211860481 @swill Again UI changes related to l10n. I think these can be merged. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: L10n update master 20160127

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-211859221 The changes are only limited to UI, so there is no need to run smoke/integration tests. Cursory look at the changes looks fine. @swill I think these

[DISCUSS] Network offerings for Isolated Networks / VPCs

Hi all, Currently, there is no reliable way to tell whether an offering was created for an Isolated Network or for tiers in a VPC. This is determined based on providers. (ConfigurationManagerImpl.isOfferingForVpc) In the UI, you have the possibility to check a flag for "VPC" during creation of a

[GitHub] cloudstack pull request: Check the existence of 'forceencap' param...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1402#issuecomment-211818185 @alexandrelimassantana @swill Thanks guys, fixed it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60193647 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self): "Check the

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60193089 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self): "Check the

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-211786322 @swill thanks Will, I'll try to split this PR into two to isolate what changes are causing this in your env and see if we can get it fixed/improved separately

  1   2   >