Bug 14364

Summary: [Fix 21] kio - Display preview for locally mounted remote directories
Product: [ROSA-based products] ROSA Fresh Reporter: VictorR2007 <victorr2007>
Component: Packages from MainAssignee: ROSA Linux Bugs <bugs>
Status: VERIFIED FIXED QA Contact: ROSA Linux Bugs <bugs>
Severity: normal    
Priority: Normal CC: a.proklov, irgaliev01, m.novosyolov, pastordidi, pribytkovav02, sulmpx60, temap, v.potapov, victorr2007, vladislav.tarakanov
Version: Plasma5Flags: v.potapov: qa_verified+
a.proklov: published+
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Platform: --- ROSA Vulnerability identifier:
RPM Package: ISO-related:
Bad POT generating: Upstream:
Attachments: Папки

Description VictorR2007 2024-03-20 23:14:33 MSK
В баге на обновление пакетов, два пул реквеста к пакетам
kio и plasma5-kio-extras.
Изменение позволяет отображать превью для удаленных папок, которые были смонтированы вручную через mount (а не через kio-fuse).

По умолчанию превью для удаленных папок выключено.

Это настраиваемо:
в файле kdeglobals,
секция [PreviewSettings],
параметр EnableRemoteFolderThumbnail.

https://abf.io/import/kio/pull_requests/2
https://abf.io/import/plasma5-kio-extras/pull_requests/6

Сборки kio
https://abf.rosalinux.ru/build_lists/5007796
https://abf.rosalinux.ru/build_lists/5007797
https://abf.rosalinux.ru/build_lists/5007798

