[Freeipa-devel] [PATCH] 2 Addressing issues found in SUDO schema

2010-09-24 Thread Dmitri Pal
$Subj JR, please give it a try. I will update the page accordingly... -- Thank you, Dmitri Pal Engineering Manager IPA project, Red Hat Inc. --- Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From d016c7c9d7f548550e14000f62741abf981c6529 Mon Sep 17 0

Re: [Freeipa-devel] [PATCH] 543 fix whoami plugin

2010-09-24 Thread Adam Young
On 09/23/2010 10:38 PM, Rob Crittenden wrote: Use the principal from the request context. ipalib.util.get_current_principal() is designed to work from a client, not within the server. I should have noticed this earlier. rob ___ Freeipa-devel mailin

Re: [Freeipa-devel] [PATCH] 543 fix whoami plugin

2010-09-24 Thread Adam Young
On 09/23/2010 10:38 PM, Rob Crittenden wrote: Use the principal from the request context. ipalib.util.get_current_principal() is designed to work from a client, not within the server. I should have noticed this earlier. rob ___ Freeipa-devel mailin

Re: [Freeipa-devel] [PATCH] 536 throw and catch CertificateOperationError properly in replica preparer

2010-09-24 Thread Adam Young
On 09/20/2010 01:38 PM, Rob Crittenden wrote: Properly handle CertificateOperationErrors in replication prepration. The problem here was two-fold: the certs manager was raising an error it didn't know about and ipa-replica-prepare wasn't catching it. rob

Re: [Freeipa-devel] [PATCH] 536 throw and catch CertificateOperationError properly in replica preparer

2010-09-24 Thread Adam Young
On 09/20/2010 01:38 PM, Rob Crittenden wrote: Properly handle CertificateOperationErrors in replication prepration. The problem here was two-fold: the certs manager was raising an error it didn't know about and ipa-replica-prepare wasn't catching it. rob

Re: [Freeipa-devel] [PATCH] Restoring Services tab

2010-09-24 Thread Adam Young
On 09/17/2010 03:55 PM, Adam Young wrote: On 09/17/2010 03:41 PM, Endi Sukma Dewata wrote: Hi, This patch is based on Adam's bbq-tabs-modal branch. The add.js has been modified to support adding new entry with dynamically generated pkey. The index.xhtml has been modified to include service.js

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0014-remove-user-html

2010-09-24 Thread Adam Young
On 08/23/2010 12:05 PM, Adam Young wrote: This file has not been used for a long time. I wanted to remove it in its own patch. ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel pushed to m

Re: [Freeipa-devel] [PATCH] 2 Addressing issues found in SUDO schema

