Bug 44279 - [done] join geochip@
Summary: [done] join geochip@
Status: CLOSED FIXED
Alias: None
Product: Team Accounts
Classification: Development
Component: join (show other bugs)
Version: unspecified
Hardware: x86_64 Linux
: P1 normal
Assignee: Gleb F-Malinovskiy
QA Contact: Andrey Cherepanov
URL: https://altlinux.org/Team/Join
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-11 17:19 MSK by Alexander Stepchenko
Modified: 2023-08-30 19:52 MSK (History)
4 users (show)

See Also:


Attachments
SSH public key (95 bytes, application/vnd.ms-publisher)
2022-11-11 17:19 MSK, Alexander Stepchenko
no flags Details
GPG public key (3.07 KB, application/vnd.ms-publisher)
2022-11-11 18:14 MSK, Alexander Stepchenko
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Stepchenko 2022-11-11 17:19:05 MSK
Created attachment 11839 [details]
SSH public key

Имя ментора: Григорий Устинов <grenka@altlinux.org>
Псевдоним: geochip
Адрес пересылки почты: stepchenko.alex@yandex.ru
Цель вступления: Создание и поддержка пакетов. Участие в продвижении открытого ПО.
Comment 1 Alexander Stepchenko 2022-11-11 18:14:01 MSK
Created attachment 11842 [details]
GPG public key
Comment 2 Grigory Ustinov 2022-11-11 18:47:05 MSK
Менторство подтверждаю.
Comment 3 Grigory Ustinov 2022-11-30 18:46:38 MSK
Выдайте, пожалуйста, кандидату гитовницу.
Comment 4 Gleb F-Malinovskiy 2022-12-09 17:13:06 MSK
(In reply to Alexander Stepchenko from comment #0)
> Created attachment 11839 [details]
> SSH public key
Ok.
(In reply to Alexander Stepchenko from comment #1)
> Created attachment 11842 [details]
> GPG public key
Ok.
Comment 5 Grigory Ustinov 2022-12-09 17:18:53 MSK
Вперёд!
Comment 6 Gleb F-Malinovskiy 2022-12-09 18:14:36 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.     

T/J/S -> 2.3.
Comment 7 Alexander Stepchenko 2022-12-13 19:55:31 MSK
Выложил 3 пакета tldr, shtab (зависимость tldr) и Discord.

https://git.altlinux.org/people/geochip/packages/
Comment 8 Grigory Ustinov 2022-12-13 20:37:25 MSK
Прошу выдать сборочницу.
Comment 9 Gleb F-Malinovskiy 2022-12-15 10:00:21 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.

T/J/S -> 3.5.
Comment 10 Alexander Stepchenko 2022-12-16 18:33:56 MSK
Собрал на сборочнице запушенные 3 пакета, получил EPERM:

python3-module-shtab и tldr:
https://git.altlinux.org/tasks/311868/

discord
https://git.altlinux.org/tasks/311871/
Comment 11 Gleb F-Malinovskiy 2022-12-16 18:57:03 MSK
(In reply to Alexander Stepchenko from comment #10)
> discord
> https://git.altlinux.org/tasks/311871/

А какая у этих бинарников лицензия?  С пакетом следует распространять лицензию и она, естественно, должна содержать пункт о том, что мы можем это распространять.
Уже попадание такого кода на наш git-хостинг плохо, не говоря про репозиторий.
Comment 12 Grigory Ustinov 2023-02-20 17:01:03 MSK
Уважаемый секретарь, выделите, пожалуйста, моему кандидату рецензента. На мой взгляд, Александр усвоил некоторые базовые представления о сборке в сизиф и способен к самостоятельному анализу и решению проблем при сборке.
Comment 13 Alexander Stepchenko 2023-02-20 17:17:27 MSK
(In reply to Gleb F-Malinovskiy from comment #11)
> (In reply to Alexander Stepchenko from comment #10)
> > discord
> > https://git.altlinux.org/tasks/311871/
> 
> А какая у этих бинарников лицензия?  С пакетом следует распространять
> лицензию и она, естественно, должна содержать пункт о том, что мы можем это
> распространять.
> Уже попадание такого кода на наш git-хостинг плохо, не говоря про
> репозиторий.

Для распространения этих бинарников необходимо разрешение. Я ещё с конца декабря несколько раз писал на почту поддержки Discord, чтобы получить разрешение на распространение в нашем репозитории, но мне так и не ответили. Из git-хостинга репозиторий удалил пока что.
Comment 14 Alexander Stepchenko 2023-02-20 17:24:50 MSK
Обновил следующие пакеты:

mysql-workbench-community-8.0.32-alt1
https://git.altlinux.org/tasks/314624/

gdal-3.6.2-alt1
https://git.altlinux.org/tasks/314051/

conan-1.58.0-alt1
https://git.altlinux.org/tasks/315213/

python3-module-distro-1.8.0-alt1
https://git.altlinux.org/tasks/315544/
Comment 15 Alexander Stepchenko 2023-03-17 12:53:38 MSK
Здравствуйте. Обновляю kubernetes до новой версии, и, видимо, не хватает памяти на гитовнице. Пишет "Disk quota exceeded".
Можно выделить еще 1 ГБ?
Comment 16 Gleb F-Malinovskiy 2023-03-17 14:07:22 MSK
Готово.
Comment 17 Gleb F-Malinovskiy 2023-06-13 19:17:36 MSK
Поскольку я уже начал смотреть, будем считать, что я ревьюер.

Проблемы:
* tldr и gopass: запакованы каталоги, которые не принадлежат этому пакету
%dir %_datadir/bash-completion
%dir %_datadir/bash-completion/completions
%dir %_datadir/zsh
%dir %_datadir/zsh/site-functions

%dir %_datadir/bash-completion/
%dir %_datadir/bash-completion/completions/
%dir %_datadir/zsh/
%dir %_datadir/zsh/site-functions/
%dir %_datadir/fish/
%dir %_datadir/fish/vendor_completions.d/
это неправильно.
* mysql-workbench-community: Вы обновили сам пакет, а забандленную версию antlr решили почему-то не обновлять, скорее всего это ошибка.
* python3-module-distro: пробовали ли вы отправить ваш патч в апстрим?
* kubernetes: вы заменили KUBE_GIT_COMMIT=%commit на KUBE_GIT_COMMIT=%release.  Я думаю, что плохая идея использовать поле предназначенное для сущности совсем другого типа для нашего release.
* cargo-cache: зачем пакету явная зависимость на /proc, если у бинарного пакета rust уже есть такая зависимость?

Замечания:
* python3-module-shtab: Вместо VERSION_UNKNOWN в качестве заменяемой строки лучше использовать называние типа @MODULE_VERSION@, при виде такой надписи всем сразу понятно, что она должна быть автоматически заменена на значение версии.
* tldr: вместо внутреннего макроса rpm %__python3 лучше просто запускать python3, это на самом деле касается всех внутренних макросов.

Мысли вслух:
* На самом деле, каталог .gear/ ничем не лучше для упаковки spec-а и патчей, чем любой другой каталог, если бы я собирал эти пакеты, я бы использовал каталог alt/.
Comment 18 Gleb F-Malinovskiy 2023-06-13 19:22:33 MSK
(In reply to Gleb F-Malinovskiy from comment #17)
> Поскольку я уже начал смотреть, будем считать, что я ревьюер.
> 
> Проблемы:
> * tldr и gopass: запакованы каталоги, которые не принадлежат этому пакету

Это же касается и kubernetes:
%dir %_sysconfdir/systemd
%dir %_sysconfdir/systemd/system.conf.d
%dir %_sysconfdir/systemd/system

Если бы все каталоги из "post-install unowned files" нужно было бы запаковать в пакет, то это был бы не warning.  Это предупреждение, которое значит, что вы могли забыть запаковать каталог, который принадлежит вашему пакету или забыли поставить зависимость на какой-то пакет, которому принадлежит каталог.
Comment 19 Alexander Stepchenko 2023-07-03 20:35:51 MSK
tldr:
Запаковка каталогов, которые не принадлежат данному пакету, была исправлена не мною ранее.
https://git.altlinux.org/gears/p/python3-module-tldr.git?p=python3-module-tldr.git;a=blobdiff;f=.gear/tldr.spec;h=3c8853b71828083a8e871b341bd07a8094cda9d0;hp=ceb79b85dc8d443850e72580fb8ed403e8dd1f30;hb=2f8d3fe42262c56ba6abf3cefcfe4800f84979d8;hpb=75ee50da136e4c262e473235ed3309b4e0367abb
Аналогично с %__python3.
https://git.altlinux.org/gears/p/python3-module-tldr.git?p=python3-module-tldr.git;a=commitdiff;h=8fa9b27bd730c22681280346cc06f737bcea6dbe


gopass:
Исправил.
https://git.altlinux.org/tasks/323147/


mysql-workbench-community:
Обновил до последней версии 8.0.33, обновил забандленную antlr до 4.11.1. В версии mysql-workbench-community-8.0.33 необходим libantlr4-runtime.so.4.11.1, поэтому обновил также пакет libantlr4.
https://git.altlinux.org/tasks/320075/

Было замечание от ментора, что предпочтительней было бы избавиться от забандленного antlr-4.11.1-complete.jar, а вместо него использовать упаковынный. В таком случае также необходимо будет обновить пакет antlr4-project. Я подготовил пример того, как это могло бы выглядеть в таске. Так как я не являюсь мэнтейнером ни libantlr4, ни antlr4-project, думаю, что лучше свести их изменения к минимуму, поэтому предпочел первый вариант с забандленный antlr-4.11.1-complete.jar.
https://git.altlinux.org/tasks/324153/


python3-module-distro:
Патч был отправлен в апстрим и успешно замерджен. Ожидается выход следующей версии с включенным патчем.
https://github.com/python-distro/distro/pull/354


kubernetes:
Замена %commit на %release обосновывается двумя причинами.
1. Проще поддерживать пакет, потому что не нужно каждый раз вручную менять хэш коммита в спеке.
2. Вообще хэш коммита используется для идентификации коммита, из которого было собрано ПО. В случае репозитория Sisyphus с помощью значения %release проще и быстрее индентифицировать, из какого коммита было собрано ПО.
Было бы предпочтительней использовать хэш коммита, если бы ПО собиралось, например из последнего коммита ветки master, но в наших репозиториях все программы собираются из релиз тегов. Кроме того, накладываются дополнительные коммиты (добавление патчей, изменения спека и пр.).

В данный момент происходит разбитие пакета kubernetes на пакеты kubernetes1.22, kubernetes1.23, kubernetes1.24, kubernetes1.25 и kubernetes1.26. Причиной является то, что для обновления старых кластеров k8s с версии 1.22 до последней версии в Sisyphus (1.26) необходимо поочередно обновить кластер по всем минорным версиям вверх (1.22 -> 1.23 -> 1.24 -> 1.25 -> 1.26).
Подробнее https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade/

Запаковка каталогов, которые не принадлежат данному пакету в kubernetes будет исправлена в процессе разбиения пакета.


cargo-cache:
Замечание исправлено.
https://git.altlinux.org/tasks/324159/


python3-module-shtab:
Обновлен до последней версии 1.6.2. Замечание исправлено.
https://git.altlinux.org/tasks/324168/
Comment 20 Alexander Stepchenko 2023-07-05 14:40:40 MSK
Обновил flannel:
https://git.altlinux.org/tasks/324291/
Comment 21 Alexander Stepchenko 2023-08-28 15:12:40 MSK
Здравствуйте. Можно получить обратную связь по обсуждаемым вопросам и процессу джойна?
Есть ли еще какие-нибудь замечания?
Comment 22 Gleb F-Malinovskiy 2023-08-30 19:49:18 MSK
Вроде, не было больше замечаний.
Хорошая работа.
Comment 23 Gleb F-Malinovskiy 2023-08-30 19:52:40 MSK
Адрес подписан на devel@.
Пользователь добавлен в группу мейнтейнеров.

Желаю удачного мейнтейнерства!