URL: https://github.com/freeipa/freeipa/pull/496
Author: stlaz
Title: #496: Use newer Certificate.serial_number in krainstance.py
Action: opened
PR body:
"""
This bit was missed in https://github.com/freeipa/freeipa/pull/458
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://g
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
stlaz commented:
"""
I see where this is going, added the ticket to the commit message.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/495#issuecomment-281912763
--
Manage your subscripti
URL: https://github.com/freeipa/freeipa/pull/495
Author: stlaz
Title: #495: Fix ipa-server-upgrade
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/495/head:pr495
git checkout pr495
From 499ce6c6ef6f921f0be8
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
HonzaCholasta commented:
"""
@MartinBasti, https://fedorahosted.org/freeipa/ticket/5959.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/495#issuecomment-281905518
--
Manage your subscript
On Wed, Feb 22, 2017 at 10:00:04AM -0500, Simo Sorce wrote:
> On Wed, 2017-02-22 at 10:59 +, Oucema Bellagha wrote:
> > I want to figure out a solution which allow user"a" to authenticate to
> > a host only when user"b" is accessing the host for security reasons.
> >
> >
> > Easy explanation:
URL: https://github.com/freeipa/freeipa/pull/398
Title: #398: Support for Certificate Identity Mapping
flo-renaud commented:
"""
Hi @sumit-bose ,
I am not able to reproduce this issue:
`[root@vm-161 ~]# kinit -k
[root@vm-161 ~]# klist
Ticket cache: KEYRING:persistent:0:krb_ccache_h6XRpeK
Default
URL: https://github.com/freeipa/freeipa/pull/398
Title: #398: Support for Certificate Identity Mapping
sumit-bose commented:
"""
It looks like the ACis on the latest version do not allow hosts to access the
rules. When I do 'kinit -k' on the IPA server or a client and call
ldapsearch -H ld
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
lslebodn commented:
"""
>Thanks for your contribution. I added your patch to my PR. On my system I ran
>into a minor issue. >Some C99 types like uint8_t were not defined and I had to
>include
URL: https://github.com/freeipa/freeipa/pull/492
Title: #492: [WIP] config: remove meaningless defaults
tiran commented:
"""
It's probably easier to always define options like ```'ldap_uri``` but use
```None``` as default.
```
cd .; ./makeaci --validate
./makeaci: ipaserver/plugins/dogtag.py:2
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
tiran commented:
"""
@MartinBasti I have rebased the branch and added wheel + placeholder building
to make check. The pylint violations have disappeared.
"""
See the full comment at
https://gith
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From 4adaffb
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
MartinBasti commented:
"""
LGTM, please rebase and I will test it.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/472#issuecomment-281749499
--
Manage your subscription for t
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
MartinBasti commented:
"""
Does this belong to any ticket which caused this regression?
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/495#issuecomment-281748907
--
Manage your subscripti
URL: https://github.com/freeipa/freeipa/pull/457
Author: martbab
Title: #457: adtrustinstance: use LDAPI/EXTERNAL to retrieve CIFS keytab
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/457/head:pr457
git checkou
URL: https://github.com/freeipa/freeipa/pull/457
Title: #457: adtrustinstance: use LDAPI/EXTERNAL to retrieve CIFS keytab
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeip
URL: https://github.com/freeipa/freeipa/pull/457
Title: #457: adtrustinstance: use LDAPI/EXTERNAL to retrieve CIFS keytab
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/af998c4d30175fb3ecc148e1b3a7aca03ef9239a
https://fedorahosted.org/freeipa/changes
URL: https://github.com/freeipa/freeipa/pull/457
Title: #457: adtrustinstance: use LDAPI/EXTERNAL to retrieve CIFS keytab
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.o
URL: https://github.com/freeipa/freeipa/pull/367
Title: #367: Remove nsslib from IPA
stlaz commented:
"""
First set of fixes to comments arrived, throwing it to Travis.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/367#issuecomment-281710491
--
Manage your subscription f
On Wed, 2017-02-22 at 10:59 +, Oucema Bellagha wrote:
> I want to figure out a solution which allow user"a" to authenticate to
> a host only when user"b" is accessing the host for security reasons.
>
>
> Easy explanation: authenticate to hostx needs (user a + user b)
>
>
> I'm brainstorming
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
pvoborni commented:
"""
Also I added section to FreeIPA wiki:
http://www.freeipa.org/page/V4/Build_system_refactoring#Packager_-_client_only_build
"""
See the full comment at
https://github.
URL: https://github.com/freeipa/freeipa/pull/468
Title: #468: Remove non-sensical kdestroy on https stop
martbab commented:
"""
@pvoborni should the fix go also into 4-4 branch? see
https://fedorahosted.org/freeipa/ticket/6673#comment:3
"""
See the full comment at
https://github.com/freeipa/
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
pvoborni commented:
"""
#364 was pushed.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/494#issuecomment-281690828
--
Manage your subscription for the Freeipa-devel mailing list:
http
URL: https://github.com/freeipa/freeipa/pull/494
Author: lslebodn
Title: #494: Support client-only build
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/494/head:pr494
git checkout pr494
--
Manage your subscript
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
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/Code
URL: https://github.com/freeipa/freeipa/pull/468
Title: #468: Remove non-sensical kdestroy on https stop
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/b8f304c66994ae82ea484a4e8bd057d4ccf1e6bd
"""
See the full comment at
https://github.com/freeipa/free
URL: https://github.com/freeipa/freeipa/pull/468
Title: #468: Remove non-sensical kdestroy on https stop
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/Contri
URL: https://github.com/freeipa/freeipa/pull/468
Author: simo5
Title: #468: Remove non-sensical kdestroy on https stop
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/468/head:pr468
git checkout pr468
--
Manage
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
pvoborni commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/70554938d4f9ba5b347cd4bc8001428e905198e4
https://fedorahosted.org/freeipa/changeset/41d7ae54fafc6deb602
URL: https://github.com/freeipa/freeipa/pull/364
Author: tiran
Title: #364: Client-only builds with --disable-server
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/364/head:pr364
git checkout pr364
--
Manage yo
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
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/364
Title: #364: Client-only builds with --disable-server
lslebodn commented:
"""
On (22/02/17 06:16), Simo Sorce wrote:
>So this is the reasoning and why I am approving this PR and not #494.
>
>When you build all components, including server bits, te
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 01:52), Tomas Krizek wrote:
>@lslebodn My bad, there was some leftover stuff that `git clean -dfx` didn't
>clear for some reason.
>
>Nevertheless, this does work and allo
URL: https://github.com/freeipa/freeipa/pull/490
Author: HonzaCholasta
Title: #490: [WIP] certdb: use certutil and match_hostname for cert
verification
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/490/h
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
simo5 commented:
"""
So this is the reasoning and why I am approving this PR and not #494.
When you build all components, including server bits, tests are installed,
therefore when we build j
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
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/434
Title: #434: csrgen: Automate full cert request flow
HonzaCholasta commented:
"""
Thank you. LGTM, but please squash the fixup commit.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/434#issuecomment-281679144
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 03:24), Christian Heimes wrote:
>python-requests is a bad example because it suffers from the same issue as IPA.
>
>A better example is any other modern Python project lik
On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote:
> On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
> > On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote:
> >> On 02/21/2017 04:24 PM, Tomas Krizek wrote:
> >>> On 02/21/2017 03:23 PM, Rob Crittenden wrote:
> Standa Lazn
On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
> On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote:
>> On 02/21/2017 04:24 PM, Tomas Krizek wrote:
>>> On 02/21/2017 03:23 PM, Rob Crittenden wrote:
Standa Laznicka wrote:
> Hello,
>
> Since we're trying to make FreeIPA
URL: https://github.com/freeipa/freeipa/pull/367
Title: #367: Remove nsslib from IPA
HonzaCholasta commented:
"""
Besides what I wrote in inline comments, we need to get rid of
`/var/lib/ipa/radb` now that it's unused.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/367#is
On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote:
On 02/21/2017 04:24 PM, Tomas Krizek wrote:
On 02/21/2017 03:23 PM, Rob Crittenden wrote:
Standa Laznicka wrote:
Hello,
Since we're trying to make FreeIPA work in FIPS we got to th
URL: https://github.com/freeipa/freeipa/pull/479
Title: #479: Merge AD trust installer into composite ones
martbab commented:
"""
I have added a basic integration tests for the built-in AD trust installation,
you can run them on 3 machines (master + 2 replicas) by running
```bash
# ipa-run-tes
URL: https://github.com/freeipa/freeipa/pull/479
Author: martbab
Title: #479: Merge AD trust installer into composite ones
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/479/head:pr479
git checkout pr479
F
URL: https://github.com/freeipa/freeipa/pull/468
Title: #468: Remove non-sensical kdestroy on https stop
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/Contribut
URL: https://github.com/freeipa/freeipa/pull/468
Title: #468: Remove non-sensical kdestroy on https stop
martbab commented:
"""
I have also noticed that the ccache is not created there, strange. However I
think it is better to explicitly specify file-based ccache anyway just to be
one the safe
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 03:24), Christian Heimes wrote:
>python-requests is a bad example because it suffers from the same issue as IPA.
>
>A better example is any other modern Python project lik
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
tiran commented:
"""
Looks totally reasonable.
I checked, ```SimpleServiceInstance('ipa_memcached')``` does not raise an
exception if systemd has no service file for IPA memcached at all.
"""
See the full com
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
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/495
Author: stlaz
Title: #495: Fix ipa-server-upgrade
Action: opened
PR body:
"""
I was to eager to ACK https://github.com/freeipa/freeipa/pull/471.
Running ipa-server-upgrade would fail to stop ipa_memcached if
it's already uninstalled.
"""
To pu
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tiran commented:
"""
python-requests is a bad example because it suffers from the same issue as IPA.
A better example is any other modern Python project like cryptography. It runs
tests with installed files
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
lslebodn commented:
"""
>Thanks for your contribution. I added your patch to my PR. On my system I ran
>into a minor issue. >Some C99 types like uint8_t were not defined and I had to
>include
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 03:04), Christian Heimes wrote:
>You are aware that your example code checks the wrong code? It is testing
>in-tree sources, not the actual sources that get packaged and
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tiran commented:
"""
You are aware that your example code checks the wrong code? It is testing
in-tree sources, not the actual sources that get packaged and installed.
"""
See the full comment at
https://g
I want to figure out a solution which allow user"a" to authenticate to a host
only when user"b" is accessing the host for security reasons.
Easy explanation: authenticate to hostx needs (user a + user b)
I'm brainstorming some ideas using Yubikey or ssh-keys.. Is there any
application which a
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 02:51), Christian Heimes wrote:
>You assumption is incorrect. ```ipatests``` does not depend on
>```ipaserver```,
>https://github.com/freeipa/freeipa/blob/master/ipatest
URL: https://github.com/freeipa/freeipa/pull/457
Title: #457: adtrustinstance: use LDAPI/EXTERNAL to retrieve CIFS keytab
martbab commented:
"""
Bump for review.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/457#issuecomment-281635160
--
Manage your subscription for the
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tiran commented:
"""
You assumption is incorrect. ```ipatests``` does not depend on ```ipaserver```,
https://github.com/freeipa/freeipa/blob/master/ipatests/setup.py#L61
```
install_requires=[
URL: https://github.com/freeipa/freeipa/pull/479
Title: #479: Merge AD trust installer into composite ones
martbab commented:
"""
Bump for review.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/479#issuecomment-281635086
--
Manage your subscription for the Freeipa-devel m
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 02:09), Christian Heimes wrote:
>There are two reasons we decided on ```--without-ipatests```:
>
>* ```--with-tests``` / ```--without-tests``` is technically not correct.
On 22.2.2017 11:28, Sumit Bose wrote:
On Wed, Feb 22, 2017 at 10:02:24AM +0100, Petr Vobornik wrote:
On 02/22/2017 12:43 AM, Fraser Tweedale wrote:
On Tue, Feb 21, 2017 at 06:12:23PM +0100, Petr Vobornik wrote:
On 02/21/2017 05:15 PM, Florence Blanc-Renaud wrote:
Hi,
related to the Certifica
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 02:23), Christian Heimes wrote:
>tiran requested changes on this pull request.
>
>see comments
>
>> -CFLAGS="$bck_cflags"
>-
>-LIBPDB_NAME=""
>-AC_CHECK_LIB([samba-passdb]
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 02:16), Christian Heimes wrote:
>NACK on 42fb9b1c
>
>* Either use ```--with-ipaplatform=redhat``` on CentOS
>* Or implement a proper way to fill ipaplatfrom from ```/etc/o
On Wed, Feb 22, 2017 at 10:02:24AM +0100, Petr Vobornik wrote:
> On 02/22/2017 12:43 AM, Fraser Tweedale wrote:
> > On Tue, Feb 21, 2017 at 06:12:23PM +0100, Petr Vobornik wrote:
> > > On 02/21/2017 05:15 PM, Florence Blanc-Renaud wrote:
> > > > Hi,
> > > >
> > > > related to the Certificate Ident
URL: https://github.com/freeipa/freeipa/pull/494
Author: lslebodn
Title: #494: Support client-only build
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/494/head:pr494
git checkout pr494
From b4e0d5ed62bfdb
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tiran commented:
"""
NACK on 42fb9b1c
* Either use ```--with-ipaplatform=redhat``` on CentOS
* Or implement a proper way to fill ipaplatfrom from ```/etc/os-relase``` value
```ID_LIKE```, https://www.freede
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tiran commented:
"""
There are two reasons we decided on ```--without-ipatests```:
* ```--with-tests``` / ```--without-tests``` is technically not correct. We
still compile C tests. The flag is about the co
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 01:43), Christian Heimes wrote:
>NACK on aece4c3c
>
>We compromised on ```--without-ipatests``` with installation of ipatests
>defaulting to true. The compromose was alre
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
tomaskrizek commented:
"""
The PR works and the `--without-ipatests` option omits the ipatests directory.
However, #494 doesn't install extra dependencies with `mock --without=server`.
"""
Se
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tomaskrizek commented:
"""
@lslebodn My bad, there was some leftover stuff that `git clean -dfx` didn't
clear for some reason.
Nevertheless, this does work and allows a client only, as well as installing
t
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tiran commented:
"""
NACK on aece4c3c
We compromised on ```--without-ipatests``` with installation of ipatests
defaulting to true. The compromose was already ACKed by @simo5
"""
See the full comment at
h
URL: https://github.com/freeipa/freeipa/pull/476
Author: HonzaCholasta
Title: #476: vault: cache the transport certificate on client
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/476/head:pr476
git checko
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
MartinBasti commented:
"""
Thank you.
I see errors reported by pylint
```
* Module ipaserver.install.installutils
ipaserver/install/installutils.py:1209: [E1101(no-member), store_versio
On 02/20/2017 06:03 AM, Fraser Tweedale wrote:
> On Fri, Feb 10, 2017 at 11:48:39AM +0100, Martin Kosek wrote:
>> On 02/10/2017 10:37 AM, Fraser Tweedale wrote:
>>> On Fri, Feb 10, 2017 at 09:23:10AM +0100, Martin Kosek wrote:
On 02/09/2017 10:44 PM, Fraser Tweedale wrote:
> On Thu, Feb 09
URL: https://github.com/freeipa/freeipa/pull/397
Title: #397: Improve wheel building and provide ipaserver wheel for local
testing
HonzaCholasta commented:
"""
The trust plugin and other trust bits are optional. The cert plugin, which
depends on `pyhbac`, is *not* optional, so you can't apply
URL: https://github.com/freeipa/freeipa/pull/485
Author: simo5
Title: #485: Fix session logout
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/485/head:pr485
git checkout pr485
--
Manage your subscription for th
URL: https://github.com/freeipa/freeipa/pull/485
Title: #485: Fix session logout
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/485
Title: #485: Fix session logout
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/908d2eaba46f5f123b49af400a8b696545c62b54
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/485#issuecommen
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
BTW I tested client-only build on fedora24, fedora25, fedora rawhide,
epel7, debian stable, debian testing, debian unstable
"""
See the full comment at
https://github.com/freeipa/fr
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
lslebodn commented:
"""
On (22/02/17 00:59), Tomas Krizek wrote:
>I'm not able to run autoreconf, it fails with the following error:
>
>```
>configure.ac:447: error: required file 'init/tmpfilesd/Makefile.in'
On 02/22/2017 12:43 AM, Fraser Tweedale wrote:
On Tue, Feb 21, 2017 at 06:12:23PM +0100, Petr Vobornik wrote:
On 02/21/2017 05:15 PM, Florence Blanc-Renaud wrote:
Hi,
related to the Certificate Identity Mapping feature, a new CLI will be
needed to find all the users matching a given certificat
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
lslebodn commented:
"""
>Thanks for your contribution. I added your patch to my PR. On my system I ran
>into a minor issue. >Some C99 types like uint8_t were not defined and I had to
>include
URL: https://github.com/freeipa/freeipa/pull/494
Title: #494: Support client-only build
tomaskrizek commented:
"""
I'm not able to run autoreconf, it fails with the following error:
```
configure.ac:447: error: required file 'init/tmpfilesd/Makefile.in' not found
asn1/Makefile.am: installing '.
URL: https://github.com/freeipa/freeipa/pull/364
Title: #364: Client-only builds with --disable-server
lslebodn commented:
"""
>Thanks for your contribution. I added your patch to my PR. On my system I ran
>into a minor issue. >Some C99 types like uint8_t were not defined and I had to
>include
URL: https://github.com/freeipa/freeipa/pull/494
Author: lslebodn
Title: #494: Support client-only build
Action: opened
PR body:
"""
How to test:
* autoreconf -if
* ./configure --disable-server
* make srpms
* mock --rebuild dist/rpms/freeipa-4.4.90.*.src.rpm --resultdir .
* mock --rebuild dist
URL: https://github.com/freeipa/freeipa/pull/397
Title: #397: Improve wheel building and provide ipaserver wheel for local
testing
tiran commented:
"""
@HonzaCholasta FreeIPA has conditional imports for SSSD modules in several
places, e.g. in the trust plugin. 96f614e closes the gap and applie
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
tiran commented:
"""
OK, you got ```with_wheels``` in ```freeipa.spec.in``` now. ```with_wheels```
is more logical than ```with_pypi``` because wheels have more uses than just
PyPI upload.
"""
Se
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From 1d3e58c
URL: https://github.com/freeipa/freeipa/pull/492
Title: #492: [WIP] config: remove meaningless defaults
HonzaCholasta commented:
"""
Sure.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/492#issuecomment-281597461
--
Manage your subscription for the Freeipa-devel mailing l
URL: https://github.com/freeipa/freeipa/pull/492
Title: #492: [WIP] config: remove meaningless defaults
tiran commented:
"""
Can you add a comment to explain the order of checks and assignments? Without
explanation, it's going to confuse the next poor developer.
"""
See the full comment at
ht
URL: https://github.com/freeipa/freeipa/pull/301
Author: HonzaCholasta
Title: #301: scripts, tests: explicitly set confdir in the rest of server code
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/301/head:pr301
URL: https://github.com/freeipa/freeipa/pull/301
Title: #301: scripts, tests: explicitly set confdir in the rest of server code
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.
URL: https://github.com/freeipa/freeipa/pull/301
Title: #301: scripts, tests: explicitly set confdir in the rest of server code
HonzaCholasta commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/fe6f2b6f6effcf9f3c58e1e3f6d0874609c10c25
"""
See the full comment at
ht
URL: https://github.com/freeipa/freeipa/pull/301
Title: #301: scripts, tests: explicitly set confdir in the rest of server code
tiran commented:
"""
My philosophy is: _Don't fix it it it ain't broken._
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/301#issuecomment-2815966
URL: https://github.com/freeipa/freeipa/pull/301
Title: #301: scripts, tests: explicitly set confdir in the rest of server code
tiran commented:
"""
My philosophy is: _Don't fix it it it ain't broken._
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/301#issuecomment-2815966
94 matches
Mail list logo
