Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Support for Mesos resource-limits #3231

Closed
wants to merge 29 commits into from
Closed

Conversation

timcharper
Copy link
Contributor

@timcharper timcharper commented Jan 31, 2020

JIRA Issues: D2IQ-62959, D2IQ-62961

@kaiwalyajoshi
Copy link
Contributor

@timcharper 👍
Was thinking about adding more tests for the side-car task case where I think we'll get significant gains in terms of resource-reuse in our current payloads.

Add test to assert that empty resource limit values are handled
@timcharper
Copy link
Contributor Author

Yeah, when Mesos support is ready it'd be great to have an actual integration test with Mesos to assert that the limits actually work

@timcharper timcharper changed the title Initial resource-limits support Support resource-limits Apr 22, 2020
@timcharper timcharper marked this pull request as ready for review April 22, 2020 19:50
@timcharper timcharper changed the title Support resource-limits Support for Mesos resource-limits Apr 22, 2020
kaiwalyajoshi
kaiwalyajoshi previously approved these changes Oct 26, 2020
@kaiwalyajoshi
Copy link
Contributor

This was merged in #3320 and scale tested in MWT-26. SDK 0.58.0 rc1-4 has this feature.
Closing this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants