Skip to content

Commit 481f65e

Browse files
Merge branch 'main' into pkar/cps-api-calls-no-use-skipun
2 parents 9615637 + 4bcc353 commit 481f65e

File tree

22 files changed

+692
-75
lines changed

22 files changed

+692
-75
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
steps:
2+
- label: ":pipeline: Generate steps"
3+
command: bash .buildkite/scripts/generate-pr-performance-benchmark.sh | buildkite-agent pipeline upload

.buildkite/pipelines/pull-request/performance-benchmark.yml

Lines changed: 0 additions & 3 deletions
This file was deleted.

.buildkite/pull-requests.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@
2727
"set_commit_status": false,
2828
"build_on_commit": false,
2929
"build_on_comment": true,
30-
"trigger_comment_regex": "^(buildkite|@elastic(search)?machine) benchmark this with (?<benchmark>\\w+)( please)?$"
30+
"target_branch": "main",
31+
"trigger_comment_regex": "^(buildkite|@elastic(search)?machine) benchmark this with (?<benchmark>\\S+)( please)?$"
3132
}
3233
]
3334
}
Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
#!/bin/bash
2+
3+
set -euo pipefail
4+
5+
env_id_baseline=$(python3 -c 'import uuid; print(uuid.uuid4())')
6+
env_id_contender=$(python3 -c 'import uuid; print(uuid.uuid4())')
7+
merge_base=$(git merge-base "${GITHUB_PR_TARGET_BRANCH}" HEAD)
8+
9+
buildkite-agent meta-data set pr_comment:custom-body:body \
10+
"This build attempted two ${GITHUB_PR_COMMENT_VAR_BENCHMARK} benchmarks to evaluate performance impact of this PR."
11+
buildkite-agent meta-data set pr_comment:custom-baseline:head \
12+
"* Baseline: ${merge_base} (env ID ${env_id_baseline})"
13+
buildkite-agent meta-data set pr_comment:custom-contender:head \
14+
"* Contender: ${GITHUB_PR_TRIGGERED_SHA} (env ID ${env_id_contender})"
15+
16+
cat << _EOF_
17+
steps:
18+
- label: Trigger baseline benchmark
19+
trigger: elasticsearch-performance-esbench-pr
20+
build:
21+
message: Baseline benchmark for PR${GITHUB_PR_NUMBER}
22+
branch: master
23+
env:
24+
CONFIGURATION_NAME: ${GITHUB_PR_COMMENT_VAR_BENCHMARK}
25+
ENV_ID: ${env_id_baseline}
26+
REVISION: ${merge_base}
27+
- label: Trigger contender benchmark
28+
trigger: elasticsearch-performance-esbench-pr
29+
build:
30+
message: Contender benchmark for PR${GITHUB_PR_NUMBER}
31+
branch: master
32+
env:
33+
CONFIGURATION_NAME: ${GITHUB_PR_COMMENT_VAR_BENCHMARK}
34+
ENV_ID: ${env_id_contender}
35+
ES_REPO_URL: https://github.com/${GITHUB_PR_OWNER}/${GITHUB_PR_REPO}.git
36+
REVISION: ${GITHUB_PR_TRIGGERED_SHA}
37+
- wait: ~
38+
- label: Modify PR comment
39+
command: buildkite-agent meta-data set pr_comment:custom-comparison:head "* [Benchmark results](<https://esbench-metrics.kb.us-east-2.aws.elastic-cloud.com:9243/app/dashboards#/view/d9079962-5866-49ef-b9f5-145f2141cd31?_a=(query:(language:kuery,query:'user-tags.env-id:${env_id_baseline} or user-tags.env-id:${env_id_contender}'))>)"
40+
_EOF_

catalog-info.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ spec:
323323
name: elasticsearch / pull-request / performance-benchmark
324324
spec:
325325
repository: elastic/elasticsearch
326-
pipeline_file: .buildkite/pipelines/pull-request/performance-benchmark.yml
326+
pipeline_file: .buildkite/pipelines/pull-request-performance-benchmark.yml
327327
env:
328328
ELASTIC_PR_COMMENTS_ENABLED: "true"
329329
ELASTIC_SLACK_NOTIFICATIONS_ENABLED: "true"

distribution/packages/build.gradle

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -354,10 +354,6 @@ tasks.register('buildDeb', Deb) {
354354
configure(commonDebConfig('x64'))
355355
}
356356

