[Sysadmins] I: altlinux p6 branch openvz template caches online

[Sysadmins] I: altlinux p6 branch openvz template caches online

[Michael Shigorin]

	Здравствуйте.
Для большего удобства развёртывания OpenVZ-контейнеров
с ALT Linux выложил подготовленные шаблоны на базе p6:

http://ftp.linux.kiev.ua/pub/Linux/ALT/people/mike/ovz/altlinux-p6-20120321-i586.tar.gz
http://ftp.linux.kiev.ua/pub/Linux/ALT/people/mike/ovz/altlinux-p6-20120321-x86_64.tar.gz

Они же доступны в контрибе на официальном сайте ovz:

http://download.openvz.org/template/precreated/contrib/altlinux-p6-20120321-i586.tar.gz
http://download.openvz.org/template/precreated/contrib/altlinux-p6-20120321-x86_64.tar.gz

При vzctl >= 3.0.30.2 и доступности интернета можно воспользоваться:

vzctl create 123 --ostemplate contrib/altlinux-p6-20120321-i586

Сделано mkimage по mkimage-profiles (ve/generic.tar.gz
из пары мелких правок поверх 0.6.0-alt1: забыл openssh):

http://www.altlinux.org/Mkimage/Profiles/m-p/examples

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Michael Shigorin]

On Thu, Mar 22, 2012 at 07:21:21PM +0200, I wrote:
> При vzctl >= 3.0.30.2 и доступности интернета можно воспользоваться:
> vzctl create 123 --ostemplate contrib/altlinux-p6-20120321-i586

Соврамши (отвлёкся и не проверил собственно завершение операции);
по результатам осмотра /usr/share/vzctl/scripts/vps-download
повесил http://bugzilla.openvz.org/2222 с патчиком, но он сам
по себе не решает вопрос автовыбора конфигурации дистрибутива.

Или же можно добавить в /etc/vz/download.conf:

TMPL_REPO_PREFIX="http://download.openvz.org/template/precreated"
TEMPLATE_REPOS="${TMPL_REPO_PREFIX}"
TEMPLATE_REPOS="${TEMPLATE_REPOS} ${TMPL_REPO_PREFIX}/contrib/"

и затем выполнить сообразно желаемому:

vzctl create 123 --ostemplate altlinux-p6-20120321-i586
vzctl create 123 --ostemplate altlinux-p6-20120321-x86_64

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Dubrovskiy Viacheslav]

[-- Attachment #1: Type: text/plain, Size: 1079 bytes --]

22.03.2012 15:26, Michael Shigorin пишет:
> On Thu, Mar 22, 2012 at 07:21:21PM +0200, I wrote:
>> При vzctl >= 3.0.30.2 и доступности интернета можно воспользоваться:
>> vzctl create 123 --ostemplate contrib/altlinux-p6-20120321-i586
> Соврамши (отвлёкся и не проверил собственно завершение операции);
> по результатам осмотра /usr/share/vzctl/scripts/vps-download
> повесил http://bugzilla.openvz.org/2222 с патчиком, но он сам
> по себе не решает вопрос автовыбора конфигурации дистрибутива.
Ты имеешь ввиду, что это вызывает вот такую ругань?
Warning: configuration file for distribution
contrib/altlinux-p6-20120321-i586 not found, using defaults from
/etc/vz/dists/default

Это вызвано тем, что OSTEMPLATE="contrib/altlinux-p6-20120321-i586"
получается в конфиге.


-- 
WBR,
Dubrovskiy Viacheslav


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 4903 bytes --]

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Michael Shigorin]

On Sun, Mar 25, 2012 at 12:59:31AM -0400, Dubrovskiy Viacheslav wrote:
> >> При vzctl >= 3.0.30.2 и доступности интернета можно воспользоваться:
> >> vzctl create 123 --ostemplate contrib/altlinux-p6-20120321-i586
> > Соврамши (отвлёкся и не проверил собственно завершение операции);
> > по результатам осмотра /usr/share/vzctl/scripts/vps-download
> > повесил http://bugzilla.openvz.org/2222 с патчиком, но он сам
> > по себе не решает вопрос автовыбора конфигурации дистрибутива.
> Ты имеешь ввиду, что это вызывает вот такую ругань?
> Warning: configuration file for distribution
> contrib/altlinux-p6-20120321-i586 not found, using defaults
> from /etc/vz/dists/default

Ага.

> Это вызвано тем, что OSTEMPLATE="contrib/altlinux-p6-20120321-i586"
> получается в конфиге.

Я предложил дефолты искать по basename, а затем по dirname
темплейта -- но патч соответствующий не добавил (страшновато
всё-таки в чужой сишный код лезть без адекватных навыков),
а Кирилл будто и не заметил соответствующую хотелку :)
Ну хоть шелльный патчик оперативно взял, уже хорошо.

