[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ > The passwords should have around the same entropy now. SHA-1 actually > produces 160bit outputs (hence 40-characters long hexadecimal digests),

[Freeipa-devel] [freeipa PR#292][comment] Increase the timeout waiting for certificate issuance in installer

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/292 Title: #292: Increase the timeout waiting for certificate issuance in installer mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/9e3c17c6ded868b4261aa76137c703a4fb866578 ""&

[Freeipa-devel] [freeipa PR#292][+pushed] Increase the timeout waiting for certificate issuance in installer

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/292 Title: #292: Increase the timeout waiting for certificate issuance in installer 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-devel] [freeipa PR#292][closed] Increase the timeout waiting for certificate issuance in installer

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/292 Author: flo-renaud Title: #292: Increase the timeout waiting for certificate issuance in installer Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/292/head:pr292 g

[Freeipa-devel] [freeipa PR#292][+ack] Increase the timeout waiting for certificate issuance in installer

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/292 Title: #292: Increase the timeout waiting for certificate issuance in installer Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.fr

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ Ok if Petr agree we can go with your proposal """ See the full comment at https://github.com/fre

[Freeipa-devel] [freeipa PR#284][+pushed] ipautil: check for open ports on all resolved IPs

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/284 Title: #284: ipautil: check for open ports on all resolved IPs 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

[Freeipa-devel] [freeipa PR#284][comment] ipautil: check for open ports on all resolved IPs

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/284 Title: #284: ipautil: check for open ports on all resolved IPs mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/a24cd01304aaef77b66d0e178585c9ec8bbce9b5 """ See the full

[Freeipa-devel] [freeipa PR#284][closed] ipautil: check for open ports on all resolved IPs

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/284 Author: tomaskrizek Title: #284: ipautil: check for open ports on all resolved IPs Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/284/head:pr284 git checkout pr28

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ I had discussion with Petr, and currently we cannot run both pylints in build system and it is not easy to add it th

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ Or we can run both pylints as far as we wants py2/3 compatible versions """ See the full comment at

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ But we build both 2/3 versions at once """ See the full comment at https://github.com/freeipa/freei

[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ Please replace this by something sane, ``` return sha1(ipautil.ipa_generate_password()).hexdigest() ``` security by obscurity worked well in Roman

[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ NACK You replaced os.random() by ipa_generate_password, but ipa_generate password does not generate random bytes but random printable characters (ent

[Freeipa-devel] [freeipa PR#316][closed] Fix error in permission-find post_callback search

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/316 Author: stlaz Title: #316: Fix error in permission-find post_callback search Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/316/head:pr316 git checkout pr316 --

[Freeipa-devel] [freeipa PR#316][comment] Fix error in permission-find post_callback search

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/316 Title: #316: Fix error in permission-find post_callback search mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/0c044cb084780ee45860169dd5d12689cf05fa49 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#316][+pushed] Fix error in permission-find post_callback search

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/316 Title: #316: Fix error in permission-find post_callback search 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

[Freeipa-devel] [freeipa PR#316][+ack] Fix error in permission-find post_callback search

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/316 Title: #316: Fix error in permission-find post_callback search 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/Co

[Freeipa-devel] [freeipa PR#209][comment] Enumerate available options in IPA installer

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/209 Title: #209: Enumerate available options in IPA installer mbasti-rh commented: """ @jcholast any update? Should reject this PR and wait for `argparse` or fix it with `optparse` as well? IMO fixing it now is better for UX, we don

[Freeipa-devel] [freeipa PR#284][comment] ipautil: check for open ports on all resolved IPs

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/284 Title: #284: ipautil: check for open ports on all resolved IPs mbasti-rh commented: """ needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/284#issuecomment-265426083 -- Manage you

[Freeipa-devel] [freeipa PR#294][comment] client, platform: Use paths.SSH* instead of get_config_dir().

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/294 Title: #294: client, platform: Use paths.SSH* instead of get_config_dir(). mbasti-rh commented: """ @tjaalton just one nitpick, otherwise it looks good to me """ See the full comment at https://github.com/freeipa

[Freeipa-devel] [freeipa PR#312][closed] bindinstance: use data in named.conf to determine configuration status

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/312 Author: martbab Title: #312: bindinstance: use data in named.conf to determine configuration status Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/312/head:pr312

[Freeipa-devel] [freeipa PR#312][comment] bindinstance: use data in named.conf to determine configuration status

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/312 Title: #312: bindinstance: use data in named.conf to determine configuration status mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/f0e09c42b76f229486e5dea097cd2b660243

[Freeipa-devel] [freeipa PR#312][+pushed] bindinstance: use data in named.conf to determine configuration status

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/312 Title: #312: bindinstance: use data in named.conf to determine configuration status Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http:

[Freeipa-devel] [freeipa PR#206][comment] Properly handle multiple cookies in rpcclient

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/206 Title: #206: Properly handle multiple cookies in rpcclient mbasti-rh commented: """ could you please fix PEP8? ``` ./ipalib/rpc.py:702:80: E501 line too long (93 > 79 characters) ``` """ See the full comme

[Freeipa-devel] [freeipa PR#293][comment] Run out-of-tree tests in Travis CI

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/293 Title: #293: Run out-of-tree tests in Travis CI mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/5ecaea6bc4f49c2665597ca38fc52f4fae8a9d24 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#293][+pushed] Run out-of-tree tests in Travis CI

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/293 Title: #293: Run out-of-tree tests in Travis CI 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/Cod

[Freeipa-devel] [freeipa PR#293][closed] Run out-of-tree tests in Travis CI

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/293 Author: martbab Title: #293: Run out-of-tree tests in Travis CI Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/293/head:pr293 git checkout pr293 -- Manage your s

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ I would like rather explicit pylint version than autodetection """ See the full comment at https://g

[Freeipa-devel] [freeipa PR#288][+pushed] Fix missing translation string

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Title: #288: Fix missing translation string 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#288][closed] Fix missing translation string

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Author: shanyin Title: #288: Fix missing translation string Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/288/head:pr288 git checkout pr288 -- Manage your subsc

[Freeipa-devel] [freeipa PR#288][comment] Fix missing translation string

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Title: #288: Fix missing translation string mbasti-rh commented: """ I pushed just commit `fix missing translation string` * 0499ba5795cf483756ac980604fd2c26fda7ba39 fix missing translation string Thank you! """

[Freeipa-devel] [freeipa PR#288][+ack] Fix missing translation string

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Title: #288: Fix missing translation string 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#276][comment] replica-conncheck: improve error msg + logging

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/276 Title: #276: replica-conncheck: improve error msg + logging mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/eb6905bbb45d246f9beddeea69da2236cf67bc68 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#276][+pushed] replica-conncheck: improve error msg + logging

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/276 Title: #276: replica-conncheck: improve error msg + logging 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

[Freeipa-devel] [freeipa PR#276][closed] replica-conncheck: improve error msg + logging

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/276 Author: tomaskrizek Title: #276: replica-conncheck: improve error msg + logging Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/276/head:pr276 git checkout pr276 -

[Freeipa-devel] [freeipa PR#312][+ack] bindinstance: use data in named.conf to determine configuration status

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/312 Title: #312: bindinstance: use data in named.conf to determine configuration status Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://w

[Freeipa-devel] [freeipa PR#309][+pushed] ipa-replica-conncheck: fix race condition

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/309 Title: #309: ipa-replica-conncheck: fix race condition 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/Contrib

[Freeipa-devel] [freeipa PR#309][comment] ipa-replica-conncheck: fix race condition

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/309 Title: #309: ipa-replica-conncheck: fix race condition mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/a44974cdf8a6c53f2e86dac92aa579ba45f666e4 """ See the full comment

[Freeipa-devel] [freeipa PR#309][closed] ipa-replica-conncheck: fix race condition

2016-12-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/309 Author: tomaskrizek Title: #309: ipa-replica-conncheck: fix race condition Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/309/head:pr309 git checkout pr309 -- Ma

[Freeipa-devel] [freeipa PR#292][comment] Increase the timeout waiting for certificate issuance in installer

2016-12-02 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/292 Title: #292: Increase the timeout waiting for certificate issuance in installer mbasti-rh commented: """ +1 """ See the full comment at https://github.com/freeipa/freeipa/pull/292#issuecomment-264478024 -- Ma

[Freeipa-devel] [freeipa PR#288][comment] Fix missing translation string

2016-12-02 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Title: #288: Fix missing translation string mbasti-rh commented: """ @shanyin could you force push to github remote branch to have updated PR? """ See the full comment at https://github.com/freeipa/freeipa/p

[Freeipa-devel] [freeipa PR#267][comment] ipa-replica-conncheck: do not close listening ports until required

2016-12-01 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/267 Title: #267: ipa-replica-conncheck: do not close listening ports until required mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/af0ba661889c2e2c9a35d4cff9681c2abab73649 ""&

[Freeipa-devel] [freeipa PR#267][+pushed] ipa-replica-conncheck: do not close listening ports until required

2016-12-01 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/267 Title: #267: ipa-replica-conncheck: do not close listening ports until required 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-devel] [freeipa PR#263][+pushed] Backwards compatibility with setuptools 0.9.8

2016-12-01 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/263 Title: #263: Backwards compatibility with setuptools 0.9.8 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/Con

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

2016-12-01 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/263 Title: #263: Backwards compatibility with setuptools 0.9.8 mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/027fc32fe0424659c5aecb4531299fe8d4a503d3 """ See the full

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

2016-12-01 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/263 Author: tiran Title: #263: Backwards compatibility with setuptools 0.9.8 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/263/head:pr263 git checkout pr263 -- Mana

[Freeipa-devel] [freeipa PR#288][comment] Fix missing translation string

2016-12-01 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Title: #288: Fix missing translation string mbasti-rh commented: """ ``` git rebase -i master fix-missing-translation-string ``` it will open interactive mode and you can remove all lines (commits) that you want to remove.

[Freeipa-devel] [freeipa PR#287][+pushed] Wheel bundles fixes

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/287 Title: #287: Wheel bundles 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

[Freeipa-devel] [freeipa PR#287][closed] Wheel bundles fixes

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/287 Author: tiran Title: #287: Wheel bundles fixes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/287/head:pr287 git checkout pr287 -- Manage your subscription for t

[Freeipa-devel] [freeipa PR#287][comment] Wheel bundles fixes

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/287 Title: #287: Wheel bundles fixes mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/235f68524767c1eb2e12fb6d1d9f6a520414c583 """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#285][+pushed] Check the result of cert request in replica installer

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/285 Title: #285: Check the result of cert request in replica installer 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/

[Freeipa-devel] [freeipa PR#285][closed] Check the result of cert request in replica installer

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/285 Author: flo-renaud Title: #285: Check the result of cert request in replica installer Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/285/head:pr285 git checkout p

[Freeipa-devel] [freeipa PR#285][comment] Check the result of cert request in replica installer

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/285 Title: #285: Check the result of cert request in replica installer mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/dbb98765d73519289ee22f3de1a5ccde140f6f5d """

[Freeipa-devel] [freeipa PR#267][comment] ipa-replica-conncheck: do not close listening ports until required

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/267 Title: #267: ipa-replica-conncheck: do not close listening ports until required mbasti-rh commented: """ needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/267#issuecomment-26390328

[Freeipa-devel] [freeipa PR#200][+pushed] Test: basic kerberos over http functionality

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/200 Title: #200: Test: basic kerberos over http functionality 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

[Freeipa-devel] [freeipa PR#200][comment] Test: basic kerberos over http functionality

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/200 Title: #200: Test: basic kerberos over http functionality mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c7fd46e42a9f5b4676415910b800e0340f77dc88 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#285][comment] Check the result of cert request in replica installer

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/285 Title: #285: Check the result of cert request in replica installer mbasti-rh commented: """ LGTM """ See the full comment at https://github.com/freeipa/freeipa/pull/285#issuecomment-263870742 -- Manage your subs

[Freeipa-devel] [freeipa PR#286][comment] fix miss translation in Chinese

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/286 Title: #286: fix miss translation in Chinese mbasti-rh commented: """ Thanks """ See the full comment at https://github.com/freeipa/freeipa/pull/286#issuecomment-263859193 -- Manage your subscription for the Fre

[Freeipa-devel] [freeipa PR#288][comment] Fix missing translation string

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/288 Title: #288: Fix missing translation string mbasti-rh commented: """ Hello, could you please remove `fix miss translation in Chinese` and `Delete zh_CN.po` from this PR? """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#285][comment] Check the result of cert request in replica installer

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/285 Title: #285: Check the result of cert request in replica installer mbasti-rh commented: """ Can we add cert state to error message? `raise RuntimeError("Certificate issuance failed")` is not too much detailed in `

[Freeipa-devel] [freeipa PR#286][comment] fix miss translation in Chinese

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/286 Title: #286: fix miss translation in Chinese mbasti-rh commented: """ This: ``` -label='Group search fields',  +label=_('Group search fields'), ``` """ See the full comme

[Freeipa-devel] [freeipa PR#286][+rejected] fix miss translation in Chinese

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/286 Title: #286: fix miss translation in Chinese Label: +rejected -- 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#286][comment] fix miss translation in Chinese

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/286 Title: #286: fix miss translation in Chinese mbasti-rh commented: """ We automatically add translations to IPA from zanata before releasing. If it is translated in zanata it will appear in next release. """

[Freeipa-devel] [freeipa PR#286][closed] fix miss translation in Chinese

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/286 Author: shanyin Title: #286: fix miss translation in Chinese Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/286/head:pr286 git checkout pr286 -- Manage your subs

[Freeipa-devel] [freeipa PR#271][comment] Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/271 Title: #271: Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient mbasti-rh commented: """ master: 9117a5d5a6ae7b3b97407e46f81a06c387974d7f paths: remove DEV_NULL 8e5d2c7014ff6371a3b306e666c301aea1f7a488 c

[Freeipa-devel] [freeipa PR#271][closed] Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient

2016-11-30 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/271 Author: jcholast Title: #271: Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/271/head:

[Freeipa-devel] [freeipa PR#269][+pushed] Prevent denial of replication updates during CA replica install

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/269 Title: #269: Prevent denial of replication updates during CA replica install Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.fr

[Freeipa-devel] [freeipa PR#269][comment] Prevent denial of replication updates during CA replica install

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/269 Title: #269: Prevent denial of replication updates during CA replica install mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/73d0d03891c8585a925f5b49739990c57f6e https://fedorahos

[Freeipa-devel] [freeipa PR#269][closed] Prevent denial of replication updates during CA replica install

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/269 Author: martbab Title: #269: Prevent denial of replication updates during CA replica install Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/269/head:pr269 git che

[Freeipa-devel] [freeipa PR#268][+pushed] Build system must regenerate file when template changes

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/268 Title: #268: Build system must regenerate file when template changes 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.or

[Freeipa-devel] [freeipa PR#268][comment] Build system must regenerate file when template changes

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/268 Title: #268: Build system must regenerate file when template changes mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/ba6ae666acaf8b930d18f45efc7c9c9faad3526b https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#268][closed] Build system must regenerate file when template changes

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/268 Author: pspacek Title: #268: Build system must regenerate file when template changes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/268/head:pr268 git checkout pr

[Freeipa-devel] [freeipa PR#271][comment] Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/271 Title: #271: Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient mbasti-rh commented: """ Ticket updated. """ See the full comment at https://github.com/freeipa/freeipa/pull/271#issuecom

[Freeipa-devel] [freeipa PR#271][comment] Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/271 Title: #271: Remove hard dependency on ipaplatform from ipapython, ipalib and ipaclient mbasti-rh commented: """ Ticket https://fedorahosted.org/freeipa/ticket/6474 is closed as wontfix and even doesn't seems right to m

[Freeipa-devel] [freeipa PR#269][comment] Prevent denial of replication updates during CA replica install

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/269 Title: #269: Prevent denial of replication updates during CA replica install mbasti-rh commented: """ Patch does not apply to 4.4.3 branch """ See the full comment at https://github.com/freeipa/freeipa/pull/269#iss

[Freeipa-devel] [freeipa PR#277][closed] DNS: URI records: bump python-dns requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Author: mbasti-rh Title: #277: DNS: URI records: bump python-dns requirements Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/277/head:pr277 git checkout pr277

[Freeipa-devel] [freeipa PR#277][+pushed] DNS: URI records: bump python-dns requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Title: #277: DNS: URI records: bump python-dns requirements 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

[Freeipa-devel] [freeipa PR#277][comment] DNS: URI records: bump python-dns requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Title: #277: DNS: URI records: bump python-dns requirements mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/a291c6ded91611ea2bd1a1fdb96314721d73a75f """ See the full

[Freeipa-devel] [freeipa PR#277][synchronized] DNS: URI records: bump python-dns requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Author: mbasti-rh Title: #277: DNS: URI records: bump python-dns requirements Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/277/head:pr277 git checkout

[Freeipa-devel] [freeipa PR#277][comment] DNS: URI records: bump python-dns requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Title: #277: DNS: URI records: bump python-dns requirements mbasti-rh commented: """ Thank you, fixed. """ See the full comment at https://github.com/freeipa/freeipa/pull/277#issuecomment-263562846 -- Manage you

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ So create a separate commits: - fixes ipasetup.py - bumps python-gssapi for pypi, with proper explanation in commit message and maybe comment in code may be

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ > PS: There is no technical reason to bump the version of python-gssapi in > freeipa.spec. The enum34 dependency issues is solely a Python packagin

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ > @mbasti-rh The bumped version numbers are required. gssapi needs to be bumped > because 1.1.x has wrong dependency information for Python 3 (enum34)

[Freeipa-devel] [freeipa PR#174][comment] add log module

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/174 Title: #174: add log module mbasti-rh commented: """ Hello, what I meant was to send fixing of missing translations strings as separated PR and if you identified any parts of code that should be logged too, you can send a PR

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ @tiran You can split patch to useful part and please send unneeded bumping of requires as separate pull request, we can continue with discussion there

[Freeipa-devel] [freeipa PR#277][synchronized] DNS: URI records: bump python-dns requirements

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Author: mbasti-rh Title: #277: DNS: URI records: bump python-dns requirements Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/277/head:pr277 git checkout

[Freeipa-devel] [freeipa PR#277][opened] DNS: URI records: bump python-dns requirements

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/277 Author: mbasti-rh Title: #277: DNS: URI records: bump python-dns requirements Action: opened PR body: """ Support for DNS URI records has been added in python-dns 1.13 https://fedorahosted.org/freeipa/ticket/6344 ""

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ Shouldn't be there python3 in BuildRequires as well? At least with python3-pylint we need python3 dependenci

[Freeipa-devel] [freeipa PR#274][comment] Improve the robustness FreeIPA's i18n module and its tests

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/274 Title: #274: Improve the robustness FreeIPA's i18n module and its tests mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/211c944a353dbc241ae6e280c9474145ab48dbe4 ""&qu

[Freeipa-devel] [freeipa PR#274][closed] Improve the robustness FreeIPA's i18n module and its tests

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/274 Author: martbab Title: #274: Improve the robustness FreeIPA's i18n module and its tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/274/head:pr274 git checkout

[Freeipa-devel] [freeipa PR#274][+pushed] Improve the robustness FreeIPA's i18n module and its tests

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/274 Title: #274: Improve the robustness FreeIPA's i18n module and its 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

[Freeipa-devel] [freeipa PR#274][+ack] Improve the robustness FreeIPA's i18n module and its tests

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/274 Title: #274: Improve the robustness FreeIPA's i18n module and its 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.or

[Freeipa-devel] [freeipa PR#273][+ack] Build: workaround bug while calling parallel make from rpmbuild

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/273 Title: #273: Build: workaround bug while calling parallel make from rpmbuild Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freei

[Freeipa-devel] [freeipa PR#101][comment] Improved vault-show error message

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/101 Title: #101: Improved vault-show error message mbasti-rh commented: """ I had discussion with @jcholast and he disagrees. This weird handling of DN should stay isolated in vault code and shouldn't be spreaded across the fra

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-28 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ Better now, but commit message missing explanation why bumping requires was needed. """ See the full comment at https://github.com/freeipa

[Freeipa-devel] [freeipa PR#264][+pushed] Python3 pylint fixes

2016-11-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/264 Title: #264: Python3 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

[Freeipa-devel] [freeipa PR#264][closed] Python3 pylint fixes

2016-11-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/264 Author: tiran Title: #264: Python3 pylint fixes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/264/head:pr264 git checkout pr264 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#264][comment] Python3 pylint fixes

2016-11-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/264 Title: #264: Python3 pylint fixes mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/38e8719f728e6d54289507fe2c7f79f9272c45c0 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#264][+ack] Python3 pylint fixes

2016-11-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/264 Title: #264: Python3 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

<    1   2   3   4   5   6   7   8   9   10   >