Bug 9225

Summary: smbldap-populate неработоспособен
Product: Sisyphus Reporter: Eugene V. Horohorin <genix>
Component: smbldap-toolsAssignee: Eugene V. Horohorin <genix>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: alex_sinister, eostapets, mike
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
исправленный патч (убраны пробелы) none

Description Eugene V. Horohorin 2006-03-10 18:58:00 MSK
smbldap-populate из комплекта smbldap-tools не работает так как надо.
вернее он никак не работает, потому что то что он пытается добавить в базу LDAP
не добавляется с ошибками.

Например, если заэкспортировать данные которые пытается добавить -populate в
файл, а потом добавить в базу получаем следующее:

ldapadd -f 1.ldif -h localhost -p 389 -x -D cn=adm,dc=xxx,dc=ru -W
Enter LDAP Password:
adding new entry "dc=xxx,dc=ru"

adding new entry "ou=Users,dc=xxx,dc=ru"

adding new entry "ou=Groups,dc=xxx,dc=ru"

adding new entry "ou=Computers,dc=xxx,dc=ru"

adding new entry "ou=Idmap,dc=oct,dc=xxx,dc=ru"

adding new entry "uid=root,ou=Users,dc=oct,dc=xxx,dc=ru"
ldapadd: update failed: uid=root,ou=Users,dc=oct,dc=xxx,dc=ru
ldap_add: Invalid syntax (21)
        additional info: objectClass: value #3 invalid per syntax

в ходе ldif видим, что после
objectClass: shadowAccount
objectClass: sambaGroupMapping

присутствует лишний пробел в конце строки, удаление которого делает ldif-файл
корректны. Е.Остпец утверрждает, что это из-за того, что там определена лишняя
(неиспользуемая) переменная.

Также требует правки и другие утилиты из комплекта smbldap-tools.

P.$. Евгений когда-то уже исправлял эти баги, но, к сожалению, изменения потерялись.
Comment 1 Eugene V. Horohorin 2006-03-10 18:59:41 MSK
s/Остпец/Остапец/
Comment 2 Eugene V. Horohorin 2006-03-10 19:06:15 MSK
всё это запускалось на Master2.4 в которм был установлен сизифовский
smbldap-tools-0.9.1-alt4.1 и perl-Crypt-SmbHash-0.02-alt1.1
Comment 3 Eugene V. Horohorin 2006-05-27 14:08:07 MSD
причина оказалась в патче smbldap-inplace_idmap-gns.patch

маленькая цитата из него:

+my $appobjectclass="";
+if (defined $config{idmapinplace}) {
+                   $appobjectclass="\nobjectClass: sambaIdmapEntry";
+} else {
+                       $appobjectclass="";
+}

и дальше:

-objectClass: shadowAccount
+objectClass: shadowAccount $appobjectclass

Вопрос -- зачем стоит пробел между shadowAccount и $appobjectclass, если
последний начинается с "/n" или принимает значение ""?????
Comment 4 Nick S. Grechukh 2006-05-29 14:29:42 MSD
если Вы готовы сделать патч - я добавлю add nmu genix
Comment 5 Eugene V. Horohorin 2006-05-29 14:35:09 MSD
(In reply to comment #4)
> если Вы готовы сделать патч - я добавлю add nmu genix

пробелы в  smbldap-inplace_idmap-gns.patch
 поудалять? могу
Comment 6 Eugene V. Horohorin 2006-05-29 14:37:11 MSD
Created attachment 1506 [details]
исправленный патч (убраны пробелы)

исправленная версия патча (убрал пробелы перед $appobjects)
Comment 7 Nick S. Grechukh 2006-05-29 14:42:14 MSD
можете заливать в инкоминг, ~ через час робот примет к сведению:

$ cat > incomingnote << EOF
smbldap-tools nmu genix
EOF
$rsync --inplace incomingnote incoming:/incoming/notes/Sisyphus/gns
$

Comment 8 Nick S. Grechukh 2006-05-29 15:17:36 MSD
Dear Nick S. Grechukh !
The initial list of commands:
 1 smbldap-tools nmu genix

The status of performance:
 1 PASS: nmu smbldap-tools genix => 2006-05-29 15:10:06


заливайте :)
Comment 9 Eugene V. Horohorin 2006-05-29 15:28:16 MSD
(In reply to comment #8)

> заливайте :)

признаюсь честно, фишку с 0.9.1-%{release} не осилил (не столько я, сколько
add_changelog), подправил руками

Comment 10 Eugene V. Horohorin 2006-06-19 13:19:03 MSD
уже в сизифе