On 08/27/2015 07:21 PM, Oleg Fayans wrote:
Hi Martin,
My bad, forgot to do git add.
On 08/27/2015 06:27 PM, Martin Basti wrote:
On 08/27/2015 05:41 PM, Oleg Fayans wrote:
Hi,
I am sorry I have missed that.
Fixed. The test fails now due to this bug:
Hi,
I am sorry I have missed that.
Fixed. The test fails now due to this bug:
https://fedorahosted.org/freeipa/ticket/5222
The test output is attached together with the updated patch
On 08/26/2015 05:53 PM, Martin Basti wrote:
On 08/26/2015 05:42 PM, Martin Basti wrote:
On 08/26/2015
Hi Martin,
My bad, forgot to do git add.
On 08/27/2015 06:27 PM, Martin Basti wrote:
On 08/27/2015 05:41 PM, Oleg Fayans wrote:
Hi,
I am sorry I have missed that.
Fixed. The test fails now due to this bug:
https://fedorahosted.org/freeipa/ticket/5222
The test output is attached together
On 08/27/2015 05:41 PM, Oleg Fayans wrote:
Hi,
I am sorry I have missed that.
Fixed. The test fails now due to this bug:
https://fedorahosted.org/freeipa/ticket/5222
The test output is attached together with the updated patch
On 08/26/2015 05:53 PM, Martin Basti wrote:
On 08/26/2015
On 08/26/2015 05:42 PM, Martin Basti wrote:
On 08/26/2015 02:53 PM, Oleg Fayans wrote:
Hi,
No more short links :)
On 08/26/2015 11:50 AM, Tomas Babej wrote:
On 08/26/2015 11:44 AM, Oleg Fayans wrote:
Hi Martin,
On 08/20/2015 11:18 AM, Martin Basti wrote:
On 08/20/2015 10:26 AM,
Hi Martin,
On 08/20/2015 11:18 AM, Martin Basti wrote:
On 08/20/2015 10:26 AM, Martin Basti wrote:
On 08/19/2015 04:17 PM, Martin Basti wrote:
I got this:
https://paste.fedoraproject.org/256746/43999380/
FYI replica install failure. (I will retest it, but I'm pretty sure
that it was
On 08/26/2015 11:44 AM, Oleg Fayans wrote:
Hi Martin,
On 08/20/2015 11:18 AM, Martin Basti wrote:
On 08/20/2015 10:26 AM, Martin Basti wrote:
On 08/19/2015 04:17 PM, Martin Basti wrote:
I got this:
https://paste.fedoraproject.org/256746/43999380/
FYI replica install failure. (I
Hi,
No more short links :)
On 08/26/2015 11:50 AM, Tomas Babej wrote:
On 08/26/2015 11:44 AM, Oleg Fayans wrote:
Hi Martin,
On 08/20/2015 11:18 AM, Martin Basti wrote:
On 08/20/2015 10:26 AM, Martin Basti wrote:
On 08/19/2015 04:17 PM, Martin Basti wrote:
I got this:
On 08/26/2015 02:53 PM, Oleg Fayans wrote:
Hi,
No more short links :)
On 08/26/2015 11:50 AM, Tomas Babej wrote:
On 08/26/2015 11:44 AM, Oleg Fayans wrote:
Hi Martin,
On 08/20/2015 11:18 AM, Martin Basti wrote:
On 08/20/2015 10:26 AM, Martin Basti wrote:
On 08/19/2015 04:17 PM,
On 08/20/2015 10:26 AM, Martin Basti wrote:
On 08/19/2015 04:17 PM, Martin Basti wrote:
I got this:
https://paste.fedoraproject.org/256746/43999380/
FYI replica install failure. (I will retest it, but I'm pretty sure
that it was clean VM, test for some reason install client first)
I got this:
https://paste.fedoraproject.org/256746/43999380/
On 08/19/2015 09:00 AM, Oleg Fayans wrote:
Hi Martin,
As discussed, here is a new version with pep8-related fixes
On 08/14/2015 10:44 AM, Oleg Fayans wrote:
Hi Martin,
Already noticed that. Implemented the named groups as Tomas
Hi Martin,
As discussed, here is a new version with pep8-related fixes
On 08/14/2015 10:44 AM, Oleg Fayans wrote:
Hi Martin,
Already noticed that. Implemented the named groups as Tomas advised.
Added the third test for
Hi Martin,
Already noticed that. Implemented the named groups as Tomas advised.
Added the third test for
On 08/13/2015 05:06 PM, Martin Basti wrote:
On 08/11/2015 03:36 PM, Oleg Fayans wrote:
Hi Martin,
On 08/11/2015 02:02 PM, Martin Basti wrote:
NACK, comments inline.
On 11/08/15 13:25, Oleg Fayans wrote:
Hi Martin,
Thanks for the review!
On 08/10/2015 07:08 PM, Martin Basti wrote:
On 08/11/2015 03:36 PM, Oleg Fayans wrote:
Hi Martin,
On 08/11/2015 02:02 PM, Martin Basti wrote:
NACK, comments inline.
On 11/08/15 13:25, Oleg Fayans wrote:
Hi Martin,
Thanks for the review!
On 08/10/2015 07:08 PM, Martin Basti wrote:
Thank you for patch, I have a few nitpicks:
1)
On
NACK, comments inline.
On 11/08/15 13:25, Oleg Fayans wrote:
Hi Martin,
Thanks for the review!
On 08/10/2015 07:08 PM, Martin Basti wrote:
Thank you for patch, I have a few nitpicks:
1)
On 10/08/15 13:05, Oleg Fayans wrote:
+def create_segment(master, leftnode, rightnode):
+
Hi Martin,
Thanks for the review!
On 08/10/2015 07:08 PM, Martin Basti wrote:
Thank you for patch, I have a few nitpicks:
1)
On 10/08/15 13:05, Oleg Fayans wrote:
+def create_segment(master, leftnode, rightnode):
+create_segment(master, leftnode, rightnode)
Why do you add the name of
Hi Martin,
On 08/11/2015 02:02 PM, Martin Basti wrote:
NACK, comments inline.
On 11/08/15 13:25, Oleg Fayans wrote:
Hi Martin,
Thanks for the review!
On 08/10/2015 07:08 PM, Martin Basti wrote:
Thank you for patch, I have a few nitpicks:
1)
On 10/08/15 13:05, Oleg Fayans wrote:
+def
Thank you for patch, I have a few nitpicks:
1)
On 10/08/15 13:05, Oleg Fayans wrote:
+def create_segment(master, leftnode, rightnode):
+create_segment(master, leftnode, rightnode)
Why do you add the name of method in docstring?
2)
+def create_segment(master, leftnode, rightnode):
+
Last update to the patch.
On 08/10/2015 09:41 AM, Oleg Fayans wrote:
Hi all.
Applied pPOP8 requirements to the code, removed unused imports. Please,
disregard the first version of this patch
On 08/10/2015 08:20 AM, Oleg Fayans wrote:
Hi list,
Here are 2 integration tests for topology
20 matches
Mail list logo