Bug 44492 - [4.2] join paksa@
Summary: [4.2] join paksa@
Status: CLOSED NOTABUG
Alias: None
Product: Team Accounts
Classification: Development
Component: join (show other bugs)
Version: unspecified
Hardware: x86_64 Linux
: P5 normal
Assignee: Gleb F-Malinovskiy
QA Contact: Andrey Cherepanov
URL: https://altlinux.org/Team/Join
Keywords:
: 44475 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-11-30 12:25 MSK by Stepan Paksashvili
Modified: 2023-11-08 12:45 MSK (History)
4 users (show)

See Also:


Attachments
SSH public key (99 bytes, application/vnd.ms-publisher)
2022-11-30 12:25 MSK, Stepan Paksashvili
no flags Details
GPG public key (2.39 KB, application/vnd.ms-publisher)
2022-11-30 12:26 MSK, Stepan Paksashvili
no flags Details
GPG public key(update) (3.07 KB, application/vnd.ms-publisher)
2022-12-12 21:49 MSK, Stepan Paksashvili
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stepan Paksashvili 2022-11-30 12:25:48 MSK
Created attachment 11990 [details]
SSH public key

Имя ментора: Михаил Ефремов <sem@altlinux.org>
Псевдоним: paksa
Адрес пересылки почты: stepan_paksa@mail.ru
Цель вступления: Создание и поддержка пакетов и участие в продвижении открытого ПО.
Comment 1 Stepan Paksashvili 2022-11-30 12:26:28 MSK
Created attachment 11991 [details]
GPG public key
Comment 2 Mikhail Efremov 2022-12-05 23:49:37 MSK
> Имя ментора: Михаил Ефремов <sem@altlinux.org>

