Hi folks,
We will try BlueJeans VC (https://bluejeans.com/272369669) this time (today
3 PM PST) for our monthly community sync.
Agenda (tentative)
Optimistic offers
Fetcher cache
Release cadence
Review process
Committership
See you in a few hours,
Vinod
P.S: If
On March 5, 2015, 8:33 p.m., Vinod Kone wrote:
src/tests/disk_quota_tests.cpp, line 508
https://reviews.apache.org/r/31772/diff/1/?file=886098#file886098line508
why the if condition? is it possible for disk_used_bytes to not exist?
Yes, it's possible if the first 'du' hasn't
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31263/
---
(Updated March 5, 2015, 9:11 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31772/
---
Review request for mesos, Ben Mahler, Ian Downes, and Vinod Kone.
Bugs:
See https://builds.apache.org/job/mesos-reviewbot/4452/
--
[URLTrigger] A change within the response URL invocation (log)
[URLTrigger] A change within the response URL invocation (log)
Building remotely on H10 (Ubuntu ubuntu) in workspace
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30841/
---
(Updated March 5, 2015, 11:39 a.m.)
Review request for mesos and Jie Yu.
A few questions:
1) Was this announced on the mailing list? I'm hearing about this for
the first time and imagine others may be in the same boat
2) How was the agenda created? it would be awesome if we could track
each community meeting somehow to allow folks to suggest agenda items,
perhaps this
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29288/
---
(Updated March 5, 2015, 7:38 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31471/
---
(Updated March 5, 2015, 7:39 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31263/
---
(Updated March 5, 2015, 9:06 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31776/
---
Review request for mesos, Kapil Arya, Michael Park, and Niklas Nielsen.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31266/
---
(Updated March 5, 2015, 9:05 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/
---
(Updated March 5, 2015, 7:22 p.m.)
Review request for mesos, Adam B, Kapil
On March 5, 2015, 8:21 p.m., Vinod Kone wrote:
i'll take a look once the marked issues are resolved.
Done
- Isabel
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30952/#review75383
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31772/#review75385
---
src/slave/containerizer/mesos/containerizer.cpp
See https://builds.apache.org/job/mesos-reviewbot/4451/
--
[URLTrigger] A change within the response URL invocation (log)
[URLTrigger] A change within the response URL invocation (log)
[URLTrigger] A change within the response URL invocation (log)
Building
On March 5, 2015, 8:33 p.m., Vinod Kone wrote:
src/tests/disk_quota_tests.cpp, line 508
https://reviews.apache.org/r/31772/diff/1/?file=886098#file886098line508
why the if condition? is it possible for disk_used_bytes to not exist?
Jie Yu wrote:
Yes, it's possible if the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31775/
---
Review request for mesos, Kapil Arya, Michael Park, and Niklas Nielsen.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31262/
---
(Updated March 5, 2015, 9:05 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29289/
---
(Updated March 5, 2015, 7:38 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30952/#review75383
---
i'll take a look once the marked issues are resolved.
- Vinod Kone
See https://builds.apache.org/job/mesos-reviewbot/4450/
--
[URLTrigger] A change within the response URL invocation (log)
[URLTrigger] A change within the response URL invocation (log)
[URLTrigger] A change within the response URL invocation (log)
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31772/
---
(Updated March 5, 2015, 8:55 p.m.)
Review request for mesos, Ben Mahler, Ian
See https://builds.apache.org/job/Mesos-Ubuntu-distcheck/714/
--
Started by an SCM change
Started by an SCM change
Started by an SCM change
Started by an SCM change
Started by an SCM change
Started by an SCM change
Started by an SCM change
Started by an SCM
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31265/
---
(Updated March 5, 2015, 9:07 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31268/
---
(Updated March 5, 2015, 9:07 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31776/
---
(Updated March 5, 2015, 9:11 p.m.)
Review request for mesos, Kapil Arya,
See
https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui/2794/
--
Started by an SCM change
Building remotely on ubuntu-5 (docker Ubuntu ubuntu5 ubuntu) in workspace
See
https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/2512/changes
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31267/
---
(Updated March 5, 2015, 9:05 p.m.)
Review request for mesos, Kapil Arya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31772/#review75398
---
Ship it!
Ship It!
- Vinod Kone
On March 5, 2015, 9:51 p.m., Jie
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31263/#review75403
---
Ship it!
Ship It!
- Joerg Schad
On March 5, 2015, 9:11 p.m.,
See https://builds.apache.org/job/mesos-reviewbot/4453/
--
[URLTrigger] A change within the response URL invocation (log)
Building remotely on H10 (Ubuntu ubuntu) in workspace
https://builds.apache.org/job/mesos-reviewbot/ws/
git rev-parse
On March 5, 2015, 11:12 p.m., Jie Yu wrote:
I don't think adding `protocol` to `Filter` is a good idea. We should keep
`protocol` in `Classifier` because the `protocol` is used for classifying
packets, isn't it?
Regarding the basic filter, you need to something the following:
```
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31499/#review75408
---
I don't think adding `protocol` to `Filter` is a good idea. We
On March 5, 2015, 9:57 p.m., Jie Yu wrote:
src/linux/routing/filter/action.hpp, line 77
https://reviews.apache.org/r/31503/diff/2/?file=884497#file884497line77
Add a space before the colon. Also, how about calling it `action::Flow`
instead of `action::FlowId`. It reads better
See https://builds.apache.org/job/Mesos-Ubuntu-distcheck/715/changes
See https://builds.apache.org/job/mesos-reviewbot/4454/changes
On March 4, 2015, 10:01 p.m., Chi Zhang wrote:
src/linux/routing/filter/action.hpp, line 87
https://reviews.apache.org/r/31503/diff/1-2/?file=879070#file879070line87
could be just parent and child directly.
What is mesos preferred coding style here?
- Cong
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31772/
---
(Updated March 5, 2015, 9:51 p.m.)
Review request for mesos, Ben Mahler, Ian
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31503/#review75386
---
src/linux/routing/filter/action.hpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31262/#review75412
---
Ship it!
Ship It!
- Joerg Schad
On March 5, 2015, 9:05 p.m.,
Agenda and running notes for this month's meeting are being collected in:
https://docs.google.com/document/d/1BMX7M54HXFxSbaURaM0PcRQoFiflRSDtnpQN0sW-_HM/edit?usp=sharing
Feel free to add any new agenda items at the bottom
On Thu, Mar 5, 2015 at 1:22 PM, Dave Lester d...@davelester.org wrote:
On March 5, 2015, 9:57 p.m., Jie Yu wrote:
src/linux/routing/filter/internal.hpp, lines 247-252
https://reviews.apache.org/r/31503/diff/2/?file=884502#file884502line247
Adjust the comments accordingly (using action 'flow' consistently).
What's `Classid` in your comment?
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31769/#review75422
---
Patch looks great!
Reviews applied: [31769]
All tests passed.
-
On March 4, 2015, 10:01 p.m., Chi Zhang wrote:
src/linux/routing/filter/internal.hpp, line 269
https://reviews.apache.org/r/31503/diff/1-2/?file=879075#file879075line269
Do you need to undo the previous step if this is an error?
There is no way to unset a classid once we set it.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31504/
---
(Updated March 6, 2015, 12:19 a.m.)
Review request for mesos, Chi Zhang, Ian
On March 5, 2015, 1:36 a.m., Chi Zhang wrote:
This is a pretty signifant feature. I am pretty sure we need a slave flag
to control it. This file has examples for that.
I tend to agree with Jie here, we don't want inconsistency (some has flow
classifiers, some doesn't), therefore I don't
On March 5, 2015, 1:36 a.m., Chi Zhang wrote:
src/slave/containerizer/isolators/network/port_mapping.cpp, line 1531
https://reviews.apache.org/r/31505/diff/2/?file=884517#file884517line1531
looks like the library code uses uint32_t here but your set is
uint16_t. assuming you have
On March 5, 2015, 11:12 p.m., Jie Yu wrote:
I don't think adding `protocol` to `Filter` is a good idea. We should keep
`protocol` in `Classifier` because the `protocol` is used for classifying
packets, isn't it?
Regarding the basic filter, you need to something the following:
```
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29289/#review75442
---
Bad patch!
Reviews applied: [31677]
Failed command:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31250/#review75465
---
Patch looks great!
Reviews applied: [31250]
All tests passed.
-
On March 6, 2015, 3 a.m., Alexander Rukletsov wrote:
src/common/resources.cpp, line 26
https://reviews.apache.org/r/31664/diff/2/?file=883420#file883420line26
Why do we need this include?
Michael Park wrote:
We need the `inline std::size_t hash_value(const SlaveID slaveId);`
On March 5, 2015, 9:57 p.m., Jie Yu wrote:
src/linux/routing/filter/action.hpp, lines 83-84
https://reviews.apache.org/r/31503/diff/2/?file=884497#file884497line83
Why we need this constructor? IMO, it's not needed. We can always do
the following at the callsites, right?
On March 5, 2015, 1:36 a.m., Chi Zhang wrote:
src/slave/containerizer/isolators/network/port_mapping.hpp, line 195
https://reviews.apache.org/r/31505/diff/2/?file=884516#file884516line195
I think we probably want to rename other counter names to account for
both ingress and egree
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31268/#review75453
---
Patch looks great!
Reviews applied: [31775, 31776, 31266, 31267,
On March 6, 2015, 3 a.m., Alexander Rukletsov wrote:
src/common/resources.cpp, line 26
https://reviews.apache.org/r/31664/diff/2/?file=883420#file883420line26
Why do we need this include?
We need the `inline std::size_t hash_value(const SlaveID slaveId);` from it
for
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30841/#review75449
---
Patch looks great!
Reviews applied: [30841]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31664/#review75452
---
src/common/resources.cpp
On March 5, 2015, 11:12 p.m., Jie Yu wrote:
I don't think adding `protocol` to `Filter` is a good idea. We should keep
`protocol` in `Classifier` because the `protocol` is used for classifying
packets, isn't it?
Regarding the basic filter, you need to something the following:
```
On March 5, 2015, 11:12 p.m., Jie Yu wrote:
I don't think adding `protocol` to `Filter` is a good idea. We should keep
`protocol` in `Classifier` because the `protocol` is used for classifying
packets, isn't it?
Regarding the basic filter, you need to something the following:
```
On March 5, 2015, 11:12 p.m., Jie Yu wrote:
I don't think adding `protocol` to `Filter` is a good idea. We should keep
`protocol` in `Classifier` because the `protocol` is used for classifying
packets, isn't it?
Regarding the basic filter, you need to something the following:
```
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review75436
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
On March 5, 2015, 11:12 p.m., Jie Yu wrote:
I don't think adding `protocol` to `Filter` is a good idea. We should keep
`protocol` in `Classifier` because the `protocol` is used for classifying
packets, isn't it?
Regarding the basic filter, you need to something the following:
```
On March 3, 2015, 6:39 p.m., Jie Yu wrote:
3rdparty/libprocess/3rdparty/stout/include/stout/ip.hpp, line 62
https://reviews.apache.org/r/31471/diff/4/?file=882869#file882869line62
`struct in_addr` stores ip address in network order by standard. No
need to mention that in the
On March 3, 2015, 5:06 p.m., Dominic Hamon wrote:
3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp, line 108
https://reviews.apache.org/r/31471/diff/4/?file=882870#file882870line108
why did this change?
sorry for that, I accidentally used an old version of net.hpp
-
On March 4, 2015, 4:39 p.m., Jay Buffington wrote:
Hey Bernd,
I'm really looking forward to this feature. There's a lot here, so I was
hoping you could help me understand by responding to some of these
questions:
Why do you need the cache table data structure? Just use the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29550/#review75352
---
Ship it!
Ship It!
- Till Toenshoff
On Jan. 26, 2015, 11:09
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31538/#review75353
---
src/slave/slave.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31724/
---
(Updated March 5, 2015, 10:07 a.m.)
Review request for mesos, Dominic Hamon
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31470/
---
(Updated March 5, 2015, 3:48 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29290/
---
(Updated March 5, 2015, 3:48 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29289/#review75351
---
Bad patch!
Reviews applied: [31677]
Failed command:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31324/#review75312
---
CHANGELOG
https://reviews.apache.org/r/31324/#comment122301
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31538/
---
(Updated March 5, 2015, 11:02 a.m.)
Review request for mesos, Isabel Jimenez,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31472/
---
(Updated March 5, 2015, 3:48 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31471/
---
(Updated March 5, 2015, 4:06 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29288/
---
(Updated March 5, 2015, 4:06 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29288/
---
(Updated March 5, 2015, 3:48 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29289/
---
(Updated March 5, 2015, 3:48 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31473/
---
(Updated March 5, 2015, 3:48 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31471/
---
(Updated March 5, 2015, 3:46 p.m.)
Review request for mesos, Benjamin Hindman,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31769/
---
Review request for mesos, Ben Mahler and Jie Yu.
Bugs: MESOS-2401
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31250/#review75370
---
Ship it!
src/tests/isolator_tests.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31724/#review75322
---
Ship it!
Ship It!
- Alexander Rojas
On March 4, 2015, 9:37
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30774/
---
(Updated March 5, 2015, 3:15 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31538/#review75317
---
Patch looks great!
Reviews applied: [31538]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29550/#review75324
---
Ship it!
Ship It!
- Alexander Rojas
On Jan. 27, 2015, 12:09
BEGIN:VCALENDAR
VERSION:2.0
METHOD:REQUEST
PRODID:-//PYVOBJECT//NONSGML Version 1//EN
BEGIN:VTIMEZONE
TZID:America/Los_Angeles
TZURL:http://tzurl.org/zoneinfo-outlook/America/Los_Angeles
X-LIC-LOCATION:America/Los_Angeles
BEGIN:DAYLIGHT
TZOFFSETFROM:-0800
TZOFFSETTO:-0700
TZNAME:PDT
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31784/
---
Review request for mesos, Alexander Rukletsov, Joerg Schad, Till Toenshoff,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31250/
---
(Updated March 6, 2015, 2 a.m.)
Review request for mesos and Ian Downes.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31183/#review75451
---
include/mesos/resources.hpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31784/#review75458
---
Patch looks great!
Reviews applied: [31784]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31790/
---
Review request for mesos.
Bugs: MESOS-2444
94 matches
Mail list logo