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

Fix 8.x 9 forward compat tests #206624

Merged
merged 6 commits into from
Jan 15, 2025

Conversation

jloleysens
Copy link
Contributor

@jloleysens jloleysens commented Jan 14, 2025

Summary

Fix 8.x -> 9 forward compatibility test scenarios:

src/core/server/integration_tests/saved_objects/migrations/group3/migration_from_older_v1.test.ts
src/core/server/integration_tests/saved_objects/migrations/group2/batch_size_bytes.test.ts

Using the EsVersion class to detect the version of ES that will be run and skipping these tests.

Test locally

Simulate what the buildkite pipeline is doing locally and the entire suite should report being skipped.

ES_SNAPSHOT_MANIFEST="https://storage.googleapis.com/kibana-ci-es-snapshots-daily/9.0.0/manifest-latest-verified.json" yarn test:jest_integration ./src/core/server/integration_tests/saved_objects/migrations/group3/migration_from_older_v1.test.ts

Simulate what the buildkite pipeline is doing locally, should pass (instead of break due to incompatible data).

ES_SNAPSHOT_MANIFEST="https://storage.googleapis.com/kibana-ci-es-snapshots-daily/9.0.0/manifest-latest-verified.json" yarn test:jest_integration src/core/server/integration_tests/saved_objects/migrations/group2/batch_size_bytes.test.ts

@jloleysens jloleysens added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0 labels Jan 14, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@jloleysens jloleysens added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Jan 14, 2025
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/test 305 306 +1
Unknown metric groups

API count

id before after diff
@kbn/test 360 361 +1

Comment on lines +73 to +75
willRunESv9
? '8.18.0_xpack_sample_saved_objects.zip'
: '7.14.0_xpack_sample_saved_objects.zip'
Copy link
Member

Choose a reason for hiding this comment

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

🧡

@jloleysens jloleysens enabled auto-merge (squash) January 15, 2025 11:09
@jloleysens jloleysens merged commit a3f5ba6 into elastic:8.x Jan 15, 2025
8 checks passed
@jloleysens jloleysens deleted the fix-8.x-9-forward-compat-tests branch January 15, 2025 12:53
jloleysens added a commit that referenced this pull request Jan 22, 2025
## Summary

Follow up from #206624 aimed at
fixing failing compat tests:


https://buildkite.com/elastic/kibana-es-forward-compatibility-testing-9-dot-0/builds?branch=8.x

### Core

* Fixed jest integration tests, generated new archives
* Skipped the UA tests for 8.x -> 9 if ES >8, I assume these tests only
make sense if ES is on v8

### Security solution ES|QL

* Made the `metadata [...` test only run when ES is v8

### Kibana management

* Made the unfreeze test only run when ES is v8
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 release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants