Re: [Freeipa-devel] [Freeipa-users] Announcing FreeIPA v2 Server Release Candidate 2 Release
Hi, I updated my IPA test servers last night without a problem. I have only the default Fedora 14 repo + Fedora 14 updates-testing repo and the Freeipa-devel repo enabled on my IPA test servers. Rgds, Siggi On Tue, March 1, 2011 01:32, Steven Jones wrote: I have tried to download the rpms by hand and the dependencies are all broken ie pythonwell stuffed by the looks of it... regards ___ Freeipa-users mailing list freeipa-us...@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-users ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] Localization patches.
Rob Crittenden wrote: Pavel Zuna wrote: On 02/23/2011 07:09 PM, Pavel Zůna wrote: On 2011-02-22 20:16, Rob Crittenden wrote: Pavel Zůna wrote: On 2011-02-17 22:52, Rob Crittenden wrote: Pavel Zůna wrote: On 2011-02-17 05:09, Rob Crittenden wrote: Pavel Zůna wrote: My efforts in fixing localization all around the framework and preparing it for localizing docstrings have resulted in a lot of patches. Because I understand they have become a bit hard to track, I decided to post them all together in this thread to make review easier. After this is committed, there will be one more patch that switches xgettext for pygettext. Then hopefully, we'll be pretty much set when it comes to i18n. Pavel Patch 81 isn't applying for me. Help is not working for me either, this is due to patch 80. $ ipa help user ipa: ERROR: NameError: global name '_' is not defined Traceback (most recent call last): File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 1087, in run api.finalize() File /home/rcrit/redhat/freeipa-version/ipalib/plugable.py, line 619, in finalize plugin_iter(base, (magic[k] for k in magic)) File /home/rcrit/redhat/freeipa-version/ipalib/base.py, line 397, in __init__ sorted(members, key=lambda m: getattr(m, name_attr)) File /home/rcrit/redhat/freeipa-version/ipalib/plugable.py, line 608, in plugin_iter plugins[klass] = PluginInstance(klass) File /home/rcrit/redhat/freeipa-version/ipalib/plugable.py, line 585, in __init__ self.instance = klass() File /home/rcrit/redhat/freeipa-version/ipalib/plugable.py, line 184, in __init__ self.doc = _(inspect.getdoc(cls)) NameError: global name '_' is not defined ipa: ERROR: an internal error has occurred Patches 69, 71 and 73 are still working fine. What is switching from xgettext to pygettext going to do? This was answered by John Dennis: xgettext doesn't parse python docstrings. rob Rebased version of 81 attached. It should also fix the traceback you're getting. Pavel Something is still not working. I'm having a hard time reproducing how I got this but with LANG=es_US.UTF-8 for a while I was getting this with every ipa user-* request: ipa: ERROR: UnicodeEncodeError: 'ascii' codec can't encode character u'\xf1' in position 20: ordinal not in range(128) Traceback (most recent call last): File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 1090, in run sys.exit(api.Backend.cli.run(argv)) File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 917, in run rv = cmd.output_for_cli(self.api.Backend.textui, result, *args, **options) File /home/rcrit/redhat/freeipa-version/ipalib/frontend.py, line 953, in output_for_cli textui.print_entries(result, order, labels, flags, print_all) File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 346, in print_entries self.print_entry(entry, order, labels, flags, print_all, format, indent) File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 378, in print_entry label, value, format, indent, one_value_per_line File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 309, in print_attribute self.print_indented(format % (attr, text[0]), indent) File /home/rcrit/redhat/freeipa-version/ipalib/cli.py, line 232, in print_indented print (CLI_TAB * indent + text) UnicodeEncodeError: 'ascii' codec can't encode character u'\xf1' in position 20: ordinal not in range(128) ipa: ERROR: ha ocurrido un error interno I think it is blowing up on this user: User login: jose First name: Jose Last name: contraseñas Home directory: /home/jose Login shell: /bin/sh Account disabled: TRUE Member of groups: ipausers Then all of a sudden things started working fine, so I'm not sure what's going on. Is this traceback meaningful to you? rob This looks like a bug in the textui backend. You get this error when you do something like this: a = u'\xf1' a.decode('utf-8') Traceback (most recent call last): File stdin, line 1, in module File /usr/lib/python2.6/encodings/utf_8.py, line 16, in decode return codecs.utf_8_decode(input, errors, True) UnicodeEncodeError: 'ascii' codec can't encode character u'\xf1' in position 0: ordinal not in range(128) It means we're not handling encoding/decoding from/to the CLI right somewhere. The character \xf1 corresponds to the small N with tilde in Jose's last name. I'm going to look into it, but I don't think it's related to the localization patches. Pavel I'm seeing 2 test failures: == FAIL: Test the `ipalib.plugable.Plugin.__init__` method. -- Traceback (most recent call last): File /usr/lib/python2.7/site-packages/nose/case.py, line 186, in runTest self.test(*self.arg) File /home/rcrit/redhat/freeipa-tests/tests/test_ipalib/test_plugable.py, line 237, in test_init assert o.summary == 'Do sub-classy things.' AssertionError == FAIL: Test gettext translation
Re: [Freeipa-devel] [PATCH] 0089 Fix user synchronization in ipa winsync
On Mon, 28 Feb 2011 08:34:15 -0700 Rich Megginson rmegg...@redhat.com wrote: On 02/26/2011 10:27 AM, Simo Sorce wrote: Apparently synchronizing new users down from AD didn't work as the account didn't have uidNumber added, an attribute required by the posixAccount objectclass. ack Pushed to master. Simo. -- Simo Sorce * Red Hat, Inc * New York ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0086 add loginShell to winsynced users
On Mon, 28 Feb 2011 22:14:31 -0500 Rob Crittenden rcrit...@redhat.com wrote: Rich Megginson wrote: On 02/18/2011 03:10 PM, Simo Sorce wrote: Fixes #266 I haven't been able to test this as the Windows machine we have available decided to not behave today. I may try again next week assuming I have time. ack Second ack. I tested the patch and it worked fine. Pushed to master. Simo. -- Simo Sorce * Red Hat, Inc * New York ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0088 Fix ipa winsync plugin
On Mon, 28 Feb 2011 10:42:48 +0100 Jakub Hrozek jhro...@redhat.com wrote: On 02/26/2011 06:26 PM, Simo Sorce wrote: When the plugin was adjusted to not use LDAP_DEPRECATED it was broken and DNs where generated withouth the RDN attribute name part. Simo. I broke this one.. Ack Thanks. Pushed to master. Simo. -- Simo Sorce * Red Hat, Inc * New York ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0090 Make use of (in)activate groups optional
On Mon, 28 Feb 2011 08:33:44 -0700 Rich Megginson rmegg...@redhat.com wrote: On 02/26/2011 10:28 AM, Simo Sorce wrote: Since we remove the use of CoS for (in)active users, the ipa_winsync plugin was broken when configured to synchronize (in)active user status (the default). ack Pushed to master. Thanks, Simo. -- Simo Sorce * Red Hat, Inc * New York ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0091 Make wrappers for sasl binds
On Mon, 28 Feb 2011 22:10:59 -0500 Rob Crittenden rcrit...@redhat.com wrote: This is needed in conjuction with patch 0092 to fix managing replicas with krb credentials Simo. ack Pushed to master. Simo. -- Simo Sorce * Red Hat, Inc * New York ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
[Freeipa-devel] [PATCH] 117 Removed association facets based on memberofindirect.
Association facets based on memberofindirect attribute have been removed because the attribute is non-assignable. Ticket 1027 -- Endi S. Dewata From fc741c0437d0e9358d99a520332f5ead851007e3 Mon Sep 17 00:00:00 2001 From: Endi S. Dewata edew...@redhat.com Date: Tue, 1 Mar 2011 12:30:29 -0600 Subject: [PATCH] Removed association facets based on memberofindirect. Association facets based on memberofindirect attribute have been removed because the attribute is non-assignable. Ticket 1027 --- install/ui/entity.js |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/install/ui/entity.js b/install/ui/entity.js index 01ad47eb71c63228b10e6387ccb7bbd4050bc1ad..f8e61af8edd4a7ea2688c0b819f449044bdd10ac 100644 --- a/install/ui/entity.js +++ b/install/ui/entity.js @@ -194,6 +194,7 @@ IPA.entity = function (spec) { // skip non-assignable associations if (attribute_member === 'memberindirect') continue; +if (attribute_member === 'memberofindirect') continue; if (attribute_member === 'enrolledby') continue; var other_entities = attribute_members[attribute_member]; -- 1.6.6.1 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 117 Removed association facets based on memberofindirect.
On 03/01/2011 02:36 PM, Endi Sukma Dewata wrote: Association facets based on memberofindirect attribute have been removed because the attribute is non-assignable. Ticket 1027 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel ACK. Pushed to master ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO
Adam Young wrote: On 02/28/2011 03:28 PM, Endi Sukma Dewata wrote: On 2/28/2011 12:51 PM, Endi Sukma Dewata wrote: On 2/28/2011 10:47 AM, Rob Crittenden wrote: Use Sudo instead of SUDO in labels, descriptions, etc. ticket 1005 rob This patch is ACKed. The capitalization is now consistent in the CLI. However, the UI capitalizes the labels in the action panel and the title of association facets, so we still see a mix of Sudo and SUDO in the UI. There are still some SUDO leftover in the UI test data, but that can be fixed in a separate patch. The attached patch fixes the UI test data. ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel ACK pushed to master ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO
Endi Sukma Dewata wrote: On 2/28/2011 10:47 AM, Rob Crittenden wrote: Use Sudo instead of SUDO in labels, descriptions, etc. ticket 1005 rob This patch is ACKed. The capitalization is now consistent in the CLI. However, the UI capitalizes the labels in the action panel and the title of association facets, so we still see a mix of Sudo and SUDO in the UI. There are still some SUDO leftover in the UI test data, but that can be fixed in a separate patch. pushed to master ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] Revert-Set-hard-limit-on-number-of-commands-in-batch
Adam Young wrote: I have not tested this, just ran: git revert 79d22f8341026450ba7ca564e24812c9351c7e70 Please test before ACKing. I will test as well now. ack ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] Revert-Set-hard-limit-on-number-of-commands-in-batch
On 03/01/2011 05:34 PM, Rob Crittenden wrote: Adam Young wrote: I have not tested this, just ran: git revert 79d22f8341026450ba7ca564e24812c9351c7e70 Please test before ACKing. I will test as well now. ack ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Pushed to master ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
[Freeipa-devel] [PATCH] admiyo-0206-2-Use-modified-entity-find-commands-for-associations
Not a 100% solution, but keeps the groups-user facet from exploding. https://fedorahosted.org/freeipa/ticket/1011 From 9b59bebceb79fd1f8101a4e08ed853c65784da9d Mon Sep 17 00:00:00 2001 From: Adam Young ayo...@redhat.com Date: Tue, 1 Mar 2011 21:21:19 -0500 Subject: [PATCH] Use modified entity find commands for associations --- install/ui/associate.js | 46 ++ install/ui/host.js |8 +--- 2 files changed, 27 insertions(+), 27 deletions(-) diff --git a/install/ui/associate.js b/install/ui/associate.js index 600cd55c3abb99f810c8e322da83262c21b5e2c3..47b5b968d6b14110016b435bbfd130c1d96cbf47 100644 --- a/install/ui/associate.js +++ b/install/ui/associate.js @@ -394,8 +394,12 @@ IPA.association_table_widget = function (spec) { 'on_success': on_success, 'on_error': on_error }); +var length = that.values.length; +if (length 100){ +length = 100; +} -for (var i=0; ithat.values.length; i++) { +for (var i=0; i length; i++) { var value = that.values[i]; var command = IPA.command({ @@ -597,6 +601,8 @@ IPA.association_facet = function (spec) { that.adder_columns = []; that.adder_columns_by_name = {}; +that.relationship_filter = spec.relationship_filter; + that.get_column = function(name) { return that.columns_by_name[name]; }; @@ -881,28 +887,28 @@ IPA.association_facet = function (spec) { if (!pkeys.length) return; -var batch = IPA.batch_command({ -'name': that.entity_name+'_'+that.name, + +var options = { +'all': true, +'rights': true +}; + +var pkey = $.bbq.getState(that.entity_name + '-pkey', true) || ''; +var args =[]; +var relationship_filter = 'in_' + that.entity_name; +options[relationship_filter] = pkey; + +var command = IPA.command({ 'on_success': on_success, -'on_error': on_error +'on_error': on_error, +'method': that.other_entity+'_find', +'args': args, +options: options }); -for (var i=0; ipkeys.length; i++) { -var pkey = pkeys[i]; +command.execute(); -var command = IPA.command({ -'method': that.other_entity+'_show', -'args': [pkey], -'options': { -'all': true, -'rights': true -} -}); -batch.add_command(command); -} - -batch.execute(); }; that.refresh = function() { @@ -926,9 +932,9 @@ IPA.association_facet = function (spec) { that.get_records( pkeys, function(data, text_status, xhr) { -var results = data.result.results; +var results = data.result.result; for (var i=0; iresults.length; i++) { -var record = results[i].result; +var record = results[i]; that.table.add_record(record); } } diff --git a/install/ui/host.js b/install/ui/host.js index 981bfc021924bc740379009e62ef5b8345b262d5..adaf2c1cb003f1ff97d646eab236b8a8e9ff0025 100644 --- a/install/ui/host.js +++ b/install/ui/host.js @@ -50,7 +50,7 @@ IPA.entity_factories.host = function () { that.add_facet(facet); facet = IPA.host_managedby_host_facet({ -'name': 'managedby_host' +name: 'managedby_host' }); that.add_facet(facet); @@ -503,18 +503,12 @@ IPA.host_managedby_host_facet = function (spec) { }).appendTo(container); }; -that.create_column({name: 'description'}); - that.create_adder_column({ name: 'fqdn', primary_key: true, width: '100px' }); -that.create_adder_column({ -name: 'description', -width: '100px' -}); that.association_facet_init(); }; -- 1.7.4 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] admiyo-0206-2-Use-modified-entity-find-commands-for-associations
On 03/01/2011 09:23 PM, Adam Young wrote: Not a 100% solution, but keeps the groups-user facet from exploding. https://fedorahosted.org/freeipa/ticket/1011 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Includes fixes for services From c33490c208678520a587a0488dc5840d3368f58e Mon Sep 17 00:00:00 2001 From: Adam Young ayo...@redhat.com Date: Tue, 1 Mar 2011 21:21:19 -0500 Subject: [PATCH] Use modified entity find commands for associations --- install/ui/associate.js | 46 ++ install/ui/host.js |8 +--- install/ui/service.js |6 -- 3 files changed, 27 insertions(+), 33 deletions(-) diff --git a/install/ui/associate.js b/install/ui/associate.js index 600cd55c3abb99f810c8e322da83262c21b5e2c3..47b5b968d6b14110016b435bbfd130c1d96cbf47 100644 --- a/install/ui/associate.js +++ b/install/ui/associate.js @@ -394,8 +394,12 @@ IPA.association_table_widget = function (spec) { 'on_success': on_success, 'on_error': on_error }); +var length = that.values.length; +if (length 100){ +length = 100; +} -for (var i=0; ithat.values.length; i++) { +for (var i=0; i length; i++) { var value = that.values[i]; var command = IPA.command({ @@ -597,6 +601,8 @@ IPA.association_facet = function (spec) { that.adder_columns = []; that.adder_columns_by_name = {}; +that.relationship_filter = spec.relationship_filter; + that.get_column = function(name) { return that.columns_by_name[name]; }; @@ -881,28 +887,28 @@ IPA.association_facet = function (spec) { if (!pkeys.length) return; -var batch = IPA.batch_command({ -'name': that.entity_name+'_'+that.name, + +var options = { +'all': true, +'rights': true +}; + +var pkey = $.bbq.getState(that.entity_name + '-pkey', true) || ''; +var args =[]; +var relationship_filter = 'in_' + that.entity_name; +options[relationship_filter] = pkey; + +var command = IPA.command({ 'on_success': on_success, -'on_error': on_error +'on_error': on_error, +'method': that.other_entity+'_find', +'args': args, +options: options }); -for (var i=0; ipkeys.length; i++) { -var pkey = pkeys[i]; +command.execute(); -var command = IPA.command({ -'method': that.other_entity+'_show', -'args': [pkey], -'options': { -'all': true, -'rights': true -} -}); -batch.add_command(command); -} - -batch.execute(); }; that.refresh = function() { @@ -926,9 +932,9 @@ IPA.association_facet = function (spec) { that.get_records( pkeys, function(data, text_status, xhr) { -var results = data.result.results; +var results = data.result.result; for (var i=0; iresults.length; i++) { -var record = results[i].result; +var record = results[i]; that.table.add_record(record); } } diff --git a/install/ui/host.js b/install/ui/host.js index 981bfc021924bc740379009e62ef5b8345b262d5..adaf2c1cb003f1ff97d646eab236b8a8e9ff0025 100644 --- a/install/ui/host.js +++ b/install/ui/host.js @@ -50,7 +50,7 @@ IPA.entity_factories.host = function () { that.add_facet(facet); facet = IPA.host_managedby_host_facet({ -'name': 'managedby_host' +name: 'managedby_host' }); that.add_facet(facet); @@ -503,18 +503,12 @@ IPA.host_managedby_host_facet = function (spec) { }).appendTo(container); }; -that.create_column({name: 'description'}); - that.create_adder_column({ name: 'fqdn', primary_key: true, width: '100px' }); -that.create_adder_column({ -name: 'description', -width: '100px' -}); that.association_facet_init(); }; diff --git a/install/ui/service.js b/install/ui/service.js index ff642a808b9d1c6aebf02d7c9385c7a245cba1aa..ee3a8b2718f82fcc56a5e01cf0f92732d3650b02 100644 --- a/install/ui/service.js +++ b/install/ui/service.js @@ -395,7 +395,6 @@ IPA.service_managedby_host_facet = function(spec) { }).appendTo(container); }; -that.create_column({name: 'description'}); that.create_adder_column({ name: 'fqdn', @@ -403,11 +402,6 @@