Сборки plasma5-kio-extras
https://abf.rosalinux.ru/build_lists/5007799
https://abf.rosalinux.ru/build_lists/5007800
https://abf.rosalinux.ru/build_lists/5007800
Comment 1 Dmitry Postnikov 2024-03-28 20:03:51 MSK
(In reply to VictorR2007 from comment #0)
 
> Это настраиваемо:
> в файле kdeglobals,
> секция [PreviewSettings],
> параметр EnableRemoteFolderThumbnail.
> 
Вот не вижу этих параметров в kdeglobals после обновления.
Comment 2 Sergey Katunin 2024-03-28 21:19:25 MSK
(In reply to Dmitry Postnikov from comment #1)
> (In reply to VictorR2007 from comment #0)
>  
> > Это настраиваемо:
> > в файле kdeglobals,
> > секция [PreviewSettings],
> > параметр EnableRemoteFolderThumbnail.
> > 
> Вот не вижу этих параметров в kdeglobals после обновления.

Его нужно создать вручную и установить в True, чтобы включить превью для удаленных папок. 

А отсутствие параметра аналогично значению false, т.е по умолчанию поведение прежнее - превью для удаленных папок не рисуется.
Comment 3 Dmitry Postnikov 2024-03-29 11:01:38 MSK
Я понял. Но у меня обратный эффект.
Раньше, в Дольфине в локальных папках был превью, и в смонтированных через mount тоже было превью.
После обнов, в Дольфине в локальных папках нет преью, и в смонтированных через mount тоже нет превью, даже с добавление этого параметра в true.
Comment 4 Sergey Katunin 2024-03-29 13:47:14 MSK
(In reply to Dmitry Postnikov from comment #3)
> Я понял. Но у меня обратный эффект.
> Раньше, в Дольфине в локальных папках был превью, и в смонтированных через
> mount тоже было превью.
> После обнов, в Дольфине в локальных папках нет преью, и в смонтированных
> через mount тоже нет превью, даже с добавление этого параметра в true.

После обнов в локальных папках нет превью? Вообще превью всего или превью папок? Это странно, обычные локальные папки не должны быть затронуты. Лимиты в Dolphin для отображения локальных превью выставлены корректно? Нужно видео с демонстрацией этого поведения и настройками.

По поводу того, что в смонтированных через mount нет превью для удаленных папок даже с добавлением параметра, возможно дело в лимитах для удаленных файлов (он по умолчанию стоит в 0, т.е не рисовать превью для удаленных):

https://forum.rosalinux.ru/download/file.php?id=13697&sid=0e567695b4e4db71140e4776a4f1c31e&mode=view
Comment 5 Dmitry Postnikov 2024-03-29 18:09:33 MSK
(In reply to Sergey Katunin from comment #4)
> (In reply to Dmitry Postnikov from comment #3)
> > Я понял. Но у меня обратный эффект.
> > Раньше, в Дольфине в локальных папках был превью, и в смонтированных через
> > mount тоже было превью.
> > После обнов, в Дольфине в локальных папках нет преью, и в смонтированных
> > через mount тоже нет превью, даже с добавление этого параметра в true.
> 
> После обнов в локальных папках нет превью? Вообще превью всего или превью
> папок? Это странно, обычные локальные папки не должны быть затронуты. Лимиты
> в Dolphin для отображения локальных превью выставлены корректно? Нужно видео
> с демонстрацией этого поведения и настройками.
> 
> По поводу того, что в смонтированных через mount нет превью для удаленных
> папок даже с добавлением параметра, возможно дело в лимитах для удаленных
> файлов (он по умолчанию стоит в 0, т.е не рисовать превью для удаленных):
> 
> https://forum.rosalinux.ru/download/file.
> php?id=13697&sid=0e567695b4e4db71140e4776a4f1c31e&mode=view

https://cloud.mail.ru/public/urnJ/4UMTC1EjL
Вот видяха. И локальные папки, нет превью, и другой раздел примонтированный через mount тоже не показывает превью. После отката , все показывается.
Comment 6 VictorR2007 2024-03-29 20:19:27 MSK
Created attachment 6073 [details]
Папки

(In reply to Dmitry Postnikov from comment #5)
> Вот видяха. И локальные папки, нет превью, и другой раздел примонтированный
> через mount тоже не показывает превью. После отката , все показывается.

Ты на видео совсем не то показываешь.
Не показываешь миниатюры файлов.
Ты показываешь миниатюры на папках.
Которые вообще только мешают работе.
Я наоборот их всегда отключаю.
Заявленная задача патчей в этом баге,
отображение качественных миниатюр файлов, не папок.
Comment 7 VictorR2007 2024-03-29 20:23:55 MSK
(In reply to VictorR2007 from comment #6)
> Ты на видео совсем не то показываешь.

Впрочем, больше не суюсь.
Ты сам знаешь, как нужно тестировать.
Просто не утерпел.
Сорри.:)
Comment 8 Sergey Katunin 2024-03-29 20:44:08 MSK
(In reply to VictorR2007 from comment #6)

> Заявленная задача патчей в этом баге,
> отображение качественных миниатюр файлов, не папок.

Нет нет, все правильно, в этом баге как раз патчи для миниатюр для папок.

(In reply to Dmitry Postnikov from comment #5)

> И локальные папки, нет превью...

Для локальных папок проблема обнаружена. В патче был изменен тип считываемого значения максимального размера из конфига с знакового на беззнаковый (чтобы больше соответствовать коду из основного проекта `KIO`), но сама переменная, в которую потом сохранялся прочитанный размер - осталась со знаковым типом. 

В итоге это приводило к переполнению и в итоге отсутствующий "MaximumSize" распознавался не как максимальный возможный размер для локальных файлов, а как `-1` - поэтому ничего и не отображалось для локальных папок. Если выставить любой другой лимит для локальных файлов, то превью начнут рисоваться.

Скоро отправлю патч в апстрим и в ABF. Но на сетевые папки это (смонтированные через mount) не должно влиять.

(In reply to Dmitry Postnikov from comment #5)

> и другой раздел примонтированный через mount тоже не показывает превью

А другой раздел - это диск? Или сетевая папка? Речь в этих изменениях именно про превью для удаленных сетевых папок, смонтированных через mount, т.е NFS, SMB, webdav и так далее.
Comment 9 VictorR2007 2024-03-29 20:52:59 MSK
(In reply to Sergey Katunin from comment #8)
> в этом баге как раз патчи для миниатюр для папок.
Неожиданно.
Comment 10 Sergey Katunin 2024-03-29 21:01:27 MSK
(In reply to Sergey Katunin from comment #8)

> Скоро отправлю патч в апстрим и в ABF.

https://abf.io/import/plasma5-kio-extras/pull_requests/7
https://invent.kde.org/network/kio-extras/-/merge_requests/342
Comment 11 VictorR2007 2024-03-29 21:19:04 MSK
(In reply to Sergey Katunin from comment #10)
> https://abf.io/import/plasma5-kio-extras/pull_requests/7
> https://invent.kde.org/network/kio-extras/-/merge_requests/342

Перенес сюда самое первое сообщение,
и заменил ссылки на сборки plasma5-kio-extras


Сборки kio
https://abf.rosalinux.ru/build_lists/5007796
https://abf.rosalinux.ru/build_lists/5007797
https://abf.rosalinux.ru/build_lists/5007798

Сборки plasma5-kio-extras
https://abf.io/build_lists/5011050
https://abf.io/build_lists/5011051
https://abf.io/build_lists/5011052
Comment 12 Dmitry Postnikov 2024-04-01 09:48:55 MSK
(In reply to VictorR2007 from comment #11)
> (In reply to Sergey Katunin from comment #10)
> > https://abf.io/import/plasma5-kio-extras/pull_requests/7
> > https://invent.kde.org/network/kio-extras/-/merge_requests/342
> 
> Перенес сюда самое первое сообщение,
> и заменил ссылки на сборки plasma5-kio-extras
> 
> 
> Сборки kio
> https://abf.rosalinux.ru/build_lists/5007796
> https://abf.rosalinux.ru/build_lists/5007797
> https://abf.rosalinux.ru/build_lists/5007798
> 
> Сборки plasma5-kio-extras
> https://abf.io/build_lists/5011050
> https://abf.io/build_lists/5011051
> https://abf.io/build_lists/5011052

Видяха https://cloud.mail.ru/public/tWWn/ioELmbEAj
Что-то не получается предпросмотр удаленных шар по smb , все через mount подцеплено. Но уже получше, локальные папки остались с превьюшкой, и на другом диске тоже с превьюшкой. Но в smb шаре так и не заработала превьюшка.
Comment 13 Sergey Katunin 2024-04-01 15:04:39 MSK
(In reply to Dmitry Postnikov from comment #12)

> Видяха https://cloud.mail.ru/public/tWWn/ioELmbEAj
> Что-то не получается предпросмотр удаленных шар по smb , все через mount
> подцеплено. Но уже получше, локальные папки остались с превьюшкой, и на
> другом диске тоже с превьюшкой. Но в smb шаре так и не заработала превьюшка.

На видео не видно настроек (в Dolphin) лимитов максимального размера сетевого файла для создания миниатюр. Этот параметр точно не 0?
Comment 14 Dmitry Postnikov 2024-04-01 15:07:32 MSK
(In reply to Sergey Katunin from comment #13)
> (In reply to Dmitry Postnikov from comment #12)
> 
> > Видяха https://cloud.mail.ru/public/tWWn/ioELmbEAj
> > Что-то не получается предпросмотр удаленных шар по smb , все через mount
> > подцеплено. Но уже получше, локальные папки остались с превьюшкой, и на
> > другом диске тоже с превьюшкой. Но в smb шаре так и не заработала превьюшка.
> 
> На видео не видно настроек (в Dolphin) лимитов максимального размера
> сетевого файла для создания миниатюр. Этот параметр точно не 0?

Может и 0. У меня все по дефолту. Тестовая система должна быть по дефолту. Для ломания есть другие.
Даже если и 0, в локальных папках и на другом диске все превьюшки ведь видны.
Comment 15 Sergey Katunin 2024-04-01 15:31:43 MSK
(In reply to Dmitry Postnikov from comment #14)

> Может и 0. У меня все по дефолту. Тестовая система должна быть по дефолту.
> Для ломания есть другие.

Не особо понял к чему это. В системе (до этого патча) были и есть параметры конфигурации, которые позволяют менять поведение в зависимости от параметров. В данном случае, это лимиты на размер файлов для построения превью. По умолчанию настроено так, что превью для локальных файлов - строятся, т.е лимита для локальных файлов нет, а превью для сетевых удаленных файлов - не строятся, т.е лимит установлен на 0. Сделано это для того, что чаще превью для удаленных систем не нужны, чем наоборот, а также это экономит сеть, ресурсы компьютера и прочее. Но при этом у пользователя есть возможность изменить эти параметры, чтобы настроить систему так, как он хочет. А именно, настроить лимит для сетевых файлов чтобы видеть превью для сетевых файлов.

Соответственно, для тестирования таких недефолтных возможностей все же параметры изменить необходимо.

В данном обсуждаемом изменении была добавлена возможность отрисовки превью для удаленных папок (которые смонтированы вручную через mount), чего никогда в KDE не было (по крайней мере последние года).

Следовательно, для проверки этой возможности необходимо:
а) настроить лимиты для сетевых файлов, чтобы превью отображались для сетевых файлов
б) установить параметр `EnableRemoteFolderThumbnail` чтобы отображались превью для сетевых папок

Без настроек лимитов для сетевых файлов, очевидно, превью для сетевых папок тоже не появятся, поскольку сетевая папка - это тоже сетевой файл. Таким образом, это является необходимым (но недостаточным) критерием для проверки обсуждаемой возможности.



(In reply to Dmitry Postnikov from comment #14)

> Даже если и 0, в локальных папках и на другом диске все превьюшки ведь видны.

Потому-что это отдельный лимит. В настройках два лимита: для локальных файлов и для сетевых. Для локальных по умолчанию лимита нет (т.е он не 0), а для сетевых файлов - он равен 0. 

Причем в этой теме я уже упоминал и про необходимость настройки лимитов, и прикладывал ссылку на скриншот с тем, где они находятся и как выглядят: https://forum.rosalinux.ru/download/file.php?id=13697&sid=0e567695b4e4db71140e4776a4f1c31e&mode=view
Comment 16 Dmitry Postnikov 2024-04-01 16:19:41 MSK
Видяха https://cloud.mail.ru/public/UWHf/yYG5Jifhc
После обновления, поставил лимит 5Мб. Все равно обе шары smb не показывают превью. Одна smb на Дебиан, другоая на Винде.
Comment 17 Dmitry Postnikov 2024-04-01 16:26:46 MSK
Видяха https://cloud.mail.ru/public/rrnP/miHeRseZC
Заработало! Ребут помог. Видимо в kdeglobals плазма подхватывает после ребута, т.к. после перелогивания не сработало. А ребут помог. Может что-то закешировалось.
В общем работает.
Comment 18 Dmitry Postnikov 2024-04-01 16:31:26 MSK
(In reply to VictorR2007 from comment #11)
> (In reply to Sergey Katunin from comment #10)
> > https://abf.io/import/plasma5-kio-extras/pull_requests/7
> > https://invent.kde.org/network/kio-extras/-/merge_requests/342
> 
> Перенес сюда самое первое сообщение,
> и заменил ссылки на сборки plasma5-kio-extras
> 
> 
> Сборки kio
> https://abf.rosalinux.ru/build_lists/5007796
> https://abf.rosalinux.ru/build_lists/5007797
> https://abf.rosalinux.ru/build_lists/5007798
> 
> Сборки plasma5-kio-extras
> https://abf.io/build_lists/5011050
> https://abf.io/build_lists/5011051
> https://abf.io/build_lists/5011052

**********************************
Обновление опубликовано в Тестинг
Comment 19 Vladimir Potapov 2024-04-10 12:37:06 MSK
kio-5.112.0-3
https://abf.rosalinux.ru/build_lists/5007796
https://abf.rosalinux.ru/build_lists/5007797
https://abf.rosalinux.ru/build_lists/5007798

plasma5-kio-extras-23.08.4-7
https://abf.io/build_lists/5011050
https://abf.io/build_lists/5011051
https://abf.io/build_lists/5011052
************************** Advisory ****************************
Preview for mounted foldres added
****************************************************************
QA Verified