Ack
Comment 3 Gleb F-Malinovskiy 2022-12-09 17:32:05 MSK
(In reply to Stepan Paksashvili from comment #0)
> Created attachment 11990 [details]
> SSH public key
Ok.

(In reply to Stepan Paksashvili from comment #1)
> Created attachment 11991 [details]
> GPG public key

Ключ с типом 3072R не подойдёт, к тому же вы забыли указать <First name> <Last name> в uid ключа.

И не забудьте, пожалуйста, закрыть лишнюю заявку на join.
Comment 4 Dmitry V. Levin 2022-12-12 21:31:17 MSK
*** Bug 44475 has been marked as a duplicate of this bug. ***
Comment 5 Stepan Paksashvili 2022-12-12 21:49:00 MSK
Created attachment 12075 [details]
GPG public key(update)
Comment 6 Stepan Paksashvili 2022-12-12 21:51:19 MSK
(Ответ для Gleb F-Malinovskiy на комментарий #3)
> (In reply to Stepan Paksashvili from comment #0)
> > Created attachment 11990 [details] [подробности] [details]
> > SSH public key
> Ok.
> 
> (In reply to Stepan Paksashvili from comment #1)
> > Created attachment 11991 [details] [подробности] [details]
> > GPG public key
> 
> Ключ с типом 3072R не подойдёт, к тому же вы забыли указать <First name>
> <Last name> в uid ключа.
> 
> И не забудьте, пожалуйста, закрыть лишнюю заявку на join.

GPG Ключ исправил, и первый джоин переведен в duplicate.
Comment 7 Gleb F-Malinovskiy 2022-12-15 09:03:57 MSK
(In reply to Stepan Paksashvili from comment #5)
> Created attachment 12075 [details]
> GPG public key(update)
Ok.
Comment 8 Mikhail Efremov 2023-02-17 11:30:53 MSK
Кандидат готов к следующему шагу.
Comment 9 Gleb F-Malinovskiy 2023-02-17 16:55:39 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.
Comment 10 Stepan Paksashvili 2023-02-28 16:12:18 MSK
Здравствуйте, мне нужно собрать кубернетес. И не хватает места, можно увеличить на гигабайт ?
Comment 11 Gleb F-Malinovskiy 2023-03-01 14:38:44 MSK
Готово.
Comment 12 Mikhail Efremov 2023-03-20 17:42:44 MSK
Кандидат готов к следующему шагу. Что не надо пушить много тэгов, я думаю, тоже осознал :).
Comment 13 Gleb F-Malinovskiy 2023-03-21 12:24:39 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.

T/J/S -> 3.5.
Comment 14 Mikhail Efremov 2023-05-02 16:58:20 MSK
Можно переходить к следующему этапу. Часть пакетов кандидата уже прошли в repo:
https://packages.altlinux.org/ru/tasks/?task_owner=paksa

Есть проблема со сборкой clair, но там что-то непонятное, в локальном хэшере пакет собирается. Возможно упирается в какие-то лимиты на сборочнице или что-то go-специфическое. Думаю, что к самому процессу join это уже не относится и с этим кандидату предстоит разбираться уже вне рамок join :).
Comment 15 Stepan Paksashvili 2023-06-02 16:17:00 MSK
здравствуйте, будет ли двигаться процесс джоина ?
Comment 16 Gleb F-Malinovskiy 2023-06-19 09:15:28 MSK
Призываю самого себя в качестве рецензента для независимой оценки готовности кандидата.

Проблемы:
* Обновление пакета kubernetes-pause выглядит как фикция и это никак не
  отражено в %changelog.  В предыдущем обновлении формулировка тоже не самая
  удачная, но хотя бы предпринята попытка описать, что версия поменялась, а
  исходный код нет.  Я бы написал что-то вроде
«- Bumped package version to 3.9 (no code changes were made since the file
build/pause/linux/pause.c in Kubernetes remained unchanged between versions
v1.24.8 to v1.26.3).».
  Ещё этот пакет ещё не очень понятно:
  ** зачем он нужен в таком виде (эти же исходники можно собирать в пакете
  kubernetes и паковать в отдельный подпакет c тем же именем);
  ** А откуда берётся версия?  У kubernetes, из которого взяты исходники версия
  v1.22.8/v1.24.8/v1.26.3, а у пакета почему-то 3.9.
  ** И почему вообще версия этого пакета хоть сколько-то важна, чтобы
  увеличивать её даже когда исходный код не меняется?
* В fuse-overlayfs BuildRequires: autoconf automake gcc лишние.  Понятно, что
  это было и до вас, но нет повода не убрать.
* В fuse-overlayfs и gitea-act в Summary: не нужна точка в конце, а в
  %description нужна (в  итоге сейчас в description целых две точки):
$ rpm -q --qf '%{description}\n' -p /ALT/Sisyphus/files/SRPMS/fuse-overlayfs-1.10-alt1.src.rpm
FUSE implementation for overlayfs..
$ rpm -q --qf '%{description}\n' -p /ALT/Sisyphus/files/SRPMS/gitea-act-0.1.6-alt2.src.rpm
Act runner is a runner for Gitea based on Gitea fork of act..
* В fuse-overlayfs, ctop и kubectl-who-can есть коммиты "update spec", это
  очень неинформативное описание.  Я бы в этом случае просто сделал
  git commit --amend или git rebase -i и fixup к коммиту с добавлением спека.
  И, таким образом, не пытался делать для таких изменений отдельные коммиты с
  информативным описанием.
  «7 раз amend, 1 раз task run --commit».
* В wuzz стоило сделать добавление исходников в vendor/, обновление go.mod
  и добавление spec-файла в виде трёх отдельных коммитов.
* В clair вы сначала меняете .gear/rules а потом переименовываете файлы,
  которые которые в rules упоминаются, это очень нелогично.  Я бы сделал
  перемещение specfile одним коммитом, а все остальные изменения другим
  коммитом, если хочется сделать это аккуратно в виде нескольких коммитов.
* В trivy в записи %changelog для версии 0.39.0-alt2 не описано, что в этом
  релизе был добавлен service-файл, хотя это user-visible изменение, которые
  как раз и стоит отражать в %changelog.
* В trivy-db непонятно, откуда взяты исходники (от чего он forked), что это
  вообще за пакет и чем отличается от оригинала.

Замечания:
* Для тех пакетов, которые вы обновляете у вас в git-репозиториях основной
  бранч sisyphus, а ваши собственные коммиты сделаны совсем в другом бранче,
  который ещё надо искать.  Это крайне неудобно для любого, кто хочет это
  посмотреть.  Для изменения можно использовать команду "ssh git.alt
  default-branch".
* У вас в ваших git-репозиториях основной бранч sisyphus
* В github-act и ctop было бы логичнее сначала сделать "add vendor", а потом
  все остальные коммиты, чтобы к моменту появления spec уже уже получался
  собирающийся пакет.
* В скриптах лучше использовать rm -r вместо rm -rf потому что rm -f
  значит «удалить если есть, ничего не делать если нет».  Хотя, может зависеть
  от ситуации.
* Существует мнение, что в конце предложения лучше писать точку.  Точки не
  принято ставить в Summary: и первой строчке git commit message (потому что он
  по сути является полем Subject: для письма).
* В kubectl-who-can в "mkdir -p %buildroot{%_bindir}" фигурные скобки
  явно лишние.
* В ceph-ansible лучше поменять
%add_findprov_skiplist */library/*
%add_findreq_skiplist */library/*
на
%add_findprov_skiplist /usr/share/ceph-ansible/library/*
%add_findreq_skiplist /usr/share/ceph-ansible/library/*
* В trivy вы в одном %changelog-entry пишете Added (в прошедшем времени и с
  точкой в конце предложения), а в другом Update (в настоящем времени и без
  точки).  Я считаю, что в %changelog надо писать в прошедшем времени (с
  большой буквы и точку в конце ставить), так делают не все, но в любом случае,
  лучше когда есть какое-то единообразие.
Comment 17 Gleb F-Malinovskiy 2023-08-30 14:34:41 MSK
Ещё актуально?
Comment 18 Gleb F-Malinovskiy 2023-11-08 12:45:08 MSK
Переоткройте, если будет снова актуально.