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 Bast
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 Ba
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 wrot
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 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 Bast
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 wro
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 wro
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:
+
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 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
find_rev
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 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/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
+fo
...
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/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:
Be careful, reviewed on friday! :-)
1)
whitespace error + pep8 error
patch:76: trailing whitespace.
# there is revers
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/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:
> +
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:
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
On 16/09/14 15:59, David Kupka wrote:
On 09/12/2014 07:24 PM, Martin Basti wrote:
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/
On 09/12/2014 07:24 PM, Martin Basti wrote:
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 comme
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) (server insta
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 09/02
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 work
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, Davi
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 accordin
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(), Fr
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 r
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 :-)
___
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 https://bugzilla.r
https://fedorahosted.org/freeipa/ticket/3575
Also should fix https://bugzilla.redhat.com/show_bug.cgi?id=1128380 as
installation is no longer interrupted when multiple IPs are resolved.
But it does not add the option to change the IP address during second run.
--
David Kupka
From 5785d3ed20514
30 matches
Mail list logo
