Skip to content

Commit fc4328d

Browse files
[8.19] [Obs AI Assistant] Unskip summarize tests (#218728) (#218734)
# Backport This will backport the following commits from `main` to `8.19`: - [[Obs AI Assistant] Unskip summarize tests (#218728)](#218728) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Viduni Wickramarachchi","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-04-21T16:07:54Z","message":"[Obs AI Assistant] Unskip summarize tests (#218728)\n\nCloses https://github.com/elastic/kibana/issues/218497\n\n## Summary\n\nThe summarize tests were skipped last week because they were failing and\nblocking pipelines.\n\nThese tests have started to fail after the merge of\nhttps://github.com/elastic/elasticsearch/pull/126635\nThese changes were reverted last week in\nhttps://github.com/elastic/elasticsearch/pull/127075\n\nTherefore, this PR unskips the tests.\n\n### Checklist\n\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"766cd471765c1466c4b12005e04ef39e5f5ed04b","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Obs AI Assistant","ci:project-deploy-observability","backport:version","v9.1.0","v8.19.0"],"title":"[Obs AI Assistant] Unskip summarize tests","number":218728,"url":"https://github.com/elastic/kibana/pull/218728","mergeCommit":{"message":"[Obs AI Assistant] Unskip summarize tests (#218728)\n\nCloses https://github.com/elastic/kibana/issues/218497\n\n## Summary\n\nThe summarize tests were skipped last week because they were failing and\nblocking pipelines.\n\nThese tests have started to fail after the merge of\nhttps://github.com/elastic/elasticsearch/pull/126635\nThese changes were reverted last week in\nhttps://github.com/elastic/elasticsearch/pull/127075\n\nTherefore, this PR unskips the tests.\n\n### Checklist\n\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"766cd471765c1466c4b12005e04ef39e5f5ed04b"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/218728","number":218728,"mergeCommit":{"message":"[Obs AI Assistant] Unskip summarize tests (#218728)\n\nCloses https://github.com/elastic/kibana/issues/218497\n\n## Summary\n\nThe summarize tests were skipped last week because they were failing and\nblocking pipelines.\n\nThese tests have started to fail after the merge of\nhttps://github.com/elastic/elasticsearch/pull/126635\nThese changes were reverted last week in\nhttps://github.com/elastic/elasticsearch/pull/127075\n\nTherefore, this PR unskips the tests.\n\n### Checklist\n\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"766cd471765c1466c4b12005e04ef39e5f5ed04b"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Viduni Wickramarachchi <[email protected]>
1 parent c48b21a commit fc4328d

File tree

2 files changed

+2
-4
lines changed
  • x-pack/test
    • api_integration/deployment_agnostic/apis/observability/ai_assistant/complete/functions
    • observability_ai_assistant_functional/tests/knowledge_base_management

2 files changed

+2
-4
lines changed

x-pack/test/api_integration/deployment_agnostic/apis/observability/ai_assistant/complete/functions/summarize.spec.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@ export default function ApiTest({ getService }: DeploymentAgnosticFtrProviderCon
2424
const es = getService('es');
2525
const observabilityAIAssistantAPIClient = getService('observabilityAIAssistantApi');
2626

27-
// Failing test: https://github.com/elastic/kibana/issues/218497
28-
describe.skip('summarize', function () {
27+
describe('summarize', function () {
2928
// Fails on MKI: https://github.com/elastic/kibana/issues/205581
3029
this.tags(['failsOnMKI']);
3130
let proxy: LlmProxy;

x-pack/test/observability_ai_assistant_functional/tests/knowledge_base_management/index.spec.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,7 @@ export default function ApiTest({ getService, getPageObjects }: FtrProviderConte
4848
}).expect(200);
4949
}
5050

51-
// Failing test: https://github.com/elastic/kibana/issues/218327
52-
describe.skip('Knowledge management tab', () => {
51+
describe('Knowledge management tab', () => {
5352
before(async () => {
5453
await clearKnowledgeBase(es);
5554

0 commit comments

Comments
 (0)