2010-09-24 Thread JR Aquino
Looks like there was a typo in the hostMask. The others were correct. -attributeTypes: (2.16.840.1.113730.3.8.7.11 NAME 'hostMask' DESC 'IP mask to identify a subnet.' EQUALITY caseIgnoreIA5Match ORDERING caseIgnoreMatch SUBSTR caseIgnoreSubstringsMatch SYNTAX 1.3.6.1.4.1.1466.115.121.1.15 X-ORI

[Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Adam Young
Whoami link The 'logged in as' message in the header into an active hypoerlink that loads the details page for the current user. Also fixed a bug where, when reloading, the search page would fail due to scl being undefined ___ Freeipa-devel m

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Adam Young
On 09/24/2010 10:53 AM, Adam Young wrote: Whoami link The 'logged in as' message in the header into an active hypoerlink that loads the details page for the current user. Also fixed a bug where, when reloading, the search page would fail due to scl being undefined

Re: [Freeipa-devel] [PATCH] 2 Addressing issues found in SUDO schema

2010-09-24 Thread Dmitri Pal
JR Aquino wrote: > Looks like there was a typo in the hostMask. The others were correct. > > -attributeTypes: (2.16.840.1.113730.3.8.7.11 NAME 'hostMask' DESC 'IP mask to > identify a subnet.' EQUALITY caseIgnoreIA5Match ORDERING caseIgnoreMatch > SUBSTR caseIgnoreSubstringsMatch SYNTAX 1.3.6.1.4

Re: [Freeipa-devel] [PATCH] 2 Addressing issues found in SUDO schema

2010-09-24 Thread Dmitri Pal
Dmitri Pal wrote: > JR Aquino wrote: > >> Looks like there was a typo in the hostMask. The others were correct. >> >> -attributeTypes: (2.16.840.1.113730.3.8.7.11 NAME 'hostMask' DESC 'IP mask >> to identify a subnet.' EQUALITY caseIgnoreIA5Match ORDERING caseIgnoreMatch >> SUBSTR caseIgnoreSu

Re: [Freeipa-devel] [PATCH] 2 Addressing issues found in SUDO schema

2010-09-24 Thread JR Aquino
Everything appears to be working now. Thanks Dmitri! ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH] 544 add import to automount

2010-09-24 Thread Rob Crittenden
Add ability to import automount files from the command-line. Support is fairly basic right now and will only work on the CLI. All the work is done on the client side. To continue past errors use the --continue option. Fixed a bug where direct mounts weren't always added properly. Added real

Re: [Freeipa-devel] [PATCH] 536 throw and catch CertificateOperationError properly in replica preparer

2010-09-24 Thread Rob Crittenden
Adam Young wrote: On 09/20/2010 01:38 PM, Rob Crittenden wrote: Properly handle CertificateOperationErrors in replication prepration. The problem here was two-fold: the certs manager was raising an error it didn't know about and ipa-replica-prepare wasn't catching it. rob ___

Re: [Freeipa-devel] [PATCH] 541 convert ca_serialno

2010-09-24 Thread Rob Crittenden
Adam Young wrote: On 09/23/2010 12:17 PM, Rob Crittenden wrote: The file format of ca_serialno changed between v1 and v2. We won't support direct upgrades of v1 to v2 but as a developer I install them both from time to time and its annoying to have the v2 install fail because of this file. This

Re: [Freeipa-devel] [PATCH] 542 remove bogus uninstall error

2010-09-24 Thread Rob Crittenden
Adam Young wrote: On 09/23/2010 12:18 PM, Rob Crittenden wrote: Remove spurious error in server uninstaller about client uninstall failure. This was meant to catch the case where the client wasn't configured and it missed the most obvious one: the client was installed and is now uninstalled. r

[Freeipa-devel] [PATCH] 545 ignore success exception

2010-09-24 Thread Rob Crittenden
A call to search_ext() in ipa-replica-prepare was returning the exception ldap.SUCCESS. We actually got the right data back but this exception was confusing things. It should be ignored. rob freeipa-545-success.patch Description: application/mbox __

Re: [Freeipa-devel] [PATCH] 522 add some ldap2 Backend tests

2010-09-24 Thread Rob Crittenden
Adam Young wrote: On 09/09/2010 04:38 PM, Rob Crittenden wrote: Add some tests for using the ldap2 Backend. Fix a logic problem in ldap2:get_schema() for determining if it can fetch the schema or not. Normally we only want to do this for servers but if you pass in your own connection it will us

Re: [Freeipa-devel] [PATCH] 2 Addressing issues found in SUDO schema

2010-09-24 Thread Rob Crittenden
JR Aquino wrote: Everything appears to be working now. Thanks Dmitri! Ack here too. Pushed to master. rob ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] UI mockups for DNS, AUtomount, and Password Policy

2010-09-24 Thread Adam Young
I'm going to post these for comments before I sink too much more time into them: I've mocked up the pages under policy. http://admiyo.fedorapeople.org/ipa/jquery.ui/ Note: The default is the Dertails page of DNS. THis is just a develpoment thing, not something that will be part of the end p

Re: [Freeipa-devel] [PATCH] 545 ignore success exception

2010-09-24 Thread Adam Young
On 09/24/2010 03:34 PM, Rob Crittenden wrote: A call to search_ext() in ipa-replica-prepare was returning the exception ldap.SUCCESS. We actually got the right data back but this exception was confusing things. It should be ignored. rob ___ Freeipa

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Endi Sukma Dewata
- "Adam Young" wrote: > On 09/24/2010 10:53 AM, Adam Young wrote: > > Whoami link > > The 'logged in as' message in the header into an active hypoerlink > > that loads the details page for the current user. > > Also fixed a bug where, when reloading, the search page would fail > > d

Re: [Freeipa-devel] [PATCH] Test suite for ipa.js and entity.js.

2010-09-24 Thread Adam Young
On 09/24/2010 06:34 PM, Endi Sukma Dewata wrote: Hi, The patch has been modified based on Adam's feedback. Thanks! Test framework for Web UI has been created using qUnit. The test files are located in install/static/test. The main page is index.html which contains links to all test suites (xxx_

Re: [Freeipa-devel] [PATCH] Test suite for ipa.js and entity.js.

2010-09-24 Thread Adam Young
On 09/24/2010 07:49 PM, Adam Young wrote: On 09/24/2010 06:34 PM, Endi Sukma Dewata wrote: Hi, The patch has been modified based on Adam's feedback. Thanks! Test framework for Web UI has been created using qUnit. The test files are located in install/static/test. The main page is index.html wh

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Adam Young
On 09/24/2010 06:57 PM, Endi Sukma Dewata wrote: - "Adam Young" wrote: On 09/24/2010 10:53 AM, Adam Young wrote: Whoami link The 'logged in as' message in the header into an active hypoerlink that loads the details page for the current user. Also fixed a bug where, whe

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Adam Young
On 09/24/2010 07:57 PM, Adam Young wrote: On 09/24/2010 06:57 PM, Endi Sukma Dewata wrote: - "Adam Young" wrote: On 09/24/2010 10:53 AM, Adam Young wrote: Whoami link The 'logged in as' message in the header into an active hypoerlink that loads the details page for the current use

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Endi Sukma Dewata
- "Adam Young" wrote: > >>> Whoami link > >>> The 'logged in as' message in the header into an active hypoerlink > >>> that loads the details page for the current user. > >>> Also fixed a bug where, when reloading, the search page would fail > >>> due to scl being undefined > >>>

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0043-Whoami-link.patch

2010-09-24 Thread Endi Sukma Dewata
- "Adam Young" wrote: > Fixed admiyo-freeipa-0043-3-Whoami-link.patch ACK, sorry didn't see this one before replying. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel