ing sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
://reviews.apache.org/r/57487/diff/6-7/
Testing
---
Tested on local vagrant for following scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
/57487/diff/6/
Changes: https://reviews.apache.org/r/57487/diff/5-6/
Testing
---
Tested on local vagrant for following scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
sure offer is unreserved
Thanks,
Dmitriy Shirchenko
ith
@serb's nice suggestion are simply expired if offer is unused. To collect them,
we can bring back the `OfferReconciler` if the complexity warrants it.
- Dmitriy
---
This is an automatically generated e-mail. To reply, visit:
https:
scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
://reviews.apache.org/r/57487/diff/1-2/
Testing
---
Tested on local vagrant for following scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
/TaskAssignerImplTest.java
cf2d25ec2e407df7159e0021ddb44adf937e1777
Diff: https://reviews.apache.org/r/57487/diff/1/
Testing
---
Tested on local vagrant for following scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
---
Tested on local vagrant for following scenarios:
Reserving a task
Making sure returned offer comes back
Making sure offer is unreserved
Thanks,
Dmitriy Shirchenko
://reviews.apache.org/r/56690/diff/5/
Testing
---
Thanks,
Dmitriy Shirchenko
e04f6113c43eca4555ee0719f8208d7c4ebb8d61
src/test/java/org/apache/aurora/scheduler/state/TaskAssignerImplTest.java
cf2d25ec2e407df7159e0021ddb44adf937e1777
Diff: https://reviews.apache.org/r/56690/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
49d4e82cc03144b80292fe43066a6cc4d7aed88f
src/test/java/org/apache/aurora/scheduler/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing (updated)
---
Tested locally and a devel cluster.
Thanks,
Dmitriy Shirchenko
/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
gt; > Could the `offerAdded` event handler handle the exception instead of
> > letting it go up to the `EventBus`? We could log an errror and increment an
> > metric.
Done.
- Dmitriy
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56691/
/scheduler/state/TaskAssignerImplTest.java
cf2d25ec2e407df7159e0021ddb44adf937e1777
Diff: https://reviews.apache.org/r/56690/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
ld need to become private.
Done.
> On Feb. 24, 2017, 9 p.m., Zameer Manji wrote:
> > src/main/java/org/apache/aurora/scheduler/TierManager.java, line 105
> > <https://reviews.apache.org/r/56690/diff/3/?file=1646597#file1646597line105>
> >
> > In line wit
/resources/ResourceTestUtil.java
e04f6113c43eca4555ee0719f8208d7c4ebb8d61
src/test/java/org/apache/aurora/scheduler/state/TaskAssignerImplTest.java
cf2d25ec2e407df7159e0021ddb44adf937e1777
Diff: https://reviews.apache.org/r/56690/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
> >
> > `getTier` is on a hot code path and I therefore recently rewrote the
> > `checkArgument` here to only construct the String if the argument is
> > invalid. This yielded a significant perf improvement in our scheduling
> > benchmarks.
Yeah, I could n
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56691/#review166450
---
@ReviewBot retry
- Dmitriy Shirchenko
On Feb. 22, 2017, 10:35
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56691/#review166434
---
@ReviewBot retry
- Dmitriy Shirchenko
On Feb. 22, 2017, 10:35
/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
/scheduler/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
/OfferManagerImplTest.java
49d4e82cc03144b80292fe43066a6cc4d7aed88f
src/test/java/org/apache/aurora/scheduler/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
/OfferManagerImplTest.java
49d4e82cc03144b80292fe43066a6cc4d7aed88f
src/test/java/org/apache/aurora/scheduler/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
/OfferManagerImplTest.java
49d4e82cc03144b80292fe43066a6cc4d7aed88f
src/test/java/org/apache/aurora/scheduler/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
/resources/ResourceTestUtil.java
e04f6113c43eca4555ee0719f8208d7c4ebb8d61
src/test/java/org/apache/aurora/scheduler/state/TaskAssignerImplTest.java
cf2d25ec2e407df7159e0021ddb44adf937e1777
Diff: https://reviews.apache.org/r/56690/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
5e570b6341c55be8ef27469077932d1ea8378b55
src/test/java/org/apache/aurora/scheduler/offers/OfferReconcilerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/56691/diff/
Testing
---
Ran locally on vagrant.
Thanks,
Dmitriy Shirchenko
7dc8c179736fab93ca03e174b16e104ba0118127
src/test/java/org/apache/aurora/scheduler/state/TaskAssignerImplTest.java
f8bb25b612ae42748205901dd4b8cfcdedf5b136
Diff: https://reviews.apache.org/r/56690/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55902/#review163004
---
@ReviewBot retry
- Dmitriy Shirchenko
On Jan. 25, 2017, 6:47
attached.
File Attachments
Updated screenshot
https://reviews.apache.org/media/uploaded/files/2017/01/25/90d6ff4f-84f9-4b4d-9b3d-56dadf7027ae__Screen_Shot_2017-01-24_at_5.32.08_PM.png
Thanks,
Dmitriy Shirchenko
attached.
File Attachments
Updated screenshot
https://reviews.apache.org/media/uploaded/files/2017/01/25/90d6ff4f-84f9-4b4d-9b3d-56dadf7027ae__Screen_Shot_2017-01-24_at_5.32.08_PM.png
Thanks,
Dmitriy Shirchenko
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55902/#review162894
---
@ReviewBot retry
- Dmitriy Shirchenko
On Jan. 25, 2017, 1:34
unit tests
e2e tests
screenshot attached.
File Attachments (updated)
Updated screenshot
https://reviews.apache.org/media/uploaded/files/2017/01/25/90d6ff4f-84f9-4b4d-9b3d-56dadf7027ae__Screen_Shot_2017-01-24_at_5.32.08_PM.png
Thanks,
Dmitriy Shirchenko
-9b3d-56dadf7027ae__Screen_Shot_2017-01-24_at_5.32.08_PM.png
Thanks,
Dmitriy Shirchenko
Screenshot
https://reviews.apache.org/media/uploaded/files/2017/01/25/c4e69424-71ad-4d71-b1f4-895bc6a7821e__Screen_Shot_2017-01-24_at_5.03.06_PM.png
Thanks,
Dmitriy Shirchenko
/MemTaskStore.java (lines
116 - 120)
<https://reviews.apache.org/r/55217/#comment231758>
:nit any reason why you didn't extract second index declration on 117:120
- Dmitriy Shirchenko
On Jan. 5, 2017, 6:59 p.m., Mehrdad Nurola
e rest of the
code base.
- Dmitriy Shirchenko
On Dec. 30, 2016, 9:18 p.m., Mehrdad Nurolahzade 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/52437/#review154818
---
Ship it!
Ship It!
- Dmitriy Shirchenko
On Oct. 12, 2016, 11
image name as a
label and using bento/debian-8.2 instead?
[1]
https://github.com/apache/aurora-packaging/blob/master/builder/deb/debian-jessie/Dockerfile#L13
- Dmitriy Shirchenko
On Oct. 4, 2016, 10:16 p.m., Renan DelValle wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52437/#review151091
---
Ship it!
Ship It!
- Dmitriy Shirchenko
On Sept. 30, 2016, 7
tps://reviews.apache.org/r/52437/#comment219178>
For my own curiousity: is the step of adding gpg keys strictly necessary?
Why does ubuntu need it but not debian?
- Dmitriy Shirchenko
On Sept. 30, 2016, 7:23 p.m., Renan DelValle
: https://reviews.apache.org/r/52276/diff/
Testing
---
Verified in vagrant + added unit tests.
Thanks,
Dmitriy Shirchenko
---
Verified in vagrant + added unit tests.
Thanks,
Dmitriy Shirchenko
also needs to be documented.
- Dmitriy Shirchenko
On Sept. 20, 2016, 7:43 p.m., Kai Huang wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> http
a739bce1226d9435fa7d0b18e411064a4e78e49e
Diff: https://reviews.apache.org/r/52074/diff/
Testing
---
src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
Thanks,
Dmitriy Shirchenko
a739bce1226d9435fa7d0b18e411064a4e78e49e
Diff: https://reviews.apache.org/r/52074/diff/
Testing
---
src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
Thanks,
Dmitriy Shirchenko
a739bce1226d9435fa7d0b18e411064a4e78e49e
Diff: https://reviews.apache.org/r/52074/diff/
Testing
---
src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
Thanks,
Dmitriy Shirchenko
/OfferManagerImplTest.java
(lines 50 - 54)
<https://reviews.apache.org/r/52074/#comment217606>
Agreed. I have been told that Aurora's philosophy is convention first, so
I'm following what has been set.
- Dmitriy Shirchenko
On Sept. 20, 2016, 7:14 p.m., Dmitriy
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52074/#review149714
-------
On Sept. 20, 2016, 7:14 p.m., Dmitriy Shirchenko wrote:
>
>
24>
> >
> > nit: add newline above
Done.
- Dmitriy
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52074/#review149862
--------
a739bce1226d9435fa7d0b18e411064a4e78e49e
Diff: https://reviews.apache.org/r/52074/diff/
Testing
---
src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
Thanks,
Dmitriy Shirchenko
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52087/#review149707
---
Ship it!
Ship It!
- Dmitriy Shirchenko
On Sept. 20, 2016, 6
a739bce1226d9435fa7d0b18e411064a4e78e49e
Diff: https://reviews.apache.org/r/52074/diff/
Testing
---
src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
Thanks,
Dmitriy Shirchenko
s_leader_redirect = True?
- Dmitriy Shirchenko
On Sept. 20, 2016, 5:32 p.m., Joshua Cohen wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
>
925c025d21bf1d44e0c1d319f6653ecfa8899481
src/test/java/org/apache/aurora/scheduler/offers/OfferManagerImplTest.java
a739bce1226d9435fa7d0b18e411064a4e78e49e
Diff: https://reviews.apache.org/r/52074/diff/
Testing
---
src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
Thanks,
Dmitriy Shirchenko
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51980/#review149530
---
@ReviewBot retry
- Dmitriy Shirchenko
On Sept. 19, 2016, 6:28
/
Testing
---
Thanks,
Dmitriy Shirchenko
.
Repository: aurora
Description
---
Due to spam bots, JIRA accounts need to be whitelisted before they can
self-assign tasks.
Diffs
-
CONTRIBUTING.md 13ebbe4be281f869d276ebe97bfae75123899cfb
Diff: https://reviews.apache.org/r/52049/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
allow easier testing so cleaner unit tests
were added with more code coverage.
Thanks,
Dmitriy Shirchenko
testing so cleaner unit tests
were added with more code coverage.
Thanks,
Dmitriy Shirchenko
/apache/aurora/scheduler/events/WebhookTest.java
488eefd14c3e67a41a75c809397c8d19f83cc08a
Diff: https://reviews.apache.org/r/51980/diff/
Testing
---
Part of reason for refactor is to allow easier testing so cleaner unit tests
were added with more code coverage.
Thanks,
Dmitriy
H_PORT_NAME if requested.
- Dmitriy Shirchenko
On Aug. 31, 2016, 10:08 p.m., Kai Huang wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://rev
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50826/#review144853
---
Ship it!
plz add a test and then LGTM
- Dmitriy Shirchenko
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50685/#review144653
---
Ship it!
Ship It!
- Dmitriy Shirchenko
On Aug. 3, 2016, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50478/#review143764
---
Ship it!
Ship It!
- Dmitriy Shirchenko
On July 27, 2016, 1
like one of them is not necessary.
```json
reservation: {
labels: {
labels: [
{
key: "job",
value: "devcluster/www-data/prod/hello"
}
]
}
}
```
- Dmitriy Shirchenko
On July 19, 2016, 2:12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48492/#review136840
---
Ship it!
Ship It!
- Dmitriy Shirchenko
On June 9, 2016, 3
; > According to the docs, `getErrorStream` _may_ return `null`, so we need
> > a guard here.
> > https://docs.oracle.com/javase/7/docs/api/java/net/HttpURLConnection.html#getErrorStream()
Good call. Done.
- Dmitriy
---
Th
sting
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
vents/WebhookModule.java
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
vents/WebhookModule.java
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
.apache.org/r/47440/#review135233
---
On June 3, 2016, 10:17 p.m., Dmitriy Shirchenko wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> h
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47440/#review135105
---
@ReviewBot retry
- Dmitriy Shirchenko
On May 26, 2016, 10:29
, lines 73-74
> > <https://reviews.apache.org/r/47440/diff/8/?file=1395852#file1395852line73>
> >
> > -4 spaces
Done.
- Dmitriy
---
This is an automatically generated e-mail. To reply, visit:
https:
ATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
.java
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
.java
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
;https://reviews.apache.org/r/47440/#comment199788>
Done.
src/main/java/org/apache/aurora/scheduler/events/WebhookInfo.java (line 75)
<https://reviews.apache.org/r/47440/#comment199789>
Done,
- Dmitriy Shirchenko
On May 25, 2016,
gt;
> >
> > Would it make sense to move this into WebhookInfo instead?
Done.
- Dmitriy
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47440/#review134392
-----
.java
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
eg tiers.json. Is that OK? It's not mixed in with
the code.
src/main/java/org/apache/aurora/scheduler/events/WebhookModule.java (line 73)
<https://reviews.apache.org/r/47440/#comment199485>
Done.
- Dmitriy Shirchenko
On May 22, 20
ke
> > sense to establish an explicit schema.
Good point. Essentially these are IScheduledTask objects which I believe
already require a deprecation cycle. Is this correct?
- Dmitriy
-------
This is an automatically generated e-ma
urces/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
er thread or added to a queue for
> > future delivery? AFAICT the distribution of events is synchronous, so it
> > seems like operations that might block for any significant amount of time
> > could cause issues.
>
> Dmitriy Shirchenko wrote:
> Yea, I'm not sure. Do you
ote:
> > src/main/java/org/apache/aurora/scheduler/events/Webhook.java, line 30
> > <https://reviews.apache.org/r/47440/diff/3/?file=1385244#file1385244line30>
> >
> > Missing javadoc for class and public methods.
Done.
> On May 17, 2016, 5:09 p.m., Maxi
urces/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
-------
On May 18, 2016, 9:12 p.m., Dmitriy Shirchenko wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47440/
> --
er thread or added to a queue for
> > future delivery? AFAICT the distribution of events is synchronous, so it
> > seems like operations that might block for any significant amount of time
> > could cause issues.
>
> Dmitriy Shirchenko wrote:
> Yea, I'm not sure. Do you
urces/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
urces/org/apache/aurora/scheduler/webhook.json PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
tomatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47440/#review133482
-------
On May 17, 2016, 2:10 a.m., Dmitriy Shirchenko wrote:
>
> -
n PRE-CREATION
src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47440/diff/
Testing
---
Need to fix tests.
Thanks,
Dmitriy Shirchenko
Diff: https://reviews.apache.org/r/46361/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
/test/python/apache/aurora/executor/common/test_health_checker.py
ff39e869634830b0eb59ddfe52a5a48ca1605a66
Diff: https://reviews.apache.org/r/46361/diff/
Testing
---
Thanks,
Dmitriy Shirchenko
tor/common/test_health_checker.py
4ab7a2fab651abd5ab8a0f312d10c444800e8b7a
Diff: https://reviews.apache.org/r/46290/diff/
Testing
---
- end to end tests
- added unit tests
Thanks,
Dmitriy Shirchenko
tor/common/test_health_checker.py
4ab7a2fab651abd5ab8a0f312d10c444800e8b7a
Diff: https://reviews.apache.org/r/46290/diff/
Testing
---
- end to end tests
- added unit tests
Thanks,
Dmitriy Shirchenko
tor/common/test_health_checker.py
4ab7a2fab651abd5ab8a0f312d10c444800e8b7a
Diff: https://reviews.apache.org/r/46290/diff/
Testing
---
- end to end tests
- added unit tests
Thanks,
Dmitriy Shirchenko
rora/executor/common/health_checker.py
d8675beb8f16ebdd2d6946367784411fe84a5cfc
src/test/python/apache/aurora/executor/common/test_health_checker.py
4ab7a2fab651abd5ab8a0f312d10c444800e8b7a
Diff: https://reviews.apache.org/r/46290/diff/
Testing
---
- end to end tests
- added unit test
1 - 100 of 192 matches
Mail list logo