Как думаешь, само предложение о поддержке иерархических
имён темплейтов осмысленное?

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Dubrovskiy Viacheslav]

[-- Attachment #1: Type: text/plain, Size: 2652 bytes --]

25.03.2012 04:51, Michael Shigorin пишет:
> On Sun, Mar 25, 2012 at 12:59:31AM -0400, Dubrovskiy Viacheslav wrote:
>>>> При vzctl >= 3.0.30.2 и доступности интернета можно воспользоваться:
>>>> vzctl create 123 --ostemplate contrib/altlinux-p6-20120321-i586
>>> Соврамши (отвлёкся и не проверил собственно завершение операции);
>>> по результатам осмотра /usr/share/vzctl/scripts/vps-download
>>> повесил http://bugzilla.openvz.org/2222 с патчиком, но он сам
>>> по себе не решает вопрос автовыбора конфигурации дистрибутива.
>> Ты имеешь ввиду, что это вызывает вот такую ругань?
>> Warning: configuration file for distribution
>> contrib/altlinux-p6-20120321-i586 not found, using defaults
>> from /etc/vz/dists/default
> Ага.
>
>> Это вызвано тем, что OSTEMPLATE="contrib/altlinux-p6-20120321-i586"
>> получается в конфиге.
> Я предложил дефолты искать по basename, а затем по dirname
> темплейта -- но патч соответствующий не добавил (страшновато
> всё-таки в чужой сишный код лезть без адекватных навыков),
> а Кирилл будто и не заметил соответствующую хотелку :)
> Ну хоть шелльный патчик оперативно взял, уже хорошо.
Сказал А говори и Б :)

Я тоже в C не копенгаген.
> Как думаешь, само предложение о поддержке иерархических
> имён темплейтов осмысленное?
Имеет. Собираю новую версию. Туда попал твой патч. Дай думаю проверю,
благо ты подогнал имидж. Попробовал - не работает. Нужно или убрать или
доделать

src/lib/create.c
    374         /* Store full ostemplate name */
    375         if (tmpl->ostmpl != NULL) {
    376                 if (cmd_p->res.tmpl.ostmpl != NULL)
    377                         free(cmd_p->res.tmpl.ostmpl);
    378                 cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
    379         }

Вот тут нужно через basename прогнать, чтобы сохранять не полный путь, а
только имя шаблона.

-- 
WBR,
Dubrovskiy Viacheslav


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 4903 bytes --]

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Michael Shigorin]

On Sun, Mar 25, 2012 at 11:09:47AM -0400, Dubrovskiy Viacheslav wrote:
> > а Кирилл будто и не заметил соответствующую хотелку :)
> > Ну хоть шелльный патчик оперативно взял, уже хорошо.
> Сказал А говори и Б :)
> Я тоже в C не копенгаген.

Так тут не Б, а как раз Ц :)

> > Как думаешь, само предложение о поддержке иерархических
> > имён темплейтов осмысленное?
> Имеет. Собираю новую версию. Туда попал твой патч. Дай думаю
> проверю, благо ты подогнал имидж. Попробовал - не работает.

Об этом и написал.

> Нужно или убрать или доделать

Если убрать, будет опять неконсистентность в vps-download.

> src/lib/create.c
>     374         /* Store full ostemplate name */
>     375         if (tmpl->ostmpl != NULL) {
>     376                 if (cmd_p->res.tmpl.ostmpl != NULL)
>     377                         free(cmd_p->res.tmpl.ostmpl);
>     378                 cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
>     379         }
> 
> Вот тут нужно через basename прогнать, чтобы сохранять не
> полный путь, а только имя шаблона.

Я уже опять боюсь трогать строки в сях после бэкпортирования
propagator -- если бы не ldv@ и особенно gns@, был бы там и
fd leak, и сегфолт.

Хотелку внятно описать могу. :)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Dubrovskiy Viacheslav]

[-- Attachment #1: Type: text/plain, Size: 1583 bytes --]

25.03.2012 12:14, Michael Shigorin пишет:
>> Нужно или убрать или доделать
> Если убрать, будет опять неконсистентность в vps-download.
>
>> src/lib/create.c
>>     374         /* Store full ostemplate name */
>>     375         if (tmpl->ostmpl != NULL) {
>>     376                 if (cmd_p->res.tmpl.ostmpl != NULL)
>>     377                         free(cmd_p->res.tmpl.ostmpl);
>>     378                 cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
>>     379         }
>>
>> Вот тут нужно через basename прогнать, чтобы сохранять не
>> полный путь, а только имя шаблона.
> Я уже опять боюсь трогать строки в сях после бэкпортирования
> propagator -- если бы не ldv@ и особенно gns@, был бы там и
> fd leak, и сегфолт.
>
> Хотелку внятно описать могу. :)
Кажется вот так правильно?

