[GitHub] rdelval closed pull request #23: Fix bug. Do not increase current_consecutive_successes if .healthchec…

2018-07-30 Thread GitBox
rdelval closed pull request #23: Fix bug. Do not increase current_consecutive_successes if .healthchec… URL: https://github.com/apache/aurora/pull/23 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rdelval closed pull request #7: vagrant example on exoscale Open Cloud

2018-07-30 Thread GitBox
rdelval closed pull request #7: vagrant example on exoscale Open Cloud URL: https://github.com/apache/aurora/pull/7 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] rdelval closed pull request #2: Fix bad indentation causing bad layout in deploying-aurora-scheduler.md

2018-07-30 Thread GitBox
rdelval closed pull request #2: Fix bad indentation causing bad layout in deploying-aurora-scheduler.md URL: https://github.com/apache/aurora/pull/2 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] rdelval closed pull request #6: Deleting message on PR that would discourage making a PR

2018-07-30 Thread GitBox
rdelval closed pull request #6: Deleting message on PR that would discourage making a PR URL: https://github.com/apache/aurora-packaging/pull/6 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] rdelval opened a new pull request #6: Deleting message on PR that would discourage making a PR as we could …

2018-07-30 Thread GitBox
rdelval opened a new pull request #6: Deleting message on PR that would discourage making a PR as we could … URL: https://github.com/apache/aurora-packaging/pull/6 We can take PRs now so this message no longer makes sense :)

[GitHub] rdelval closed pull request #5: Added dependencies to build aurora 0.20.0

2018-07-30 Thread GitBox
rdelval closed pull request #5: Added dependencies to build aurora 0.20.0 URL: https://github.com/apache/aurora-packaging/pull/5 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] rdelval closed pull request #26: Deleting message discouraging PRs on github

2018-07-31 Thread GitBox
rdelval closed pull request #26: Deleting message discouraging PRs on github URL: https://github.com/apache/aurora/pull/26 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] rdelval opened a new pull request #26: Deleting message discouraging PRs on github

2018-07-31 Thread GitBox
rdelval opened a new pull request #26: Deleting message discouraging PRs on github URL: https://github.com/apache/aurora/pull/26 Also testing out our newly acquired capacity to take PRs on github  This is an automated

[GitHub] StephanErb commented on issue #29: Add Travis file for Aurora tests

2018-08-09 Thread GitBox
StephanErb commented on issue #29: Add Travis file for Aurora tests URL: https://github.com/apache/aurora/pull/29#issuecomment-411736575 We probably need to file an infra ticket to enable this feature if we want it. I am lacking the necessary permissions to enable it for the apache/aurora

[GitHub] StephanErb opened a new pull request #29: Add Travis file for Aurora tests

2018-08-09 Thread GitBox
StephanErb opened a new pull request #29: Add Travis file for Aurora tests URL: https://github.com/apache/aurora/pull/29 We could also continue to use the Apache Jenkins but I think Travis is simpler. This is an automated

[GitHub] StephanErb opened a new pull request #28: Fix sandbox permission errors with Mesos 1.6.0

2018-08-09 Thread GitBox
StephanErb opened a new pull request #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28 This change makes Aurora compatible with Mesos 1.6.0 which has changed the permission of sandbox folders (see

[GitHub] DavidMcLaughlin commented on issue #28: Fix sandbox permission errors with Mesos 1.6.0

2018-08-10 Thread GitBox
DavidMcLaughlin commented on issue #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28#issuecomment-412233951 Does this degrade gracefully to older versions of Mesos? This is

[GitHub] DavidMcLaughlin opened a new issue #30: Count number of times partitioned tasks reenter the cluster as healthy

2018-08-14 Thread GitBox
DavidMcLaughlin opened a new issue #30: Count number of times partitioned tasks reenter the cluster as healthy URL: https://github.com/apache/aurora/issues/30 Currently when a task is PARTITIONED and LOST, Aurora reschedules a replacement. Later on, the task can send a message saying it

[GitHub] jordanly opened a new pull request #33: Fix possible NullPointerException in InstanceActionHandler

