мир и поэтому теперь надо писать так
===
Уважаемый коллега, вы проделали классную работу, спасибо за код, который показали. Всё отлично, но есть пара нюансов. Для того, чтобы он стал совсем бриллиантового качества, советую вам прочитать вот этот (ссылка) и вот этот (ссылка) раздел документации ansible, а так же best practice вот тут (ссылка).
Обратите внимание на hostvars[node1], это плохо потому и поэтому. На server_id==1, это плохо потому и поэтому. Ход с delegate_to localhost отличный, я бы не догадался, ведь обычно так никто не делает, но это потому что они токсичные душнилы, а вы молодец - какой свежий ход!
Немного не хватает идемпотентности, но если кто-то вам скажет, что это плохо, нажмите на нем игнор, чтобы не расстраивал и снижал вам мотивацию, а другим коллегам объясните, что вам этот код нужен для того, чтобы запустить один раз, поэтому незачем токсить душноту.
Успехов!
===
так было бы лучше? все целуют взасос жопы единорогов, объема много, смысла не больше ни на байт. по факту хотя бы из-за идемпотентности это не ансибл, а баш, о чем сразу было и написано. если это playbook на один раз, поэтому упарываться не надо, так надо залогиниться шеллом в машину и скомандовать, чего уж стесняться.
аргументы "сам покажи, как умеешь / сделай вместо меня сам" обычно проходят в районе старшей группы детского сада, когда ими оперируют взрослые люди в отрасли, это странно.
хочешь нормального ревью (а не чтобы вместо тебя написали), относись к коллегам с уважением. хуякнуть сначала скриншот, потом остроумно (nyet) пошутить про мониторы, потом ебануть в чат портянку с переносом строк и без подсветки (хотя не то что намекнули, а прямо сказали про gist) - это конечно софтскилльно, единорожно и розовопонье. а все вокруг - токсичные мудаки, которые почему-то не хотят "показать, как надо"
Взасос жопы ебидорогов
Обсуждают сегодня