[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 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. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Allow+non-r

[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 your

[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 feat

[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: ``` 20

[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 using

[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 an

[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 th

[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 'ro

[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 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 wh

[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 and

[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 did

[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 r

[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 alter

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

Re: [jira] [Commented] (CLOUDSTACK-8339) Allow non-root credentials for adding KVM hypervisor

2015-05-24 Thread Marcus
On Sun, May 24, 2015 at 8:46 PM, ASF GitHub Bot (JIRA) wrote: > > [ > https://issues.apache.org/jira/browse/CLOUDSTACK-8339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14557951#comment-14557951 > ] > > ASF GitHub Bot commented on CLOUDSTACK-8339: > -

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

2015-05-24 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/285#discussion_r30964594 --- Diff: test/integration/component/maint/testpath_disable_enable_zone.py --- @@ -429,71 +380,1093 @@ def test_01_disable_enable_zone(self):

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

2015-05-24 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/285#discussion_r30964684 --- Diff: test/integration/component/maint/testpath_disable_enable_zone.py --- @@ -429,71 +380,1093 @@ def test_01_disable_enable_zone(self):