[Freeipa-devel] [freeipa PR#191][edited] Exclude testing ipa.pot file from zanata

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/191 Author: mbasti-rh Title: #191: Exclude testing ipa.pot file from zanata Action: edited Changed field: title Original value: """ Zanata fix 4 4 """ -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#193][opened] [ipa-4-4] Make httpd publish its CA certificate on DL1

2016-10-26 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/193 Author: stlaz Title: #193: [ipa-4-4] Make httpd publish its CA certificate on DL1 Action: opened PR body: """ httpd did not publish its certificate on DL1 which could cause issues during client installation in a rare corner case where there

[Freeipa-devel] [freeipa PR#163][comment] Do not create Object Signing certificate

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/163 Title: #163: Do not create Object Signing certificate mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/eb6bfd82f363405e3377b2a912b1152ba76625ae """ See the full comment at

[Freeipa-devel] [freeipa PR#163][closed] Do not create Object Signing certificate

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/163 Author: frasertweedale Title: #163: Do not create Object Signing certificate Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/163/head:pr163 git checkout pr163 --

[Freeipa-devel] [freeipa PR#163][+pushed] Do not create Object Signing certificate

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/163 Title: #163: Do not create Object Signing certificate Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#165][comment] Tests: Verify that cert-find show CA without --all

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/165 Title: #165: Tests: Verify that cert-find show CA without --all mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/42d1a06bd1e856c14110c06ba0d9d946df36331d ipa-4-4:

[Freeipa-devel] [freeipa PR#176][+ack] cert-show: show validity in default output

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/176 Title: #176: cert-show: show validity in default output Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#176][comment] cert-show: show validity in default output

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/176 Title: #176: cert-show: show validity in default output mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/b6a3c9dc74ccef6f8e7df4123670d7e11269198c ipa-4-4:

[Freeipa-devel] [freeipa PR#191][synchronized] Zanata fix 4 4

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/191 Author: mbasti-rh Title: #191: Zanata fix 4 4 Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/191/head:pr191 git checkout pr191 From

[Freeipa-devel] [freeipa PR#192][opened] server-del: fix incorrect check for one IPA master

2016-10-26 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/192 Author: martbab Title: #192: server-del: fix incorrect check for one IPA master Action: opened PR body: """ make the check more robust against returned container types for multivalued attributes (lists vs. tuples).

[Freeipa-devel] [freeipa PR#151][synchronized] Make httpd publish its CA certificate on DL1

2016-10-26 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/151 Author: stlaz Title: #151: Make httpd publish its CA certificate on DL1 Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/151/head:pr151 git checkout pr151

[Freeipa-devel] [freeipa PR#193][+pushed] [ipa-4-4] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/193 Title: #193: [ipa-4-4] Make httpd publish its CA certificate on DL1 Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#193][comment] [ipa-4-4] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/193 Title: #193: [ipa-4-4] Make httpd publish its CA certificate on DL1 mbasti-rh commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/c84d920ce8b4ca634d72d7bd99652f93f98b0959 """ See the full comment at

[Freeipa-devel] [freeipa PR#193][closed] [ipa-4-4] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/193 Author: stlaz Title: #193: [ipa-4-4] Make httpd publish its CA certificate on DL1 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/193/head:pr193 git checkout

[Freeipa-devel] [freeipa PR#193][+ack] [ipa-4-4] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/193 Title: #193: [ipa-4-4] Make httpd publish its CA certificate on DL1 Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#163][comment] Do not create Object Signing certificate

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/163 Title: #163: Do not create Object Signing certificate mbasti-rh commented: """ Works for me """ See the full comment at https://github.com/freeipa/freeipa/pull/163#issuecomment-256402862 -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] [freeipa PR#186][+ack] replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/186 Title: #186: replicainstall: log ACI and LDAP errors in promotion check Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#186][closed] replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/186 Author: pvoborni Title: #186: replicainstall: log ACI and LDAP errors in promotion check Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/186/head:pr186 git

[Freeipa-devel] [freeipa PR#176][+pushed] cert-show: show validity in default output

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/176 Title: #176: cert-show: show validity in default output Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#192][comment] server-del: fix incorrect check for one IPA master

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/192 Title: #192: server-del: fix incorrect check for one IPA master mbasti-rh commented: """ LGTM, but I wrote possible improvement inline, please check. """ See the full comment at https://github.com/freeipa/freeipa/pull/192#issuecomment-256376902

[Freeipa-devel] [freeipa PR#151][comment] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/151 Title: #151: Make httpd publish its CA certificate on DL1 mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/5d15626b4db8f5e777e037680623badc86b6c31d """ See the full comment at

[Freeipa-devel] [freeipa PR#151][closed] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/151 Author: stlaz Title: #151: Make httpd publish its CA certificate on DL1 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/151/head:pr151 git checkout pr151 --

[Freeipa-devel] [freeipa PR#151][+pushed] Make httpd publish its CA certificate on DL1

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/151 Title: #151: Make httpd publish its CA certificate on DL1 Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#186][+pushed] replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/186 Title: #186: replicainstall: log ACI and LDAP errors in promotion check Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#186][comment] replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/186 Title: #186: replicainstall: log ACI and LDAP errors in promotion check mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/d0c17b4d9afb95db2abcd93096fa6626fd61870e """ See the full comment at

[Freeipa-devel] [freeipa PR#139][comment] WebUI: Vault Management

2016-10-26 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management pvomacka commented: """ Fixed PEP8 errors. """ See the full comment at https://github.com/freeipa/freeipa/pull/139#issuecomment-256271405 -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH] 0102-0104: webui: Add support for setting custom table pagination size

2016-10-26 Thread Martin Basti
On 11.08.2016 16:18, Pavel Vomacka wrote: Hello, please review attached patches. https://fedorahosted.org/freeipa/ticket/5742 bump for review -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] [PATCH] restrict setkeytab operation

2016-10-26 Thread Martin Basti
On 31.08.2016 14:36, Martin Basti wrote: On 26.07.2016 13:38, Simo Sorce wrote: On Mon, 2016-07-25 at 11:26 -0400, Simo Sorce wrote: On Mon, 2016-07-25 at 11:10 -0400, Rob Crittenden wrote: Simo Sorce wrote: On Mon, 2016-07-25 at 10:55 -0400, Rob Crittenden wrote: Simo Sorce wrote: As

Re: [Freeipa-devel] [PATCH] 956 replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread Petr Vobornik
On 10/26/2016 09:53 AM, Martin Basti wrote: > > > On 30.03.2016 10:06, Martin Basti wrote: >> >> >> On 24.03.2016 15:27, Petr Vobornik wrote: >>> to enable debugging of such errors. >>> >>> E.g.: https://fedorahosted.org/freeipa/ticket/5741 >>> >>> >> Can we log the whole traceback to get exact

[Freeipa-devel] [freeipa PR#186][opened] replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/186 Author: pvoborni Title: #186: replicainstall: log ACI and LDAP errors in promotion check Action: opened PR body: """ to enable debugging of such errors. E.g.: https://fedorahosted.org/freeipa/ticket/5741 """ To pull the PR as Git branch: git

[Freeipa-devel] [freeipa PR#139][synchronized] WebUI: Vault Management

2016-10-26 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/139 Author: pvomacka Title: #139: WebUI: Vault Management Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/139/head:pr139 git checkout pr139 From

Re: [Freeipa-devel] [PATCH] 956 replicainstall: log ACI and LDAP errors in promotion check

2016-10-26 Thread Martin Basti
On 30.03.2016 10:06, Martin Basti wrote: On 24.03.2016 15:27, Petr Vobornik wrote: to enable debugging of such errors. E.g.: https://fedorahosted.org/freeipa/ticket/5741 Can we log the whole traceback to get exact place where error happened? Martin^2 bump -- Manage your

[Freeipa-devel] [freeipa PR#188][comment] Move Python build artefacts to top level directory

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/188 Title: #188: Move Python build artefacts to top level directory tiran commented: """ I ran into some issues with ```make rpm```. ```setup.py``` started to pick up build artefacts of other packages. Now every package uses its own subdirectory

Re: [Freeipa-devel] [PATCH] webui: 0084, 0101: refactoring rpc module

2016-10-26 Thread Martin Basti
On 09.08.2016 13:29, Pavel Vomacka wrote: Hello, please review attached patches. The rpc module is now separated from display layer and changing activity text while loading metadata. https://fedorahosted.org/freeipa/ticket/6144 bump for review -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#187][opened] Register entry points of Custodia plugins

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/187 Author: tiran Title: #187: Register entry points of Custodia plugins Action: opened PR body: """ With setuptools in place FreeIPA is able to register its Custodia plugins. Custodia 0.1 ignores the plugins directives. Custodia 0.2 uses the entry

[Freeipa-devel] [freeipa PR#188][comment] Move Python build artefacts to top level directory

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/188 Title: #188: Move Python build artefacts to top level directory tiran commented: """ The PR will also simplifies @pspacek next branch a bit. """ See the full comment at https://github.com/freeipa/freeipa/pull/188#issuecomment-256312577 --

[Freeipa-devel] [freeipa PR#189][comment] Create relative symbol links

2016-10-26 Thread rcritten
URL: https://github.com/freeipa/freeipa/pull/189 Title: #189: Create relative symbol links rcritten commented: """ I think you should add the reasoning for switching the link type to the commit message and if this is related to some higher-level ticket that should be included as well. """

[Freeipa-devel] [freeipa PR#184][comment] Minor install script fixes

2016-10-26 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/184 Title: #184: Minor install script fixes martbab commented: """ Note that the installer class has no access to debug options. These are only used in the containing ConfigureTool class to set up loggers (see

[Freeipa-devel] [freeipa PR#190][opened] [4.4] Fix tests install dom0

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/190 Author: mbasti-rh Title: #190: [4.4] Fix tests install dom0 Action: opened PR body: """ Backport PR #136 to ipa-4-4 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#184][reopened] Minor install script fixes

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/184 Author: simo5 Title: #184: Minor install script fixes Action: reopened To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/184/head:pr184 git checkout pr184 -- Manage your

[Freeipa-devel] [freeipa PR#188][synchronized] Move Python build artefacts to top level directory

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/188 Author: tiran Title: #188: Move Python build artefacts to top level directory Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/188/head:pr188 git checkout

[Freeipa-devel] [freeipa PR#189][opened] Create relative symbol links

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/189 Author: tiran Title: #189: Create relative symbol links Action: opened PR body: """ Instead of absolute symbolic links to /usr/bin/COMMAND the RPM spec now creates relative symbolic links. Signed-off-by: Christian Heimes

[Freeipa-devel] [freeipa PR#136][closed] Fix KRA install tests

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/136 Author: mbasti-rh Title: #136: Fix KRA install tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/136/head:pr136 git checkout pr136 -- Manage your

[Freeipa-devel] [freeipa PR#136][comment] Fix KRA install tests

2016-10-26 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/136 Title: #136: Fix KRA install tests mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/84ca1fc220c329b58fb6a22c8eb9bf17d3622c55

[Freeipa-devel] [freeipa PR#189][comment] Create relative symbol links

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/189 Title: #189: Create relative symbol links tiran commented: """ I ran into some strange issues while I was working on PR #188. The symlinks were both dangling symlinks and pointed to a directory that my user could not write to. Eventually I

[Freeipa-devel] [freeipa PR#136][+ack] Fix KRA install tests

2016-10-26 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/136 Title: #136: Fix KRA install tests 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

[Freeipa-devel] [freeipa PR#184][closed] Minor install script fixes

2016-10-26 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/184 Author: simo5 Title: #184: Minor install script fixes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/184/head:pr184 git checkout pr184 -- Manage your

[Freeipa-devel] [freeipa PR#184][comment] Minor install script fixes

2016-10-26 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/184 Title: #184: Minor install script fixes martbab commented: """ Reverted the Fix install scripts debugging commit. master: * dc873007f8616ab9e77f903e235ba49f45ecde37 Revert "Fix install scripts debugging" """ See the full comment at

[Freeipa-devel] [freeipa PR#188][opened] Move Python build artefacts to top level directory

2016-10-26 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/188 Author: tiran Title: #188: Move Python build artefacts to top level directory Action: opened PR body: """ All setup.py use the same build, dist and *.egg-info directory on top level. Build artefacts are no longer placed in local build

[Freeipa-devel] [freeipa PR#184][comment] Minor install script fixes

2016-10-26 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/184 Title: #184: Minor install script fixes abbra commented: """ I'm fine with that (revert --debug commit). Either alternative (make Configurable be aware of the debug or do a refactoring of an installer) is roughly going into the same direction.