HomeMesosphereNo notifications. 4 unresolved issues.

removing references to marathon cmdline options which have been removed
AbandonedAll Users

Authored by kensipe on Sep 8 2017, 12:08 AM.

Details

Summary

removed the following:

  • event_subscriber -> docs/docs/rest-api/public/api/v2/examples/info.json:34:
  • http_event_callback_slow_consumer_timeout -> tests/scale/mom.json:3:
  • save_tasks_to_launch_timeout -> tests/scale/mom.json:3:
  • max_tasks_per_offer -> tests/scale/mom.json:3:

tests/system/fixtures/mom-ee-disabled-1.3.json:3:
tests/system/fixtures/mom-ee-disabled-1.4.json:3:
tests/system/fixtures/mom-ee-permissive-1.3.json:3:
tests/system/fixtures/mom-ee-permissive-1.4.json:3:
tests/system/fixtures/mom-ee-strict-1.3.json:3:
tests/system/fixtures/mom-ee-strict-1.4.json:3:

Test Plan

none

Diff Detail

Repository
rMARATHON marathon
Branch
ken/cmdline-flag-removal
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3454
Build 6604: Marathon (revised)Jenkins
Build 6603: arc lint + arc unit
kensipe created this revision.Sep 8 2017, 12:08 AM
kensipe edited the summary of this revision. (Show Details)
jenkins requested changes to this revision.Sep 8 2017, 12:11 AM
This revision now requires changes to proceed.Sep 8 2017, 12:11 AM
kensipe updated this revision to Diff 4227.Sep 8 2017, 12:12 AM
  • updating to default
jenkins requested changes to this revision.Sep 8 2017, 12:18 AM
This revision now requires changes to proceed.Sep 8 2017, 12:18 AM
ichernetsky accepted this revision.Sep 8 2017, 12:20 AM
✗ Build of 4226 failed jenkins-public-marathon-phabricator-974.

Error message:

Stage Package Docker Image, Debian and RedHat Packages failed.

(๑′°︿°๑)

✗ Build of 4227 failed jenkins-public-marathon-phabricator-975.

Error message:

Stage Package Docker Image, Debian and RedHat Packages failed.

(๑′°︿°๑)

zen-dog accepted this revision.Sep 8 2017, 9:18 AM
jdef added a comment.Sep 8 2017, 5:44 PM

I'm confused why 1.3 and 1.4 JSONs are being changed from max_tasks_per_offer to max_instances_per_offer; the old flag was deprecated in 1.4 (but the docs don't indicate that it no longer works).

kensipe abandoned this revision.Sep 21 2017, 6:41 PM