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 kb migration test suite to serverless #205537

Closed
1 task done
neptunian opened this issue Jan 3, 2025 · 1 comment · Fixed by #205631
Closed
1 task done

[Obs AI Assistant] move kb migration test suite to serverless #205537

neptunian opened this issue Jan 3, 2025 · 1 comment · Fixed by #205631
Assignees
Labels
Team:Obs AI Assistant Observability AI Assistant

Comments

@neptunian
Copy link
Contributor

neptunian commented Jan 3, 2025

Knowledge base migration tests are missing in serverless. We should duplicate it into serverless and make sure it can pass, so it can be included in migrating tests that can run in both environments to deployment agnostic #205194. Or it can be done as part of that issue or after that issue is merged. We should address the flaky test first #202926

Dependencies

Preview Give feedback
  1. Team:Obs AI Assistant
    arturoliduena
@botelastic botelastic bot added the needs-team Issues missing a team label label Jan 3, 2025
@neptunian neptunian added Team:Obs AI Assistant Observability AI Assistant and removed needs-team Issues missing a team label labels Jan 3, 2025
@elasticmachine
Copy link
Contributor

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

@viduni94 viduni94 assigned viduni94 and unassigned viduni94 Jan 6, 2025
viduni94 added a commit to viduni94/kibana that referenced this issue Jan 6, 2025
@viduni94 viduni94 self-assigned this Jan 6, 2025
viduni94 added a commit to viduni94/kibana that referenced this issue Jan 6, 2025
viduni94 added a commit to viduni94/kibana that referenced this issue Jan 7, 2025
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jan 8, 2025
…s test suite (elastic#205631)

Closes elastic#205537

## Summary

The knowledge base migration test suite is missing in serverless. This
PR adds it to the serverless test suite.
- This has a dependancy to elastic#205194
since we are removing all serverless tests and adding them to DA tests.
- If the DA tests PR gets merged first, I'll refactor this PR to add it
there.

### Checklist

- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit e32ff8e)
crespocarlos pushed a commit to crespocarlos/kibana that referenced this issue Jan 8, 2025
…s test suite (elastic#205631)

Closes elastic#205537

## Summary

The knowledge base migration test suite is missing in serverless. This
PR adds it to the serverless test suite.
- This has a dependancy to elastic#205194
since we are removing all serverless tests and adding them to DA tests.
- If the DA tests PR gets merged first, I'll refactor this PR to add it
there.

### Checklist

- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Jan 13, 2025
…s test suite (elastic#205631)

Closes elastic#205537

## Summary

The knowledge base migration test suite is missing in serverless. This
PR adds it to the serverless test suite.
- This has a dependancy to elastic#205194
since we are removing all serverless tests and adding them to DA tests.
- If the DA tests PR gets merged first, I'll refactor this PR to add it
there.

### Checklist

- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Obs AI Assistant Observability AI Assistant
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants