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 @@
