Bug 3779

Summary: Проверить etermount из пакета etercifs
Product: CIFS@Etersoft Reporter: Vitaly Lipatov <lav>
Component: прочееAssignee: Elena V. Gurevich <lbeasty>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: baraka, kipruss, sin
Version: не указана   
Target Milestone: ---   
Hardware: PC   
OS: All   
Whiteboard:
Заявки RT: Связано с:
Дата напоминания:
Bug Depends on:    
Bug Blocks: 1217    

Description Vitaly Lipatov 2009-04-04 16:06:39 MSD
В пакете etercifs появилась команда etermount.
Предлагаю
1. Описать её в документации (Костя)
2. Составить план тестирования (Лена)
3. Проверить и сообщить о найденных недочётах (Лена)
4. Сообщить, что etermount работает (Денис)
Comment 1 Konstantin Baev 2009-04-05 01:34:30 MSD
(In reply to comment #0)
> В пакете etercifs появилась команда etermount.
> Предлагаю
> 1. Описать её в документации (Костя)

Описал. См. http://git.etersoft.ru/people/kipruss/packages/?p=wine-etersoft-docs.git;a=commitdiff;h=1bfc01bc2bad634d7858d68f9c5f7b1039e4ce24

Обращаю внимание, что предыдущий коммит не был пока взят. то есть надо брать оба.

Ещё я не указал, с какой версии появился скрипт etermount. Наверное, стоит написать, что с 4.3.3, чтобы никого не путать релизом alt2.

Также я закомментировал фразу про case sensitive для Ubuntu 8.04/8.10, так как это уже решено как у нас, так и в самой Убунте (в 9.04 по крайней мере обещано), а ставить явно этот параметр вредно.

Также есть подобное описание в пакете etersoft-docs, ярлык от которого висит на рабочем столе нашего дистра.
Comment 2 Elena V. Gurevich 2009-04-07 14:55:52 MSD
Тесты прошли. Долго чтото мучалась, отрывая указание опций монтирования в командной строке. В общем случае запуск выглядит как 
rect --Mount.Root='/tmp/rect' --Mount.Command='/usr/sbin/etermount' --Mount.Suffix='' --Mount.Umount='sudo cifsumount'
Comment 3 Денис Баранов 2009-04-07 14:58:14 MSD
Лена, сделай пож план: что должно работать и работает ли.
Comment 4 Elena V. Gurevich 2009-04-07 14:58:37 MSD
*мучилась
Comment 5 Elena V. Gurevich 2009-04-07 15:06:08 MSD
Ээм. Не совсем поняла. О чем план сделать?
Comment 6 Денис Баранов 2009-04-07 15:28:03 MSD
(In reply to comment #5)
> Ээм. Не совсем поняла. О чем план сделать?
> 
Все, отбой. Не то имел ввиду.-)

(In reply to comment #1)
> Описал. См.
> http://git.etersoft.ru/people/kipruss/packages/?p=wine-etersoft-docs.git;a=commitdiff;h=1bfc01bc2bad634d7858d68f9c5f7b1039e4ce24

Наверное туда стоит еще описать про то где и как задаются опции монтирования?!
Comment 7 Денис Баранов 2009-04-14 20:07:15 MSD
Проверил.
Из замечаний:
1) Слишком моного выводится всего ненужного в консоль при неудачном монтировании:
etermount //win-test/baraka /mnt/123
Info: the folder /mnt/123 exists and empty
Password:
Usage: mount -V                 : print version
       mount -h                 : print this help
       mount                    : list mounted filesystems
       mount -l                 : idem, including volume labels
So far the informational part. Next the mounting.
The command is `mount [-t fstype] something somewhere'.
Details found in /etc/fstab may be omitted.
       mount -a [-t|-O] ...     : mount all stuff from /etc/fstab
       mount device             : mount device at the known place
       mount directory          : mount known device here
       mount -t type dev dir    : ordinary mount command
Note that one does not really mount a device, one mounts
a filesystem (of the given type) found on the device.
One can also mount an already visible directory tree elsewhere:
       mount --bind olddir newdir
or move a subtree:
       mount --move olddir newdir
One can change the type of mount containing the directory dir:
       mount --make-shared dir
       mount --make-slave dir
       mount --make-private dir
       mount --make-unbindable dir
One can change the type of all the mounts in a mount subtree
containing the directory dir:
       mount --make-rshared dir
       mount --make-rslave dir
       mount --make-rprivate dir
       mount --make-runbindable dir
A device can be given by name, say /dev/hda1 or /dev/cdrom,
or by label, using  -L label  or by uuid, using  -U uuid .
Other options: [-nfFrsvw] [-o options] [-p passwdfd].
For many more details, say  man 8 mount .
Warning: error while mount of share //win-test/baraka in mountpoint /mnt/123!
Почему нельзя сделать только 1,2,3 и последнюю строчку!?
(Пояснение: при монтировании был неуказан пароль пользователя)

2) Виталик уже писал: etermount -h (etermount --help) не реализованы.
Comment 8 Konstantin Baev 2009-04-15 02:11:59 MSD
(In reply to comment #7)
> Проверил.
> Из замечаний:
> 1) Слишком моного выводится всего ненужного
> в консоль при неудачном монтировании:

Это не я, это mount.  Но он тоже больше не будет ругаться.

> Почему нельзя сделать только 1,2,3 и
> последнюю строчку!?

Можно и нужно. Но для этого я должен узнать об этом. Для этого и существует тестирование. Вот, протестировали сегодня, сказали - я сделал.

> 
> 2) Виталик уже писал: etermount -h (etermount --help) не
> реализованы.
> 

Где писал? Я или не видел или пропустил.

Сделал --help.

см. http://git.etersoft.ru/people/kipruss/packages/?p=etercifs.git;a=commitdiff;h=391aadd227d22d21b6ee2baf58d5adb64fae169a

Пакетить пока не буду - завтра все равно выпускать версию с исправлением 3807.
Если вдруг кому-то захочется срочно проверить, то можно внести изменения прямо в установленный /usr/sbin/etermount руками, там их немного.
Comment 9 Vitaly Lipatov 2009-04-15 13:18:51 MSD
(In reply to comment #8)
> > 2) Виталик уже писал: etermount -h (etermount --help) не
> > реализованы.
> > 
> 
> Где писал? Я или не видел или пропустил.
Не дошло в джаббере. Исправлюсь, буду писать в баги :)

[18:42:47] <lav> Тем не менее не ясно, зачем проверять на пустой каталог
# etermount //server/tmp /net/cifs
Error: the folder /net/cifs is not empty!

И так реагировать на запрос справки:
# etermount -h
Info: the folder /net/sharebase was created
/usr/sbin/etermount: line 46: Usage:: command not found
Warning: error while mount of share -h in mountpoint /net/sharebase
[18:43:33] <lav> А главное что из etermount не узнать, с какими параметрами он монтирует. Возможно это стоит выводить в справке
Comment 10 Konstantin Baev 2009-04-15 15:24:19 MSD
(In reply to comment #9)
> (In reply to comment #8)
> > > 2) Виталик уже писал: etermount -h (etermount --help) не
> > > реализованы.
> > > 
> > 
> > Где писал? Я или не видел или пропустил.
> Не дошло в джаббере. Исправлюсь, буду
> писать в баги :)
> 
> [18:42:47] <lav> Тем не менее не ясно, зачем
> проверять на пустой каталог
> # etermount //server/tmp /net/cifs
> Error: the folder /net/cifs is not empty!

Я считаю, что непустой каталог не может быть точкой монтирования. Если я ошибаюсь, то переделаю - не вопрос.

> И так реагировать на запрос справки:
> # etermount -h
> Info: the folder /net/sharebase was created
> /usr/sbin/etermount: line 46: Usage:: command not found
> Warning: error while mount of share -h in mountpoint /net/sharebase

С этим уже все хорошо.

> [18:43:33] <lav> А главное что из etermount не узнать, с
> какими параметрами он монтирует. Возможно
> это стоит выводить в справке
> 

С этим уже все хорошо.
Comment 11 Konstantin Baev 2009-04-15 15:27:08 MSD
Посмотрел в истории копыта - да, действительно, были сообщения. Но у меня копыто падало.

> И монтировать в /net/sharebase по умолчанию

Это могу оторвать. Это для ленивых.

direct будет
Comment 12 Vitaly Lipatov 2009-04-15 15:50:49 MSD
(In reply to comment #10)
> > [18:42:47] <lav> Тем не менее не ясно, зачем
> > проверять на пустой каталог
> > # etermount //server/tmp /net/cifs
> > Error: the folder /net/cifs is not empty!
> 
> Я считаю, что непустой каталог не может
> быть точкой монтирования. Если я ошибаюсь,
> то переделаю - не вопрос.
Смотря в каком смысле не может быть.
В принципе, обычно каталог монтирования пустой...
Я думаю, что etermount
- да, всё же не должен монтировать в непустой каталог
- не должен создавать целевой каталог, пусть он уже существует, это нормально.
- в нашем дистрибутиве /net/sharebase лучше уже иметь заранее


Ждём тогда сборку с direct в etermount и мелкими исправлениями?
Желательно без изменения кода, чтобы это был точно работающий
модуль, который мы уже вчера проверяли.

Comment 13 Konstantin Baev 2009-04-15 17:57:00 MSD
> Ждём тогда сборку с direct в etermount и мелкими
> исправлениями?
> Желательно без изменения кода, чтобы это
> был точно работающий
> модуль, который мы уже вчера проверяли.
> 

Можно тестить:

ftp://ftp.etersoft.ru/pub/Etersoft/LINUX@Etersoft/Boxes/etercifs/noarch/RPMS.default/etercifs-4.3.6-alt1.noarch.rpm

Это типа релиз. Без нововведений. Изменения: Убран код, вызывающий ошибку 3807, вернул direct и дописал в etermount рекомендованные вещи.
Comment 14 Konstantin Baev 2009-04-15 18:10:11 MSD
(In reply to comment #12)
> - не должен создавать целевой каталог,
> пусть он уже существует, это нормально.

Покамест он все же создаёт каталог. Если это правда надо оторвать, то я это сделаю потом, в следующем релизе.
Comment 15 Евгений Синельников 2009-04-16 17:23:56 MSD
В число стандартных опций etermount входит user=guest. Правильнее просто guest. Но даже этот вариант ущербен. Нужно уметь ходить не только гостём, а то получается, что работать можно только открытой всем на запись базой... Вообще в 1С хранится конфиденциальная информация обычно...
Comment 16 Vitaly Lipatov 2009-04-16 17:30:05 MSD
(In reply to comment #15)
> В число стандартных опций etermount входит
> user=guest. Правильнее просто guest. Но даже этот
> вариант ущербен. Нужно уметь ходить не
> только гостём, а то получается, что
> работать можно только открытой всем на
> запись базой... Вообще в 1С хранится
> конфиденциальная информация обычно...
Тот, кто хранит конфиденциальную информацию, сможет смонтировать и без
этого скрипта.
Этот вариант - максимально простой, когда мы на винде ставим галочку "расшарить", а здесь - монтируем. Всё остальное - подробности.

Comment 17 Денис Баранов 2009-04-22 18:54:32 MSD
etermount протестирован и принят.