Skip to content

Commit

Permalink
enhance: only use streaming service on e2e and nightly e2e (milvus-io…
Browse files Browse the repository at this point in the history
…#38743)

issue: milvus-io#38399

Signed-off-by: chyezh <[email protected]>
  • Loading branch information
chyezh authored and NicoYuan1986 committed Dec 26, 2024
1 parent a5a267c commit 930bfd1
Show file tree
Hide file tree
Showing 13 changed files with 29 additions and 378 deletions.
2 changes: 1 addition & 1 deletion ci/jenkins/Nightly2.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ pipeline {
axes {
axis {
name 'milvus_deployment_option'
values 'standalone', 'distributed-pulsar', 'distributed-kafka', 'standalone-authentication', 'standalone-one-pod', 'distributed-streaming-service'
values 'standalone', 'distributed-pulsar', 'distributed-kafka', 'standalone-authentication', 'standalone-one-pod'
}
}
stages {
Expand Down
37 changes: 10 additions & 27 deletions ci/jenkins/PR.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ pipeline {
axes {
axis {
name 'milvus_deployment_option'
values 'standalone', 'distributed', 'standalone-kafka-mmap', 'distributed-streaming-service'
values 'standalone', 'distributed', 'standalone-kafka-mmap'
}
}
stages {
Expand All @@ -100,32 +100,15 @@ pipeline {
def helm_release_name = get_helm_release_name milvus_deployment_option
// pvc name would be <pod-name>-volume-0, used for pytest result archiving
def pvc = env.JENKINS_AGENT_NAME + '-volume-0'

if (milvus_deployment_option == 'distributed-streaming-service') {
try {
tekton.pytest helm_release_name: helm_release_name,
pvc: pvc,
milvus_helm_version: milvus_helm_chart_version,
ciMode: 'e2e',
milvus_image_tag: milvus_image_tag,
pytest_image: pytest_image,
helm_image: helm_image,
milvus_deployment_option: milvus_deployment_option,
verbose: 'false'
} catch (Exception e) {
println e
}
} else {
tekton.pytest helm_release_name: helm_release_name,
pvc: pvc,
milvus_helm_version: milvus_helm_chart_version,
ciMode: 'e2e',
milvus_image_tag: milvus_image_tag,
pytest_image: pytest_image,
helm_image: helm_image,
milvus_deployment_option: milvus_deployment_option,
verbose: 'false'
}
tekton.pytest helm_release_name: helm_release_name,
pvc: pvc,
milvus_helm_version: milvus_helm_chart_version,
ciMode: 'e2e',
milvus_image_tag: milvus_image_tag,
pytest_image: pytest_image,
helm_image: helm_image,
milvus_deployment_option: milvus_deployment_option,
verbose: 'false'
}
}
}
Expand Down
2 changes: 2 additions & 0 deletions tests/_helm/values/e2e/distributed
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ affinity:
weight: 1
cluster:
enabled: true
streaming:
enabled: true
dataCoordinator:
resources:
limits:
Expand Down
274 changes: 0 additions & 274 deletions tests/_helm/values/e2e/distributed-streaming-service

This file was deleted.

2 changes: 2 additions & 0 deletions tests/_helm/values/e2e/standalone
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ affinity:
weight: 1
cluster:
enabled: false
streaming:
enabled: true
dataCoordinator:
resources:
limits:
Expand Down
2 changes: 2 additions & 0 deletions tests/_helm/values/e2e/standalone-kafka-mmap
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ affinity:
weight: 1
cluster:
enabled: false
streaming:
enabled: true
dataCoordinator:
resources:
limits:
Expand Down
2 changes: 2 additions & 0 deletions tests/_helm/values/e2e/standalone-one-pod
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ affinity:
weight: 1
cluster:
enabled: false
streaming:
enabled: true
etcd:
enabled: false
metrics:
Expand Down
2 changes: 2 additions & 0 deletions tests/_helm/values/nightly/distributed-kafka
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
cluster:
enabled: true
streaming:
enabled: true
common:
security:
authorizationEnabled: false
Expand Down
2 changes: 2 additions & 0 deletions tests/_helm/values/nightly/distributed-pulsar
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
cluster:
enabled: true
streaming:
enabled: true
common:
security:
authorizationEnabled: false
Expand Down
Loading

0 comments on commit 930bfd1

Please sign in to comment.