URL: https://github.com/freeipa/freeipa/pull/187
Title: #187: Register entry points of Custodia plugins
jcholast commented:
"""
Just curious, why is the store class called `iSecStore` and not `IPASecStore`
or something similar to make it apparent that it comes from IPA?
"""
See the full
URL: https://github.com/freeipa/freeipa/pull/221
Title: #221: gitignore: ignore tar ball
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
URL: https://github.com/freeipa/freeipa/pull/219
Title: #219: Refactor installer code requesting certificates
frasertweedale commented:
"""
Although there are conflicts with `master`, there are problems when the patches
are rebased. Server installation (CA-ful) fails when requesting the RA
URL: https://github.com/freeipa/freeipa/pull/219
Title: #219: Refactor installer code requesting certificates
frasertweedale commented:
"""
Although there are no conflicts with `master`, there are problems when the
patches are rebased. Server installation (CA-ful) fails when requesting the RA
URL: https://github.com/freeipa/freeipa/pull/217
Author: frasertweedale
Title: #217: change certificate processing code to use python-cryptography
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa
URL: https://github.com/freeipa/freeipa/pull/195
Author: tiran
Title: #195: [WIP] Make ipaclient pip install-able
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/195/head:pr195
git checkout pr195
From
URL: https://github.com/freeipa/freeipa/pull/195
Author: tiran
Title: #195: [WIP] Make ipaclient pip install-able
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/195/head:pr195
git checkout pr195
From
URL: https://github.com/freeipa/freeipa/pull/182
Author: tiran
Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/182/head:pr182
git
URL: https://github.com/freeipa/freeipa/pull/182
Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context
tiran commented:
"""
ipapython.admintool.ScriptError still prints the full traceback:
```
$ IPA_CONFDIR=/tmp/ipa ./ipa
[2016-11-09T16:35:38Z ipa] : ScriptError: IPA_CONFDIR
URL: https://github.com/freeipa/freeipa/pull/143
Title: #143: Issue6386 nss dir
tiran commented:
"""
I have fixed all places that don't depend on hard-coded paths. The other places
are used for client enrolment and depend on hard-coded paths for sysrestore.
Some places use the path before
URL: https://github.com/freeipa/freeipa/pull/143
Author: tiran
Title: #143: Issue6386 nss dir
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/143/head:pr143
git checkout pr143
From
URL: https://github.com/freeipa/freeipa/pull/180
Author: tiran
Title: #180: Make api.env.nss_dir relative to api.env.confdir
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/180/head:pr180
git checkout
URL: https://github.com/freeipa/freeipa/pull/187
Author: tiran
Title: #187: Register entry points of Custodia plugins
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/187/head:pr187
git checkout pr187
From
URL: https://github.com/freeipa/freeipa/pull/197
Author: tiran
Title: #197: Make setup.py files PyPI compatible
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/197/head:pr197
git checkout pr197
From
URL: https://github.com/freeipa/freeipa/pull/188
Title: #188: Move Python egg-info to top level directory
tiran commented:
"""
I'm closing this PR because the issue can't be fixed w/o patching setuptools
heavily. I got it mostly right except for SOURCES.txt. We need to revise the
project
URL: https://github.com/freeipa/freeipa/pull/188
Author: tiran
Title: #188: Move Python egg-info to top level directory
Action: closed
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 pr188
--
URL: https://github.com/freeipa/freeipa/pull/215
Title: #215: Add script to setup krb5 NFS exports
mbasti-rh commented:
"""
Hello,
can you please provide user cases for this, or some kind of top level design
what problem are you solving, why and how do you plan resolve it?
We may help you
URL: https://github.com/freeipa/freeipa/pull/223
Author: tomaskrizek
Title: #223: LDAP refactoring: remove admin_conn
Action: opened
PR body:
"""
This first commit removes the admin_conn alias for api.Backend.ldap2 that was
previously used in services.
When trying to get rid of it, I found
On 10/25/2016 10:24 AM, Oleg Fayans wrote:
Integration part of the tests is ready. 2 tests:
1. Adds a cert to idoverride of a windows user
2. sssd part - looks up user by his certificate using dbus-sssd
Second and third dbus call are executed as a string insted of as array
of strings because
URL: https://github.com/freeipa/freeipa/pull/222
Author: flo-renaud
Title: #222: Fix ipa-replica-install when upgrade from ca-less to ca-full
Action: opened
PR body:
"""
When ipa-replica-prepare is run on a master upgraded from CA-less to
CA-full, it creates the replica file with a copy of
On 11/03/2016 04:56 PM, Oleg Fayans wrote:
Hi Martin,
The commit message was updated with the correct ticket link
Thanks for review!
On 11/03/2016 04:22 PM, Martin Basti wrote:
almost ACK, but the ticket in commit message is closed as invalid. So
I'm quite puzzled now what to do.
On
URL: https://github.com/freeipa/freeipa/pull/218
Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/218
Author: martbab
Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/218/head:pr218
git
URL: https://github.com/freeipa/freeipa/pull/218
Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/3ecda74d14066f6609d72422041bcc0c6499de77
"""
See the full comment at
On 11/09/2016 04:34 PM, Milan Kubík wrote:
On 11/03/2016 04:56 PM, Oleg Fayans wrote:
Hi Martin,
The commit message was updated with the correct ticket link
Thanks for review!
On 11/03/2016 04:22 PM, Martin Basti wrote:
almost ACK, but the ticket in commit message is closed as invalid. So
URL: https://github.com/freeipa/freeipa/pull/221
Author: tomaskrizek
Title: #221: gitignore: ignore tar ball
Action: opened
PR body:
"""
Add tar ball generated by build to gitignore.
https://fedorahosted.org/freeipa/ticket/6418
"""
To pull the PR as Git branch:
git remote add ghfreeipa
URL: https://github.com/freeipa/freeipa/pull/218
Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/218
Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests
pspacek commented:
"""
Jenkins tests now pass.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/218#issuecomment-259439106
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/216
Title: #216: libexec scripts: ldap conn management
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/216
Title: #216: libexec scripts: ldap conn management
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/33f7b8dc32bc95e0db067ac4df49807ee2b5120e
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/216
Author: tomaskrizek
Title: #216: libexec scripts: ldap conn management
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/216/head:pr216
git checkout pr216
--
URL: https://github.com/freeipa/freeipa/pull/217
Title: #217: change certificate processing code to use python-cryptography
frasertweedale commented:
"""
The travis-ci failure is due to two minor pep8 violations, which I intend :)
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/217
Author: frasertweedale
Title: #217: change certificate processing code to use python-cryptography
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa
URL: https://github.com/freeipa/freeipa/pull/195
Author: tiran
Title: #195: [WIP] Make ipaclient pip install-able
Action: edited
Changed field: body
Original value:
"""
## proof of concept
This makes ipaclient and dependencies pip install-able by adding install
requirements to all
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
memo for me:
- [ ] /freeipa*.tar.gz is not removed
- [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__
and other pyc/pyo. add ```clean-local: rm -rf
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
pspacek commented:
"""
@tiran
> autoconf and automake files are not removed (Makefile.in, /config.sub ...)
According to [Automake manual section 13 What Gets
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
pspacek commented:
"""
@tiran
> autoconf and automake files are not removed (Makefile.in, /config.sub ...)
According to [Automake manual section 13 What Gets
URL: https://github.com/freeipa/freeipa/pull/220
Author: pspacek
Title: #220: Build: fix make clean to remove build artifacts from top-level
directory
Action: opened
PR body:
"""
make lint and make dist were generating files which were not removed by make
clean.
URL: https://github.com/freeipa/freeipa/pull/216
Title: #216: libexec scripts: ldap conn management
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/216
Title: #216: libexec scripts: ldap conn management
flo-renaud commented:
"""
Thanks for the update. Works for me.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/216#issuecomment-259406309
--
Manage your subscription for
URL: https://github.com/freeipa/freeipa/pull/195
Author: tiran
Title: #195: [WIP] Make ipaclient pip install-able
Action: edited
Changed field: body
Original value:
"""
## proof of concept
This makes ipaclient and dependencies pip install-able by adding install
requirements to all
Hi FreeIPA gang,
we just pushed Build system refactoring to master.
Most visible change is that you need to use command "./makerpms.sh" instead of
"make rpms" when building FreeIPA from clean Git tree.
"make rpms" will work as usual after initial autoreconf -i && ./configure
combo so this
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/c48e5fd811326dc64e19490f88003e442815a052
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
URL: https://github.com/freeipa/freeipa/pull/213
Author: pspacek
Title: #213: Build system refactoring phase 3
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/213/head:pr213
git checkout pr213
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
mbasti-rh commented:
"""
ACK
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/213#issuecomment-259399255
--
Manage your subscription for the Freeipa-devel mailing list:
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
memo for me:
- [ ] /freeipa*.tar.gz is not removed
- [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__
and other pyc/pyo. add ```clean-local: rm -rf
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
pspacek commented:
"""
@tiran
I've tested the find command and it works. The trick is `-o` which acts like OR
and allows you to specify different conditions and associate different actions
to them.
URL: https://github.com/freeipa/freeipa/pull/216
Author: tomaskrizek
Title: #216: libexec scripts: ldap conn management
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/216/head:pr216
git checkout pr216
URL: https://github.com/freeipa/freeipa/pull/219
Author: flo-renaud
Title: #219: Refactor installer code requesting certificates
Action: edited
Changed field: body
Original value:
"""
With this PR, the certificates requested during server installation are now
consistently obtained through
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
pspacek commented:
"""
@tiran
> add ipasetup.py to dist_noinst_SCRIPTS ?
`ipasetup.py` file is auto-generated from `ipasetup.py.in` so it should not be
part of distibution tarball. I've marked this
URL: https://github.com/freeipa/freeipa/pull/219
Author: flo-renaud
Title: #219: Refactor installer code requesting certificates
Action: opened
PR body:
"""
With this PR, the certificates requested during server installation are now
consistently obtained through certmonger (applies to
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
memo for me:
- [ ] /freeipa*.tar.gz is not removed
- [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__
and other pyc/pyo. add ```clean-local: rm -rf
URL: https://github.com/freeipa/freeipa/pull/218
Author: martbab
Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests
Action: opened
PR body:
"""
Since /etc/ipa/ca.crt should be always present on the test runner, we should
use it in bind method tests and not rely on
URL: https://github.com/freeipa/freeipa/pull/177
Title: #177: Add options to write lightweight CA cert or chain to file
frasertweedale commented:
"""
Bump for review
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/177#issuecomment-259387372
--
Manage your subscription for
URL: https://github.com/freeipa/freeipa/pull/217
Author: frasertweedale
Title: #217: change certificate processing code to use python-cryptography
Action: opened
PR body:
"""
This commit changes certificate processing code to use python-cryptography
instead of NSS.
Part of the refactoring
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
memo for me:
- [ ] /freeipa*.tar.gz is not removed
- [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__
and other pyc/pyo. add ```clean-local: rm -rf
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
memo for me:
- [ ] /freeipa*.tar.gz is not removed
- [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__
and other pyc/pyo. add ```clean-local: rm -rf
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
memo for me:
- [ ] /freeipa*.tar.gz is not removed
- [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__
and other pyc/pyo. add ```clean-local: rm -rf
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
tiran commented:
"""
The patch has some minor creases but works. Let's merge it to master and iron
out the remaining small issues with PRs.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
URL: https://github.com/freeipa/freeipa/pull/213
Title: #213: Build system refactoring phase 3
mbasti-rh commented:
"""
commit `Build: add rpms target and makerpms.sh script` misses makerpms.sh and
ticket, and it looks like you forgot to squash this commit
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/216
Author: tomaskrizek
Title: #216: libexec scripts: ldap conn management
Action: opened
PR body:
"""
Certificate renewal scripts require connection to LDAP. Properly
handle connects and disconnects from LDAP.
64 matches
Mail list logo