Re: [Freeipa-devel] client-only FreeIPA build

On 22.11.2016 18:10, Petr Vobornik wrote: On 11/22/2016 05:25 PM, Rob Crittenden wrote: Lukas Slebodnik wrote: On (22/11/16 16:29), Petr Spacek wrote: On 22.11.2016 16:27, Jan Cholasta wrote: Hi, On 22.11.2016 16:04, Petr Spacek wrote: Hello, the recent changes with regard to

Re: [Freeipa-devel] client-only FreeIPA build

On 22.11.2016 16:59, Lukas Slebodnik wrote: On (22/11/16 16:29), Petr Spacek wrote: On 22.11.2016 16:27, Jan Cholasta wrote: Hi, On 22.11.2016 16:04, Petr Spacek wrote: Hello, the recent changes with regard to http://www.freeipa.org/page/V4/Integration_Improvements beg a question whether we

Re: [Freeipa-devel] client-only FreeIPA build

On ke, 23 marras 2016, Jan Cholasta wrote: On 22.11.2016 18:10, Petr Vobornik wrote: On 11/22/2016 05:25 PM, Rob Crittenden wrote: Lukas Slebodnik wrote: On (22/11/16 16:29), Petr Spacek wrote: On 22.11.2016 16:27, Jan Cholasta wrote: Hi, On 22.11.2016 16:04, Petr Spacek wrote: Hello,

