каких рисков?

при чём тут mkdir, если речь про replace => false?

что может произойти если ему не поставить replace => 'no',

mkdir был про создание директории через команду

что может произойти если ему не поставить replace => 'no',

конфигурация может стать неконтролируемой

@Andorka а как ты предлагаешь депенденси у ресурсов Аля конфиг паттерн + сервис выстраивать ?

да как хочешь :)

где-то они на классы разбиты, где-то в одном классе всё в кучу

А причем тут классы и чейнинг ресурсов ?

ну классы же так же чейнятся

какая разница

Классы с точки зрения депенденси такой же ресурс

чота я уже упустил мысль, что мы обсуждаем

Так. И почему тогда классы можно чейнить а ресурсы нет ? )

Ты говорил вроде что чейнинг ресурсов это плохо

чоо

не говорил

А, ты имел ввиду что чейнинг лучше не у самих ресурсов делать а через ресурс тайпы где то внизу манифеста или через обращение через инстанцированное имя (квадратные скобки, большие буквы)

Д ?

да, но нет

не всегда

Ну всмысле вцелом это убирает слепоту, когда стрелочки не замечаешь.

зависит от сложности класса и вообще common sense

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

А еще можно заебаться с неймингом

Так то

Живёшь и думаешь что всё ок а потом выстреливает из глубины

бывает

в хороших модулях редкость

Ой...

но и сами такие модули тоже редкость

Нет, встроенная CA одна и тупенькая, но автосайн с 4.x можно делать умным (но тоже не без проблем): Policy-based autosigning. Я за внешнюю CA через cfssl какой-нибадь, но пока так и не довелось нигде сделать.

Спс. Policy-based autosign вчера потыкал уже. На первый взгляд работает. Скорее всего на нём и остановимся

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

и получается в 3\4 случаев всё собирается , а в одном каком то особенном варианте пересечения параметров вылетает undefined var

в хороших модулях редкость

ловил в паппетлабовских такое

я не говорил что совсем не существует

но оно вообще расстроило тогда

сидел сам писал по итогу аналог

а не проще было запатчить?

или унаследовать и переопределить

я очень жалею, что раньше этого делать не начал, и сначала пытался модули патчить или свои велосипедить

а так можно оторвать зависимость в классе который наследуешь?

я очень жалею, что раньше этого делать не начал, и сначала пытался модули патчить или свои велосипедить

хмммм

почитаю еще не делал такого

а так можно оторвать зависимость в классе который наследуешь?

class own::foo inherits module::foo { Resource['title'] { require => [], } примерно так. не всегда, конечно, можно

А я уже привык делать обертки к модулям :) боль-страдания

@ptchol твой выход

обёрткам не обязательно быть жирными просто. совсем без них не выйдет. но когда модуль нормально разбит на классы, можно выебнуться и вместо обёртки сделать попроще: include ::module include ::fixes::module::foo

чаще всего, правда, всё равно жирные врапперы, конечно

вот на такую репку $ find site -type f | wc -l 2035 вот столько "умных" маленьких фиксов с inherits: $ g grep -l lint:ignore:inheri | wc -l 8

вангую что половину уже можно грохнуть

ну и это всё-таки с шаблонами и типами

find site -name '*.pp' | wc -l

очень много вещей было бы проще, если б так делал сразу. а то уже три года не могу съехать на апстримный puppetlabs-rabbitmq, потому что мне понадобилось в самом начале пару мелочей "улучшить" там

find site -name '*.pp' | wc -l 1433 там темплейтов ещё куча :)

вангую что половину уже можно грохнуть

пока меньше. я пытался чистить, но слишком много усилий требует, потому что говнокод

господа - как вам класс

file { "$isdns_dir": ensure => 'present', replace => 'no', # this is the important property content => "From Puppet\n", owned => '$ora_user', group => '$ora_group', mode => '0644', }

говно

)))))))))))))))))))))))))))))))

syntax error

говно

+1

lint ругаться будет минимум на 3 строчки

а точно надо переменные в одинарные кавычки?

и тайтл тоже

а replace - разве не true/false?

там можно on/off

и в puppet cookbook почему-то это рекомендуют

и тайтл тоже

ну это хотя бы просто линтер ругнётся, оно хоть работать будет как ожидают

на одинарные же вроде тоже ругается

и в puppet cookbook почему-то это рекомендуют

его вообще обновляют? он же появился ещё когда 0.24 был

хз

говно

А как по феншую будет?

код не читал

на одинарные же вроде тоже ругается

ругается, но оно ещё и работать будет не так, как ожидают

сразу говном назвал

сразу говном назвал

т.к. паппет ?

только мне странным кажется, что переменная в тайтле называется isdns_DIR, а ensure => file, да ещё и с контентом?

т.к. паппет ?

ну лично мне папет как язык нравится больше чем например salt или terraform

Ну вобщем все написали что код не очень (говно), но никто не предложил своё. У меня кстати такое же "гавно" работает.

ты точно читал сообщения?

все (ха-ха), написали, что выглядит подозрительно, и может потом взорваться

Ну вобщем все написали что код не очень (говно), но никто не предложил своё. У меня кстати такое же "гавно" работает.

Это нормально, ведь любой код, который писал не ты или писал давно - говно

??? спасибо всем

причём "давно" нередко начинается с получаса

Это нормально, ведь любой код, который писал не ты или писал давно - говно

Ну да не я писал.

Все говно все сломать

@ptchol твой выход

А чо мой, ты молодец , врапперы модулей часто полезны

Я про то, что надо все докеризовать

Только люди не понимают часто разницы между враппером модуля и каким нить профилем

А. Это да

Уменьшает объем кода где то раз в 8

Я про то, что надо все докеризовать

Кому?

Кому?

SRE

Я про то, что надо все докеризовать

ты имеешь ввиду DOCKER ?

Да, докеризовать и кубернетизировать

Спасибо всем написал через файлы ГОВНО код который работает без екзеков