Bug 30929 - Остался старый код после мерджинга изменений upstream
Summary: Остался старый код после мерджинга изменений upstream
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: python-module-salt (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Valentin Rosavitskiy
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-08 18:05 MSK by Владимир Диденко
Modified: 2015-05-18 08:31 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Владимир Диденко 2015-04-08 18:05:11 MSK
После мерджинга изменений upstream для 2015.2, осталась часть изменений от 2014.7. Т.е. пакет собирается не из чистых исходников, а из 2015.2 + ошметки от 2014.7. Убедиться в этом достаточно просто. 

1. Клонируем salt

   git clone http://git.altlinux.org/gears/s/salt.git

2. Переходим на ветку sisyphus

   git checkout sisyphus

3. Смотрим diff 

   git diff v2015.2 sisyphus

4. Видим ошметки от старых версий. Например в файле salt/runners/state.py есть
   дублирующий import

   from __future__ import print_function 

   который приводит к ошибке во время запуска salt-master. 

Я для себя локально все не alt-специфичные файлы потер, оставив только одну правку в salt/modules/ssh.py, в которой правился путь к ssh ключам (/etc/ssh заменялся на /etc/openssl). Не стал разбираться это осталось с прошлых версий или нет. 

Вобщем, мои предложения оформлены в виде 2015.2-alt3

http://git.altlinux.org/people/cow/packages/salt.git

Если устроят, то могу отправить на сборку.
Comment 1 Valentin Rosavitskiy 2015-04-18 11:19:13 MSK
(В ответ на комментарий №0)
> После мерджинга изменений upstream для 2015.2, осталась часть изменений от
> 2014.7. Т.е. пакет собирается не из чистых исходников, а из 2015.2 + ошметки от
> 2014.7. Убедиться в этом достаточно просто. 
> 
> 1. Клонируем salt
> 
>    git clone http://git.altlinux.org/gears/s/salt.git
> 
> 2. Переходим на ветку sisyphus
> 
>    git checkout sisyphus
> 
> 3. Смотрим diff 
> 
>    git diff v2015.2 sisyphus
> 
> 4. Видим ошметки от старых версий. Например в файле salt/runners/state.py есть
>    дублирующий import
> 
>    from __future__ import print_function 
> 
>    который приводит к ошибке во время запуска salt-master. 
> 
> Я для себя локально все не alt-специфичные файлы потер, оставив только одну
> правку в salt/modules/ssh.py, в которой правился путь к ssh ключам (/etc/ssh
> заменялся на /etc/openssl). Не стал разбираться это осталось с прошлых версий
> или нет. 
> 
> Вобщем, мои предложения оформлены в виде 2015.2-alt3
> 
> http://git.altlinux.org/people/cow/packages/salt.git
> 
> Если устроят, то могу отправить на сборку.
Да, есть такое, самому интересно как оно так получилось.
Отправил на сборку новую версию с исправлениями, сверялся с чистим апстримом.
Comment 2 Repository Robot 2015-04-18 11:25:14 MSK
salt-2015.2-alt3 -> sisyphus:

* Sat Apr 18 2015 Valentin Rosavitskiy <valintinr@altlinux> 2015.2-alt3
- Removed code from previous tag (ALT 30929)
Comment 3 Владимир Диденко 2015-04-18 11:54:18 MSK
(В ответ на комментарий №1)

> Да, есть такое, самому интересно как оно так получилось.

Обычное дело, когда upstream в stable ветку коммитит изменения, которых нет в последующих ветках. Я на такие же грабли при обновлении mongo наступил. Возможно, корифеи знают, как такие ситуации получше разруливать - надо будет спросить. 

> Отправил на сборку новую версию с исправлениями, сверялся с чистим апстримом.

Спасибо!