2018-08-16 Thread GitBox
jordanly opened a new pull request #33: Fix possible NullPointerException in InstanceActionHandler URL: https://github.com/apache/aurora/pull/33 See #31 for a description of the issue. Ensure that we properly handle possible null values when checking the SlaPolicy of an update.

[GitHub] shanmugh commented on a change in pull request #33: Fix possible NullPointerException in InstanceActionHandler

2018-08-16 Thread GitBox
shanmugh commented on a change in pull request #33: Fix possible NullPointerException in InstanceActionHandler URL: https://github.com/apache/aurora/pull/33#discussion_r210765698 ## File path: src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh ## @@ -223,6 +223,17 @@

[GitHub] jordanly opened a new issue #32: Generic exceptions within storage.write statements are not caught potentially causing inconsistent state

2018-08-16 Thread GitBox
jordanly opened a new issue #32: Generic exceptions within storage.write statements are not caught potentially causing inconsistent state URL: https://github.com/apache/aurora/issues/32 A finding from https://github.com/apache/aurora/issues/31. A user created an update to remove

[GitHub] jordanly opened a new issue #31: Possible NullPointerException in InstanceActionHandler

2018-08-16 Thread GitBox
jordanly opened a new issue #31: Possible NullPointerException in InstanceActionHandler URL: https://github.com/apache/aurora/issues/31 After introducing the 'slaAware' field in [this commit](https://github.com/apache/aurora/commit/4e28e73bb83b75f1369a705d15c96e40fb7098e1), there has

[GitHub] DavidMcLaughlin closed pull request #33: Fix possible NullPointerException in InstanceActionHandler

2018-08-16 Thread GitBox
DavidMcLaughlin closed pull request #33: Fix possible NullPointerException in InstanceActionHandler URL: https://github.com/apache/aurora/pull/33 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] DavidMcLaughlin closed issue #31: Possible NullPointerException in InstanceActionHandler

2018-08-16 Thread GitBox
DavidMcLaughlin closed issue #31: Possible NullPointerException in InstanceActionHandler URL: https://github.com/apache/aurora/issues/31 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] edi33416 opened a new issue #27: Update custom executor example

2018-08-07 Thread GitBox
edi33416 opened a new issue #27: Update custom executor example URL: https://github.com/apache/aurora/issues/27 I've been trying to run the `mesos-executor` as the custom executor, by following the [instructions](http://aurora.apache.org/documentation/latest/features/custom-executors/)

[GitHub] rdelval closed pull request #29: Add Travis file for Aurora tests

2018-08-11 Thread GitBox
rdelval closed pull request #29: Add Travis file for Aurora tests URL: https://github.com/apache/aurora/pull/29 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] jsirois opened a new pull request #34: Fixup the pycharm setup script.

2018-08-21 Thread GitBox
jsirois opened a new pull request #34: Fixup the pycharm setup script. URL: https://github.com/apache/aurora/pull/34 With modern PyCharm (2018+), virtualenv support for wheels and namespace packages works, so kill the obsolete `--egg` in our pip installs. Also leverage `./pants

[GitHub] rdelval closed pull request #34: Fixup the pycharm setup script.

2018-08-21 Thread GitBox
rdelval closed pull request #34: Fixup the pycharm setup script. URL: https://github.com/apache/aurora/pull/34 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] StephanErb commented on issue #28: Fix sandbox permission errors with Mesos 1.6.0

2018-08-21 Thread GitBox
StephanErb commented on issue #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28#issuecomment-414761435 Given our default Aurora and Thermos setup where the executor is launched as `root`: * In Mesos < 1.6: *

[GitHub] jsirois commented on issue #34: Fixup the pycharm setup script.

2018-08-21 Thread GitBox
jsirois commented on issue #34: Fixup the pycharm setup script. URL: https://github.com/apache/aurora/pull/34#issuecomment-414789036 This worked for me on PyCharm 2018.2.1, where worked means code navigation through namespace packages (twitter.common.*) and unit test running via

