URL: https://github.com/freeipa/freeipa/pull/209
Title: #209: Enumerate available options in IPA installer
jcholast commented:
"""
@mbasti-rh: `knob()` already handles choices, it's the built-in `optparse`
module which does not display them. Once the installer code is migrated to
`argparse`, t
URL: https://github.com/freeipa/freeipa/pull/245
Title: #245: Allow full customisability of IPA CA subject DN
tiran commented:
"""
@frasertweedale I don't have a clue, either. Let's ignore the message.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/245#issuecomment-2618743
URL: https://github.com/freeipa/freeipa/pull/209
Title: #209: Enumerate available options in IPA installer
mbasti-rh commented:
"""
@jcholast I know, but it doesn't fill `metavar` with choices. I don't know when
we will migrate to argparse, so I think until that we can extend it to show
choice
URL: https://github.com/freeipa/freeipa/pull/113
Title: #113: ipalib.constants: Remove default domain, realm, basedn,
xmlrpc_uri, ldap_uri
jcholast commented:
"""
Actually it should be created from domain name, which is the primary identifier
of an IPA domain, not from realm name.
"""
See the
URL: https://github.com/freeipa/freeipa/pull/247
Title: #247: Add 'ipa local-env' subcommand
jcholast commented:
"""
Sorry, but this is wrong. `ipa env` is supposed to return local settings unless
run with `--server`. Why was it not fixed instead of adding a new redundant
command?
"""
See the
URL: https://github.com/freeipa/freeipa/pull/247
Title: #247: Add 'ipa local-env' subcommand
tiran commented:
"""
@jcholast Please open a ticket.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/247#issuecomment-261879797
--
Manage your subscription for the Freeipa-devel ma
URL: https://github.com/freeipa/freeipa/pull/259
Author: tiran
Title: #259: Minor fixes for IPAVersion class
Action: opened
PR body:
"""
Py3: classes with __eq__ must provide __hash__ function or set __hash__
to None.
Comparison function like __eq__ must signal unsupported types by
returning N
URL: https://github.com/freeipa/freeipa/pull/259
Title: #259: Minor fixes for IPAVersion class
tiran commented:
"""
I pulled the fix from PR #254
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/259#issuecomment-261881543
--
Manage your subscription for the Freeipa-devel ma
URL: https://github.com/freeipa/freeipa/pull/247
Title: #247: Add 'ipa local-env' subcommand
jcholast commented:
"""
Reopened 6490.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/247#issuecomment-261881844
--
Manage your subscription for the Freeipa-devel mailing list:
ht
On 11.11.2016 18:28, Christian Heimes wrote:
On 2016-11-11 17:46, Martin Basti wrote:
On 11.11.2016 15:25, Christian Heimes wrote:
Hello,
I have released the first version of a new design document. It describes
how I'm going to improve integration of FreeIPA's client libraries
(ipalib, ipapy
URL: https://github.com/freeipa/freeipa/pull/254
Author: tiran
Title: #254: Replace LooseVersion with parse_ipa_version()
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/254/head:pr254
git checkout pr254
Fr
URL: https://github.com/freeipa/freeipa/pull/254
Author: tiran
Title: #254: Replace LooseVersion with pkg_resource.parse_version
Action: edited
Changed field: title
Original value:
"""
Replace LooseVersion with parse_ipa_version()
"""
--
Manage your subscription for the Freeipa-devel mailin
URL: https://github.com/freeipa/freeipa/pull/254
Title: #254: Replace LooseVersion with pkg_resource.parse_version
tiran commented:
"""
Back to ```parse_version```!
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/254#issuecomment-261886678
--
Manage your subscription for
On 2016-11-21 10:26, Jan Cholasta wrote:
> On 11.11.2016 18:28, Christian Heimes wrote:
>> On 2016-11-11 17:46, Martin Basti wrote:
>>>
>>>
>>> On 11.11.2016 15:25, Christian Heimes wrote:
Hello,
I have released the first version of a new design document. It
describes
how I
URL: https://github.com/freeipa/freeipa/pull/258
Author: tiran
Title: #258: Break ipaplatform / ipalib import cycle of hell
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/258/head:pr258
git checkout pr258
URL: https://github.com/freeipa/freeipa/pull/258
Title: #258: Break ipaplatform / ipalib import cycle of hell
tiran commented:
"""
thx @mbasti-rh . I took care of it.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/258#issuecomment-261889771
--
Manage your subscription for
On 21.11.2016 10:32, Christian Heimes wrote:
On 2016-11-21 10:26, Jan Cholasta wrote:
On 11.11.2016 18:28, Christian Heimes wrote:
On 2016-11-11 17:46, Martin Basti wrote:
On 11.11.2016 15:25, Christian Heimes wrote:
Hello,
I have released the first version of a new design document. It
des
URL: https://github.com/freeipa/freeipa/pull/212
Title: #212: KRA: don't add KRA container when KRA replica
stlaz commented:
"""
ACK, works on both DLs.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/212#issuecomment-261890178
--
Manage your subscription for the Freeipa-d
URL: https://github.com/freeipa/freeipa/pull/212
Title: #212: KRA: don't add KRA container when KRA replica
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contri
On 21.11.2016 11:32, Christian Heimes wrote:
> On 2016-11-21 10:26, Jan Cholasta wrote:
>> On 11.11.2016 18:28, Christian Heimes wrote:
>>> On 2016-11-11 17:46, Martin Basti wrote:
On 11.11.2016 15:25, Christian Heimes wrote:
> Hello,
>
> I have released the first version
URL: https://github.com/freeipa/freeipa/pull/223
Author: tomaskrizek
Title: #223: LDAP refactoring: remove admin_conn
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/223/head:pr223
git checkout pr223
From b
On 2016-11-21 10:46, Jan Cholasta wrote:
> On 21.11.2016 10:32, Christian Heimes wrote:
>> On 2016-11-21 10:26, Jan Cholasta wrote:
>>> On 11.11.2016 18:28, Christian Heimes wrote:
On 2016-11-11 17:46, Martin Basti wrote:
>
>
> On 11.11.2016 15:25, Christian Heimes wrote:
>> He
URL: https://github.com/freeipa/freeipa/pull/254
Author: tiran
Title: #254: Replace LooseVersion with pkg_resource.parse_version
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/254/head:pr254
git checkout p
On 21.11.2016 11:04, Christian Heimes wrote:
On 2016-11-21 10:46, Jan Cholasta wrote:
On 21.11.2016 10:32, Christian Heimes wrote:
On 2016-11-21 10:26, Jan Cholasta wrote:
On 11.11.2016 18:28, Christian Heimes wrote:
On 2016-11-11 17:46, Martin Basti wrote:
On 11.11.2016 15:25, Christian H
On 2016-11-21 11:38, Jan Cholasta wrote:
> On 21.11.2016 11:04, Christian Heimes wrote:
>> On 2016-11-21 10:46, Jan Cholasta wrote:
>>> On 21.11.2016 10:32, Christian Heimes wrote:
On 2016-11-21 10:26, Jan Cholasta wrote:
> On 11.11.2016 18:28, Christian Heimes wrote:
>> On 2016-11-11
URL: https://github.com/freeipa/freeipa/pull/260
Author: pspacek
Title: #260: Build: fix path in ipa-ods-exporter.socket unit file
Action: opened
PR body:
"""
This fixes regression caused by incorrect
daemons/dnssec/ipa-ods-exporter.socket.in path template introduced
in commit 312e780041fc9025
Hi,
On 11.11.2016 15:25, Christian Heimes wrote:
Hello,
I have released the first version of a new design document. It describes
how I'm going to improve integration of FreeIPA's client libraries
(ipalib, ipapython, ipaclient, ipaplatform) for third party developers.
http://www.freeipa.org/pag
URL: https://github.com/freeipa/freeipa/pull/260
Title: #260: Build: fix path in ipa-ods-exporter.socket unit file
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page
URL: https://github.com/freeipa/freeipa/pull/238
Title: #238: Build system refactoring phase 8: update translation system
pspacek commented:
"""
@tiran You are right, I forgot to remove the `config.rpath` when
`AM_GNU_GETTEXT_VERSION` macro was introduced. This version fixes this problem
by re
URL: https://github.com/freeipa/freeipa/pull/261
Author: martbab
Title: #261: upgrade: do not set HTTP and DS principals explicitly
Action: opened
PR body:
"""
In ipa-server-upgrade, HTTP and DS principals are explicitly constructed from
realm, fqdn, and service prefix. This is no neither requ
On 2016-11-21 13:31, Jan Cholasta wrote:
> Hi,
>
> On 11.11.2016 15:25, Christian Heimes wrote:
>> Hello,
>>
>> I have released the first version of a new design document. It describes
>> how I'm going to improve integration of FreeIPA's client libraries
>> (ipalib, ipapython, ipaclient, ipaplatfo
URL: https://github.com/freeipa/freeipa/pull/181
Title: #181: Tests : User Tracker creation of user with minimal values
apophys commented:
"""
I think in this case we can go with keyword arguments only. Most of the uses of
the tracker in the tests do it already. What I will need in the case of
URL: https://github.com/freeipa/freeipa/pull/196
Title: #196: ipatests: unresolvable nested netgroups
martbab commented:
"""
I am strongly opposed to keeping this particular test in XMLRPC suite since it
actually does not test any XMLRPC calls but is, in fact, an integration test
for SSSD netg
On 21.11.2016 14:15, Christian Heimes wrote:
> On 2016-11-21 13:31, Jan Cholasta wrote:
>> Hi,
>>
>> On 11.11.2016 15:25, Christian Heimes wrote:
>>> Hello,
>>>
>>> I have released the first version of a new design document. It describes
>>> how I'm going to improve integration of FreeIPA's client
URL: https://github.com/freeipa/freeipa/pull/191
Title: #191: Exclude testing ipa.pot file from zanata
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/
URL: https://github.com/freeipa/freeipa/pull/191
Title: #191: Exclude testing ipa.pot file from zanata
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/ad32bf147ed6996c0967bb8e8cfb803113ceaf5f
ipa-4-4:
https://fedorahosted.org/freeipa/changeset/76d4368ff98
URL: https://github.com/freeipa/freeipa/pull/191
Title: #191: Exclude testing ipa.pot file from zanata
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribu
URL: https://github.com/freeipa/freeipa/pull/191
Author: mbasti-rh
Title: #191: Exclude testing ipa.pot file from zanata
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/191/head:pr191
git checkout pr191
--
Manag
URL: https://github.com/freeipa/freeipa/pull/231
Title: #231: Do not log DM password in ca/kra installation logs
martbab commented:
"""
Well that is what I was pointing at, by adding both DM and admin passwords to
the parent method's default `nolog_list`, you are future-proofing the code
becau
On 2016-11-21 14:44, Petr Spacek wrote:
>>> 3.3 ipaplatform auto-configuration
>>>
>>> I'm not sure if guessing platform from ID_LIKE is really a good idea. It
>>> might work fine for centos -> rhel, but in general we can't really
>>> assume it will always work, as the platforms listed in ID_LIKE m
URL: https://github.com/freeipa/freeipa/pull/62
Title: #62: Configure Anonymous PKINIT on server install
simo5 commented:
"""
@splashx you would have to manually configure each KDC and give them certs, it
is doable.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/62#issuec
On 21.11.2016 15:07, Christian Heimes wrote:
On 2016-11-21 14:44, Petr Spacek wrote:
3.3 ipaplatform auto-configuration
I'm not sure if guessing platform from ID_LIKE is really a good idea. It
might work fine for centos -> rhel, but in general we can't really
assume it will always work, as the
On 21.11.2016 14:15, Christian Heimes wrote:
On 2016-11-21 13:31, Jan Cholasta wrote:
Hi,
On 11.11.2016 15:25, Christian Heimes wrote:
Hello,
I have released the first version of a new design document. It describes
how I'm going to improve integration of FreeIPA's client libraries
(ipalib, ip
URL: https://github.com/freeipa/freeipa/pull/223
Title: #223: LDAP refactoring: remove admin_conn
tomaskrizek commented:
"""
Depends on #262
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/223#issuecomment-261956124
--
Manage your subscription for the Freeipa-devel mailin
URL: https://github.com/freeipa/freeipa/pull/262
Title: #262: upgrade: do not explicitly set principal for services
martbab commented:
"""
Heh, I have opened PR#261 for this same issue recently. Since you assigned
yourself to the ticket and filled all required fields you win this race :).
"""
URL: https://github.com/freeipa/freeipa/pull/262
Author: tomaskrizek
Title: #262: upgrade: do not explicitly set principal for services
Action: opened
PR body:
"""
After installer refactoring, principal is a property of service.
https://fedorahosted.org/freeipa/ticket/6500
"""
To pull the PR
URL: https://github.com/freeipa/freeipa/pull/223
Author: tomaskrizek
Title: #223: LDAP refactoring: remove admin_conn
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/223/head:pr223
git checkout pr223
From b
URL: https://github.com/freeipa/freeipa/pull/262
Title: #262: upgrade: do not explicitly set principal for services
martbab commented:
"""
Heh, I have opened https://github.com/freeipa/freeipa/pull/262 for this same
issue recently. Since you assigned yourself to the ticket and filled all
requi
URL: https://github.com/freeipa/freeipa/pull/261
Author: martbab
Title: #261: upgrade: do not set HTTP and DS principals explicitly
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/261/head:pr261
git checkout pr26
URL: https://github.com/freeipa/freeipa/pull/261
Title: #261: upgrade: do not set HTTP and DS principals explicitly
martbab commented:
"""
Closing this as duplicate of https://github.com/freeipa/freeipa/pull/262
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/261#issuecomme
URL: https://github.com/freeipa/freeipa/pull/262
Title: #262: upgrade: do not explicitly set principal for services
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/pag
URL: https://github.com/freeipa/freeipa/pull/262
Title: #262: upgrade: do not explicitly set principal for services
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/
URL: https://github.com/freeipa/freeipa/pull/262
Title: #262: upgrade: do not explicitly set principal for services
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/2793cdc8593c40d8318ec3685408ade6bf9a5320
"""
See the full comment at
https://github.com/f
URL: https://github.com/freeipa/freeipa/pull/262
Author: tomaskrizek
Title: #262: upgrade: do not explicitly set principal for services
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/262/head:pr262
git checkout
On 21.11.2016 15:25, Jan Cholasta wrote:
On 21.11.2016 15:07, Christian Heimes wrote:
On 2016-11-21 14:44, Petr Spacek wrote:
3.3 ipaplatform auto-configuration
I'm not sure if guessing platform from ID_LIKE is really a good
idea. It
might work fine for centos -> rhel, but in general we can't
On 10/10/2016 08:47 AM, Standa Laznicka wrote:
On 10/10/2016 07:53 AM, Jan Cholasta wrote:
On 7.10.2016 12:23, Standa Laznicka wrote:
On 10/07/2016 08:31 AM, Jan Cholasta wrote:
On 17.8.2016 13:47, Stanislav Laznicka wrote:
On 08/11/2016 02:59 PM, Stanislav Laznicka wrote:
On 08/11/2016 07:4
URL: https://github.com/freeipa/freeipa/pull/251
Author: tiran
Title: #251: Add rebuild rule for template files
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/251/head:pr251
git checkout pr251
--
Manage your su
URL: https://github.com/freeipa/freeipa/pull/251
Title: #251: Add rebuild rule for template files
pspacek commented:
"""
I'm going to provide a proper fix as agreed on meeting today.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/251#issuecomment-261987023
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/251
Title: #251: Add rebuild rule for template files
Label: +rejected
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/
URL: https://github.com/freeipa/freeipa/pull/260
Title: #260: Build: fix path in ipa-ods-exporter.socket unit file
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/5862eaa1a0e3fbced79d6c209016c1138e692888
"""
See the full comment at
https://github.com/
URL: https://github.com/freeipa/freeipa/pull/260
Author: pspacek
Title: #260: Build: fix path in ipa-ods-exporter.socket unit file
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/260/head:pr260
git checkout pr260
URL: https://github.com/freeipa/freeipa/pull/261
Title: #261: upgrade: do not set HTTP and DS principals explicitly
Label: +rejected
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.or
URL: https://github.com/freeipa/freeipa/pull/259
Title: #259: Minor fixes for IPAVersion class
mbasti-rh commented:
"""
LGTM, I'll test tomorrow
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/259#issuecomment-261995775
--
Manage your subscription for the Freeipa-devel mai
URL: https://github.com/freeipa/freeipa/pull/258
Title: #258: Break ipaplatform / ipalib import cycle of hell
mbasti-rh commented:
"""
LGTM, except the inline comment I made, I'll test it tomorrow
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/258#issuecomment-261995638
-
URL: https://github.com/freeipa/freeipa/pull/256
Title: #256: Pylint: whitelist packages with extension modules
mbasti-rh commented:
"""
LGTM, I'll test it tomorrow
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/256#issuecomment-261997895
--
Manage your subscription for t
URL: https://github.com/freeipa/freeipa/pull/212
Title: #212: KRA: don't add KRA container when KRA replica
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/61094a2a20f5cacdb7c87940d0db8d8593a87505
"""
See the full comment at
https://github.com/freeipa
URL: https://github.com/freeipa/freeipa/pull/212
Author: mbasti-rh
Title: #212: KRA: don't add KRA container when KRA replica
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/212/head:pr212
git checkout pr212
--
URL: https://github.com/freeipa/freeipa/pull/212
Title: #212: KRA: don't add KRA container when KRA replica
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Con
URL: https://github.com/freeipa/freeipa/pull/249
Title: #249: Remove references to ds_newinst.pl
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/687ebd18a1927cd6dcbb6cb884b979096c8a44aa
"""
See the full comment at
https://github.com/freeipa/freeipa/pu
URL: https://github.com/freeipa/freeipa/pull/249
Author: frasertweedale
Title: #249: Remove references to ds_newinst.pl
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/249/head:pr249
git checkout pr249
--
Manage
URL: https://github.com/freeipa/freeipa/pull/244
Title: #244: Add templating to ipaplatform path [RFC]
mbasti-rh commented:
"""
Can you finish this PR? It can be tested and possibly merged
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/244#issuecomment-261997481
--
Manage
URL: https://github.com/freeipa/freeipa/pull/249
Title: #249: Remove references to ds_newinst.pl
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Cod
URL: https://github.com/freeipa/freeipa/pull/255
Title: #255: Adjustments for setup requirements
mbasti-rh commented:
"""
I found some changes in versions of dependencies I don't like, because there is
no explanation why it is needed and it is out of sync between specfile and
setup.py
"""
Se
URL: https://github.com/freeipa/freeipa/pull/254
Title: #254: Replace LooseVersion with pkg_resource.parse_version
martbab commented:
"""
It seems that your changes broke IPA upgrade:
```
Done configuring the web interface (httpd).
Applying LDAP updates
Upgrading IPA:
[1/9]: stopping director
URL: https://github.com/freeipa/freeipa/pull/223
Title: #223: LDAP refactoring: remove admin_conn
mbasti-rh commented:
"""
LGTM and Works for me, but I have to make sure that things I wrote inline
didn't happen
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/223#issuecomme
URL: https://github.com/freeipa/freeipa/pull/223
Title: #223: LDAP refactoring: remove admin_conn
mbasti-rh commented:
"""
LGTM and Works for me, but I have to make sure that things I wrote inline
won''t happen
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/223#issuecomme
URL: https://github.com/freeipa/freeipa/pull/254
Title: #254: Replace LooseVersion with pkg_resource.parse_version
tiran commented:
"""
setuptool's version parser does not support slicing. I need to find another
solution for ```verify_client_version()```.
"""
See the full comment at
https://g
URL: https://github.com/freeipa/freeipa/pull/258
Title: #258: Break ipaplatform / ipalib import cycle of hell
jcholast commented:
"""
The original code is broken by design IMO. The API object is used only to get
the configured service startup timeout and to guess our DS instance name. None
of
URL: https://github.com/freeipa/freeipa/pull/244
Title: #244: Add templating to ipaplatform path [RFC]
jcholast commented:
"""
Also LGTM.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/244#issuecomment-262162424
--
Manage your subscription for the Freeipa-devel mailing li
URL: https://github.com/freeipa/freeipa/pull/231
Title: #231: Do not log DM password in ca/kra installation logs
tomaskrizek commented:
"""
I agree. We need to re-add `self.dm_password` to `nolog_list`, just like it was
before I removed it
[here](https://github.com/freeipa/freeipa/commit/9340a
80 matches
Mail list logo