Skip to content
Snippets Groups Projects

Add support for Docker '--cpus' option

Merged Tomasz Maczukin requested to merge feature/support-for-docker-cpus-option into master
All threads resolved!

This is a general Merge Request template. Consider to choose a template from the list above if it will match your case more.

What does this MR do?

Adds support for --cpus option for Docker executor. Successor of !469 (closed).

Why was this MR needed?

Please read #2101 (closed) for a reference

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

Closes #2101 (closed)

Edited by Tomasz Maczukin

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • assigned to @tmaczukin

  • Looks good overall @tmaczukin, just a couple of comments.

  • Tomasz Maczukin resolved all discussions

    resolved all discussions

  • Author Maintainer

    @nick.thomas I've updated the MR

  • Nick Thomas
  • assigned to @tmaczukin

  • Just a couple of tiny things now @tmaczukin

  • Tomasz Maczukin resolved all discussions

    resolved all discussions

  • Nick Thomas approved this merge request

    approved this merge request

  • merged

  • mentioned in issue #2491 (closed)

  • Please register or sign in to reply
    Loading