Lukas Slebodnik wrote:
> On (03/03/17 17:07), Lukas Slebodnik wrote:
>> ehlo,
>>
>> This is a small continuation fo discussin from pull request
>> "Make pylint and jsl optional" #502[1]
>>
>> Pylint and jslint are already optional because some downstream distributions
>> does not have such packages
URL: https://github.com/freeipa/freeipa/pull/532
Title: #532: Fix cookie with Max-Age processing
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/532
Title: #532: Fix cookie with Max-Age processing
simo5 commented:
"""
LGTM, please merge
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/532#issuecomment-284055799
--
Manage your subscription for the Freeipa-devel mailing lis
URL: https://github.com/freeipa/freeipa/pull/538
Author: tiran
Title: #538: Run test_ipaclient test suite
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/538/head:pr538
git checkout pr538
From 5dfb17168972e
URL: https://github.com/freeipa/freeipa/pull/537
Title: #537: test_csrgen: adjusted comparison test scripts for CSRGenerator
apophys commented:
"""
Ack.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/537#issuecomment-284012793
--
Manage your subscription for the Freeipa-d
URL: https://github.com/freeipa/freeipa/pull/537
Title: #537: test_csrgen: adjusted comparison test scripts for CSRGenerator
Label: +ack
--
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/537
Title: #537: test_csrgen: adjusted comparison test scripts for CSRGenerator
apophys commented:
"""
Ack.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/537#issuecomment-284012793
--
Manage your subscription for the Freeipa-d
URL: https://github.com/freeipa/freeipa/pull/537
Author: Rezney
Title: #537: test_csrgen: adjusted comparison test scripts for CSRGenerator
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/537/head:pr537
git
URL: https://github.com/freeipa/freeipa/pull/532
Title: #532: Fix cookie with Max-Age processing
apophys commented:
"""
Hi, can this PR get little more attention? The issue seems to be a cause for a
lot of failures in our integration tests. (I'm not 100% sure though)
"""
See the full comment a
URL: https://github.com/freeipa/freeipa/pull/537
Author: Rezney
Title: #537: test_csrgen: adjusted comparison test scripts for CSRGenerator
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/537/head:pr537
git
On (03/03/17 17:07), Lukas Slebodnik wrote:
>ehlo,
>
>This is a small continuation fo discussin from pull request
>"Make pylint and jsl optional" #502[1]
>
>Pylint and jslint are already optional because some downstream distributions
>does not have such packages. This is a reason why desing documen
ehlo,
This is a small continuation fo discussin from pull request
"Make pylint and jsl optional" #502[1]
Pylint and jslint are already optional because some downstream distributions
does not have such packages. This is a reason why desing document[2]
mention configuration options for disabling th
URL: https://github.com/freeipa/freeipa/pull/475
Title: #475: Add options to run only ipaclient unittests
martbab commented:
"""
I like the second approach better. If you squash the commits I will Ack the PR.
I still think we need a substantial reorganization of the test suites but that
needs
URL: https://github.com/freeipa/freeipa/pull/538
Author: tiran
Title: #538: Run test_ipaclient test suite
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/538/head:pr538
git checkout pr538
From 4dc1dabc4fc68
URL: https://github.com/freeipa/freeipa/pull/519
Title: #519: WebUI: add sizelimit:0 to cert-find
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/519
Title: #519: WebUI: add sizelimit:0 to cert-find
flo-renaud commented:
"""
Hi @pvomacka ,
thank you, the fix works as expected.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/519#issuecomment-283949286
--
Manage your subsc
URL: https://github.com/freeipa/freeipa/pull/507
Title: #507: Use https to get security domain from Dogtag
tomaskrizek commented:
"""
If backport for 4.4 is needed, please open another PR against `ipa-4-4`. Thanks.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/507#issueco
URL: https://github.com/freeipa/freeipa/pull/516
Title: #516: IdM Server: list all Employees with matching Smart Card
flo-renaud commented:
"""
@abbra ,
Thanks for your comment. Running in permissive mode I did not see any AVC
logged in the journal.
@HonzaCholasta
thanks for the tips re. writ
URL: https://github.com/freeipa/freeipa/pull/507
Title: #507: Use https to get security domain from Dogtag
tomaskrizek commented:
"""
master:
* d1c5d92897d3e262edd2e43295c1270590aebd3d Use https to get security domain
from Dogtag
"""
See the full comment at
https://github.com/freeipa/freeipa
URL: https://github.com/freeipa/freeipa/pull/507
Title: #507: Use https to get security domain from Dogtag
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/Cont
URL: https://github.com/freeipa/freeipa/pull/507
Author: tiran
Title: #507: Use https to get security domain from Dogtag
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/507/head:pr507
git checkout pr507
--
Manag
URL: https://github.com/freeipa/freeipa/pull/507
Title: #507: Use https to get security domain from Dogtag
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/Contrib
URL: https://github.com/freeipa/freeipa/pull/502
Title: #502: Make pylint and jsl optional
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/502
Title: #502: Make pylint and jsl optional
lslebodn commented:
"""
I am still expect some comment from @rcritten
LS
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/502#issuecomment-283938711
--
Manage your subscription for
URL: https://github.com/freeipa/freeipa/pull/538
Author: tiran
Title: #538: Run test_ipaclient test suite
Action: opened
PR body:
"""
Depends on PR #537 to fix the test suite first.
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreei
URL: https://github.com/freeipa/freeipa/pull/502
Author: tiran
Title: #502: Make pylint and jsl optional
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/502/head:pr502
git checkout pr502
From 4a642a9366acc4
URL: https://github.com/freeipa/freeipa/pull/502
Title: #502: Make pylint and jsl optional
tomaskrizek commented:
"""
@tiran Needs rebase.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/502#issuecomment-283931719
--
Manage your subscription for the Freeipa-devel mailing l
URL: https://github.com/freeipa/freeipa/pull/502
Title: #502: Make pylint and jsl optional
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/502
Title: #502: Make pylint and jsl optional
tomaskrizek commented:
"""
Issues found by @HonzaCholasta were addressed and no one has raised any serious
concern that this patch should not be accepted.
"""
See the full comment at
https://github.com/
URL: https://github.com/freeipa/freeipa/pull/523
Title: #523: cert-request: minor refactors
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/523
Author: frasertweedale
Title: #523: cert-request: minor refactors
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/523/head:pr523
git checkout pr523
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/523
Title: #523: cert-request: minor refactors
tomaskrizek commented:
"""
master:
* 2066a80be21258d9311ae374fe124d9ac3b79acd Remove redundant principal_type
argument
* 11c9df25774fbc8ed24b30f75c205d12ca3c5b90 Extract method to map principal to
prin
URL: https://github.com/freeipa/freeipa/pull/523
Title: #523: cert-request: minor refactors
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/400
Title: #400: WebUI: Certificate Mapping
flo-renaud commented:
"""
Hi @pvomacka
thank you, LGTM.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/400#issuecomment-283923415
--
Manage your subscription for the Freeipa-devel ma
URL: https://github.com/freeipa/freeipa/pull/537
Author: Rezney
Title: #537: test_csrgen: adjusted comparison test scripts for CSRGenerator
Action: opened
PR body:
"""
Commit ada91c2 introduced changes in "csrgen/templates/openssl_base.tmpl"
which broke the following 2 tests:
test_CSRGene
URL: https://github.com/freeipa/freeipa/pull/536
Author: flo-renaud
Title: #536: ipa systemd unit should define Wants=network instead of
Requires=network
Action: opened
PR body:
"""
The file ipa.service defines
Requires=network.target
which means that ipa stack will be restarted each time
36 matches
Mail list logo