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

Enable summarisation spec #199134

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

sorenlouv
Copy link
Member

@sorenlouv sorenlouv commented Nov 6, 2024

This enables the api test that covers summarisation. It was originally skipped because tiny_elser was not available on CI. Now it is we can effectively test storing and retrieving entries from the knowledge base.

@sorenlouv sorenlouv requested a review from a team as a code owner November 6, 2024 13:15
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:Obs AI Assistant Observability AI Assistant labels Nov 6, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

Copy link
Contributor

github-actions bot commented Nov 6, 2024

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@sorenlouv sorenlouv added backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) and removed Team:Obs AI Assistant Observability AI Assistant ci:project-deploy-observability Create an Observability project labels Nov 6, 2024
Comment on lines -167 to -169
if (this.config.enableKnowledgeBase) {
addLensDocsToKb({ service, logger: this.logger.get('kb').get('lens') });
}
Copy link
Member Author

@sorenlouv sorenlouv Nov 6, 2024

Choose a reason for hiding this comment

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

This is removed to make testing easier. Right now Lens docs are imported automatically and pollutes tests. In order not to bother with this I simply got rid of it.
It will be properly removed (and cleaned up) in #191043

@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:Obs AI Assistant Observability AI Assistant labels Nov 6, 2024
Copy link
Contributor

github-actions bot commented Nov 6, 2024

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

@sorenlouv sorenlouv added the release_note:skip Skip the PR/issue when compiling release notes label Nov 6, 2024
@sorenlouv sorenlouv merged commit e399483 into elastic:main Nov 6, 2024
39 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11713577210

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 6, 2024
This enables the api test that covers summarisation. It was originally
skipped because tiny_elser was not available on CI. Now it is we can
effectively test storing and retrieving entries from the knowledge base.

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

💚 All backports created successfully

Status Branch Result
8.x

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 Nov 7, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Enable summarisation spec
(#199134)](#199134)

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

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

<!--BACKPORT [{"author":{"name":"Søren
Louv-Jansen","email":"soren.louv@elastic.co"},"sourceCommit":{"committedDate":"2024-11-06T23:25:30Z","message":"Enable
summarisation spec (#199134)\n\nThis enables the api test that covers
summarisation. It was originally\r\nskipped because tiny_elser was not
available on CI. Now it is we can\r\neffectively test storing and
retrieving entries from the knowledge
base.","sha":"e3994831ff3c6f337a209bcd3d2f93344eb787a4","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","Team:Obs
AI Assistant","ci:project-deploy-observability"],"title":"Enable
summarisation
spec","number":199134,"url":"https://github.com/elastic/kibana/pull/199134","mergeCommit":{"message":"Enable
summarisation spec (#199134)\n\nThis enables the api test that covers
summarisation. It was originally\r\nskipped because tiny_elser was not
available on CI. Now it is we can\r\neffectively test storing and
retrieving entries from the knowledge
base.","sha":"e3994831ff3c6f337a209bcd3d2f93344eb787a4"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199134","number":199134,"mergeCommit":{"message":"Enable
summarisation spec (#199134)\n\nThis enables the api test that covers
summarisation. It was originally\r\nskipped because tiny_elser was not
available on CI. Now it is we can\r\neffectively test storing and
retrieving entries from the knowledge
base.","sha":"e3994831ff3c6f337a209bcd3d2f93344eb787a4"}}]}] BACKPORT-->

Co-authored-by: Søren Louv-Jansen <soren.louv@elastic.co>
mgadewoll pushed a commit to mgadewoll/kibana that referenced this pull request Nov 7, 2024
This enables the api test that covers summarisation. It was originally
skipped because tiny_elser was not available on CI. Now it is we can
effectively test storing and retrieving entries from the knowledge base.
neptunian added a commit that referenced this pull request Dec 18, 2024
…#204790)

Closes #192497

- Unskips summarize.spec.ts. It was originally skipped because
tiny_elser was not available on CI.
- Adds `this.tags(['skipMKI']` due to
#192751

Related: #199134
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 18, 2024
…elastic#204790)

Closes elastic#192497

- Unskips summarize.spec.ts. It was originally skipped because
tiny_elser was not available on CI.
- Adds `this.tags(['skipMKI']` due to
elastic#192751

Related: elastic#199134
(cherry picked from commit 4af2706)
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this pull request Dec 19, 2024
…elastic#204790)

Closes elastic#192497

- Unskips summarize.spec.ts. It was originally skipped because
tiny_elser was not available on CI.
- Adds `this.tags(['skipMKI']` due to
elastic#192751

Related: elastic#199134
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:Obs AI Assistant Observability AI Assistant v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants