Skip to content

Kibana action connectors #306

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

Merged
merged 62 commits into from
May 23, 2023
Merged

Conversation

dimuon
Copy link
Contributor

@dimuon dimuon commented Apr 13, 2023

Fixes: #46.

Adds support for a Kibana action connectors based on a generated Kibana client. The client is generated from Kibana OpenAPI specs similar to #292.

Please refer to README for more details and known issues.

dimuon added 9 commits April 12, 2023 17:34
The generated code still has some drawbacks:
- ModelMap structure is missing (it's defined manually)
- OneOf generates a golang struct with promoted fields
  where each field represent one of the options.
  The generated client cannot properly encode such structs.
@dimuon dimuon marked this pull request as ready for review May 16, 2023 15:43
@dimuon
Copy link
Contributor Author

dimuon commented May 16, 2023

@tobio , can you please have a look at this?

@dimuon dimuon requested review from Kushmaro and tobio May 16, 2023 15:44
Copy link
Member

@tobio tobio left a comment

Choose a reason for hiding this comment

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

LGTM, couple of small questions. There's so much work here 👍 👍 👍

@dimuon
Copy link
Contributor Author

dimuon commented May 22, 2023

@tobio , I addressed all comments, I believe. Can you please have a look and approve it?

Copy link
Member

@tobio tobio left a comment

Choose a reason for hiding this comment

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

❤️ ❤️ ❤️ ❤️ ❤️ ❤️

@dimuon dimuon merged commit e4bcf44 into elastic:main May 23, 2023
@dimuon dimuon deleted the kibana-action-connectors branch May 23, 2023 08:09
@Kushmaro
Copy link
Contributor

GO TEAM @dimuon @tobio <3

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.

[Feature] Support for Alert Connectors
3 participants