357-
tasks.named('assemble'){
358-
dependsOn 'buildDeb', 'buildAarch64Deb'
359-
}
360-
361357
Closure commonRpmConfig(String architecture) {
362358
return {
363359
configure(commonPackageConfig('rpm', architecture))
@@ -391,11 +387,6 @@ tasks.register('buildRpm', Rpm) {
391387
configure(commonRpmConfig('x64'))
392388
}
393389

394-
tasks.named('assemble'){
395-
dependsOn 'buildRpm', 'buildAarch64Rpm'
396-
}
397-
398-
399390
Closure dpkgExists = { it -> new File('/bin/dpkg-deb').exists() || new File('/usr/bin/dpkg-deb').exists() || new File('/usr/local/bin/dpkg-deb').exists() }
400391
Closure rpmExists = { it -> new File('/bin/rpm').exists() || new File('/usr/bin/rpm').exists() || new File('/usr/local/bin/rpm').exists() }
401392

@@ -409,6 +400,11 @@ subprojects {
409400

410401
String buildTask = "build${it.name.replaceAll(/-[a-z]/) { it.substring(1).toUpperCase() }.capitalize()}"
411402
ext.buildDist = parent.tasks.named(buildTask)
403+
tasks.named('assemble').configure {
404+
dependsOn buildDist
405+
}
406+
407+
// deprecated here for backwards compatibility of DistroTestPlugin and DistributionDownloadPlugin
412408
artifacts {
413409
'default' buildDist
414410
}

docs/changelog/74274.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
pr: 74274
2+
summary: Introduce new rescorer based on script
3+
area: Search
4+
type: feature
5+
issues:
6+
- 52338

muted-tests.yml

Lines changed: 9 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -299,12 +299,6 @@ tests:
299299
- class: org.elasticsearch.compute.operator.LimitOperatorTests
300300
method: testEarlyTermination
301301
issue: https://github.com/elastic/elasticsearch/issues/128721
302-
- class: org.elasticsearch.xpack.esql.qa.single_node.GenerativeForkIT
303-
method: test {csv-spec:lookup-join.EnrichLookupStatsBug}
304-
issue: https://github.com/elastic/elasticsearch/issues/129228
305-
- class: org.elasticsearch.xpack.esql.qa.single_node.GenerativeForkIT
306-
method: test {lookup-join.MultipleBatches*
307-
issue: https://github.com/elastic/elasticsearch/issues/129210
308302
- class: org.elasticsearch.entitlement.runtime.policy.FileAccessTreeTests
309303
method: testWindowsMixedCaseAccess
310304
issue: https://github.com/elastic/elasticsearch/issues/129167
@@ -471,9 +465,6 @@ tests:
471465
- class: org.elasticsearch.xpack.ml.integration.RevertModelSnapshotIT
472466
method: testRevertModelSnapshot
473467
issue: https://github.com/elastic/elasticsearch/issues/132733
474-
- class: org.elasticsearch.xpack.esql.qa.single_node.GenerativeForkIT
475-
method: test {csv-spec:lookup-join.MvJoinKeyFromRowExpanded}
476-
issue: https://github.com/elastic/elasticsearch/issues/132778
477468
- class: org.elasticsearch.gradle.internal.transport.TransportVersionManagementPluginFuncTest
478469
method: definitions have primary ids which cannot change
479470
issue: https://github.com/elastic/elasticsearch/issues/132788
@@ -588,15 +579,6 @@ tests:
588579
- class: org.elasticsearch.gradle.internal.transport.TransportVersionValidationFuncTest
589580
method: definitions have primary ids which cannot change
590581
issue: https://github.com/elastic/elasticsearch/issues/133131
591-
- class: org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests
592-
method: testBlockLoader {preference=Params[syntheticSource=true, preference=NONE]}
593-
issue: https://github.com/elastic/elasticsearch/issues/133216
594-
- class: org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests
595-
method: testBlockLoader {preference=Params[syntheticSource=true, preference=DOC_VALUES]}
596-
issue: https://github.com/elastic/elasticsearch/issues/133217
597-
- class: org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests
598-
method: testBlockLoader {preference=Params[syntheticSource=true, preference=STORED]}
599-
issue: https://github.com/elastic/elasticsearch/issues/133218
600582
- class: org.elasticsearch.xpack.esql.action.RandomizedTimeSeriesIT
601583
method: testGroupBySubset
602584
issue: https://github.com/elastic/elasticsearch/issues/133220
@@ -615,6 +597,15 @@ tests:
615597
- class: org.elasticsearch.test.rest.yaml.RcsCcsCommonYamlTestSuiteIT
616598
method: test {p0=search.vectors/100_knn_nested_search/nested kNN search inner_hits & profiling}
617599
issue: https://github.com/elastic/elasticsearch/issues/133273
600+
- class: org.elasticsearch.xpack.security.authc.AuthenticationServiceTests
601+
method: testInvalidToken
602+
issue: https://github.com/elastic/elasticsearch/issues/133328
603+
- class: org.elasticsearch.test.rest.yaml.RcsCcsCommonYamlTestSuiteIT
604+
method: test {p0=search/160_exists_query/Test exists query on unmapped byte field}
605+
issue: https://github.com/elastic/elasticsearch/issues/133331
606+
- class: org.elasticsearch.xpack.esql.ccq.MultiClusterSpecIT
607+
method: test {csv-spec:change_point.Values null column}
608+
issue: https://github.com/elastic/elasticsearch/issues/133334
618609

619610
# Examples:
620611
#

0 commit comments

Comments
 (0)