Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Периодическое падение разных команд, если включена переменная среды "RUNNER_ENV" #192

Closed
artbear opened this issue Nov 30, 2017 · 0 comments
Assignees
Milestone

Comments

@artbear
Copy link
Collaborator

artbear commented Nov 30, 2017

Периодическое падение разных команд, если включена переменная среды "RUNNER_ENV"

Из гиттера

Artur Ayukhanov @artbear 12:49
@/all Кто-нибудь сталкивался с проблемой падения команды ванесса-раннер из-за неудачной попытки удаления временных файлов?
на одном рабочем месте у клиента один из лог-файлов залочен и при попытке удаления команда падает 😟

Evhen Sosna @pumbaEO 12:50
нечего дебаг логи включать.

Artur Ayukhanov @artbear 12:51
что лучше сделать?
добавить доп.ключик для того, чтобы не падать при попытке удаления временных файлов
или добавить ключик, чтобы сделать некий таймаут перед повторной попыткой удаления
?
@pumbaEO ты же понимаешь, что наличие/отсутствие дебаг-логов не должно влиять на работоспособность продукта ?

Evhen Sosna @pumbaEO 12:52
Какие вообще временніе файлі там не может удалить? Для какой команді?

Artem Kuznetsov @arkuznetsov 13:00
@artbear Я бы просто информацию выдавал, что не удалось удалить файлы с указанием пути.

Artur Ayukhanov @artbear 13:00
для разных команд.
Удалить не может как раз отладочный лог ХХХ.vanessa-runner.log
я так понимаю, что лог не закрыт, и файл залочен.
если лог закрыть, то проблема должна уйти
но поведение странное - я же люблю логи и в других местах на эту ошибку не нарывался
а вообще этот файл не должен удаляться вообще 😀

Artur Ayukhanov @artbear 13:08
@pumbaEO А виноват-то остаток твоего кода 😀

Процедура ИнициализироватьЛоги(Знач КаталогЛогов)

    РежимРаботы = ПолучитьПеременнуюСреды("RUNNER_ENV");
    Если ЗначениеЗаполнено(РежимРаботы) И НРег(РежимРаботы) = "debug" Тогда
        УровеньЛога = УровниЛога.Отладка;
        Лог.УстановитьУровень(УровеньЛога);

        Аппендер = Новый ВыводЛогаВФайл();
        ИмяВременногоФайла = ОбщиеМетоды.ПолучитьИмяВременногоФайлаВКаталоге(КаталогЛогов, СтрШаблон("%1.log", ПараметрыСистемы.ИмяПродукта()));
        Аппендер.ОткрытьФайл(ИмяВременногоФайла);
        Лог.ДобавитьСпособВывода(Аппендер);

    КонецЕсли;

    Лог.УстановитьРаскладку(ЭтотОбъект);

КонецПроцедуры

оказывается, у нас в инструкции на ванесса-тфс на скриншоте стоит эта переменная 😟
ошибочно стоит 😟

Evhen Sosna @pumbaEO 13:09
Нет дебаг логи виноваты, они не нужны 99% времени работы скриптов.
единственно применения для дебагг логов я вижу только "накапливаем сообщения" и если необходимо выполнить исключение или выход из программы, только тогда выводим их.

Artur Ayukhanov @artbear 13:11
Женя, я ведь проверил код, файл с именем ХХХ,vanessa-runner.log создается только в этом блоке.
и переменная Runner_ENV была включена 😟

Artur Ayukhanov @artbear 13:30
@pumbaEO Так чего с такими логами сделаем?

Evhen Sosna @pumbaEO 13:30
Убирать.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant