Skip to content

Commit

Permalink
[KYUUBI #6244][TEST] Fix test logs upload error
Browse files Browse the repository at this point in the history
# 🔍 Description
## Issue References 🔗

This pull request fixes #6244

## Describe Your Solution 🔧

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

## Types of changes 🔖

- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

#### Behavior Without This Pull Request ⚰️

#### Behavior With This Pull Request 🎉

#### Related Unit Tests

---

# Checklist 📝

- [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #6351 from beryllw/kyuubi-6244.

Closes #6244

6034310 [wangjunbo] [KYUUBI #6244] Fix test logs upload error
460b259 [wangjunbo] [KYUUBI #6244] Fix test logs upload error

Authored-by: wangjunbo <wangjunbo@qiyi.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit c73ae71)
Signed-off-by: Cheng Pan <chengpan@apache.org>
  • Loading branch information
wangjunbo authored and pan3793 committed Apr 30, 2024
1 parent 5942686 commit 7523a4e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ trait WithEtcdCluster extends KyuubiFunSuite {
override def beforeAll(): Unit = {
etcdCluster = new Etcd.Builder()
.withNodes(1)
.withMountedDataDirectory(false)
.build()
etcdCluster.start()
super.beforeAll()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ class EtcdDiscoveryClientSuite extends DiscoveryClientTests {
if (hasDockerEnv) {
etcdCluster = new Etcd.Builder()
.withNodes(2)
.withMountedDataDirectory(false)
.build()
etcdCluster.start()
conf = new KyuubiConf()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ class EngineRefWithEtcdSuite extends EngineRefTests {
override def beforeAll(): Unit = {
etcdCluster = new Etcd.Builder()
.withNodes(1)
.withMountedDataDirectory(false)
.build()
etcdCluster.start()
super.beforeAll()
Expand Down

0 comments on commit 7523a4e

Please sign in to comment.