Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-26 Thread Martin Kosek
On Wed, 2012-04-25 at 11:24 +0200, Jan Cholasta wrote: On 24.4.2012 16:57, Nalin Dahyabhai wrote: On Tue, Apr 24, 2012 at 01:02:44PM +0200, Jan Cholasta wrote: I'm just curious, why you do this: default:nis-keys-format: %mregsub(%{macAddress}

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-25 Thread Jan Cholasta
On 24.4.2012 16:57, Nalin Dahyabhai wrote: On Tue, Apr 24, 2012 at 01:02:44PM +0200, Jan Cholasta wrote: I'm just curious, why you do this: default:nis-keys-format: %mregsub(%{macAddress} %{fqdn},(..[:\\\|-]..[:\\\|-]..[:\\\|-]..[:\\\|-]..[:\\\|-]..) (.*),%1) and not simply this:

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-24 Thread Jan Cholasta
On 23.4.2012 23:18, Nalin Dahyabhai wrote: On Mon, Apr 23, 2012 at 05:40:27PM +0200, Jan Cholasta wrote: On 23.4.2012 17:21, Jan Cholasta wrote: On 16.4.2012 22:51, Nalin Dahyabhai wrote: The ethers.byname and ethers.byaddr NIS maps pair host names and hardware network addresses. This should

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-24 Thread Petr Viktorin
On 04/24/2012 01:02 PM, Jan Cholasta wrote: The error is: Traceback (most recent call last): File /usr/lib/python2.7/site-packages/ipaserver/install/ldapupdate.py, line 652, in __update_record self.conn.addEntry(entry) File /usr/lib/python2.7/site-packages/ipaserver/ipaldap.py, line 495, in

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-24 Thread Nalin Dahyabhai
On Tue, Apr 24, 2012 at 01:02:44PM +0200, Jan Cholasta wrote: I'm just curious, why you do this: default:nis-keys-format: %mregsub(%{macAddress} %{fqdn},(..[:\\\|-]..[:\\\|-]..[:\\\|-]..[:\\\|-]..[:\\\|-]..) (.*),%1) and not simply this: default:nis-keys-format: ${macAddress} ?

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-23 Thread Jan Cholasta
On 16.4.2012 22:51, Nalin Dahyabhai wrote: The ethers.byname and ethers.byaddr NIS maps pair host names and hardware network addresses. This should close ticket #2259. Nalin Please add this to install/updates/50-nis.update as well. Besides that, ACK on all 3 patches. I have checked only if

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-23 Thread Jan Cholasta
On 23.4.2012 17:21, Jan Cholasta wrote: On 16.4.2012 22:51, Nalin Dahyabhai wrote: The ethers.byname and ethers.byaddr NIS maps pair host names and hardware network addresses. This should close ticket #2259. Nalin Please add this to install/updates/50-nis.update as well. Besides that, ACK

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-23 Thread Rob Crittenden
Jan Cholasta wrote: On 23.4.2012 17:21, Jan Cholasta wrote: On 16.4.2012 22:51, Nalin Dahyabhai wrote: The ethers.byname and ethers.byaddr NIS maps pair host names and hardware network addresses. This should close ticket #2259. Nalin Please add this to install/updates/50-nis.update as

Re: [Freeipa-devel] [PATCH] add ethers.byname and ethers.byaddr NIS maps

2012-04-23 Thread Nalin Dahyabhai
On Mon, Apr 23, 2012 at 05:40:27PM +0200, Jan Cholasta wrote: On 23.4.2012 17:21, Jan Cholasta wrote: On 16.4.2012 22:51, Nalin Dahyabhai wrote: The ethers.byname and ethers.byaddr NIS maps pair host names and hardware network addresses. This should close ticket #2259. Please add this to