[GitHub] rdelval closed pull request #36: Removing MD5 check from verify release candidate script and release script

2018-09-04 Thread GitBox
rdelval closed pull request #36: Removing MD5 check from verify release candidate script and release script URL: https://github.com/apache/aurora/pull/36 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rdelval opened a new pull request #36: Removing MD5 check from verify release candidate script and release script

2018-09-04 Thread GitBox
rdelval opened a new pull request #36: Removing MD5 check from verify release candidate script and release script URL: https://github.com/apache/aurora/pull/36 No longer needed as Apache policy recommends at least SHA-256

[GitHub] rdelval edited a comment on issue #28: Fix sandbox permission errors with Mesos 1.6.0

2018-09-05 Thread GitBox
rdelval edited a comment on issue #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28#issuecomment-418752624 Giving notice to other PMC members in the interest of getting 0.21.0 out the door: Given that this PR has been open for 27 days and

[GitHub] rdelval commented on issue #28: Fix sandbox permission errors with Mesos 1.6.0

2018-09-05 Thread GitBox
rdelval commented on issue #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28#issuecomment-418752624 Giving fair warning to other PMC members in the interest of getting 0.21.0 out the door: Given that this PR has been open for 27 days and is

[GitHub] rdelval edited a comment on issue #28: Fix sandbox permission errors with Mesos 1.6.0

2018-09-05 Thread GitBox
rdelval edited a comment on issue #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28#issuecomment-418752624 Notice to other PMC members in the interest of getting 0.21.0 out the door: Given that this PR has been open for 27 days and is

[GitHub] rdelval closed pull request #28: Fix sandbox permission errors with Mesos 1.6.0

2018-09-05 Thread GitBox
rdelval closed pull request #28: Fix sandbox permission errors with Mesos 1.6.0 URL: https://github.com/apache/aurora/pull/28 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] rdelval opened a new pull request #37: Staggered (Variable batch) Updates

2018-09-10 Thread GitBox
rdelval opened a new pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37 Adding support for variable group sizes when executing an update. Design doc for this change is here:

[GitHub] rdelval opened a new pull request #35: Removing MD5 checksum and making SHA-512 our default release checksum

2018-08-29 Thread GitBox
rdelval opened a new pull request #35: Removing MD5 checksum and making SHA-512 our default release checksum URL: https://github.com/apache/aurora/pull/35 According to new Apache policy, we should only distribute a SHA-256 or SHA-512 checksum along with new releases. Removing MD5

[GitHub] shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r220977471 ## File path: src/test/python/apache/aurora/client/api/test_updater_util.py ## @@ -46,15 +71,94 @@ def

[GitHub] shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r220977389 ## File path: src/test/python/apache/aurora/client/api/test_updater_util.py ## @@ -46,15 +71,94 @@ def

[GitHub] shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r220977423 ## File path: src/test/python/apache/aurora/client/api/test_updater_util.py ## @@ -46,15 +71,94 @@ def

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221010750 ## File path: src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java ## @@ -792,9 +796,51 @@

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221013225 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221020555 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221013639 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221013032 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221017239 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221012433 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221012761 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0

[GitHub] jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
jordanly commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221011282 ## File path: src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java ## @@ -1026,7 +1051,6

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221020632 ## File path: src/main/java/org/apache/aurora/scheduler/updater/UpdateFactory.java ## @@ -76,12 +77,10 @@ Update

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221019490 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0 +1,152

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221021619 ## File path: src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh ## @@ -364,7 +364,6 @@ test_update() { # Check

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221020786 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0 +1,152

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221020703 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0 +1,152

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221021099 ## File path: src/main/python/apache/aurora/client/api/updater_util.py ## @@ -80,6 +84,18 @@ def

[GitHub] shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221069282 ## File path: src/main/java/org/apache/aurora/scheduler/updater/UpdateFactory.java ## @@ -76,12 +77,10 @@ Update

[GitHub] shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
shanmugh commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221069680 ## File path: src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh ## @@ -364,7 +364,6 @@ test_update() { # Check

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-11 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-429174966 @StephanErb almost have this figured out. My idea is to pass the uid of the user vagrant from the vagrant box to the container. Once we have

