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.

Sign up for GitHub

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

Jump to bottom

в именах переменных среды символ дефиса не корректно работает в сборочных линиях #343

Closed
thedemoncat opened this issue Feb 5, 2020 · 3 comments
Closed

в именах переменных среды символ дефиса не корректно работает в сборочных линиях #343

thedemoncat opened this issue Feb 5, 2020 · 3 comments
Labels
technical debt
Milestone
1.10.0

Comments

@thedemoncat
Copy link
Contributor

thedemoncat commented Feb 5, 2020 •
edited
Loading

такой блок в дженкинсе работать не будет:
environment{
RUNNER_storage-user = "user"
RUNNER_storage-pwd = "СекретныйПароль"
}
но, тут https://github.com/silverbulleters/vanessa-runner/blob/develop/src/main.os#L40 заявлены именно такие переменные среды
И вообще не совсем корректные имена. Обычно они пишутся заглавными буквами.

Предлагаю дополнить структуру новыми параметрами, продублировав текущие.
@artbear
Copy link
Collaborator

artbear commented Feb 6, 2020

Да, согласен с новыми именами параметров. Сделаешь ПР?
@thedemoncat

@thedemoncat
Copy link
Contributor Author

Да.

thedemoncat added a commit to thedemoncat/vanessa-runner that referenced this issue Feb 10, 2020
@thedemoncat
Добавил альтернативные параметры vanessa-opensource#343
15d1f34
artbear added a commit that referenced this issue Feb 14, 2020
@thedemoncat @artbear
Альтернативные имена переменных среды (#346)
7d5b1cf 
* Изменено описание параметров конфигурационных файлов

* Обеспечиваем существование каталога при формировании отчета #194

* Текст "Ошибок не обнаружено" не является ошибкой проверки синтаксиса #334

* Поправил описание параметров

* убрал ненужный комментарий

* удалил запятую

* Добавил английское No syntax errors found

* Использую ФС.ОбеспечитьПустойКаталог вместо собственного велосипеда

* избавился от каталога отчета junit

* Еще одно английское сообщение No errors found

* Добавил альтернативные параметры #343

* Исправил ошибку RUNNER_UCCODE вместо RUNNER_UCODE

* Исправил на RUNNER_STORAGE_PWD вместо RUNNER_STORAGE_PSW

Co-authored-by: Artur Ayukhanov <[email protected]>
@artbear
Copy link
Collaborator

artbear commented Feb 14, 2020

исправлено в #346

@artbear artbear closed this as completed Feb 14, 2020
@artbear artbear added this to the 1.10.0 milestone Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
technical debt
Projects
None yet
Milestone
1.10.0
Development

No branches or pull requests
2 participants
@artbear @thedemoncat