Add builder dev report for 2017-07-10 #34089
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AkihiroSuda
reviewed
Jul 13, 2017
reports/builder/2017-07-10.md
Outdated
|
|
||
| The build definition solver was updated to detect the identical parts of the graph sent by different clients and synchronize their processing. This is important when multiple targets of the same project are built on the same time and removes any duplication of work. | ||
|
|
||
| Running build jobs now has support for graceful canceling and clear error reporting in case some build steps fail or are canceled. Bugs that may have left state dir in an inconsistent state of server shutdown were fixes. |
|
|
||
| The first version of instruction cache was implemented. This caching scheme has many benefits compared to the current `docker build` caching as it doesn't require all data to be locally available to determine the cache match. The interface for the cache implementation is much simpler and could be implemented remotely as it only needs to store the cache keys and doesn't need to understand or compare their values. Content-based caching will be implemented on top of this work later. | ||
|
|
||
| Separate source implementation for git repositories is currently in review. Using this source for accessing source code in git repositories has many performance and caching advantages. All the build jobs using the same git remote will use a shared local repository where updates will be pulled. All the nodes based on a git source will be cached using the commit ID of the current checkout. |
There was a problem hiding this comment.
I wrote this on Monday. Sorry for posting late.
thaJeztah
requested changes
Jul 13, 2017
reports/builder/2017-07-10.md
Outdated
|
|
||
| Many new features have been added since the last report. | ||
|
|
||
| The build definition solver was updated to detect the identical parts of the graph sent by different clients and synchronize their processing. This is important when multiple targets of the same project are built on the same time and removes any duplication of work. |
There was a problem hiding this comment.
s/on the same time/at the same time/
reports/builder/2017-07-10.md
Outdated
|
|
||
| ### Quality: Dependency interface switch | ||
|
|
||
| No updates for this week. Metadata commands need to be updated but it probably easier to do it after https://github.com/moby/moby/pull/33492 has been merged. |
There was a problem hiding this comment.
s/but it probably easier/but it is probably easier/
c36305e
to
747bd0c
Compare
thaJeztah
approved these changes
Jul 13, 2017
cpuguy83
requested changes
Jul 13, 2017
reports/builder/2017-07-10.md
Outdated
|
|
||
| [PR](https://github.com/moby/moby/pull/33492) | ||
|
|
||
| The PR is in code review and waiting for feedback. Hopefully ready to me merged this week. |
Signed-off-by: Tonis Tiigi <[email protected]>
747bd0c
to
2689f9c
Compare
|
I see @cpuguy83's comment was addressed; I'll merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@vieux @thaJeztah
Signed-off-by: Tonis Tiigi [email protected]