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

Ошибка в master в КомандаУправлениеСеансами.os #36

Closed
1cgh opened this issue Jul 17, 2018 · 15 comments
Closed

Comments

@1cgh
Copy link

1cgh commented Jul 17, 2018

Ошибка в master в КомандаУправлениеСеансами.os
Ну и, соответственно, в хабе в пакете то же самое.

Функция ПолучитьСписокРабочихПроцессов()

	КомандаЗапускаПроцессы = СтрокаЗапускаКлиента() + СтрШаблон("process list --cluster=""%1""%2",
	ИдентификаторКластера(), 
	КлючиАвторизацииВКластере()) + " " + мНастройки.АдресСервераАдминистрирования;
	
	Лог.Информация("Получаю список рабочих процессов...");
	СписокПроцессов = ВыполнитьКоманду(КомандаЗапускаПроцессы);

Падает на
СписокПроцессов = ВыполнитьКоманду(КомандаЗапускаПроцессы);
Там ожидается коллекция параметров, а передается строка КомандаЗапускаПроцессы

@EvilBeaver
Copy link
Member

СписокПроцессов = ВыполнитьКоманду(КомандаЗапускаПроцессы);
Там ожидается коллекция параметров, а передается строка КомандаЗапускаПроцессы

Я вот эту фразу не понял... Что там происходит?

@1cgh
Copy link
Author

1cgh commented Jul 17, 2018

@EvilBeaver

https://github.com/oscript-library/deployka/blob/master/src/%D0%9A%D0%BB%D0%B0%D1%81%D1%81%D1%8B/%D0%9A%D0%BE%D0%BC%D0%B0%D0%BD%D0%B4%D0%B0%D0%A3%D0%BF%D1%80%D0%B0%D0%B2%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5%D0%A1%D0%B5%D0%B0%D0%BD%D1%81%D0%B0%D0%BC%D0%B8.os#L441
Функция ПолучитьСписокРабочихПроцессов()
СписокПроцессов = ВыполнитьКоманду(КомандаЗапускаПроцессы);

https://github.com/oscript-library/deployka/blob/master/src/%D0%9A%D0%BB%D0%B0%D1%81%D1%81%D1%8B/%D0%9A%D0%BE%D0%BC%D0%B0%D0%BD%D0%B4%D0%B0%D0%A3%D0%BF%D1%80%D0%B0%D0%B2%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5%D0%A1%D0%B5%D0%B0%D0%BD%D1%81%D0%B0%D0%BC%D0%B8.os#L62
Функция ВыполнитьКоманду(Знач ПараметрыКоманды) Экспорт
ПрочитатьПараметры(ПараметрыКоманды);

https://github.com/oscript-library/deployka/blob/master/src/%D0%9A%D0%BB%D0%B0%D1%81%D1%81%D1%8B/%D0%9A%D0%BE%D0%BC%D0%B0%D0%BD%D0%B4%D0%B0%D0%A3%D0%BF%D1%80%D0%B0%D0%B2%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5%D0%A1%D0%B5%D0%B0%D0%BD%D1%81%D0%B0%D0%BC%D0%B8.os#L86
Процедура ПрочитатьПараметры(Знач ПараметрыКоманды)
Для Каждого КЗ Из ПараметрыКоманды Цикл

Из ПолучитьСписокРабочихПроцессов() вызывается ВыполнитьКоманду(), откуда уже вызывается ПрочитатьПараметры()

И в ПрочитатьПараметры() ожидается коллекция, а приходит строка

@1cgh
Copy link
Author

1cgh commented Jul 17, 2018

Получаем
{Модуль C:\Program Files (x86)\OneScript\lib\deployka\src\Классы\КомандаУправлениеСеансами.os / Ошибка в строке: 86 / Значение не является значением объектного типа}

@EvilBeaver
Copy link
Member

Блин. Это же 100% исправлялось, и давно.

@EvilBeaver
Copy link
Member

@nixel2007
Copy link
Member

image

@EvilBeaver
Copy link
Member

а ospx собрался и запушился с библиотеки, походу?

@nixel2007
Copy link
Member

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

@EvilBeaver
Copy link
Member

Я вижу конкретное расхождение кода в двух деплойках... Сейчас попытаюсь собрать воедино

@EvilBeaver
Copy link
Member

Собрал релиз на базе upstream. Разошедшиеся возможности частично черри-пикнул из os-library, а конфликтующие изменения в классах СерверАдминистрирования и команде управления рег.заданиями оставил в ветке conflicted-scheduler

@EvilBeaver
Copy link
Member

@1cgh Огромная просьба скачать и посмотреть как работает

@1cgh
Copy link
Author

1cgh commented Jul 18, 2018

@EvilBeaver реп этот, ветка master?

@EvilBeaver
Copy link
Member

EvilBeaver commented Jul 18, 2018

Лучше из opm поставить

@1cgh
Copy link
Author

1cgh commented Jul 18, 2018

@EvilBeaver у нас всё ок. можем закрыть или еще понаблюдать
пока вот думаем, что делать с #34
аналогичная есть проблема

@EvilBeaver
Copy link
Member

Ну я там отписался уже. ЕМНИП деплойка убивает и сеансы и соединения, но если соединения обращались в этот момент к СУБД, то они не убиваются. Аналогично происходит и в консоли кластера, так что можно считать это штатным поведением.

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

No branches or pull requests

3 participants