Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/591
---
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 user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121603506
Added same logic for fedora20/21 spec files as well. Merged on master now.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121602657
LGTM, good catch
---
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
awesome!
agree on the context... thanks!
-Original Message-
From: wilderrodrigues [mailto:g...@git.apache.org]
Sent: Wednesday, July 15, 2015 4:18 PM
To: dev@cloudstack.apache.org
Subject: [GitHub] cloudstack pull request: CLOUDSTACK-8633: Changing file
permission...
Github user
I was trying to bring up MS with xencluster.
On Wed, Jul 15, 2015 at 4:10 PM, pvr9712 wrote:
> Github user pvr9712 commented on the pull request:
>
> https://github.com/apache/cloudstack/pull/591#issuecomment-121571026
>
> BTW, here is my email on the Dev list with the details for this i
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121573911
Hi @pvr9712
Thanks for the commit IDs, a bit of context always help.
It would have saved us some time if the PR would have the commit ID whi
Github user pvr9712 commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121571026
BTW, here is my email on the Dev list with the details for this issues -
including the earlier commit - Using Master builds, MS logs shows following
error -
"ma
Github user pvr9712 commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121566177
@wilderrodrigues if you look at the earlier commit, the code was refactored
and I feel it is a miss. If this worked for someone, will be interested to
know.
b
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121563084
@pvr9712 the question here is not if it works or not, because it's quite
clear that once the permission changes it will suffice @sanju1010's need. The
point
Github user pvr9712 commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121559068
I have tested this changes and it works.
---
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 pr
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121512748
@wilderrodrigues currently I am on holiday and not able to look into stuff.
As far as I know sudo was implemented for KVM agent only.
See ya in about 1
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/591#issuecomment-121511254
@bhaisaab @DaanHoogland @remibergsma could you have a look at this?
Wasn't ACS management-server supposed to be executed as SUDO? If I'm right,
there
GitHub user sanju1010 opened a pull request:
https://github.com/apache/cloudstack/pull/591
CLOUDSTACK-8633: Changing file permissions from 755 to 440
With 0755 permissions on /etc/sudoers.d/cloudstack-management we are
getting following error in bringing up CS:
Failed to start b
13 matches
Mail list logo