URL: https://github.com/freeipa/freeipa/pull/226
Title: #226: Build refactoring phase 5
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/Code
URL: https://github.com/freeipa/freeipa/pull/226
Author: pspacek
Title: #226: Build refactoring phase 5
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/226/head:pr226
git checkout pr226
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/226
Author: pspacek
Title: #226: Build refactoring phase 5
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/226/head:pr226
git checkout pr226
From
URL: https://github.com/freeipa/freeipa/pull/214
Title: #214: ipaldap: remove do_bind from LDAPClient
jcholast commented:
"""
DM password may be `None` in `dns_container_exists()` and
`dnssec_container_exists()` (for example in `BindInstance.setup()`), so you
can't simple bind
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: reopened
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/214/head:pr214
git checkout pr214
--
URL: https://github.com/freeipa/freeipa/pull/226
Title: #226: Build refactoring phase 5
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/Code
URL: https://github.com/freeipa/freeipa/pull/226
Title: #226: Build refactoring phase 5
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/394edf5f055766fa0cdf70afab6d263f75d0d065
URL: https://github.com/freeipa/freeipa/pull/233
Author: pspacek
Title: #233: Build phase 6: %install cleanup
Action: opened
PR body:
"""
This patch set is based on phase 5 - PR #226.
Now all the installation steps (except Python2/3) are handled by make install.
Python 2/3 support will
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/214/head:pr214
git checkout pr214
URL: https://github.com/freeipa/freeipa/pull/232
Title: #232: Installer refactoring
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/Code
URL: https://github.com/freeipa/freeipa/pull/232
Title: #232: Installer refactoring
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/Code
URL: https://github.com/freeipa/freeipa/pull/233
Author: pspacek
Title: #233: Build phase 6: %install cleanup
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/233/head:pr233
git checkout pr233
From
URL: https://github.com/freeipa/freeipa/pull/233
Title: #233: Build phase 6: %install cleanup
pspacek commented:
"""
Rebased on top of current master.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/233#issuecomment-259937908
--
Manage your subscription for the
URL: https://github.com/freeipa/freeipa/pull/234
Author: martbab
Title: #234: Always use GSSAPI to set up initial replication
Action: opened
PR body:
"""
This PR makes DS replica use common method to set up initial replication in
both domain levels, namely GSSAPI. Since the workflow was
URL: https://github.com/freeipa/freeipa/pull/232
Title: #232: Installer refactoring
mbasti-rh commented:
"""
ACK, we may ignore those minor PEP8 issues, it is mainly caused by copying code
to other parts.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/207
Title: #207: Provide user hint about IP address in IPA install
mbasti-rh commented:
"""
LGTM, I'll test later
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/207#issuecomment-259938389
--
Manage your subscription for the
URL: https://github.com/freeipa/freeipa/pull/174
Title: #174: add log module
mbasti-rh commented:
"""
@shanyin Did centralized logging meet your requirements?
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/174#issuecomment-259939617
--
Manage your subscription for the
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/214/head:pr214
git checkout pr214
URL: https://github.com/freeipa/freeipa/pull/232
Title: #232: Installer refactoring
jcholast commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/eac6f52957c361c219ad6048b515ddb62da31154
URL: https://github.com/freeipa/freeipa/pull/232
Author: jcholast
Title: #232: Installer refactoring
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/232/head:pr232
git checkout pr232
--
Manage your subscription
URL: https://github.com/freeipa/freeipa/pull/233
Title: #233: Build phase 6: %install cleanup
mbasti-rh commented:
"""
Build failed
```
Failed to open: 'freeipa.spec.in', not a valid spec file.
```
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/233#issuecomment-259939261
URL: https://github.com/freeipa/freeipa/pull/213
Author: pspacek
Title: #213: Build system refactoring phase 3
Action: edited
Changed field: body
Original value:
"""
This monster patch-set refactors most of build system and moves most of the
logic from SPEC file to build system.
It is not
URL: https://github.com/freeipa/freeipa/pull/234
Title: #234: Always use GSSAPI to set up initial replication
mbasti-rh commented:
"""
```
Traceback (most recent call last):
File "/sbin/ipa-server-install", line 23, in
from ipaserver.install import ipa_server_install
File
URL: https://github.com/freeipa/freeipa/pull/214
Title: #214: ipaldap: remove do_bind from LDAPClient
mbasti-rh commented:
"""
```
ipaserver/install/ca.py:226: [W0612(unused-variable), install_step_1] Unused
variable 'dm_password')
```
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/207
Title: #207: Provide user hint about IP address in IPA install
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/207
Author: Akasurde
Title: #207: Provide user hint about IP address in IPA install
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/207/head:pr207
git checkout pr207
URL: https://github.com/freeipa/freeipa/pull/207
Title: #207: Provide user hint about IP address in IPA install
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/28bc54f91dfbd76887180fa67ceecb46977a4fb8
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/233
Title: #233: Build phase 6: %install cleanup
pspacek commented:
"""
@tiran Please re-review and set review status accordingly. Thanks!
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/233#issuecomment-259955838
--
Manage
URL: https://github.com/freeipa/freeipa/pull/212
Title: #212: KRA: don't add KRA container when KRA replica
stlaz commented:
"""
configure_instance and configure_replica codes were merged, you'll need to
check for self.clone value instead.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/214/head:pr214
git checkout pr214
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/214/head:pr214
git checkout pr214
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
URL: https://github.com/freeipa/freeipa/pull/233
Author: pspacek
Title: #233: Build phase 6: %install cleanup
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/233/head:pr233
git checkout pr233
From
URL: https://github.com/freeipa/freeipa/pull/233
Author: pspacek
Title: #233: Build phase 6: %install cleanup
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/233/head:pr233
git checkout pr233
From
URL: https://github.com/freeipa/freeipa/pull/233
Author: pspacek
Title: #233: Build phase 6: %install cleanup
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/233/head:pr233
git checkout pr233
From
URL: https://github.com/freeipa/freeipa/pull/233
Title: #233: Build phase 6: %install cleanup
pspacek commented:
"""
I've fixed incorrect use of `-D` in `install` calls above.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/233#issuecomment-259960522
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/196
Title: #196: ipatests: unresolvable nested netgroups
martbab commented:
"""
Is there any particular reason why this is among XML RPC tests and not a
separate integration test? IMHO it should be a CI test as it tests integration
with SSSD. I get
URL: https://github.com/freeipa/freeipa/pull/233
Title: #233: Build phase 6: %install cleanup
pspacek commented:
"""
> Failed to open: 'freeipa.spec.in', not a valid spec file.
Damn it! I added the last commit with comment in SPEC file and did not run
tests on that. Surprise surprise, RPM is
URL: https://github.com/freeipa/freeipa/pull/207
Title: #207: Provide user hint about IP address in IPA install
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/233
Title: #233: Build phase 6: %install cleanup
pspacek commented:
"""
This version fixes the fixable issues, i.e. everything mentioned above except
changing file ownership.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/230
Title: #230: cert-request: accept CSRs with extraneous data
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/230
Title: #230: cert-request: accept CSRs with extraneous data
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/e1df2e0792a6a423563c4787215b284948f51582
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/230
Title: #230: cert-request: accept CSRs with extraneous data
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/230
Author: frasertweedale
Title: #230: cert-request: accept CSRs with extraneous data
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/230/head:pr230
git checkout
URL: https://github.com/freeipa/freeipa/pull/212
Author: mbasti-rh
Title: #212: KRA: don't add KRA container when KRA replica
Action: synchronized
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
URL: https://github.com/freeipa/freeipa/pull/209
Title: #209: Enumerate available options in IPA installer
mbasti-rh commented:
"""
Hi, we changed a lot of code during refactoring, PR doesn't apply. IMO list of
choices shown in --help should be handled in knob() if metavar is not specified
URL: https://github.com/freeipa/freeipa/pull/214
Title: #214: ipaldap: remove do_bind from LDAPClient
martbab commented:
"""
The fix works as expected. ACK.
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/f183f70e0183e51d569ada972bd3ec73cad76a30
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/234
Author: martbab
Title: #234: Always use GSSAPI to set up initial replication
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/234/head:pr234
git checkout
URL: https://github.com/freeipa/freeipa/pull/182
Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context
tiran commented:
"""
http://www.freeipa.org/page/V4/Integration_Improvements#API_for_local_configuration_directory
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/214/head:pr214
git checkout pr214
--
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/page/V4/Integration_Improvements
Regards,
Christian
URL: https://github.com/freeipa/freeipa/pull/235
Author: mbasti-rh
Title: #235: Make Knob function deprecated
Action: opened
PR body:
"""
`Knob` function is outdated and was replaced by `knob`. Make explicit
note in code about this.
https://fedorahosted.org/freeipa/ticket/6392
"""
To pull
URL: https://github.com/freeipa/freeipa/pull/235
Title: #235: Make Knob function deprecated
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/Code
URL: https://github.com/freeipa/freeipa/pull/235
Title: #235: Make Knob function deprecated
stlaz commented:
"""
ACK, there should be note about this deprecation somewhere. Deleting Knob might
be worth a ticket as well.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/235
Title: #235: Make Knob function deprecated
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/Code
URL: https://github.com/freeipa/freeipa/pull/235
Title: #235: Make Knob function deprecated
mbasti-rh commented:
"""
I would wait with ACK, I realized that Knobs with capital K are not used
anymore, so we can remove it instead of deprecating
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/236
Author: pspacek
Title: #236: Build phase 7: cleanup
Action: opened
PR body:
"""
Depends on PR #233.
- Clean-up ancient leftovers and clean minor bugs here and there.
- Support --enable-silent-rules and V=0 variable for make to make the build
URL: https://github.com/freeipa/freeipa/pull/236
Title: #236: Build phase 7: cleanup
lslebodn commented:
"""
You should the opposite of the last patch "Build: clean-up spurious NULL
variables from Makefile.am files". The NULL should be added to each list.
NACK to the last patch. It does not
URL: https://github.com/freeipa/freeipa/pull/235
Author: mbasti-rh
Title: #235: Remove unused Knob function
Action: edited
Changed field: title
Original value:
"""
Make Knob function deprecated
"""
--
Manage your subscription for the Freeipa-devel mailing list:
URL: https://github.com/freeipa/freeipa/pull/235
Author: mbasti-rh
Title: #235: Make Knob function deprecated
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/235/head:pr235
git checkout pr235
From
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.
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, ipapython, ipaclient,
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, ipapython, ipaclient, ipaplatform) for third party
On 2016-11-11 18:33, Rob Crittenden wrote:
> Martin Basti wrote:
>> 2) if I understand correctly, you want to separate client installer code
>> and client CLI code. In past we had freeipa-admintools but it was
>> removed because it was really tightly bounded to installed client. Do
>> you want to
64 matches
Mail list logo