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

[Obs AI Assistant] Move esql docs to dir that does not get deleted #174583

Merged
merged 3 commits into from
Jan 10, 2024

Conversation

dgieselaar
Copy link
Member

@dgieselaar dgieselaar commented Jan 10, 2024

any folder called docs is automatically deleted when creating a distributable. Rename it to esql_docs so this doesn't happen. We need tests, but will tackle that separately. Not sure how to write tests for the distributable.

@dgieselaar dgieselaar added release_note:skip Skip the PR/issue when compiling release notes ci:cloud-deploy Create or update a Cloud deployment ci:cloud-persist-deployment Persist cloud deployment indefinitely v8.12.0 v8.12.1 v8.13.0 labels Jan 10, 2024
@dgieselaar dgieselaar requested a review from a team as a code owner January 10, 2024 11:49
@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@dgieselaar dgieselaar enabled auto-merge (squash) January 10, 2024 12:48
@dgieselaar
Copy link
Member Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented Jan 10, 2024

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@dgieselaar dgieselaar merged commit f9c8344 into elastic:main Jan 10, 2024
21 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 10, 2024
…ic#174583)

any folder called `docs` is automatically deleted when creating a
distributable. Rename it to `esql_docs` so this doesn't happen. We need
tests, but will tackle that separately. Not sure how to write tests for
the distributable.

(cherry picked from commit f9c8344)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.12

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jan 10, 2024
…#174583) (#174603)

# Backport

This will backport the following commits from `main` to `8.12`:
- [[Obs AI Assistant] Move esql docs to dir that does not get ...
(#174583)](#174583)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Dario
Gieselaar","email":"dario.gieselaar@elastic.co"},"sourceCommit":{"committedDate":"2024-01-10T13:53:43Z","message":"[Obs
AI Assistant] Move esql docs to dir that does not get ...
(#174583)\n\nany folder called `docs` is automatically deleted when
creating a\r\ndistributable. Rename it to `esql_docs` so this doesn't
happen. We need\r\ntests, but will tackle that separately. Not sure how
to write tests for\r\nthe
distributable.","sha":"f9c83448c94713a227aa62aae223c90db852e6fa","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","ci:cloud-deploy","ci:cloud-persist-deployment","v8.12.0","v8.12.1","v8.13.0"],"title":"[Obs
AI Assistant] Move esql docs to dir that does not get
deleted","number":174583,"url":"https://github.com/elastic/kibana/pull/174583","mergeCommit":{"message":"[Obs
AI Assistant] Move esql docs to dir that does not get ...
(#174583)\n\nany folder called `docs` is automatically deleted when
creating a\r\ndistributable. Rename it to `esql_docs` so this doesn't
happen. We need\r\ntests, but will tackle that separately. Not sure how
to write tests for\r\nthe
distributable.","sha":"f9c83448c94713a227aa62aae223c90db852e6fa"}},"sourceBranch":"main","suggestedTargetBranches":["8.12"],"targetPullRequestStates":[{"branch":"8.12","label":"v8.12.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.13.0","branchLabelMappingKey":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/174583","number":174583,"mergeCommit":{"message":"[Obs
AI Assistant] Move esql docs to dir that does not get ...
(#174583)\n\nany folder called `docs` is automatically deleted when
creating a\r\ndistributable. Rename it to `esql_docs` so this doesn't
happen. We need\r\ntests, but will tackle that separately. Not sure how
to write tests for\r\nthe
distributable.","sha":"f9c83448c94713a227aa62aae223c90db852e6fa"}}]}]
BACKPORT-->

Co-authored-by: Dario Gieselaar <dario.gieselaar@elastic.co>
delanni pushed a commit to delanni/kibana that referenced this pull request Jan 11, 2024
…ic#174583)

any folder called `docs` is automatically deleted when creating a
distributable. Rename it to `esql_docs` so this doesn't happen. We need
tests, but will tackle that separately. Not sure how to write tests for
the distributable.
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
…ic#174583)

any folder called `docs` is automatically deleted when creating a
distributable. Rename it to `esql_docs` so this doesn't happen. We need
tests, but will tackle that separately. Not sure how to write tests for
the distributable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:cloud-deploy Create or update a Cloud deployment ci:cloud-persist-deployment Persist cloud deployment indefinitely release_note:skip Skip the PR/issue when compiling release notes v8.12.0 v8.12.1 v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants