You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gi...@apache.org on 2019/01/11 21:33:29 UTC
[mesos] 02/11: Updated `SlaveRecoveryTest.Reboot` to expect none
pids.
This is an automated email from the ASF dual-hosted git repository.
gilbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git
commit 43772dfa7111ebdf759b066728736921062cc4fc
Author: Qian Zhang <zh...@gmail.com>
AuthorDate: Fri Jan 11 12:21:22 2019 -0800
Updated `SlaveRecoveryTest.Reboot` to expect none pids.
After agent host is rebooted, the recovered executor's forked pid and
libprocess pid will be `NONE()` since we do not read them from agent's
meta directory which are actually obsolete after reboot.
Review: https://reviews.apache.org/r/69716/
---
src/tests/slave_recovery_tests.cpp | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/tests/slave_recovery_tests.cpp b/src/tests/slave_recovery_tests.cpp
index 0eb47e2..91fa00a 100644
--- a/src/tests/slave_recovery_tests.cpp
+++ b/src/tests/slave_recovery_tests.cpp
@@ -2798,11 +2798,8 @@ TYPED_TEST(SlaveRecoveryTest, Reboot)
EXPECT_TRUE(executorState.runs.contains(containerId2.get()));
- EXPECT_SOME_EQ(
- executorPid,
- executorState
- .runs[containerId2.get()]
- .libprocessPid);
+ EXPECT_NONE(executorState.runs[containerId2.get()].forkedPid);
+ EXPECT_NONE(executorState.runs[containerId2.get()].libprocessPid);
EXPECT_TRUE(executorState
.runs[containerId2.get()]