[GitHub] rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-15 Thread GitBox
rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-429971376 > @rdelval sorry for the delay. I have a bad gut feeling with the proposed fix as it would imply our users have to adjust their docker

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-15 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-430018282 Okay, found out why this tests fails but I'm not sure what we'll be able to do about it:

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-15 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-429971376 > @rdelval sorry for the delay. I have a bad gut feeling with the proposed fix as it would imply our users have to adjust their docker containers.

[GitHub] rdelval opened a new pull request #45: Miscellaneous cosmetic fixes and Travis CI badge

2018-10-15 Thread GitBox
rdelval opened a new pull request #45: Miscellaneous cosmetic fixes and Travis CI badge URL: https://github.com/apache/aurora/pull/45 * Removed stray spaces. * Added Travis CI badge. * Added github PR template to replicate the format we had in ReviewBoard.

[GitHub] rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-15 Thread GitBox
rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-429971376 > @rdelval sorry for the delay. I have a bad gut feeling with the proposed fix as it would imply our users have to adjust their docker

[GitHub] rdelval closed pull request #43: Custom executor UI fix

2018-10-15 Thread GitBox
rdelval closed pull request #43: Custom executor UI fix URL: https://github.com/apache/aurora/pull/43 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] rdelval commented on issue #43: Custom executor UI fix

2018-10-15 Thread GitBox
rdelval commented on issue #43: Custom executor UI fix URL: https://github.com/apache/aurora/pull/43#issuecomment-430015151 Thanks @StephanErb. The friendly neighborhood @jcohen made sure can only use the "Squash and merge" strategy so no more worries about wonky history 

[GitHub] rdelval commented on a change in pull request #44: Change devtoolset-7 (scl-rh) of CentOS6

2018-10-14 Thread GitBox
rdelval commented on a change in pull request #44: Change devtoolset-7 (scl-rh) of CentOS6 URL: https://github.com/apache/aurora/pull/44#discussion_r225005578 ## File path: build-support/python/make-mesos-native-egg ## @@ -139,38 +139,33 @@ CENTOS6_X86_64_DEPENDENCIES=(

[GitHub] rdelval commented on a change in pull request #44: Change devtoolset-7 (scl-rh) of CentOS6

2018-10-14 Thread GitBox
rdelval commented on a change in pull request #44: Change devtoolset-7 (scl-rh) of CentOS6 URL: https://github.com/apache/aurora/pull/44#discussion_r225005389 ## File path: build-support/python/make-mesos-native-egg ## @@ -139,38 +139,33 @@ CENTOS6_X86_64_DEPENDENCIES=(

[GitHub] thinker0 commented on a change in pull request #44: Change devtoolset-7 (scl-rh) of CentOS6

2018-10-15 Thread GitBox
thinker0 commented on a change in pull request #44: Change devtoolset-7 (scl-rh) of CentOS6 URL: https://github.com/apache/aurora/pull/44#discussion_r225062503 ## File path: build-support/python/make-mesos-native-egg ## @@ -139,38 +139,33 @@ CENTOS6_X86_64_DEPENDENCIES=(

[GitHub] thinker0 commented on issue #44: Change devtoolset-7 (scl-rh) of CentOS6

2018-10-15 Thread GitBox
thinker0 commented on issue #44: Change devtoolset-7 (scl-rh) of CentOS6 URL: https://github.com/apache/aurora/pull/44#issuecomment-429758817 Uploaded. https://bintray.com/thinker0/mesos/mesos-executor-egg This is an

[GitHub] StephanErb commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-15 Thread GitBox
StephanErb commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-429775655 @rdelval sorry for the delay. I have a bad gut feeling with the proposed fix as it would imply our users have to adjust their docker containers.

[GitHub] rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-17 Thread GitBox
rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-430018282 Okay, found out why this tests fails but I'm not sure what we'll be able to do about it:

