---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70186/#review213626
---
Ship it!
Ship It!
- Gastón Kleiman
On March 11, 2019, 12:18
he updated test doesn't call `result->response()`, so there is no risk of
it crashing.
src/tests/storage_local_resource_provider_tests.cpp
Lines 4658-4660 (original), 4662-4664 (patched)
<https://reviews.apache.org/r/70200/#comment299693>
`%s/ASSERT/EXPECT/g`
- Gastón Kleiman
only `RESERVE`
opertions don't need a resource provider. Removing the instantiation of the RP
would make remove a lot of boilerplate and some amount of clock manipulation,
thus making the test much shorter, readable and less prone to races.
- Gastón Kleiman
On March 13, 2019, 6:41 p.m., Greg
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70200/#review213754
---
Ship it!
Ship It!
- Gastón Kleiman
On March 15, 2019, 3:35
/operation_reconciliation_tests.cpp
Lines 284-293 (patched)
<https://reviews.apache.org/r/70208/#comment299748>
We could remove this if the first operation also reserved 0.1 CPUs
- Gastón Kleiman
On March 15, 2019, 3:53 p.m., Greg Mann
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70221/#review213755
---
Ship it!
Ship It!
- Gastón Kleiman
On March 15, 2019, 3:37
returns a response, but
it just happens to be the case that no call does that at the moment.
I would delete this comment.
- Gastón Kleiman
On March 14, 2019, 10:24 a.m., Greg Mann wrote:
>
> ---
> This is an automatically genera
tps://reviews.apache.org/r/70156/#comment299492>
Nit: I'd name this `EXPECT_METRIC_EQ`.
- Gastón Kleiman
On March 7, 2019, 8:59 a.m., Benno Evers wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
ive?
src/master/metrics.cpp
Lines 684 (patched)
<https://reviews.apache.org/r/70116/#comment299500>
Can counters for terminal states be decremented? I guess not? If I'm
correct, then we could add a `CHECK` here.
- Gastón Kleiman
On M
But I like Joseph's suggestion better =).
- Gastón Kleiman
On March 21, 2019, 3:26 p.m., Greg Mann wrote:
>
> ---
> This is an automatically generated e-mail. To rep
```
- Gastón
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69912/#review212634
-------
On Feb. 6
, visit:
https://reviews.apache.org/r/69910/#review212651
-------
On Feb. 7, 2019, 1:53 p.m., Gastón Kleiman wrote:
>
> ---
> This is an automatically gener
://reviews.apache.org/r/69913/diff/1/
Testing
---
`GLOG_v=1 bin/mesos-tests.sh --verbose
--gtest_filter="*OperationReconciliationTest*"` passed
Thanks,
Gastón Kleiman
eviews.apache.org/r/69913/#review212635
-------
On Feb. 6, 2019, 2:04 p.m., Gastón Kleiman wrote:
>
> ---
> This is an automatically generated e-mail. To re
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69911/#review212632
---
On Feb. 7, 2019, 1:55 p.m., Gastón Kleiman wrote:
>
> ---
eviews.apache.org/r/69913/#review212635
-------
On Feb. 6, 2019, 2:04 p.m., Gastón Kleiman wrote:
>
> ---
> This is an automatically generated e-mail. To re
: https://reviews.apache.org/r/69910/diff/1-2/
Testing
---
`bin/mesos-tests.sh--gtest_filter="*AgentOperationFeedbackTest*"
--gtest_repeat=5000 --gtest_break_on_failure`
Thanks,
Gastón Kleiman
status updates that it generates.
Diffs (updated)
-
src/master/master.cpp 4c9ef2528d0ab74b565dcb03d05c5189d0aa0c0f
Diff: https://reviews.apache.org/r/69912/diff/2/
Changes: https://reviews.apache.org/r/69912/diff/1-2/
Testing
---
`bin/mesos-tests.sh` passed
Thanks,
Gastón
eedback_tests.cpp PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/69910/diff/1/
>
>
> Testing
> ---
>
> `bin/mesos-tests.sh--gtest_filter="*AgentOperationFeedbackTest*"
> --gtest_repeat=5000 --gtest_break_on_failure`
>
>
> Thanks,
>
> Gastón Kleiman
>
>
/agent_operation_feedback_tests.cpp
Lines 564 (patched)
<https://reviews.apache.org/r/69922/#comment298500>
Can we also check that the agent ID is set and that the RP ID isn't?
- Gastón Kleiman
On Feb. 7, 2019, 3:23 p.m., Greg Mann
="*OperationReconciliationTest*"
--gtest_repeat=5000 --gtest_break_on_failure` passed
Thanks,
Gastón Kleiman
428e14ef2e6b9f0e622a6ce05c469dd0ed97d07e
Diff: https://reviews.apache.org/r/69920/diff/1/
Testing
---
Thanks,
Gastón Kleiman
---
Thanks,
Gastón Kleiman
4c9ef2528d0ab74b565dcb03d05c5189d0aa0c0f
Diff: https://reviews.apache.org/r/69912/diff/1/
Testing
---
`bin/mesos-tests.sh` passed
Thanks,
Gastón Kleiman
---
`GLOG_v=1 bin/mesos-tests.sh --verbose
--gtest_filter="*OperationReconciliationTest*"` passed
Thanks,
Gastón Kleiman
src/tests/agent_operation_feedback_tests.cpp PRE-CREATION
Diff: https://reviews.apache.org/r/69910/diff/1/
Testing
---
`bin/mesos-tests.sh--gtest_filter="*AgentOperationFeedbackTest*"
--gtest_repeat=5000 --gtest_break_on_failure`
Thanks,
Gastón Kleiman
/69911/diff/1/
Testing
---
`bin/mesos-tests.sh --gtest_filter="*OperationReconciliationTest*"
--gtest_repeat=5000 --gtest_break_on_failure` passed
Thanks,
Gastón Kleiman
ly, visit:
https://reviews.apache.org/r/69911/#review212654
-------
On Feb. 8, 2019, 3:12 p.m., Gastón Kleiman wrote:
>
> ---
> This is an automatically generated e-mail.
-tests.sh--gtest_filter="*AgentOperationFeedbackTest*"
--gtest_repeat=5000 --gtest_break_on_failure`
Thanks,
Gastón Kleiman
d
Thanks,
Gastón Kleiman
t;;
```
If they get that, then it might be easier for them to debug missing
capabilities.
src/master/master.cpp
Line 8725 (original)
<https://reviews.apache.org/r/69957/#comment298683>
Would the following assertion hold?
`CHE
(patched)
<https://reviews.apache.org/r/69964/#comment298686>
Should we do `GET_OPERATIONS` here and check the response?
- Gastón Kleiman
On Feb. 12, 2019, 3:42 p.m., Greg Mann wrote:
>
> ---
> This is an automatically g
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69958/#review212778
---
Ship it!
Ship It!
- Gastón Kleiman
On Feb. 12, 2019, 1:53
tps://reviews.apache.org/r/69964/#comment298687>
Can we check here that the framework doesn't get any operation status
updates?
- Gastón Kleiman
On Feb. 12, 2019, 3:42 p.m., Greg Mann wrote:
>
> ---
> This is an automatically g
of the `Slave::operationStatusAcknowledgement` method.
Diffs
-
src/slave/slave.cpp e3c2c005d865b5c333e92e50e49ef398fe06ad79
Diff: https://reviews.apache.org/r/69978/diff/1/
Testing
---
Manual testing + existing tests still pass.
Thanks,
Gastón Kleiman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69964/#review212813
---
Ship it!
Ship It!
- Gastón Kleiman
On Feb. 12, 2019, 6:39
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69969/#review212814
---
Ship it!
Ship It!
- Gastón Kleiman
On Feb. 12, 2019, 6:46
fails over while processing the
acknowledgement of a terminal operation status update.
Diffs
-
src/slave/slave.cpp e3c2c005d865b5c333e92e50e49ef398fe06ad79
Diff: https://reviews.apache.org/r/69977/diff/1/
Testing
---
Manual testing + existing tests still pass.
Thanks,
Gastón
: https://reviews.apache.org/r/69978/diff/1-2/
Testing
---
Manual testing + existing tests still pass.
Thanks,
Gastón Kleiman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69957/#review212883
---
Ship it!
Ship It!
- Gastón Kleiman
On Feb. 13, 2019, 8:09
(original), 1799 (patched)
<https://reviews.apache.org/r/69907/#comment298733>
Can you fix it here too?
- Gastón Kleiman
On Feb. 6, 2019, 8:35 a.m., Vinod Kone wrote:
>
> ---
> This is an automatically generated e-mail.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69908/#review212849
---
Ship it!
Ship It!
- Gastón Kleiman
On Feb. 6, 2019, 8:38
ed, so the task shoudl have been removed
from the master's unreachable tasks list`
- Gastón Kleiman
On Feb. 6, 2019, 8:41 a.m., Vinod Kone wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69843/#review212429
---
Ship it!
Ship It!
- Gastón Kleiman
On Jan. 25, 2019, 6:22
epeat=5000 --gtest_break_on_failure` passed on CentOS 7.4.1708.
Thanks,
Gastón Kleiman
/69795/diff/6-7/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
/r/69795/diff/7-8/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
---
Thanks,
Gastón Kleiman
new proto with both cmake and autotools.
Thanks,
Gastón Kleiman
/
Testing
---
`GLOG_v=1 bin/mesos-tests.sh --verbose
--gtest_filter="*Slave*RetryOperationStatusUpdateAfterRecovery*"
--gtest_repeat=5000 --gtest_break_on_failure` passed on CentOS 7.4.1708.
Thanks,
Gastón Kleiman
/diff/5-6/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
/slave.cpp ed92f672f5155d70a36ba3619bb6f06fa09bc836
src/tests/slave_tests.cpp 9168e06348a83dcb20400a2fe0e3bb1f26e6ff1b
Diff: https://reviews.apache.org/r/69794/diff/5/
Changes: https://reviews.apache.org/r/69794/diff/4-5/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
src/slave/slave.cpp ed92f672f5155d70a36ba3619bb6f06fa09bc836
src/tests/slave_tests.cpp 9168e06348a83dcb20400a2fe0e3bb1f26e6ff1b
Diff: https://reviews.apache.org/r/69794/diff/4/
Changes: https://reviews.apache.org/r/69794/diff/3-4/
Testing
---
Current tests still pass.
Thanks,
Gastón
.
Compiled new proto with both cmake and autotools.
Thanks,
Gastón Kleiman
: https://reviews.apache.org/r/69795/diff/1-2/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
.
Thanks,
Gastón Kleiman
eviews.apache.org/r/69794/#review212206
---
On Jan. 23, 2019, 3:23 p.m., Gastón Kleiman wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69794/
> ---
://reviews.apache.org/r/69795/diff/2-3/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
/slave/slave.cpp ed92f672f5155d70a36ba3619bb6f06fa09bc836
src/tests/slave_tests.cpp 9168e06348a83dcb20400a2fe0e3bb1f26e6ff1b
Diff: https://reviews.apache.org/r/69794/diff/3/
Changes: https://reviews.apache.org/r/69794/diff/2-3/
Testing
---
Current tests still pass.
Thanks,
Gastón
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69786/#review212351
---
Ship it!
Ship It!
- Gastón Kleiman
On Jan. 17, 2019, 6:30
slave failed recovery and hence registering as a new
// slave before the master removed the old slave from its map.
// - If the slave was shutting down while it had a registration
// retry scheduled. See MESOS-8463.
```
- Gastón Kleiman
On Feb. 4, 2019, 1:
(patched)
<https://reviews.apache.org/r/69876/#comment298353>
We should consider making the agent not recover the operation status update
manager if it isn't started with the `AGENT_OPERATION_FEEDBACK` capability.
- Gastón Kleiman
On Jan. 31, 2019, 3:08 p.m., Greg Mann
/69795/diff/4-5/
Testing
---
Current tests still pass.
Thanks,
Gastón Kleiman
600 (patched)
<https://reviews.apache.org/r/69871/#comment298345>
Nit: remove this extra line.
- Gastón Kleiman
On Jan. 31, 2019, 9:57 a.m., Greg Mann wrote:
>
> ---
> This is an automatically generated e-mail.
> On Feb. 4, 2019, 4:28 p.m., Gastón Kleiman wrote:
> > src/tests/master_tests.cpp
> > Lines 9419 (patched)
> > <https://reviews.apache.org/r/69876/diff/1/?file=2123554#file2123554line9419>
> >
> > We should consider making the agent not recover t
the presence of `RESERVATION_REFINEMENT`?
>
> Gastón Kleiman wrote:
> This framework only pushes one reservation, so I guess that there should
> only be one element for reservations made by it?
>
> The check would fail if the example framework shares a role with
so
we shouldn't get any status updates.
- Gastón
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70412/#review214481
-------
/70412/diff/2/
Changes: https://reviews.apache.org/r/70412/diff/1-2/
Testing
---
Manual testing.
Thanks,
Gastón Kleiman
://reviews.apache.org/r/70412/diff/3/
Changes: https://reviews.apache.org/r/70412/diff/2-3/
Testing
---
Manual testing.
Thanks,
Gastón Kleiman
ated e-mail. To reply, visit:
https://reviews.apache.org/r/70412/#review214468
-------
On April 5, 2019, 4:35 p.m., Gastón Kleiman wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apach
Diff: https://reviews.apache.org/r/70478/diff/1/
Testing
---
Manual testing.
Thanks,
Gastón Kleiman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70379/#review214640
---
Ship it!
Ship It!
- Gastón Kleiman
On April 10, 2019, 8:35
, 2600 (patched)
<https://reviews.apache.org/r/70408/#comment300826>
Please shift this line left to fix the indentation.
- Gastón Kleiman
On April 10, 2019, 8:29 a.m., Andrei Sekretenko wrote:
>
> ---
> This is an auto
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70377/#review214641
---
Ship it!
Ship It!
- Gastón Kleiman
On April 10, 2019, 8:31
: mesos
Description
---
Added metrics to the operation feedback example framework.
Diffs
-
src/examples/operation_feedback_framework.cpp
ab19605dda069db06b6215a95049014244fc2d4f
Diff: https://reviews.apache.org/r/70506/diff/1/
Testing
---
Thanks,
Gastón Kleiman
://reviews.apache.org/r/70506/diff/2/
Changes: https://reviews.apache.org/r/70506/diff/1-2/
Testing
---
Thanks,
Gastón Kleiman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70587/#review215039
---
Ship it!
Ship It!
- Gastón Kleiman
On May 2, 2019, 3 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70593/#review215022
---
Ship it!
Ship It!
- Gastón Kleiman
On May 3, 2019, 9:38 a.m
://reviews.apache.org/r/70593/
```
- Gastón Kleiman
On May 3, 2019, 9:38 a.m., Andrei Sekretenko wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
)
<https://reviews.apache.org/r/70528/#comment303732>
Super minor nit: s/ / / =)
- Gastón Kleiman
On July 10, 2019, 7:58 a.m., Benno Evers wrote:
>
> ---
> This is an automatically generated e-mail. To reply,
ed data structures and
I don't think that we should add a new one just for this.
src/master/master.cpp
Lines 11402 (patched)
<https://reviews.apache.org/r/71080/#comment303979>
Nit: s/ids/IDs/
- Gastón Kleiman
On July 17, 2019, 11:
701 - 781 of 781 matches
Mail list logo