mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chun-Hung Hsiao <>
Subject Re: Review Request 69866: Updated SLRP test `ProfileDisappeared` to request operation feedback.
Date Wed, 13 Feb 2019 03:08:48 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 13, 2019, 3:08 a.m.)

Review request for mesos, Benjamin Bannier, Gastón Kleiman, and Greg Mann.


Fixed a compilation error.

Bugs: MESOS-9537

Repository: mesos


This patch updates `StorageLocalResourceProviderTest.ProfileDisappeared`
to use the v1 scheduler API to request operation feedback, so MESOS-9537
would be triggered when an outstanding `UPDATE_STATE` call from the
resource provider races with an offer operation.

Diffs (updated)

  src/tests/mesos.hpp b10ec0ad62efa8980ca12693939970c2252ee814 
  src/tests/storage_local_resource_provider_tests.cpp fb001aa8d32d1a0a03014a35772fe10b65ce8d9a




`make check`

Ran the test ~1000 times.

This test will fail without r/69858:
E0130 22:48:02.790385 24040 provider.cpp:2903] Failed to update status of operation (uuid:
2bf6813c-bde9-4c0e-8831-a779f5dc50ac): Mismatched framework ID for operation status update
OPERATION_DROPPED (Status UUID: 8a625c40-96e8-46e2-8050-69843032969a) for operation UUID 2bf6813c-bde9-4c0e-8831-a779f5dc50ac
on agent bfa6e505-54b7-4bb8-a8aa-99fb12291fb1-S0 (expected bfa6e505-54b7-4bb8-a8aa-99fb12291fb1-0000
got no framework ID)
I0130 22:48:02.791590 24042 manager.cpp:163] Terminating resource provider 765391c2-9485-4614-a6e5-45e67ef7a92c
../../src/tests/storage_local_resource_provider_tests.cpp:1387: Failure
      Expected: v1::OPERATION_FINISHED
To be equal to: update->status().state()
      Which is: OPERATION_ERROR
../../src/tests/storage_local_resource_provider_tests.cpp:1393: Failure
Failed to wait 15secs for offers
../../src/tests/storage_local_resource_provider_tests.cpp:1361: Failure
Actual function call count doesn't match EXPECT_CALL(*scheduler, offers(_, v1::scheduler::OffersHaveAnyResource(
std::bind(isStoragePool, lambda::_1, "test2"))))...
         Expected: to be called once
           Actual: never called - unsatisfied and active


Chun-Hung Hsiao

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message