[Freeipa-devel] [freeipa PR#244][comment] Add templating to ipaplatform path [RFC]

URL: https://github.com/freeipa/freeipa/pull/244 Title: #244: Add templating to ipaplatform path [RFC] tiran commented: """ Please ignore this PR for now. It's not relevant for PyPI packages. """ See the full comment at https://github.com/freeipa/freeipa/pull/244#issuecomment-262220179 --

[Freeipa-devel] [freeipa PR#238][+ack] Build system refactoring phase 8: update translation system

URL: https://github.com/freeipa/freeipa/pull/238 Title: #238: Build system refactoring phase 8: update translation system 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#223][+ack] LDAP refactoring: remove admin_conn

URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn 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#254][comment] Replace LooseVersion with pkg_resource.parse_version

URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resource.parse_version tiran commented: """ @martbab more fun, the doc string of ```verify_client_version``` deviates from the actual implementation. The code does not implement the minor version check.

[Freeipa-devel] Removing ipa.pot file from git tree

Hello list, we plan to remove ipa.pot file from git tree, as this is file can be generated from code during build time, and it is required only for pushing sources to Zanata. Does anybody remember reason why this file was added into git tree? Note: Translated strings (*.po files) will

Re: [Freeipa-devel] Removing ipa.pot file from git tree

On 22/11/16 13:37, Martin Basti wrote: Hello list, we plan to remove ipa.pot file from git tree, as this is file can be generated from code during build time, and it is required only for pushing sources to Zanata. Does anybody remember reason why this file was added into git tree? Note:

[Freeipa-devel] [freeipa PR#122][synchronized] Acceptance tests

URL: https://github.com/freeipa/freeipa/pull/122 Author: dkupka Title: #122: Acceptance tests Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/122/head:pr122 git checkout pr122 From

[Freeipa-devel] [freeipa PR#231][comment] Do not log DM password in ca/kra installation logs

URL: https://github.com/freeipa/freeipa/pull/231 Title: #231: Do not log DM password in ca/kra installation logs tomaskrizek commented: """ I didn't notice `dm_password` is no longer in `DogtagInstance`, I re-added it elsewhere. In that case, as @stlaz said, passing it to `spawn_instance()`

[Freeipa-devel] [freeipa PR#223][synchronized] LDAP refactoring: remove admin_conn

URL: https://github.com/freeipa/freeipa/pull/223 Author: tomaskrizek Title: #223: LDAP refactoring: remove admin_conn Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/223/head:pr223 git checkout pr223 From

[Freeipa-devel] [freeipa PR#122][closed] Acceptance tests

URL: https://github.com/freeipa/freeipa/pull/122 Author: dkupka Title: #122: Acceptance tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/122/head:pr122 git checkout pr122 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#122][comment] Acceptance tests

URL: https://github.com/freeipa/freeipa/pull/122 Title: #122: Acceptance tests martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/3e53bbcc34bd256da36209fd8cf8ac5d33ec8093 https://fedorahosted.org/freeipa/changeset/4225484356426a73cc11211bceda7f06ee23d093

[Freeipa-devel] [freeipa PR#122][+pushed] Acceptance tests

URL: https://github.com/freeipa/freeipa/pull/122 Title: #122: Acceptance tests 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

[Freeipa-devel] [freeipa PR#231][comment] Do not log DM password in ca/kra installation logs

URL: https://github.com/freeipa/freeipa/pull/231 Title: #231: Do not log DM password in ca/kra installation logs stlaz commented: """ @martbab Oh, I thought you wanted me to re-add `dm_password` to DogtagInstance as @tomaskrizek which does not seem right as DogtagInstance is in no position to

[Freeipa-devel] [freeipa PR#231][synchronized] Do not log DM password in ca/kra installation logs

URL: https://github.com/freeipa/freeipa/pull/231 Author: stlaz Title: #231: Do not log DM password in ca/kra installation logs Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/231/head:pr231 git checkout

[Freeipa-devel] NTP in FreeIPA

Hello everyone! Is it worth to keep configuring NTP in FreeIPA? In usual environment there're no special requirements for time synchronization and the distribution default (be it ntpd, chrony or anything else) will just work. Any tampering with the configuration can't make it any better.

[Freeipa-devel] [freeipa PR#122][+ack] Acceptance tests

URL: https://github.com/freeipa/freeipa/pull/122 Title: #122: Acceptance 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#223][comment] LDAP refactoring: remove admin_conn

URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/68295bf8cfd57333deb50f58df1b336a4b48ffe7

[Freeipa-devel] [freeipa PR#223][+pushed] LDAP refactoring: remove admin_conn

URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn 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#223][closed] LDAP refactoring: remove admin_conn

URL: https://github.com/freeipa/freeipa/pull/223 Author: tomaskrizek Title: #223: LDAP refactoring: remove admin_conn Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/223/head:pr223 git checkout pr223 -- Manage

[Freeipa-devel] [freeipa PR#254][synchronized] Replace LooseVersion with pkg_resource.parse_version

URL: https://github.com/freeipa/freeipa/pull/254 Author: tiran Title: #254: Replace LooseVersion with pkg_resource.parse_version Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/254/head:pr254 git checkout

Re: [Freeipa-devel] NTP in FreeIPA

On 22.11.2016 13:06, Petr Spacek wrote: On 22.11.2016 12:15, David Kupka wrote: Hello everyone! Is it worth to keep configuring NTP in FreeIPA? In usual environment there're no special requirements for time synchronization and the distribution default (be it ntpd, chrony or anything else)

[Freeipa-devel] client-only FreeIPA build

Hello, the recent changes with regard to http://www.freeipa.org/page/V4/Integration_Improvements beg a question whether we should invest into supporting client-only builds in FreeIPA build system. Right now, FreeIPA can be built on all architectures we care about so there is no incentive to

[Freeipa-devel] [freeipa PR#263][opened] Backwards compatibility with setuptools 0.9.8

URL: https://github.com/freeipa/freeipa/pull/263 Author: tiran Title: #263: Backwards compatibility with setuptools 0.9.8 Action: opened PR body: """ Setuptools 0.9.8 does not support PEP 440 version schema with +git suffix. Signed-off-by: Christian Heimes """ To pull

Re: [Freeipa-devel] Removing ipa.pot file from git tree

On 22.11.2016 13:59, David Kupka wrote: On 22/11/16 13:37, Martin Basti wrote: Hello list, we plan to remove ipa.pot file from git tree, as this is file can be generated from code during build time, and it is required only for pushing sources to Zanata. Does anybody remember reason why this

[Freeipa-devel] [freeipa PR#238][comment] Build system refactoring phase 8: update translation system

URL: https://github.com/freeipa/freeipa/pull/238 Title: #238: Build system refactoring phase 8: update translation system mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/4842231074683ff68be50b147560f5383aa305b6

[Freeipa-devel] [freeipa PR#238][+pushed] Build system refactoring phase 8: update translation system

URL: https://github.com/freeipa/freeipa/pull/238 Title: #238: Build system refactoring phase 8: update translation system Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] client-only FreeIPA build

Hi, On 22.11.2016 16:04, Petr Spacek wrote: Hello, the recent changes with regard to http://www.freeipa.org/page/V4/Integration_Improvements beg a question whether we should invest into supporting client-only builds in FreeIPA build system. Right now, FreeIPA can be built on all architectures

Re: [Freeipa-devel] client-only FreeIPA build

On (22/11/16 16:04), Petr Spacek wrote: >Hello, > >the recent changes with regard to >http://www.freeipa.org/page/V4/Integration_Improvements >beg a question whether we should invest into supporting client-only builds in >FreeIPA build system. > >Right now, FreeIPA can be built on all

[Freeipa-devel] [freeipa PR#238][closed] Build system refactoring phase 8: update translation system

URL: https://github.com/freeipa/freeipa/pull/238 Author: pspacek Title: #238: Build system refactoring phase 8: update translation system Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/238/head:pr238 git

Re: [Freeipa-devel] client-only FreeIPA build

On 22.11.2016 16:14, Lukas Slebodnik wrote: > On (22/11/16 16:04), Petr Spacek wrote: >> Hello, >> >> the recent changes with regard to >> http://www.freeipa.org/page/V4/Integration_Improvements >> beg a question whether we should invest into supporting client-only builds in >> FreeIPA build

[Freeipa-devel] [freeipa PR#235][comment] Remove unused Knob function

URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function mbasti-rh commented: """ @jcholast ping """ See the full comment at https://github.com/freeipa/freeipa/pull/235#issuecomment-262279274 -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] client-only FreeIPA build

On (22/11/16 16:29), Petr Spacek wrote: >On 22.11.2016 16:27, Jan Cholasta wrote: >> Hi, >> >> On 22.11.2016 16:04, Petr Spacek wrote: >>> Hello, >>> >>> the recent changes with regard to >>> http://www.freeipa.org/page/V4/Integration_Improvements >>> beg a question whether we should invest into

[Freeipa-devel] [freeipa PR#235][+ack] Remove unused Knob function

URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function 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#235][comment] Remove unused Knob function

URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function jcholast commented: """ There was no reason, I just forgot, so go ahead. """ See the full comment at https://github.com/freeipa/freeipa/pull/235#issuecomment-262290033 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#235][+pushed] Remove unused Knob function

URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function 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

[Freeipa-devel] [freeipa PR#256][+pushed] Pylint: whitelist packages with extension modules

URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules 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#259][closed] Minor fixes for IPAVersion class

URL: https://github.com/freeipa/freeipa/pull/259 Author: tiran Title: #259: Minor fixes for IPAVersion class Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/259/head:pr259 git checkout pr259 -- Manage your

[Freeipa-devel] [freeipa PR#259][+pushed] Minor fixes for IPAVersion class

URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class 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

[Freeipa-devel] [freeipa PR#259][comment] Minor fixes for IPAVersion class

URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/29947fe1a304ff6f913e5d94d56d8108a7c94087 """ See the full comment at

[Freeipa-devel] [freeipa PR#256][closed] Pylint: whitelist packages with extension modules

URL: https://github.com/freeipa/freeipa/pull/256 Author: tiran Title: #256: Pylint: whitelist packages with extension modules Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/256/head:pr256 git checkout pr256 --

[Freeipa-devel] [freeipa PR#235][comment] Remove unused Knob function

URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/55b14abcb561422cf48755dae6b0638656535fe5 """ See the full comment at

[Freeipa-devel] [freeipa PR#256][comment] Pylint: whitelist packages with extension modules

URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/573eee444e1746fd5949897294c96a1793e74511 """ See the full comment at

[Freeipa-devel] [freeipa PR#259][+ack] Minor fixes for IPAVersion class

URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class 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

Re: [Freeipa-devel] client-only FreeIPA build

Lukas Slebodnik wrote: > On (22/11/16 16:29), Petr Spacek wrote: >> On 22.11.2016 16:27, Jan Cholasta wrote: >>> Hi, >>> >>> On 22.11.2016 16:04, Petr Spacek wrote: Hello, the recent changes with regard to http://www.freeipa.org/page/V4/Integration_Improvements beg a

[Freeipa-devel] [freeipa PR#256][+ack] Pylint: whitelist packages with extension modules

URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules 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#258][comment] Break ipaplatform / ipalib import cycle of hell

URL: https://github.com/freeipa/freeipa/pull/258 Title: #258: Break ipaplatform / ipalib import cycle of hell tiran commented: """ It's no longer a priority, but I still like to fix the imports No matter what the cyclic imports and cross-package dependencies are a mess. A wrong order of

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

URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management mbasti-rh commented: """ NACK 1) view My User Vaults/Add vault There is no marked radio button and all fields are shown which are mutual exclusive. A one option from radio group should be marked. It is doing

Re: [Freeipa-devel] client-only FreeIPA build

On 11/22/2016 05:25 PM, Rob Crittenden wrote: > Lukas Slebodnik wrote: >> On (22/11/16 16:29), Petr Spacek wrote: >>> On 22.11.2016 16:27, Jan Cholasta wrote: Hi, On 22.11.2016 16:04, Petr Spacek wrote: > Hello, > > the recent changes with regard to >

Re: [Freeipa-devel] NTP in FreeIPA

I would say that it is worth keeping in FreeIPA. I know myself and some customers use its functionality by having the clients sync to the IPA servers and have the servers sync to the NTP source. This way if the NTP source ever gets disrupted for long periods of time (which has happened in my

[Freeipa-devel] [freeipa PR#117][comment] Make ipa-replica-install run in interactive mode

URL: https://github.com/freeipa/freeipa/pull/117 Title: #117: Make ipa-replica-install run in interactive mode tomaskrizek commented: """ This PR needs to be rebased to reflect installer refactoring. """ See the full comment at