https://opencarbon.ru/ 2026-04-21T11:46:33-04:00 https://opencarbon.ru/ https://opencarbon.ru/lib/tpl/bootstrap3/images/favicon.ico text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:213?rev=1558379903&do=diff Текст удален без Утверждения, обратитесь к Owner ~~OWNERAPPROVE~~ text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-02-07_глухая_отправка_уведомлений_об_ошибках_из_приложения?rev=1558379903&do=diff Ошибка: Код бекапа проверяет свободное место и если его не хватает - шлет alarm и падает. if [ $node_backup_freespace -le $mysize ]; then /node/bin/alarm "no free space for bk $VM_NODE_BACKUP $VM_DIR $node_backup_freespace $mysize" exit 1 fi text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-03-20_попутно_ничего_нельзя_менять?rev=1558379903&do=diff Нарушено правило: никогда ничего нельзя править попутно при решении задачи. Постановка задачи: добавить текст сообщения «Демо-версия». text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-04-01_избыточное_решение?rev=1558379903&do=diff Нарушено правило: Не нужно решать простую задачу слишком хорошо с запасом на будущее. Постановка задачи: Добавить проверку, если было более 3 FATAL задач обращений в тех.поддержку то создать задачу для проведения аудита клиента руками разработчика. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-04-12_не_протестировал_код?rev=1558379903&do=diff Нарушено правило: Тестировать код после любых изменений. Постановка задачи: Использовать в функции определения выходного техподдержки функции из библиотеки date_lib. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-06-06_обработка_лог.файлов?rev=1558379903&do=diff Нарушено правило: Полная обработка лог.файлов неизвестного размера в неконтроллируемой системе. Постановка задачи: Анализировать в системе мониторинга /var/log/messages на предмет ошибок raid, дисковой и файловой подсистемы. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-07-18_использование_не_популярного_сахара?rev=1558379903&do=diff Использование не всем известного сахара усложняет чтение кода и приводит к потенциальным ошибкам в будущем --- [ "$date_now" -le "$date_min" ] && echo NIGHT && return 0 +++ [ "$date_now" -le "$date_min" ] && { echo NIGHT; return 0; } --- if [ "$method" == "auth" ] || [ "$method" == "web" ] || [ "$method" == "xml" ]; then +++ if [ "$method" = "auth" -o "$method" = "web" -o "$method" = "xml" ]; then ---[ -n $cert_date_new_start ] && [[ "$cert_date_new_start" -gt "$CERT_DATE_CUR" ]] && return 0… text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-09-04_всегда_нужно_проверять_свои_изменения?rev=1558379903&do=diff Нарушено правило: нужно тестировать любые изменения. Постановка задачи: мигрировать виртуальную машину на другой сервер. Ошибка: мигрировал, понадеявшись что все будет работать штатно. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-09-24_подстели_подушку_заранее_при_опасных_операциях?rev=1558379903&do=diff Ошибка: Изменение правил iptables на удаленном сервере без плана на случай ошибок. Добавили правило, из-за которого пропал доступ к продакшен-серверу. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-11-19_вредный_хардкод?rev=1558379903&do=diff Проблема: в коде сайта «захардкожен» его адрес. Например, в коде crm на странице со ссылками на различные отчеты: <a href='http://example.com/crm/carbon/report1.php'>Подробный отчет о клиентах</a><br/> <a href='https://example.com/crm/carbon/report2.php?month=&year='>Отчет оценки</a><br/> <a href='http://example.com/crm/carbon/report3.php'>Просроченные задачи</a><br/>… text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-11-21_перенаправление_всего_вывода_в_утилитах?rev=1558379903&do=diff Что произошло: Нельзя полностью перенаправлять вывод утилит в лог. Это нарушение strongbash020. Постановка задачи: Сделать вменяемый вывод /app/reductor/service restart . text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-12-04_молчаливые_ошибки?rev=1558379903&do=diff Постановка задачи: Добавить опцию в продукт Reductor, которая включает экспериментальные модули ядра. Проблема: Выяснилось, что модули несовместимы с некоторыми версиями ядра. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-12-04_скрытие_экспериментальных_опций?rev=1558379903&do=diff Постановка задачи: Добавить опцию, включающую экспериментальный функционал Проблема: Разработчик решил скрыть опцию из интерфейса конфигурации, чтобы пользователь не смог активировать ее, что может привести к проблемам text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2018-12-11_непродуктовые_опции_в_конфигурации?rev=1558379903&do=diff Постановка задачи: у клиента понадобилось повысить размеры таблицы conntrack и таблицы маршрутов в ядре для повышения производительности. При этом, текущие значения уже регулируются скриптами. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2019-01-11_отображение_ошибок_пользователю?rev=1558379903&do=diff Короткая версия статьи: Ситауция: на сайте мониторинга есть список сенсоров и их состояний «ОК» или «Есть проблема». Какое должно быть состояние, если в код сенсора завершился с ошибкой? При этом, сенсор выполняется на наших серверах, а не клиентских. text/html 2019-05-20T15:18:23-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2019-04-04_требуется_проверять_наличие_новой_функции_в_конце_работы?rev=1558379903&do=diff Постановка задачи: требовалось сделать резервный файл выгрузки, чтобы у клиентов не происходило проблем из-за старого файла выгрузки. При этом, в некоторых случаях происходила замена файла резервным. text/html 2019-06-03T11:54:48-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2019-06-03_настороженно_относиться_к_необкатанным_решениям?rev=1559577288&do=diff Постановка задачи: Нужно было добавить LOG-правило в iptables для логирования определенного трафика. Как сделали: Чтобы защитить логи от флуда - было решено использовать лимит, наиболее гибко под задачу можно было настроить hashlimit. Так и сделали: text/html 2019-07-17T11:31:05-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2019-07-17_нельзя_использовать_nice?rev=1563377465&do=diff Планировал использовать nice на ресурсоёмкие задачи, что-бы они не роняли слабые машины. Правило: Запрещено использовать nice Почему: Спросить OSV text/html 2019-08-26T05:27:27-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2019-08-26_нельзя_использовать_механизм_ошибок_для_бизнес-логики?rev=1566811647&do=diff Нарушено правило: Программы пишутся для пользователей и программистов, а не для компьютера, все должно быть понятно для человека. Задача: text/html 2019-10-19T14:04:53-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2019-09-23_обработка_результата_пайпом?rev=1571508293&do=diff Нарушено правило: StrongBash Задача: Обойти все зоны, выдаваемые другой командой. cloudfox dns info "$DNS_NAME" --zones | while read -r DNS_ZONE; do .... done Ошибка: Если при выполнении text/html 2020-01-14T09:22:36-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2020-01-14_параметры_в_bash_без_кавычек?rev=1579011756&do=diff Нарушено правило: Опасная обработка параметров. Ошибка: Написал код, который обрабатывает параметры: if [ $1 = info ]; then cloud_conf node show "$NODE_NAME" fi text/html 2020-01-28T02:13:10-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2020-01-28_использовать_импорт_переменных_вместо_параметров?rev=1580195590&do=diff Постановка задачи: При вызове супервизора передавать owner_id. Проблема: В конфигурационных файлах vm, cdn и dns записывались разные значения для параметра OWNER. text/html 2020-02-19T03:39:47-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2020-01-28_не_плодить_переменные_в_баше?rev=1582101587&do=diff Создал лишнюю переменную в баше, чего делать категорически нельзя: local hostname="${ARG_HOSTNAME:-}" Вместо этого нужно было использовать "${ARG_HOSTNAME:-}" text/html 2020-06-22T05:22:51-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2020-06-16_не_правильное_использование_tail_и_grep?rev=1592817771&do=diff Ошибка: скрипт постоянно находит трейсбеки в логах, даже исправленные. Как не надо делать: Был написан скрипт, который грепает трейсбеки из последних 2к строк логах: text/html 2021-07-07T08:38:21-04:00 https://opencarbon.ru/правила_разработки:как_не_надо_делать:2021-07-07_все_пользовательские_данные_в_учетных_системах?rev=1625661501&do=diff Все пользовательские данные в учетных системах должны быть доступны для ручного изменения администратором. Постановка задачи: Добавить новое поле в базу данных CRM, для учета новой тарифной опции.