Ping for review
On 11/03/2016 04:56 PM, Oleg Fayans wrote:
Hi Martin,
The commit message was updated with the correct ticket link
Thanks for review!
On 11/03/2016 04:22 PM, Martin Basti wrote:
almost ACK, but the ticket in commit message is closed as invalid. So
I'm quite puzzled now what to
URL: https://github.com/freeipa/freeipa/pull/207
Author: Akasurde
Title: #207: Provide user hint about IP address in IPA install
Action: synchronized
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 p
URL: https://github.com/freeipa/freeipa/pull/214
Author: tomaskrizek
Title: #214: ipaldap: remove do_bind from LDAPClient
Action: opened
PR body:
"""
Remove do_bind() method that was a relict used in IPAdmin. Replace
its uses with simple / external binds.
https://fedorahosted.org/freeipa/tick
URL: https://github.com/freeipa/freeipa/pull/215
Author: jumitche
Title: #215: Add script to setup krb5 NFS exports
Action: opened
PR body:
"""
python script to setup secure NFS exports with kerberos that relies heavily on
FreeIPA, and is in many ways the compliment to ipa-client-automount th
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
mbasti-rh commented:
"""
Why are there these whitespace errors?
```
git am ...
...
Applying: Build: stop build when a step in web UI build fails
Applying: Build: fix distribution of static files for we
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
mbasti-rh commented:
"""
I cannot build rpms
```
./makerpms.sh
...
config.status: executing libtool commands
IPA Server 4.4.90.201611071629GITbcd4f03
==
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
mbasti-rh commented:
"""
Build works for me, error was on my side
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/213#issuecomment-259128604
--
Manage your subscription for the F
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
mbasti-rh commented:
"""
Works for me
I read commits, it makes sense, works for me, but I'm not an autotools expert
so I might miss something.
I'm curious about one commit: Build: cleanup unused L
URL: https://github.com/freeipa/freeipa/pull/202
Title: #202: ipa-getkeytab enhancements
martbab commented:
"""
Done.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/202#issuecomment-259153194
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redh
URL: https://github.com/freeipa/freeipa/pull/202
Author: martbab
Title: #202: ipa-getkeytab enhancements
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/202/head:pr202
git checkout pr202
From 7b6669c9d8cf3a
URL: https://github.com/freeipa/freeipa/pull/172
Title: #172: fix pki-tomcat error after uninstall
tomaskrizek commented:
"""
I have not encountered this issue in Fedora. I guess it's related to using
FreeIPA on Ubuntu 16.04. The proposed solution is distribution dependant
workaround.
I'm clo
URL: https://github.com/freeipa/freeipa/pull/168
Title: #168: Update cli.py
tomaskrizek commented:
"""
Could you please file a ticket for the issue, describing what is the problem
and how to reproduce it?
https://fedorahosted.org/freeipa
"""
See the full comment at
https://github.com/freeip
URL: https://github.com/freeipa/freeipa/pull/168
Title: #168: Update cli.py
Garont commented:
"""
Sorry, I've already quit the job where I've implemented freeipa, so now stack
with bug is unavailable for me :)
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/168#issuecommen
URL: https://github.com/freeipa/freeipa/pull/168
Title: #168: Update cli.py
Garont commented:
"""
Sorry, I've already quit the job where I've implemented freeipa, so now stack
bug is unavailable for me :)
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/168#issuecomment-259
URL: https://github.com/freeipa/freeipa/pull/202
Title: #202: ipa-getkeytab enhancements
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/202
Title: #202: ipa-getkeytab enhancements
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/294fc3dc5645eeb7942908c3e351c06aa0af329e
https://fedorahosted.org/freeipa/changeset/0c68c27e51c2a30265a760382d7d4fab7d
URL: https://github.com/freeipa/freeipa/pull/202
Title: #202: ipa-getkeytab enhancements
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/202
Author: martbab
Title: #202: ipa-getkeytab enhancements
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/202/head:pr202
git checkout pr202
From 63a0c7c9591b9e
URL: https://github.com/freeipa/freeipa/pull/202
Author: martbab
Title: #202: ipa-getkeytab enhancements
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/202/head:pr202
git checkout pr202
--
Manage your subscript
another ping for review
On 11/08/2016 09:32 AM, Oleg Fayans wrote:
Ping for review
On 11/03/2016 04:56 PM, Oleg Fayans wrote:
Hi Martin,
The commit message was updated with the correct ticket link
Thanks for review!
On 11/03/2016 04:22 PM, Martin Basti wrote:
almost ACK, but the ticket in c
Never give up pinging :)
On 11/03/2016 12:43 PM, Martin Basti wrote:
LGTM
On 03.11.2016 09:42, Oleg Fayans wrote:
One more ping for review
On 10/27/2016 02:21 PM, Oleg Fayans wrote:
ping for review
On 10/25/2016 10:24 AM, Oleg Fayans wrote:
Integration part of the tests is ready. 2 tests:
And this one.
On 11/03/2016 09:42 AM, Oleg Fayans wrote:
One more ping for review
On 10/27/2016 02:21 PM, Oleg Fayans wrote:
ping for review
On 10/25/2016 11:29 AM, Oleg Fayans wrote:
The patch was rebased to be able to apply on top of latest version of
certs in idoverrides patch. As before,
URL: https://github.com/freeipa/freeipa/pull/195
Title: #195: [WIP] Make ipaclient pip install-able
rcritten commented:
"""
This is an important feature for integration. OpenStack uses pip to install
dependencies into virtual environments for doing multi-version python testing.
I'll need ipali
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 y
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
pspacek commented:
"""
- This version supports lint target and all configuration options for linters
listed in the design document.
- Fixes in systemd-tmpfiles call from make install.
- Updates .travi
URL: https://github.com/freeipa/freeipa/pull/195
Title: #195: [WIP] Make ipaclient pip install-able
mbasti-rh commented:
"""
Where is a ticket? :)
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/195#issuecomment-259196360
--
Manage your subscription for the Freeipa-devel m
URL: https://github.com/freeipa/freeipa/pull/205
Title: #205: Support DAL version 5 and version 6
simo5 commented:
"""
There was no upstream ticket when I created the commit :-)
I'll add.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/205#issuecomment-259221154
--
Manage
URL: https://github.com/freeipa/freeipa/pull/205
Author: simo5
Title: #205: Support DAL version 5 and version 6
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/205/head:pr205
git checkout pr205
From bad5cad
28 matches
Mail list logo