[Freeipa-devel] [freeipa PR#19] WebUI: Add 'Restore' option to action dropdown menu (comment)

stlaz commented on a pull request """ Works as expected, the code looks fine as well. """ See the full comment at https://github.com/freeipa/freeipa/pull/19#issuecomment-245509559 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-dev

[Freeipa-devel] [freeipa PR#19] WebUI: Add 'Restore' option to action dropdown menu (+ack)

pvomacka's pull request #19: "WebUI: Add 'Restore' option to action dropdown menu" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/19 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-de

[Freeipa-devel] [freeipa PR#60] Tests: extend DNS cmdline tests with lowercased record type (+ack)

mbasti-rh's pull request #60: "Tests: extend DNS cmdline tests with lowercased record type" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/60 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/f

[Freeipa-devel] [PATCH] 0108 cert-request: raise error when request fails

The attached patch fixes regression in cert-request: https://fedorahosted.org/freeipa/ticket/6309 Thanks, Fraser From b27eef53ee36b7cae70206c37dea6aaa3bcfc940 Mon Sep 17 00:00:00 2001 From: Fraser Tweedale Date: Thu, 8 Sep 2016 11:56:16 +1000 Subject: [PATCH] cert-request: raise error when reques

[Freeipa-devel] Karma Requests for pki-core-10.3.5-5

*The following updated candidate builds of pki-core 10.3.5 on Fedora 24, 25, and 26 (rawhide) consist of the following: * * *Fedora 24* o *pki-core-10.3.5-5.fc24 * * *Fedora 25* o *pki-core-10.3.5-5.fc25

[Freeipa-devel] [freeipa PR#10] Client-side CSR autogeneration (synchronize)

LiptonB's pull request #10: "Client-side CSR autogeneration" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/10 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/10/head:pr10 git checkout pr10

[Freeipa-devel] [freeipa PR#67] advise: Use `name` instead of `__name__` to get plugin names (opened)

martbab's pull request #67: "advise: Use `name` instead of `__name__` to get plugin names" was opened PR body: """ This change will allow ipa-advise to correctly handle advise plugins with custom names. """ See the full pull-request at https://github.com/freeipa/freeipa/pull/67 ... or pull the P

[Freeipa-devel] [freeipa PR#58] Ip addr validation (closed)

mbasti-rh's pull request #58: "Ip addr validation" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/58 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/58/head:pr58 git checkout pr58 -- Manage you

[Freeipa-devel] [freeipa PR#58] Ip addr validation (comment)

dkupka commented on a pull request """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/81d64d530cca148198312d3d993502575b288f63 https://fedorahosted.org/freeipa/changeset/71ad8d4fc982b5349248d50338e1d16ce45c523e https://fedorahosted.org/freeipa/changeset/f3d379071a9af50e38fcc0749

[Freeipa-devel] [freeipa PR#58] Ip addr validation (+pushed)

mbasti-rh's pull request #58: "Ip addr validation" label *pushed* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/58 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: htt

[Freeipa-devel] [freeipa PR#58] Ip addr validation (+ack)

mbasti-rh's pull request #58: "Ip addr validation" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/58 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http:/

Re: [Freeipa-devel] [PATCH 190] expose `--secret` option in radiusproxy-* commands

On 7.9.2016 16:13, Martin Babinsky wrote: On 09/07/2016 03:55 PM, Jan Cholasta wrote: On 21.7.2016 10:50, Jan Cholasta wrote: On 21.7.2016 10:13, Martin Babinsky wrote: On 07/20/2016 12:10 PM, Martin Babinsky wrote: On 07/19/2016 12:32 PM, Jan Cholasta wrote: Hi, On 18.7.2016 13:51, Martin

Re: [Freeipa-devel] [PATCH 190] expose `--secret` option in radiusproxy-* commands

On 09/07/2016 03:55 PM, Jan Cholasta wrote: On 21.7.2016 10:50, Jan Cholasta wrote: On 21.7.2016 10:13, Martin Babinsky wrote: On 07/20/2016 12:10 PM, Martin Babinsky wrote: On 07/19/2016 12:32 PM, Jan Cholasta wrote: Hi, On 18.7.2016 13:51, Martin Babinsky wrote: https://fedorahosted.org/f

Re: [Freeipa-devel] [PATCH 0183] ipa-advise: correct handling of plugin namespace iteration

On 19.7.2016 09:15, Martin Babinsky wrote: On 07/18/2016 08:46 AM, Jan Cholasta wrote: Hi, On 11.7.2016 14:18, Martin Babinsky wrote: https://fedorahosted.org/freeipa/ticket/6044 Note that you should use .name rather than .__name__ to get plugin names, otherwise the code won't work with plug

Re: [Freeipa-devel] [PATCH 190] expose `--secret` option in radiusproxy-* commands

On 21.7.2016 10:50, Jan Cholasta wrote: On 21.7.2016 10:13, Martin Babinsky wrote: On 07/20/2016 12:10 PM, Martin Babinsky wrote: On 07/19/2016 12:32 PM, Jan Cholasta wrote: Hi, On 18.7.2016 13:51, Martin Babinsky wrote: https://fedorahosted.org/freeipa/ticket/6078 I don't think we want th

Re: [Freeipa-devel] [Test][patch-0058] Fixed topology tests failures in CI

ping for review On 08/24/2016 01:58 PM, Oleg Fayans wrote: And here is how the run looks like: $ ipa-run-tests test_integration/test_topology.py WARNING: Couldn't write lextab module 'pycparser.lextab'. [Errno 13] Permission denied: 'lextab.py' WARNING: yacc table file version is out of date WA

[Freeipa-devel] [freeipa PR#58] Ip addr validation (synchronize)

mbasti-rh's pull request #58: "Ip addr validation" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/58 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/58/head:pr58 git checkout pr58 From 7fc0

[Freeipa-devel] [freeipa PR#58] Ip addr validation (comment)

martbab commented on a pull request """ @mbasti-rh you forgot to copy-paste the code to promote_check function. @pvoborni ^^ and that's why we need to refactor installer code """ See the full comment at https://github.com/freeipa/freeipa/pull/58#issuecomment-245277233 -- Manage your subscript

[Freeipa-devel] [freeipa PR#66] [master, ipa-4-4] Tests: Add cleanup to integration trust tests (opened)

mirielka's pull request #66: "[master, ipa-4-4] Tests: Add cleanup to integration trust tests" was opened PR body: """ Trust tests fail if they are executed after external trust tests. This is caused my missing cleanup. Providing cleanup that would enable correct execution of the tests regardless

[Freeipa-devel] [freeipa PR#43] Tests: Fix regex errors in integration trust tests (synchronize)

mirielka's pull request #43: "Tests: Fix regex errors in integration trust tests" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/43 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/43/head:

Re: [Freeipa-devel] [PATCH] 0106 Make host/service cert revocation aware of lightweight CAs

On 7.9.2016 11:05, Fraser Tweedale wrote: On Wed, Sep 07, 2016 at 10:39:59AM +0200, Jan Cholasta wrote: On 7.9.2016 10:28, Fraser Tweedale wrote: On Wed, Sep 07, 2016 at 08:32:42AM +0200, Jan Cholasta wrote: On 6.9.2016 19:36, Fraser Tweedale wrote: On Tue, Sep 06, 2016 at 10:19:14AM +0200, J

[Freeipa-devel] [freeipa PR#58] Ip addr validation (synchronize)

mbasti-rh's pull request #58: "Ip addr validation" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/58 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/58/head:pr58 git checkout pr58 From 7fc0

[Freeipa-devel] [freeipa PR#31] WebUI: add support for sub-CAs while revoking certificates and removing certificate hold (+rejected)

pvomacka's pull request #31: "WebUI: add support for sub-CAs while revoking certificates and removing certificate hold" label *rejected* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/31 -- Manage your subscription for the Freeipa-devel mailing list: https://

Re: [Freeipa-devel] [PATCH] 0102..0105 Better handling for cert-request to disabled CA

On 09/06/2016 04:51 PM, Fraser Tweedale wrote: On Tue, Aug 30, 2016 at 10:54:32AM +0200, Martin Babinsky wrote: On 08/26/2016 04:19 AM, Fraser Tweedale wrote: The attached patches add better handling of cert-request failure due to target CA being disabled (#6260). To do this, rather than go an

[Freeipa-devel] [freeipa PR#64] cert: fix cert-find --certificate when the cert is not in LDAP (closed)

jcholast's pull request #64: "cert: fix cert-find --certificate when the cert is not in LDAP" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/64 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/6

[Freeipa-devel] [freeipa PR#64] cert: fix cert-find --certificate when the cert is not in LDAP (comment)

dkupka commented on a pull request """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/b7b6faf14aaa8ac677ab9ebc2bcbf87e6b2a1146 ipa-4-4: https://fedorahosted.org/freeipa/changeset/5d4f7b78bc4d179544810419f73ec4d48b0a2a76 """ See the full comment at https://github.com/freeipa/fr

[Freeipa-devel] [freeipa PR#64] cert: fix cert-find --certificate when the cert is not in LDAP (+pushed)

jcholast's pull request #64: "cert: fix cert-find --certificate when the cert is not in LDAP" label *pushed* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/64 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listi

[Freeipa-devel] [freeipa PR#64] cert: fix cert-find --certificate when the cert is not in LDAP (+ack)

jcholast's pull request #64: "cert: fix cert-find --certificate when the cert is not in LDAP" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/64 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo

Re: [Freeipa-devel] [PATCH] 0101 Add ca-disable and ca-enable commands

On 09/06/2016 04:49 PM, Fraser Tweedale wrote: On Tue, Aug 30, 2016 at 10:23:10AM +0200, Martin Babinsky wrote: On 08/30/2016 10:09 AM, Jan Cholasta wrote: Hi, On 30.8.2016 09:56, Martin Babinsky wrote: On 08/25/2016 10:25 AM, Fraser Tweedale wrote: Hi team, The attached patch fixes https:/

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (closed)

pvoborni's pull request #65: "#6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/65 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (+pushed)

pvoborni's pull request #65: "#6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate" label *pushed* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/65 -- Manage your subscription for the Freeipa-devel mailing list: https://ww

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (comment)

pvoborni commented on a pull request """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/7fea3914fbfc0748f26dfe41445b5f0d12f406e6 ipa-4-4: https://fedorahosted.org/freeipa/changeset/a68da14654243821274848b9af57fec3dc2fdb39 """ See the full comment at https://github.com/freeipa/

[Freeipa-devel] [PATCH] 0107 Fix cert revocation when removing all certs via host/service-mod

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:00:18 +1000 Subject: [PATCH] Fix cert revocation when removing all certs via host/service-mod Wh

Re: [Freeipa-devel] [PATCH] 0106 Make host/service cert revocation aware of lightweight CAs

On Wed, Sep 07, 2016 at 10:39:59AM +0200, Jan Cholasta wrote: > On 7.9.2016 10:28, Fraser Tweedale wrote: > > On Wed, Sep 07, 2016 at 08:32:42AM +0200, Jan Cholasta wrote: > > > On 6.9.2016 19:36, Fraser Tweedale wrote: > > > > On Tue, Sep 06, 2016 at 10:19:14AM +0200, Jan Cholasta wrote: > > > > >

Re: [Freeipa-devel] [PATCH] 0106 Make host/service cert revocation aware of lightweight CAs

On 7.9.2016 10:28, Fraser Tweedale wrote: On Wed, Sep 07, 2016 at 08:32:42AM +0200, Jan Cholasta wrote: On 6.9.2016 19:36, Fraser Tweedale wrote: On Tue, Sep 06, 2016 at 10:19:14AM +0200, Jan Cholasta wrote: On 5.9.2016 17:30, Fraser Tweedale wrote: On Mon, Sep 05, 2016 at 11:59:11PM +1000, F

Re: [Freeipa-devel] [PATCH] 0106 Make host/service cert revocation aware of lightweight CAs

On Wed, Sep 07, 2016 at 08:32:42AM +0200, Jan Cholasta wrote: > On 6.9.2016 19:36, Fraser Tweedale wrote: > > On Tue, Sep 06, 2016 at 10:19:14AM +0200, Jan Cholasta wrote: > > > On 5.9.2016 17:30, Fraser Tweedale wrote: > > > > On Mon, Sep 05, 2016 at 11:59:11PM +1000, Fraser Tweedale wrote: > > >

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (+ack)

pvoborni's pull request #65: "#6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/65 -- Manage your subscription for the Freeipa-devel mailing list: https://www.r

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (comment)

stlaz commented on a pull request """ Seems to be working as expected. """ See the full comment at https://github.com/freeipa/freeipa/pull/65#issuecomment-245210985 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute t

[Freeipa-devel] [freeipa PR#58] Ip addr validation (comment)

dkupka commented on a pull request """ @mbasti-rh In last patch, please copy-paste the warnings also into replicainstall.py """ See the full comment at https://github.com/freeipa/freeipa/pull/58#issuecomment-245205422 -- Manage your subscription for the Freeipa-devel mailing list: https://www.

[Freeipa-devel] [freeipa PR#31] WebUI: add support for sub-CAs while revoking certificates and removing certificate hold (closed)

pvomacka's pull request #31: "WebUI: add support for sub-CAs while revoking certificates and removing certificate hold" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/31 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa

[Freeipa-devel] [freeipa PR#31] WebUI: add support for sub-CAs while revoking certificates and removing certificate hold (comment)

pvoborni commented on a pull request """ Obsoleted by pull request #65 """ See the full comment at https://github.com/freeipa/freeipa/pull/31#issuecomment-245200659 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute t

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (synchronize)

pvoborni's pull request #65: "#6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/65 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/fre

[Freeipa-devel] [freeipa PR#65] #6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate (opened)

pvoborni's pull request #65: "#6216 - webui: cert_revoke should use --cacn to set correct CA when revoking certificate" was opened PR body: """ This is Pavel's patch with changes mentioned in pull request #31 comment 1 """ See the full pull-request at https://github.com/freeipa/freeipa/pull/65 .

[Freeipa-devel] [freeipa PR#63] fix for 6238 "Unable to view certificates issued by Sub CA in Web UI" separated from pr31 (closed)

pvoborni's pull request #63: "fix for 6238 "Unable to view certificates issued by Sub CA in Web UI" separated from pr31" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/63 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa

[Freeipa-devel] [freeipa PR#63] fix for 6238 "Unable to view certificates issued by Sub CA in Web UI" separated from pr31 (+pushed)

pvoborni's pull request #63: "fix for 6238 "Unable to view certificates issued by Sub CA in Web UI" separated from pr31" label *pushed* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/63 -- Manage your subscription for the Freeipa-devel mailing list: https://w

[Freeipa-devel] [freeipa PR#63] fix for 6238 "Unable to view certificates issued by Sub CA in Web UI" separated from pr31 (comment)

martbab commented on a pull request """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/40f923f56b4777e3e18c9f76ba1a745ed69ef0a6 https://fedorahosted.org/freeipa/changeset/64ac981dddcecf1176585b6e7b729cf38b24bcea ipa-4-4: https://fedorahosted.org/freeipa/changeset/0b76ba8723d7ba6

[Freeipa-devel] [freeipa PR#50] Add cert checks in ipa-server-certinstall (synchronize)

flo-renaud's pull request #50: "Add cert checks in ipa-server-certinstall" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/50 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/50/head:pr50 gi