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

Add insight schemas API #23

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Add insight schemas API #23

merged 1 commit into from
Dec 18, 2023

Conversation

NikolayCheremnov
Copy link
Contributor

@NikolayCheremnov NikolayCheremnov commented Dec 6, 2023

Добавлена реализация schemas API для сервиса insight.
Добавлены примеры и документация.

Бамп версии будет сделан после принятия решения о ветках.

Copy link
Contributor

@slowaner slowaner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Надо ещё дополнить документацию и бампнуть версию.

Нужен новый раздел документации про работу с задачами.

cybsi/cloud/insight/schemas.py Outdated Show resolved Hide resolved
@neexee
Copy link
Member

neexee commented Dec 6, 2023

Сначала думал, что это не стоит вливать в main до того, как он insight станет доступен на https://cybsi.cloud. Лучше сделать отдельную ветку, 1.1.0-dev, например, и вливать подобные MR-ы в неё. Версию там можно пока не бампать. Бампнем версию sdk когда будем вливать 1.1.0-dev в main.

Но лучше всё-таки влить в main, видимо. Тогда смежным командам можно будет на интеграционных стендах новые фичи попробовать, пользуясь свежей версией sdk.

@nsofanat1k Что думаешь?

@slowaner
Copy link
Contributor

slowaner commented Dec 6, 2023

Но лучше всё-таки влить в main, видимо. Тогда смежным командам можно будет на интеграционных стендах новые фичи попробовать, пользуясь свежей версией sdk.

Можно из новой ветки выпустить версию типа 1.1.0a1, тогда и main сохраним и возможность для интеграции будет

@pvragov
Copy link
Collaborator

pvragov commented Dec 7, 2023

Сначала думал, что это не стоит вливать в main до того, как он insight станет доступен на https://cybsi.cloud. Лучше сделать отдельную ветку, 1.1.0-dev, например, и вливать подобные MR-ы в неё. Версию там можно пока не бампать. Бампнем версию sdk когда будем вливать 1.1.0-dev в main.

Но лучше всё-таки влить в main, видимо. Тогда смежным командам можно будет на интеграционных стендах новые фичи попробовать, пользуясь свежей версией sdk.

@nsofanat1k Что думаешь?

Я бы отводил ветку 1.1.0. Если нам нужно будет выпустить патч на 1.0.0, то с ним могут поехать недоделанные куски, чего не хотелось бы.

@slowaner
Copy link
Contributor

slowaner commented Dec 7, 2023

Я бы отводил ветку 1.1.0. Если нам нужно будет выпустить патч на 1.0.0, то с ним могут поехать недоделанные куски, чего не хотелось бы.

А не лучше ли в main вести основную разработку, а отвести, как раз, ветку 1.0?

@pvragov
Copy link
Collaborator

pvragov commented Dec 8, 2023

Я бы отводил ветку 1.1.0. Если нам нужно будет выпустить патч на 1.0.0, то с ним могут поехать недоделанные куски, чего не хотелось бы.

А не лучше ли в main вести основную разработку, а отвести, как раз, ветку 1.0?

такой подход имеет смысл, если мы поддерживаем несколько версий приложения (как в коробке cybsi, например). В клауде всегда будет актуальна только последняя версия sdk.

@slowaner
Copy link
Contributor

slowaner commented Dec 8, 2023

Создал ветку 1.1.0.

@NikolayCheremnov NikolayCheremnov force-pushed the add-insight-schemas-api branch 3 times, most recently from b8fd6d8 to f97924c Compare December 8, 2023 09:58
docs/user/enrichment.rst Outdated Show resolved Hide resolved
@neexee neexee changed the base branch from main to 1.1.0 December 10, 2023 12:13
Copy link
Collaborator

@pvragov pvragov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

можно вливать. Советую вливаться после #25.

@NikolayCheremnov NikolayCheremnov merged commit f8d61b5 into 1.1.0 Dec 18, 2023
7 checks passed
@NikolayCheremnov NikolayCheremnov deleted the add-insight-schemas-api branch December 18, 2023 06:30
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

Successfully merging this pull request may close these issues.

4 participants