On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David Kupka wrote:
On 09/23/2014 08:25 PM, Martin Basti wrote:
On 23/09/14 13:23, David Kupka wrote:
On 09/18/2014 06:34 PM, Martin Basti wrote:
...
1)
+if options.unattended:
+
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David Kupka wrote:
On 09/23/2014 08:25 PM, Martin Basti wrote:
On 23/09/14 13:23, David Kupka wrote:
On 09/18/2014 06:34 PM, Martin Basti
On 09/26/2014 09:34 AM, Jan Cholasta wrote:
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David Kupka wrote:
On 09/23/2014 08:25 PM, Martin Basti wrote:
On 23/09/14 13:23, David Kupka
On 09/26/2014 10:30 AM, David Kupka wrote:
On 09/26/2014 09:34 AM, Jan Cholasta wrote:
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David Kupka wrote:
On 09/23/2014 08:25 PM, Martin
On 09/26/2014 02:47 PM, David Kupka wrote:
On 09/26/2014 10:30 AM, David Kupka wrote:
On 09/26/2014 09:34 AM, Jan Cholasta wrote:
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David
On 26/09/14 14:47, David Kupka wrote:
On 09/26/2014 10:30 AM, David Kupka wrote:
On 09/26/2014 09:34 AM, Jan Cholasta wrote:
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David Kupka
On 09/26/2014 05:50 PM, Martin Basti wrote:
On 26/09/14 14:47, David Kupka wrote:
On 09/26/2014 10:30 AM, David Kupka wrote:
On 09/26/2014 09:34 AM, Jan Cholasta wrote:
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin
On 09/26/2014 05:50 PM, Martin Basti wrote:
On 26/09/14 14:47, David Kupka wrote:
On 09/26/2014 10:30 AM, David Kupka wrote:
On 09/26/2014 09:34 AM, Jan Cholasta wrote:
Dne 26.9.2014 v 08:28 David Kupka napsal(a):
On 09/25/2014 04:17 PM, David Kupka wrote:
On 09/24/2014 08:54 PM, Martin
On 09/24/2014 08:54 PM, Martin Basti wrote:
On 24/09/14 15:44, David Kupka wrote:
On 09/23/2014 08:25 PM, Martin Basti wrote:
On 23/09/14 13:23, David Kupka wrote:
On 09/18/2014 06:34 PM, Martin Basti wrote:
...
1)
+if options.unattended:
+for ip in ip_addresses:
+
On 09/23/2014 08:25 PM, Martin Basti wrote:
On 23/09/14 13:23, David Kupka wrote:
On 09/18/2014 06:34 PM, Martin Basti wrote:
...
1)
+if options.unattended:
+for ip in ip_addresses:
+if search_reverse_zones and find_reverse_zone(str(ip)):
+
On 24/09/14 15:44, David Kupka wrote:
On 09/23/2014 08:25 PM, Martin Basti wrote:
On 23/09/14 13:23, David Kupka wrote:
On 09/18/2014 06:34 PM, Martin Basti wrote:
...
1)
+if options.unattended:
+for ip in ip_addresses:
+if search_reverse_zones and
On 09/18/2014 06:34 PM, Martin Basti wrote:
...
1)
+if options.unattended:
+for ip in ip_addresses:
+if search_reverse_zones and find_reverse_zone(str(ip)):
+# reverse zone is already in LDAP
+continue
+
On 23/09/14 13:23, David Kupka wrote:
On 09/18/2014 06:34 PM, Martin Basti wrote:
...
1)
+if options.unattended:
+for ip in ip_addresses:
+if search_reverse_zones and find_reverse_zone(str(ip)):
+# reverse zone is already in LDAP
+
On 09/17/2014 07:25 AM, David Kupka wrote:
On 09/16/2014 06:09 PM, Martin Basti wrote:
On 16/09/14 15:59, David Kupka wrote:
On 09/12/2014 07:24 PM, Martin Basti wrote:
snip /
Be careful, reviewed on friday! :-)
1)
whitespace error + pep8 error
patch:76: trailing whitespace.
# there is
...
1)
+if options.unattended:
+for ip in ip_addresses:
+if search_reverse_zones and find_reverse_zone(str(ip)):
+# reverse zone is already in LDAP
+continue
+for rz in ret_reverse_zones:
+
On 09/16/2014 06:09 PM, Martin Basti wrote:
On 16/09/14 15:59, David Kupka wrote:
...
2)
+# check that there is IP address in every reverse zone
+if options.reverse_zones:
+for rz in options.reverse_zones:
+for ip in config.ip_addresses:
+if
On 17/09/14 07:25, David Kupka wrote:
3)
+elif options.reverse_zones or (not(options.no_reverse) and
bindinstance.create_reverse()):
OR operator, this will create additional zones (non-specified by user)
even if user write NO
When user specifies some reverse zone (using --reverse-zone)
On 09/12/2014 07:24 PM, Martin Basti wrote:
snip /
Be careful, reviewed on friday! :-)
1)
whitespace error + pep8 error
patch:76: trailing whitespace.
# there is reverse zone for every ip address
warning: 1 line adds whitespace errors.
./ipaserver/install/bindinstance.py:640:9: E265
On 16/09/14 15:59, David Kupka wrote:
On 09/12/2014 07:24 PM, Martin Basti wrote:
snip /
Be careful, reviewed on friday! :-)
1)
whitespace error + pep8 error
patch:76: trailing whitespace.
# there is reverse zone for every ip address
warning: 1 line adds whitespace errors.
On 09/16/2014 06:09 PM, Martin Basti wrote:
On 16/09/14 15:59, David Kupka wrote:
On 09/12/2014 07:24 PM, Martin Basti wrote:
snip /
Be careful, reviewed on friday! :-)
1)
whitespace error + pep8 error
patch:76: trailing whitespace.
# there is reverse zone for every ip address
warning:
On 09/08/2014 05:56 PM, Martin Basti wrote:
On 02/09/14 16:55, David Kupka wrote:
The patch now depends on freeipa-dkupka-0012 as both modifies the same
part of code.
freeipa-dkupka-0012 is now accepted and merged upstream so there is no
need to take this dependency into account.
On
snip /
Be careful, reviewed on friday! :-)
1)
whitespace error + pep8 error
patch:76: trailing whitespace.
# there is reverse zone for every ip address
warning: 1 line adds whitespace errors.
./ipaserver/install/bindinstance.py:640:9: E265 block comment should
start with '# '
2)
On 08/09/14 17:56, Martin Basti wrote:
On 02/09/14 16:55, David Kupka wrote:
The patch now depends on freeipa-dkupka-0012 as both modifies the
same part of code.
On 09/02/2014 10:29 AM, David Kupka wrote:
Forget to add str() conversion to some places when removing map(). Now
it should be
On 02/09/14 16:55, David Kupka wrote:
The patch now depends on freeipa-dkupka-0012 as both modifies the same
part of code.
On 09/02/2014 10:29 AM, David Kupka wrote:
Forget to add str() conversion to some places when removing map(). Now
it should be working again.
On 08/27/2014 02:24 PM,
Forget to add str() conversion to some places when removing map(). Now
it should be working again.
On 08/27/2014 02:24 PM, David Kupka wrote:
Patch modified according to jcholast's personally-delivered feedback:
1) use action='append' instead of that ugly parsing
2) do not use map(),
The patch now depends on freeipa-dkupka-0012 as both modifies the same
part of code.
On 09/02/2014 10:29 AM, David Kupka wrote:
Forget to add str() conversion to some places when removing map(). Now
it should be working again.
On 08/27/2014 02:24 PM, David Kupka wrote:
Patch modified
Patch modified according to jcholast's personally-delivered feedback:
1) use action='append' instead of that ugly parsing
2) do not use map(), FreeIPA doesn't like it
On 08/25/2014 05:04 PM, David Kupka wrote:
https://fedorahosted.org/freeipa/ticket/3575
Also should fix
On 08/27/2014 02:24 PM, David Kupka wrote:
...
2) do not use map(), FreeIPA doesn't like it
Well, it is not that FreeIPA does not like it, it is just more Pythonic to use
list comprehension and not functional languages remnants in Python :-)
___
On Wed, Aug 27, 2014 at 03:17:37PM +0200, Martin Kosek wrote:
On 08/27/2014 02:24 PM, David Kupka wrote:
...
2) do not use map(), FreeIPA doesn't like it
Well, it is not that FreeIPA does not like it, it is just more Pythonic to
use list comprehension and not functional languages remnants
29 matches
Mail list logo
