[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105004714 @bhaisaab Nice! The code LGTM, will give it a test-drive today or tomorrow --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack-docs-rn pull request: add note on XenServer: we depend ...

2015-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-rn/pull/21 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105045696 @bhaisaab, I tested your patch and ran to an issue where a TTY is required to run sudo, but not requested. The host cannot be added. Logs: ```

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread mlsorensen
Github user mlsorensen commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105031461 Oh, I see the original post mentions that! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread mlsorensen
Github user mlsorensen commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105031405 Note you still have to add a sudoers entry for the user to run cloudstack-setup-agent.

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105064856 Why don't we make a dedicated user (be it 'cloud' or anything) during agent install time? Makes a lot more sense to me, than relying on the usage of

[GitHub] cloudstack pull request: kvm: for disabling pxe, don't pass any fi...

2015-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/290 --- 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-8339: Allow non-root users to ...

2015-05-24 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105076586 On Mon, May 25, 2015 at 12:50 AM, Rafael da Fonseca notificati...@github.com wrote: Creating a user for this use would be worse, because it would

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105063069 Silly me.. this is running as supplied user, not as cloud. I agree remi, ssh -t is the better option :) --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-8247: Pull average Cpu util re...

2015-05-24 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/289#issuecomment-105072855 I need to fix the memory and CPU utils reporting together, holding on this PR until next fix. --- 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-24 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/238#issuecomment-105057113 looks like a couple more things got broken since last rebase with master... need to refix :| --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105056903 Hi @remibergsma If you're running into the no tty issue, that will likely happen also for the other commands the management server tries to run. I'm

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105072838 Thanks for reviewing everyone. Regarding the tty/sudo issue, I'll test and see if we can fix something in the agent post installation or setup scripts similar to

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105073554 Creating a user for this use would be worse, because it would require some sort of 'standard' credential for ssh access. Or it would require that such account

RE: support for /31 Networks

2015-05-24 Thread Singh, Devender
Hi , Can I get patch file to use /31 subnet in 4.4.2 Devender -Original Message- From: Wilder Rodrigues [mailto:wrodrig...@schubergphilis.com] Sent: 22 May 2015 18:23 To: dev@cloudstack.apache.org Subject: Re: support for /31 Networks Hi guys, We found when it was changed and the

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105115006 For now, I think the quick win is to make the proposed use of 'sudo' work, so that works what worked before (we require root credentials to add a host) and

[GitHub] cloudstack pull request: CLOUDSTACK-8339: Allow non-root users to ...

2015-05-24 Thread mlsorensen
Github user mlsorensen commented on the pull request: https://github.com/apache/cloudstack/pull/288#issuecomment-105133656 The issue now is that root credentials for the host are stored in the db, and even echoed back if you ask to list hosts with details. It's a huge step forward to

[GitHub] cloudstack pull request: Disable enable zone pod cluster and host

2015-05-24 Thread pritisarap12
Github user pritisarap12 commented on the pull request: https://github.com/apache/cloudstack/pull/285#issuecomment-105132818 Disable enable zone pod cluster and host: --Added missing strings in codes.py --- If your project is set up for it, you can reply to this email

Re: IPv6 ideas for Basic Networking

2015-05-24 Thread Remi Bergsma
At Schuberg Philis we’ve been working on a design voor IPv6 in VPC networks (so this is Advanced Networking) and I indeed had a look at your functional spec. I’ll finalise what we’ve come up with and publish it early next week so we can align and discuss and work from there. Nice to see there

[GitHub] cloudstack-docs-rn pull request: add note on XenServer: we depend ...

2015-05-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/21#issuecomment-104984303 @xenserverarmy thanks, I wanted to keep it simple and not rewrite the XenServer docs inside the CloudStack ones. BUT you're right, the timeout is not