[GitHub] rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-16 Thread GitBox
rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-429971376 > @rdelval sorry for the delay. I have a bad gut feeling with the proposed fix as it would imply our users have to adjust their docker

[GitHub] StephanErb opened a new pull request #46: Update Pystachio to 0.8.4

2018-10-16 Thread GitBox
StephanErb opened a new pull request #46: Update Pystachio to 0.8.4 URL: https://github.com/apache/aurora/pull/46 This fixes a few inconsistencies with hashing and equality https://github.com/wickman/pystachio/pull/30. This is not directly applicable for Aurora but will be for downstream

[GitHub] rdelval closed pull request #46: Update Pystachio to 0.8.4

2018-10-16 Thread GitBox
rdelval closed pull request #46: Update Pystachio to 0.8.4 URL: https://github.com/apache/aurora/pull/46 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] rdelval closed pull request #18: update output of commands #AURORA-1434

2018-10-16 Thread GitBox
rdelval closed pull request #18: update output of commands #AURORA-1434 URL: https://github.com/apache/aurora/pull/18 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] rdelval opened a new issue #47: Deprecate Thrift method of backup restoration

2018-10-19 Thread GitBox
rdelval opened a new issue #47: Deprecate Thrift method of backup restoration URL: https://github.com/apache/aurora/issues/47 Thrift backup restoration should be deprecated in 0.22.0 and removed in 0.23.0 given the fact that the restoration tool has existed since 0.20.0

[GitHub] rdelval closed pull request #44: Change devtoolset-7 (scl-rh) of CentOS6

2018-10-15 Thread GitBox
rdelval closed pull request #44: Change devtoolset-7 (scl-rh) of CentOS6 URL: https://github.com/apache/aurora/pull/44 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] rdelval opened a new issue #48: Password/Username not forwarded by followers

2018-10-29 Thread GitBox
rdelval opened a new issue #48: Password/Username not forwarded by followers URL: https://github.com/apache/aurora/issues/48 How to reproduce: Send thrift request that requires username and password to succeed to a follower.

[GitHub] rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-31 Thread GitBox
rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-434824242 @StephanErb added the following line to revert permissions to pre Mesos 1.6 levels for Docker based tasks:

[GitHub] rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-31 Thread GitBox
rdelval edited a comment on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-434824242 @StephanErb added the following line to revert permissions to pre Mesos 1.6 levels:

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-31 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-434824242 @StephanErb added the following line:

[GitHub] StephanErb closed pull request #45: Miscellaneous cosmetic fixes and Travis CI badge

2018-10-31 Thread GitBox
StephanErb closed pull request #45: Miscellaneous cosmetic fixes and Travis CI badge URL: https://github.com/apache/aurora/pull/45 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] rdelval commented on a change in pull request #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-31 Thread GitBox
rdelval commented on a change in pull request #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#discussion_r229893534 ## File path: src/main/python/apache/aurora/executor/common/sandbox.py ## @@ -182,6 +182,10 @@ def

[GitHub] StephanErb commented on issue #45: Miscellaneous cosmetic fixes and Travis CI badge

2018-10-31 Thread GitBox
StephanErb commented on issue #45: Miscellaneous cosmetic fixes and Travis CI badge URL: https://github.com/apache/aurora/pull/45#issuecomment-434865710 Thanks! (why did I not see this earlier?) This is an automated message

[GitHub] StephanErb commented on a change in pull request #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-31 Thread GitBox
StephanErb commented on a change in pull request #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#discussion_r229886233 ## File path: src/main/python/apache/aurora/executor/common/sandbox.py ## @@ -182,6 +182,10 @@ def

