https://github.com/ansible/ansible/issue...
2018-02-13 10:17:44

Участники:
@blaze - 9, @oxpa - 8, @tzirechnoy - 4, @proforg - 3, @gbdj - 2, @solhov - 1, @seeker - 1

@seeker
мимими какое
#2897887/1 2018-02-13 10:21:09
@tzirechnoy
Особенно учитывая, что 8.8.8.8 имете смысл дивертить примерно во вторую очередь...
#2897887/2 2018-02-13 10:43:00
@oxpa
Замечательная манера репортить проблемы у чувака.
#2897887/3 2018-02-13 10:43:11
@tzirechnoy
То есть вот на машынке, с которой я сейчас сижу -- во времена до блэклистов был почему-то в локалке закрыт DNS, так что как раз 8.8.8.8 шёл в vpn. Сейчас, правда, всё идёт в vpn, только выбранные -- в локалку, но вот просто как факт.
#2897887/4 2018-02-13 10:44:53
@proforg
The code in the issue seems self-descriptive to several people in this thread. Do you really need any additional feedback from issue author?
#2897887/5 → /3 2018-02-13 14:03:17
@oxpa
Сегодня ты принимаешь такой issue по очевидной проблеме, а завтра ебланавты пишут все репорты так. Важность у этой проблемы явно не настолько высокая, чтобы продолжать разговор с автором.
#2897887/6 → /5 2018-02-13 14:14:39
@blaze
Важность этой проблемы очень высокая. Этот код все равно что надпись "хуй" на лбу авторов проекта, мне кажется крайне важным не ходить с такой надписью на лбу. Хотя как показывает пример systemd, даже если этой надписью гордится, успеху это ничуть не препятствует.
#2897887/7 → /6 2018-02-13 15:45:58
@proforg
вот для таких, козлов, как вы, code of conduct и придумали!
#2897887/8 → /7 2018-02-13 15:48:06
@blaze
LOL!
#2897887/9 → /8 2018-02-13 15:49:01
@oxpa
Наверно, стоит расписать, что делает проблему действительно важной. А не вот эти ваши аналогии: "код что хуй".
#2897887/10 → /7 2018-02-13 15:50:19
@blaze
Хотя я бы, конечно, высказался более иносказательно. В стиле "мне кажется, этот код ставит под сомнение профессиональную компетенцию, душевное здоровье и видовую принадлежность автора и ревьюеров"
#2897887/11 → /9 2018-02-13 15:50:28
@blaze
Как в /11 подойдет?
#2897887/12 → /10 2018-02-13 15:54:02
@oxpa
issue Должен хоть в какой-то мере предполагать решение проблемы. Если тебя не устраивает видовая принадлежность авторов или душевное состояние разработчиков, вряд ли ты можешь это изменить. Если есть нереализованное желание поупражняться в витиеватоизысканных формулировках двусмысленных оскорблений, можно съездить в Одессу. Она славится подобным подходом. А issue в проекты не должны содержать оскорблений. А в идеале, должны быть вообще pull request'ами
#2897887/13 → /12 2018-02-13 15:57:48
@proforg
нет, issue не должен предполагать решение проблемы, но должен её демонстрировать. а уж воспринимать ли такое описание как оскорбление и отвечать в стиле "да, мы ахуели", или подходить к вопросу конструктивно - это личный выбор каждой конкретной команды разработчиков.
#2897887/14 → /13 2018-02-13 16:02:33
@oxpa
ну... а какую проблему демонстрирует issue в топике? Может стоит написать? Кажется очевидным, что люди, которые это приняли, проблему там не видят
#2897887/15 → /14 2018-02-13 16:03:49
@blaze
То есть мне предлагается считать разработчиков проекта не спьяну сделавшими глупость, а некомпетентными и нуждающимися в разъяснении типа "совершенно не факт, что маршрут к любому произвольному адресу будет идти через default route". Это точно менее оскорбительно, чем "вы ебанулись"? Мы же не о кругозоре говорим, а о компетентности в области, которая является ключевой для проекта. И это вроде как не интерны, а люди с опытом и т.д. Вообще, вспоминается история о том, как автора docker не взяли в google, кажется. Если не примут этих, я ни секунды не удивлюсь.
#2897887/16 → /13 2018-02-13 16:03:54
@oxpa
я даже больше скажу, не факт, что default route вообще существует. И непонятно, чего хочет автор, что он, собственно, считает проблемой
#2897887/17 → /16 2018-02-13 16:05:08
@blaze
Попробуй перечислить варианты, что могло бы показаться проблемой в этом коде.
#2897887/18 → /17 2018-02-13 16:10:35
@oxpa
Не хочу. Меня всё устраивает в этом коде: везде, где у меня есть default route, маршрут до 8.8.8.8 совпадает с маршрутом до 0.0.0.0. Код работает (а я его не использую, потому что bsd)
#2897887/19 → /18 2018-02-13 16:11:49
@blaze
Welcome aboard!.gif
#2897887/20 → /19 2018-02-13 16:22:02
@tzirechnoy
systemd и куча других примеров показывает, что такая надпись на лбу успеху как раз сопутствует.
#2897887/21 → /7 2018-02-13 16:22:47
@tzirechnoy
systemd и куча других примеров показывает, что такая надпись на лбу успеху как раз сопутствует.
#2897887/22 → /7 2018-02-13 16:22:51 X
@gbdj
Какого хрена в инфраструктурном проекте делает эта волшебная константа?! WTF? Это не смотря на исходник за пределами этой строчки. Но если посмотреть: какого хрена вместо лукапа дефолтного префикса по таблице марщрутизации берется итого вое решение для какоко-то адреса? А все это ради определения дефолтного интерфейса. Т.е. по факту у тебя может что нибудь сконфигурироватся не на том интерфейсе и хорошо если не ssh или firewall. И удачи тебе понять в чем проблема не вчитываясь полдня в говнокод на питоне.
#2897887/23 → /17 2018-02-13 17:48:26
@gbdj
Основная проблема в том что подобные действия в коде никто не ожидает. Поэтому при отладке оно абсолютно не укладывается в модель возможных проблем. Если причина настолько идиоткая, быстро выловить ее не получится и отладка представляет обычно долбление головой об стену. По этому за такой бред в коде надо бить ногами. Пойду git blame посмотрю.
#2897887/24 → /15 2018-02-13 18:00:08
@oxpa
А теперь сравни свой комментарий с "вы ебанулись??" и найди 10 отличий. Или представь, что я ответил "нипизди!", а ты сиди и догадывайся, что не так. Я понимаю претензию к коду. Нет необходимости мне её объяснять. Я всего лишь хочу показать, что это объяснение нужно ТАМ, вместо обсценной лексики.
#2897887/25 → /23 2018-02-14 09:31:18
@solhov
да нет там никакой проблемы. как известно, ansible -- это инструмент devops, а devops -- это такие программисты-неудачники, которые думают что они как админы будут невъебенно круты и вообще все as code. ну собственно именно это они этим кодом и демонстрируют. через жопу, неправильно и не вообще не то, что нужно. исправлять это только по рецепту луговского
#2897887/26 → /15 2018-02-14 09:56:46
@blaze
Они и интерфейсы-то определяют, парся вывод ifconfig. Я старательно сдерживался в обсуждении, но вот про devops совершенно правильно сказано.
#2897887/27 → /23 2018-02-14 17:14:35
@blaze
Все это сведется к http://i0.kym-cdn.com/entries/icons/original/000/005/908/idontwanttoliveonthisplanet.jpg
#2897887/28 → /25 2018-02-14 17:16:29