@@ -375,7 +376,7 @@ int vps_create(vps_handler *h, envid_t veid,
vps_param *vps_p, vps_param *cmd_p,
        if (tmpl->ostmpl != NULL) {
                if (cmd_p->res.tmpl.ostmpl != NULL)
                        free(cmd_p->res.tmpl.ostmpl);
-               cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
+               cmd_p->res.tmpl.ostmpl = strdup(basename(tmpl->ostmpl));
        }
        if ((ret = vps_save_config(veid, dst, cmd_p, vps_p, action)))
                goto err_names;

-- 
WBR,
Dubrovskiy Viacheslav


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 4903 bytes --]

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Michael Shigorin]

On Sun, Mar 25, 2012 at 12:36:44PM -0400, Dubrovskiy Viacheslav wrote:
> >> Вот тут нужно через basename прогнать, чтобы сохранять не
> >> полный путь, а только имя шаблона.
> > Хотелку внятно описать могу. :)
> Кажется вот так правильно?

Не, в багзиле ответил.

> @@ -375,7 +376,7 @@ int vps_create(vps_handler *h, envid_t veid,
> -               cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
> +               cmd_p->res.tmpl.ostmpl = strdup(basename(tmpl->ostmpl));

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

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Dubrovskiy Viacheslav]

[-- Attachment #1: Type: text/plain, Size: 1334 bytes --]

25.03.2012 12:51, Michael Shigorin пишет:
>> @@ -375,7 +376,7 @@ int vps_create(vps_handler *h, envid_t veid,
>> -               cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
>> +               cmd_p->res.tmpl.ostmpl = strdup(basename(tmpl->ostmpl));
> Ты теряешь информацию перед записью, а я предлагаю отбрасывать
> лишнюю при чтении.  Тогда по мере доработки утилит ей можно
> будет воспользоваться, т.к. она будет сохранена.
Мне кажется ты усложняешь.
OSTEMPLATE нужна только для определения какой дистрибутив использовался
в образе. Совершенно не важно из какой он директории был взят.
Директории могут быть любые и вешать какой-то функционал на их имя мне
кажется не очень хорошей идеей. Достаточно того что уже есть привязка к
имени образа. Да и какой функционал может быть повешен на имя
директории? Пример можешь привести?

-- 
WBR,
Dubrovskiy Viacheslav


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 4903 bytes --]

Re: [Sysadmins] I: altlinux p6 branch openvz template caches online

[Michael Shigorin]

On Sun, Mar 25, 2012 at 01:01:55PM -0400, Dubrovskiy Viacheslav wrote:
> >> -               cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl);
> >> +               cmd_p->res.tmpl.ostmpl = strdup(basename(tmpl->ostmpl));
> > Ты теряешь информацию перед записью, а я предлагаю отбрасывать
> > лишнюю при чтении.  Тогда по мере доработки утилит ей можно
> > будет воспользоваться, т.к. она будет сохранена.
> Мне кажется ты усложняешь.

Возможно.  Или же стоило сразу выписывать полную картину маслом,
но уже засыпал.

> Директории могут быть любые и вешать какой-то функционал на их
> имя мне кажется не очень хорошей идеей.

Если хламовник, то любые... мне показалось, что можно дать
интересную функциональность тем, кто использует их с головой
(при этом ничего не сломав тем, кто сваливает всё в кучу).

> Достаточно того что уже есть привязка к имени образа.

Удалённо (http://download.openvz.org/template/precreated/contrib/):

contrib/altlinux-p6
contrib/debian-6.0

Локально (/var/lib/vz/template/cache/):

altlinux/p6-kiev-x86_64
altlinux/p6-odessa-i586
opensuse/11.4-kiev-i586

А в светлом будущем(tm) ещё и управлять различиями этих образов,
генерируемых из mkimage-profiles.  Один тестовый из сузи 11.4
уже был сделан (но невоспроизводимо), а сейчас ещё и centos6
удалось запинать (и уже в базовой части воспроизводимо).
При этом можно аккуратно различать test и prod, архитектуры,
версии, дистрибутивы, назначение...

> Да и какой функционал может быть повешен на имя директории?
> Пример можешь привести?

---
to get things done properly src/lib/dist.c::get_dist_conf_name()
would need fixup too -- probably looking at basename and then at
dirname would be fine (so that contrib/* would still work for
generally available templates but e.g. centos/6.0-x86_64 or
thatdepartment/debian-7.0-ns would be feasible too).
--- http://bugzilla.openvz.org/show_bug.cgi?id=2222#c0

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/