[GitHub] StephanErb commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-11-01 Thread GitBox
StephanErb commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-434995648 @rdelval the end-to-end tests are failing for me: ``` (venv) serb@mac-0017 [10:30:48] [~/workspace/aurora/aurora] [mesosUpgrade] -> %

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-11-01 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-435222905 Funny enough out of all the errors I experienced working on this, this was never one of them. I'll try running the tests again.

[GitHub] rdelval closed pull request #42: Upgrade dev enviornment to Mesos 1.6.1

2018-11-13 Thread GitBox
rdelval closed pull request #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-11-13 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-438386798 @jordanly was kind enough to run this overnight and all test succeeded. I will go ahead and merge this by the end of today if no one opposes.

[GitHub] StephanErb commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-11-13 Thread GitBox
StephanErb commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-438421556 @rdelval sounds good to me. I still have issues getting the tests to pass but these seem to be unrelated to the changes here.

[GitHub] rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-08 Thread GitBox
rdelval commented on issue #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42#issuecomment-428026150 Looking into an issue that the end to end tests run into: ``` + test_thermos_profile devcluster/vagrant/test/http_example_docker ++ aurora

[GitHub] rdelval opened a new pull request #42: Upgrade dev enviornment to Mesos 1.6.1

2018-10-08 Thread GitBox
rdelval opened a new pull request #42: Upgrade dev enviornment to Mesos 1.6.1 URL: https://github.com/apache/aurora/pull/42 * Upgraded Mesos dependencies to 1.6.1 * Uploaded apache-aurora/dev-environment 0.0.17 up to Vagrant Cloud * Upgraded Docker2ACI to latest version. * Upgraded

[GitHub] rdelval closed pull request #37: Staggered (Variable batch) Updates

2018-10-08 Thread GitBox
rdelval closed pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] dknig1b commented on a change in pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
dknig1b commented on a change in pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#discussion_r222022790 ## File path: src/test/python/apache/aurora/config/test_loader.py ## @@ -93,6 +101,44 @@ def

[GitHub] shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#discussion_r222021637 ## File path: src/test/python/apache/aurora/config/test_loader.py ## @@ -93,6 +101,44 @@ def

[GitHub] shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#discussion_r222092818 ## File path: src/main/python/apache/aurora/config/loader.py ## @@ -56,17 +62,49 @@ def flush_schemas(cls):

[GitHub] shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#discussion_r222090928 ## File path: src/test/python/apache/aurora/config/test_loader.py ## @@ -93,6 +101,44 @@ def

[GitHub] dknig1b opened a new pull request #41: Memoize loader.load and loader.load_json

2018-10-01 Thread GitBox
dknig1b opened a new pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41 Problem: when reusing the aurora.client.cli.context to process multiple jobkeys from a config_file, the client currently reloads and processes the config file for

[GitHub] dknig1b commented on issue #41: Memoize loader.load and loader.load_json

2018-10-01 Thread GitBox
dknig1b commented on issue #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#issuecomment-425965843 adding @shanmugh @jordanly @rezmont This is an automated message from the Apache

[GitHub] dknig1b commented on a change in pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
dknig1b commented on a change in pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#discussion_r222119497 ## File path: src/main/python/apache/aurora/config/loader.py ## @@ -56,17 +62,49 @@ def flush_schemas(cls):

[GitHub] jordanly closed pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
jordanly closed pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json

2018-10-02 Thread GitBox
shanmugh commented on a change in pull request #41: Memoize loader.load and loader.load_json URL: https://github.com/apache/aurora/pull/41#discussion_r222119950 ## File path: src/main/python/apache/aurora/config/loader.py ## @@ -56,17 +62,49 @@ def flush_schemas(cls):

[GitHub] rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates

2018-09-27 Thread GitBox
rdelval commented on a change in pull request #37: Staggered (Variable batch) Updates URL: https://github.com/apache/aurora/pull/37#discussion_r221124930 ## File path: src/main/java/org/apache/aurora/scheduler/updater/strategy/VariableBatchStrategy.java ## @@ -0,0 +1,152

[GitHub] rdelval opened a new pull request #40: Move Mesos Egg generation from Vagrant to Docker

2018-09-28 Thread GitBox
rdelval opened a new pull request #40: Move Mesos Egg generation from Vagrant to Docker URL: https://github.com/apache/aurora/pull/40 Addressing #39 en route to #38. Tested all distros. CentOS6 is broken but it was also broken when using a vagrantbox. It is a candidate for removal

  1   2   3   4   >