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

[ES|QL] add docs step to sync pipeline #186011

Merged
merged 16 commits into from
Jun 13, 2024

Conversation

drewdaemon
Copy link
Contributor

@drewdaemon drewdaemon commented Jun 11, 2024

Summary

#184689 added a script to gather ES|QL function docs from Elasticsearch.

This PR adds it to our CI pipeline!

Close #173492
Close #173817

@drewdaemon
Copy link
Contributor Author

/ci

@drewdaemon drewdaemon marked this pull request as ready for review June 11, 2024 20:47
@drewdaemon drewdaemon requested review from a team as code owners June 11, 2024 20:47
@drewdaemon drewdaemon added release_note:skip Skip the PR/issue when compiling release notes Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana labels Jun 12, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

Copy link
Contributor

@delanni delanni left a comment

Choose a reason for hiding this comment

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

Wouldn't it make sense to merge this with the esql_generate_function_definitions.sh? They're doing a lot of similar things besides the exact generation - and I don't know the background to this, but maybe the docs & functions are often updated together, so why not generate them in a single step & single PR?

@stratoula
Copy link
Contributor

This is a good point but @drewdaemon knows better

@drewdaemon
Copy link
Contributor Author

@delanni good idea. I'll make this change.

Copy link
Contributor

@delanni delanni left a comment

Choose a reason for hiding this comment

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

wow, looks so much slimmer now 👍

@stratoula
Copy link
Contributor

Niiiiice

@drewdaemon
Copy link
Contributor Author

@elasticmachine merge upstream

@drewdaemon
Copy link
Contributor Author

Geez... the i18n check takes forever

@drewdaemon
Copy link
Contributor Author

Going to increase the timeout until #185035 lands

@drewdaemon drewdaemon enabled auto-merge (squash) June 12, 2024 20:12
@drewdaemon
Copy link
Contributor Author

/ci

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@drewdaemon drewdaemon merged commit ca98a8b into main Jun 13, 2024
24 checks passed
@drewdaemon drewdaemon deleted the operationalize-docs-generator-script branch June 13, 2024 00:42
@kibanamachine kibanamachine added v8.15.0 backport:skip This commit does not require backporting labels Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes Team:ESQL ES|QL related features in Kibana v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ES|QL][Meta] Keep in sync with ES [ES|QL] Automatic creation of inline documentation
7 participants