Bug 40557 - Не работает совсем с новым python3-module-samba
Summary: Не работает совсем с новым python3-module-samba
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: netcmdplus (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 blocker
Assignee: manowar@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 40398
  Show dependency tree
 
Reported: 2021-07-21 15:10 MSK by Anton V. Boyarshinov
Modified: 2021-07-21 19:04 MSK (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anton V. Boyarshinov 2021-07-21 15:10:17 MSK
$ samba-tool-plus 
Traceback (most recent call last):
  File "/usr/bin/samba-tool-plus", line 33, in <module>
    from netcmdplus.main import cmd_sambatoolplus
  File "/usr/lib/python3/site-packages/netcmdplus/main.py", line 26, in <module>
    from netcmdplus.user import cmd_userplus
  File "/usr/lib/python3/site-packages/netcmdplus/user.py", line 30, in <module>
    from samba.netcmd.user import ( cmd_user, cmd_user_create )
ImportError: cannot import name 'cmd_user_create' from 'samba.netcmd.user' (/usr/lib64/python3/site-packages/samba/netcmd/user.py)

Проблема является фатальной для alt-server в режиме сервера AD с управлением через альтератор
$ rpm -qf /usr/lib64/python3/site-packages/samba/netcmd/user.py
python3-module-samba-4.14.5-alt1.x86_64
Comment 1 Evgeny Sinelnikov 2021-07-21 18:03:34 MSK
Ждём:
#279901 PENDING #2 sisyphus netcmdplus.git=0.1.2-alt1
Comment 2 Evgeny Sinelnikov 2021-07-21 18:06:35 MSK
Следом сразу исправление в бранчи:
#279903 POSTPONED #1 c9f2 netcmdplus.git=0.1.2-alt1
#279902 POSTPONED #1 p9 netcmdplus.git=0.1.2-alt1
#279901 PENDING #2 sisyphus netcmdplus.git=0.1.2-alt1
Comment 3 manowar@altlinux.org 2021-07-21 18:15:56 MSK
(Ответ для Evgeny Sinelnikov на комментарий #1)
> Ждём:
> #279901 PENDING #2 sisyphus netcmdplus.git=0.1.2-alt1

А может быть

    class cmd_userplus_update(cmd_user.subcommands["create"])

? На случай, если снова переименуют?
Comment 4 manowar@altlinux.org 2021-07-21 18:18:27 MSK
Вообще, было бы интересно узнать у разработчиков Самба: почему переезд с user_create на user_add был плавным:

-    # take this print out after the add subcommand is removed.
-    # the add subcommand is deprecated but left in for now to allow people to
-    # migrate to create
-
-    def run(self, *args, **kwargs):
-        self.outf.write(
-            "Note: samba-tool user add is deprecated.  "
-            "Please use samba-tool user create for the same function.\n")
-        return super(cmd_user_add, self).run(*args, **kwargs)

А об обратном переезде с add на create они не позаботились?
Comment 5 Repository Robot 2021-07-21 18:21:24 MSK
netcmdplus-0.1.2-alt1 -> sisyphus:

 Wed Jul 21 2021 Evgeny Sinelnikov <sin@altlinux> 0.1.2-alt1
 - Fix using obsoleted in samba-4.14 cmd_user_create class with renamed
   cmd_user_add class (closes: 40557)
 - Add conflicts with old samba package provided python3(samba.netcmd.user)
Comment 6 Evgeny Sinelnikov 2021-07-21 18:24:39 MSK
(Ответ для manowar@altlinux.org на комментарий #4)
[...]
> А об обратном переезде с add на create они не позаботились?

Если быть честными до конца, то эту часть интерфейса использовать было как-то опрометчиво. Я бы не стал. Они никому её не предоставляляли в качестве публичного API. Это внутренний интерфейс samba-tool.

Более того, предлагаю рассмотреть детально то, что внесено в netcmdplus, разбить на патчи и начать пробрасывать в апстрим.
Comment 7 Evgeny Sinelnikov 2021-07-21 18:25:22 MSK
(Ответ для manowar@altlinux.org на комментарий #3)
[...]
> ? На случай, если снова переименуют?

Я думаю, что не стоит переусложнять. Это раз.

И готовится нужно совсем к другому. Это два:
https://download.samba.org/pub/samba/rc/samba-4.15.0rc1.WHATSNEW.txt

samba-tool available without the ad-dc
--------------------------------------

The samba-tool command is now available when samba is configured
--without-ad-dc. Not all features will work, and some ad-dc specific options
have been disabled. The samba-tool domain options, for example, are limited
when no ad-dc is present. Samba must still be built with ads in order to enable
samba-tool.
Comment 8 manowar@altlinux.org 2021-07-21 18:54:57 MSK
(Ответ для Evgeny Sinelnikov на комментарий #6)

> Более того, предлагаю рассмотреть детально то, что внесено в netcmdplus,
> разбить на патчи и начать пробрасывать в апстрим.

Начать можно с патчей для нашей сборки samba-tool и удалить после этого samba-tool-plus. Это я могу попробовать.
Comment 9 manowar@altlinux.org 2021-07-21 19:04:00 MSK
(Ответ для Evgeny Sinelnikov на комментарий #7)

> И готовится нужно совсем к другому. Это два:
> https://download.samba.org/pub/samba/rc/samba-4.15.0rc1.WHATSNEW.txt
> 
> samba-tool available without the ad-dc
> --------------------------------------
> 
> The samba-tool command is now available when samba is configured
> --without-ad-dc. Not all features will work, and some ad-dc specific options
> have been disabled. The samba-tool domain options, for example, are limited
> when no ad-dc is present. Samba must still be built with ads in order to
> enable
> samba-tool.

Звучит так, как будто бы они добавляют поддержку того же самого в урезанном варианте для специальной конфигурации. В обычной конфигурации всё должно сохраниться по-старому?