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

DockerCPUConfig Allow multiple work_dirs #4413

Merged
merged 3 commits into from
Jul 4, 2019
Merged

Conversation

maaktweluit
Copy link
Contributor

  • Renamed DockerCpuConfig.work_dir to work_dirs
  • handling it as a list
    • check if the work_dirs are unique
    • check if the work_dirs are not parent / child
  • Updated docker_machine hypervisor to handle multiple workdirs

This change is required to allow provider and requestor side of the environment to use seperate work_dirs

@maaktweluit maaktweluit requested a review from Wiezzel as a code owner July 2, 2019 13:16
@maaktweluit maaktweluit self-assigned this Jul 2, 2019
@maaktweluit maaktweluit changed the title DockerCPUConfig Allow multiple work_dirs [WIP] DockerCPUConfig Allow multiple work_dirs Jul 2, 2019
@maaktweluit maaktweluit force-pushed the mwu/list-work-dirs branch 2 times, most recently from dd611e7 to 48248c0 Compare July 2, 2019 15:23
Copy link
Contributor

@mfranciszkiewicz mfranciszkiewicz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving for a coverage check

@codecov
Copy link

codecov bot commented Jul 2, 2019

Codecov Report

Merging #4413 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #4413      +/-   ##
===========================================
- Coverage    88.42%   88.42%   -0.01%     
===========================================
  Files          224      224              
  Lines        19841    19862      +21     
===========================================
+ Hits         17544    17562      +18     
- Misses        2297     2300       +3

@maaktweluit maaktweluit force-pushed the mwu/list-work-dirs branch 2 times, most recently from d1c5d31 to 6339ef5 Compare July 2, 2019 17:20
@maaktweluit maaktweluit dismissed mfranciszkiewicz’s stale review July 2, 2019 17:35

coverage check completed

@maaktweluit maaktweluit changed the title [WIP] DockerCPUConfig Allow multiple work_dirs DockerCPUConfig Allow multiple work_dirs Jul 3, 2019
@maaktweluit
Copy link
Contributor Author

@Wiezzel @mfranciszkiewicz ready for review!

Copy link

@Wiezzel Wiezzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's that tests/key_reuse.py thing? Should it be in this PR?

@maaktweluit maaktweluit force-pushed the mwu/list-work-dirs branch from 9f1657a to 785ba62 Compare July 3, 2019 13:28
@maaktweluit
Copy link
Contributor Author

@Wiezzel thanks for the review, applied all the comments and rebased to clean the branch

Copy link

@Wiezzel Wiezzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@maaktweluit maaktweluit force-pushed the mwu/list-work-dirs branch 2 times, most recently from e3fc03e to c2ea07b Compare July 4, 2019 10:09
Co-Authored-By: Adam Wierzbicki <awierzbicki@golem.network>
@maaktweluit maaktweluit force-pushed the mwu/list-work-dirs branch from c2ea07b to 9f066c7 Compare July 4, 2019 10:52
@maaktweluit maaktweluit merged commit 1f281b5 into develop Jul 4, 2019
@maaktweluit maaktweluit deleted the mwu/list-work-dirs branch July 4, 2019 12:50
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants