URL: https://github.com/freeipa/freeipa/pull/100
Title: #100: Ipa 4 4
dkupka commented:
"""
> hi, my issue is not in deployment my issue is a bug in the software itself i
> have already submitted a ticket with redhat support which is taking too much
> time so i hoped you guys can see this sinc
URL: https://github.com/freeipa/freeipa/pull/102
Title: #102: Updated ipa-client-install info about hostname
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/Contr
URL: https://github.com/freeipa/freeipa/pull/84
Author: stlaz
Title: #84: Fix update_from_dict function testing
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/84/head:pr84
git checkout pr84
From 45b739afe8
URL: https://github.com/freeipa/freeipa/pull/84
Title: #84: Fix update_from_dict function testing
stlaz commented:
"""
Removed the method and its respective test again.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/84#issuecomment-248832033
--
Manage your subscription fo
URL: https://github.com/freeipa/freeipa/pull/84
Author: stlaz
Title: #84: Removed update_from_dict function from ldapupdate
Action: edited
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/84/head:pr84
git checkout pr84
--
Manag
URL: https://github.com/freeipa/freeipa/pull/102
Author: stlaz
Title: #102: Updated ipa-client-install info about hostname
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/102/head:pr102
git checkout pr102
--
Man
URL: https://github.com/freeipa/freeipa/pull/102
Title: #102: Updated ipa-client-install info about hostname
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/Co
URL: https://github.com/freeipa/freeipa/pull/102
Title: #102: Updated ipa-client-install info about hostname
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/2e0afab5f2a47149580b4bc79093cdbb77f489c3
ipa-4-4:
https://fedorahosted.org/freeipa/changeset/e2a
URL: https://github.com/freeipa/freeipa/pull/97
Title: #97: Pylint fixes
tomaskrizek commented:
"""
NACK, please see inline comments
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/97#issuecomment-248867576
--
Manage your subscription for the Freeipa-devel mailing list:
ht
Fixed patch N 41
On 09/21/2016 04:21 PM, Oleg Fayans wrote:
Patch-0076 rebased to current master
On 09/21/2016 02:41 PM, Oleg Fayans wrote:
Hi David,
As per your comments the patches were once again refactored. I am
attaching the full set of them, please ignore any previous versions
The patch
URL: https://github.com/freeipa/freeipa/pull/84
Title: #84: Removed update_from_dict function from ldapupdate
abbra commented:
"""
LGTM. Thanks.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/84#issuecomment-248869910
--
Manage your subscription for the Freeipa-devel mail
URL: https://github.com/freeipa/freeipa/pull/84
Title: #84: Removed update_from_dict function from ldapupdate
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/Cont
URL: https://github.com/freeipa/freeipa/pull/84
Title: #84: Removed update_from_dict function from ldapupdate
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/330a3ca93101bcec82ec5d3add14586871864bdd
ipa-4-4:
https://fedorahosted.org/freeipa/changeset/fd
URL: https://github.com/freeipa/freeipa/pull/84
Author: stlaz
Title: #84: Removed update_from_dict function from ldapupdate
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/84/head:pr84
git checkout pr84
--
Manag
URL: https://github.com/freeipa/freeipa/pull/84
Title: #84: Removed update_from_dict function from ldapupdate
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/C
URL: https://github.com/freeipa/freeipa/pull/104
Author: martbab
Title: #104: Backport XMLRPC test fixes to ipa-4-3 branch
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/104/head:pr104
git checkout pr104
F
URL: https://github.com/freeipa/freeipa/pull/97
Author: mbasti-rh
Title: #97: Pylint fixes
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/97/head:pr97
git checkout pr97
From 9c4208a527c22685fa79c32080fb036
On 21.09.2016 12:01, Jan Pazdziora wrote:
Hello,
I've recently hit again the situation of IPA installer not happy
about the provided IP address not being local to it, this time in
containerized environment:
https://bugzilla.redhat.com/show_bug.cgi?id=1377973
During the discussion, we
Hello all,
Following test is failing:
test_cert_find.test_0007_find_revocation_reason_0
self = 0x7f1bf4532f90>
def test_00
On 09/22/2016 01:41 PM, Martin Basti wrote:
Hello all,
Following test is failing:
test_cert_find.test_0007_find_revocation_reason_0
On 22.09.2016 13:56, Martin Babinsky wrote:
On 09/22/2016 01:41 PM, Martin Basti wrote:
Hello all,
Following test is failing:
test_cert_find.test_0007_find_revocation_reason_0
__
URL: https://github.com/freeipa/freeipa/pull/97
Title: #97: Pylint fixes
tomaskrizek commented:
"""
Please fix the typo.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/97#issuecomment-248884825
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.re
URL: https://github.com/freeipa/freeipa/pull/105
Author: mbasti-rh
Title: #105: Test: dont use global variable for iteration in test_cert_plugin
Action: opened
PR body:
"""
Iteration over global variable causes unwanted value changes outside
method
https://fedorahosted.org/freeipa/ticket/5755
URL: https://github.com/freeipa/freeipa/pull/97
Author: mbasti-rh
Title: #97: Pylint fixes
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/97/head:pr97
git checkout pr97
From fc4c31912df4d004f04700fa2a4f4ef
URL: https://github.com/freeipa/freeipa/pull/97
Author: mbasti-rh
Title: #97: Pylint fixes
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/97/head:pr97
git checkout pr97
From fc4c31912df4d004f04700fa2a4f4ef
URL: https://github.com/freeipa/freeipa/pull/105
Title: #105: Test: dont use global variable for iteration in test_cert_plugin
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.free
URL: https://github.com/freeipa/freeipa/pull/105
Title: #105: Test: dont use global variable for iteration in test_cert_plugin
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.f
URL: https://github.com/freeipa/freeipa/pull/105
Author: mbasti-rh
Title: #105: Test: dont use global variable for iteration in test_cert_plugin
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/105/head:pr105
git
Hi Oleg!
As we discussed this morning there're still some issues [1][2] but I
think the patch set is in reasonably good state and I'm not sure that
the issues are bugs in IPA or in tests. This can be investigated and
fixed later.
ACK.
Pushed to master:
* bbac233b5ee487ab0e035cf0b861144769a0
URL: https://github.com/freeipa/freeipa/pull/105
Title: #105: Test: dont use global variable for iteration in test_cert_plugin
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/929086e0992cc32a654b4dfa435f536ecb0c665b
"""
See the full comment at
https://g
URL: https://github.com/freeipa/freeipa/pull/97
Title: #97: Pylint fixes
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/97
Author: mbasti-rh
Title: #97: Pylint fixes
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/97/head:pr97
git checkout pr97
From 6f16460d67397c2b0aa5fd5d7066c5c
URL: https://github.com/freeipa/freeipa/pull/97
Title: #97: Pylint fixes
mbasti-rh commented:
"""
I had to rebase, there were conflicts due the latest pushed patches
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/97#issuecomment-248913108
--
Manage your subscription for t
Hello all,
In 4.5, I would like to remove all unused variables from code and enable
pylint check. Due to big amount of unused variables in the code this
will be longterm effort.
Why this?:
* better code readability
* removing dead code
* unused variable may uncover potential bug
It is cl
URL: https://github.com/freeipa/freeipa/pull/97
Author: mbasti-rh
Title: #97: Pylint fixes
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/97/head:pr97
git checkout pr97
--
Manage your subscription for the Freei
URL: https://github.com/freeipa/freeipa/pull/106
Author: mbasti-rh
Title: #106: Pylint: enable additional checks
Action: opened
PR body:
"""
Enabling:
* cyclic-imports
* global-variable-not-assigned
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
URL: https://github.com/freeipa/freeipa/pull/97
Title: #97: Pylint fixes
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/36484e8672f5ee1fdc2bd57622e330ab8dbb7671
https://fedorahosted.org/freeipa/changeset/275e85d076607ce317b3aeca467167fac55bf396
https:/
URL: https://github.com/freeipa/freeipa/pull/97
Title: #97: Pylint fixes
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/107
Author: tomaskrizek
Title: #107: Update man/help for --server option
Action: opened
PR body:
"""
The --server option now specifically mentions that
it expects the FQDN of the IPA server.
https://fedorahosted.org/freeipa/ticket/6202
"""
To pull
On 09/22/2016 04:39 PM, Martin Basti wrote:
Hello all,
In 4.5, I would like to remove all unused variables from code and
enable pylint check. Due to big amount of unused variables in the code
this will be longterm effort.
Why this?:
* better code readability
* removing dead code
* unused
On 22.09.2016 17:59, Tomas Krizek wrote:
On 09/22/2016 04:39 PM, Martin Basti wrote:
Hello all,
In 4.5, I would like to remove all unused variables from code and
enable pylint check. Due to big amount of unused variables in the
code this will be longterm effort.
Why this?:
* better code
On 09/22/2016 06:00 PM, Martin Basti wrote:
On 22.09.2016 17:59, Tomas Krizek wrote:
On 09/22/2016 04:39 PM, Martin Basti wrote:
Hello all,
In 4.5, I would like to remove all unused variables from code and
enable pylint check. Due to big amount of unused variables in the
code this will be
Hi all,
As you know, FedoraHosted.org will be decommissioned.
https://communityblog.fedoraproject.org/fedorahosted-sunset-2017-02-28/
We use Trac instance there. Let's discuss where we should migrate and
what are our requirements. Then put results on one place. For that I've
created:
http://ww
On 22.09.2016 18:05, Tomas Krizek wrote:
On 09/22/2016 06:00 PM, Martin Basti wrote:
On 22.09.2016 17:59, Tomas Krizek wrote:
On 09/22/2016 04:39 PM, Martin Basti wrote:
Hello all,
In 4.5, I would like to remove all unused variables from code and
enable pylint check. Due to big amount of
Bump for review.
Rebased patches attached (there was a trivial conflict in imports).
Thanks,
Fraser
On Tue, Sep 06, 2016 at 02:05:06AM +1000, Fraser Tweedale wrote:
> On Fri, Aug 26, 2016 at 10:28:58AM +0200, Jan Cholasta wrote:
> > On 19.8.2016 13:11, Fraser Tweedale wrote:
> > > Bump for revie
Bump for review.
On Wed, Sep 07, 2016 at 04:06:25PM +0700, Fraser Tweedale wrote:
> Attached patch fixes https://fedorahosted.org/freeipa/ticket/6305
>
> Thanks,
> Fraser
> From d4d7e77795f96a4970058e61d99c70522689b22d Mon Sep 17 00:00:00 2001
> From: Fraser Tweedale
> Date: Wed, 7 Sep 2016 19:
On 22.9.2016 16:39, Martin Basti wrote:
Hello all,
In 4.5, I would like to remove all unused variables from code and enable
pylint check. Due to big amount of unused variables in the code this
will be longterm effort.
Why this?:
* better code readability
* removing dead code
* unused variabl
On 23.9.2016 05:30, Fraser Tweedale wrote:
Bump for review.
Works for me, ACK.
Pushed to master: 97d4ffc2dc5db00fd7ed10b0b290cc97a506d0ef
--
Jan Cholasta
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIP
URL: https://github.com/freeipa/freeipa/pull/106
Title: #106: Pylint: enable additional checks
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/108
Author: frasertweedale
Title: #108: https://fedorahosted.org/freeipa/ticket/6256
Action: opened
PR body:
"""
None
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/108/head:pr
URL: https://github.com/freeipa/freeipa/pull/108
Author: frasertweedale
Title: #108: Bump pki min version and add commentary about sub-CA revocation
on delete
Action: edited
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/108/
URL: https://github.com/freeipa/freeipa/pull/108
Author: frasertweedale
Title: #108: Bump pki min version and add commentary about sub-CA revocation
on delete
Action: edited
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/108/
On 25.8.2016 15:31, Martin Basti wrote:
On 10.08.2016 07:53, Stanislav Laznicka wrote:
On 08/10/2016 07:31 AM, Jan Cholasta wrote:
On 9.8.2016 18:52, Petr Vobornik wrote:
On 08/09/2016 04:18 PM, Martin Basti wrote:
On 09.08.2016 16:07, Stanislav Laznicka wrote:
https://fedorahosted.org/f
URL: https://github.com/freeipa/freeipa/pull/109
Author: frasertweedale
Title: #109: sudorule: add SELinux transition examples to plugin doc
Action: opened
PR body:
"""
It is not obvious how to add SELinux type and role transitions to a
Sudo rule. Update the 'sudorule' plugin documentation wi
On 25.8.2016 12:08, Jan Cholasta wrote:
On 22.8.2016 07:00, Fraser Tweedale wrote:
On Fri, Aug 19, 2016 at 08:09:33PM +1000, Fraser Tweedale wrote:
On Mon, Aug 15, 2016 at 10:54:25PM +1000, Fraser Tweedale wrote:
On Mon, Aug 15, 2016 at 02:08:54PM +0200, Jan Cholasta wrote:
On 19.7.2016 12:05
55 matches
Mail list logo