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

Разделить неудачный результат выполнения команды от падения приложения #575

Open
yukon39 opened this issue May 11, 2023 · 4 comments

Comments

@yukon39
Copy link
Contributor

yukon39 commented May 11, 2023

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

В частности, это касается команд: syntax-check, edt-validate, session closed, run, xunit и vanessa

Предложение:

  • В целях обратной совместимости, при неуспехе выполнения команд возвращать exitCode = 1, а для ошибок времени исполнения использовать, например, exitCode = 255
  • ошибки выполнения от 1С останутся с кодом 1, что важно для тестирования или прогона фич
  • а ошибки, выдаваемые сразу от ванесса-раннер, пусть будут 255
@artbear
Copy link
Collaborator

artbear commented Jun 29, 2024

@yukon39 не согласен,
1 как определить, в чем разница между "неуспехе выполнения команд" и "ошибки времени исполнения"
в чем между ними разница?

@yukon39
Copy link
Contributor Author

yukon39 commented Jun 29, 2024

Первое - ошибка запуска вообще, типа неверная платформа, авторизация, ключ е найден и т.п.

Второе - предприятие запустилось, но возникла ошибка при выполнении встроенного кода.

Для чего. Например, для Дженкинса первый тип ошибок - это пометка шага ABORTED, как например, истечение таймаута.
Второй тип - пометка красным FAILED или желтым UNSTABLE в зависимости от нужд пайплайна.

@artbear
Copy link
Collaborator

artbear commented Jun 29, 2024

Тогда предлагаю эти коды поменять местами, пусть ошибки выполнения от 1С останутся 1, что важно для тестирования или прогона фич
а вот ошибки ванесса-раннер пусть будут 255

@yukon39
Copy link
Contributor Author

yukon39 commented Jun 29, 2024

Можно и так, да. Сейчас поправлю

@artbear artbear modified the milestones: 1.13.0, 1.12.0 Jun 29, 2024
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

2 participants