On 22.03.2016 10:43, Martin Basti wrote:
On 18.03.2016 11:53, Christian Heimes wrote:
On 2016-03-18 10:22, Martin Basti wrote:
On 29.02.2016 16:02, David Kupka wrote:
sorry for letting this patch rot for so long. I've forget about it
the minute Fraser replied.
To compensate a little I've fixed pep8 error, rebased it and
attaching two versions for master and for 4.3 branch.
I haven't found any missing cases and it works for me. If you're OK
with the modified patches it can be pushed.
----- Original Message -----
From: "Christian Heimes" <chei...@redhat.com>
To: "Fraser Tweedale" <ftwee...@redhat.com>
Cc: "freeipa-devel" <email@example.com>
Sent: Wednesday, January 20, 2016 11:57:42 AM
Subject: Re: [Freeipa-devel] [PATCH 0029] Move user/group constants
for PKI and DS into ipaplatform
On 2016-01-20 02:54, Fraser Tweedale wrote:
On Tue, Jan 19, 2016 at 02:20:27PM +0100, Christian Heimes wrote:
ipaplatform.constants has platform specific names for a couple of
users like Apache HTTPD. The user names for PKI_USER, PKI_GROUP,
and DS_GROUP are defined in other modules. Similar to #5587 the
patch moves the constants into the platform module.
I see a few remaining cases:
712: pent = pwd.getpwnam("dirsrv")
167: self.master.run_command(['userdel', 'dirsrv'])
168: self.master.run_command(['userdel', 'pkiuser'])
441: if name == 'pkiuser':
When these are included, ACK.
My new patch takes care of remaining cases.
Christian do you agree with proposed changes, can we push it?
Oh, the patch is still open? ACK!
Pushed to ipa-4-3: e3bf65f2df9c50873f0967b96a6a2a5975a87f79
Pushed to master: 49be6c8d3cc20902dbe8e92a74e31aed2fd21d9f
This patch broke ipa-restore.
please not that 2 modules are imported as same name in ipa_restore.py
from ipalib import api, errors, constants
from ipaplatform.constants import constants
2016-03-22T16:56:27Z DEBUG File
"/usr/lib/python2.7/site-packages/ipapython/admintool.py", line 171, in
return_value = self.run()
line 218, in run
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code