Bug 30929

Summary:	Остался старый код после мерджинга изменений upstream
Product:	Sisyphus	Reporter:	Владимир Диденко <vladimir.didenko>
Component:	python-module-salt	Assignee:	Valentin Rosavitskiy <valintinr>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	real.altlinux.org
Version:	unstable
Hardware:	all
OS:	Linux

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 наступил. Возможно, корифеи знают, как такие ситуации получше разруливать - надо будет спросить. 

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

Спасибо!