-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes the builder steps to use the new ncache.Cache. #1047
Conversation
...src/integration-test/java/com/google/cloud/tools/jib/registry/BlobPullerIntegrationTest.java
Show resolved
Hide resolved
...e/src/main/java/com/google/cloud/tools/jib/builder/steps/PushContainerConfigurationStep.java
Show resolved
Hide resolved
jib-core/src/main/java/com/google/cloud/tools/jib/configuration/BuildConfiguration.java
Show resolved
Hide resolved
try (InputStreamReader stderr = | ||
new InputStreamReader(dockerProcess.getErrorStream(), StandardCharsets.UTF_8)) { | ||
throw new IOException( | ||
"'docker load' command failed with error: " + CharStreams.toString(stderr)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you fixing #1013? Perhaps do this in another PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah right, I added this to debug something. I'll remove this from this PR.
Image<Layer> testImage = | ||
Image.builder() | ||
.addLayer( | ||
new Layer() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you prefer using a mock, you could do .addLayer(mockLayer)
and when(mockLayer.getBlob()).thenReturn(Blobs.from(fileA))
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll change to that to make the code less boiler-platey.
jib-core/src/main/java/com/google/cloud/tools/jib/builder/steps/BuildImageStep.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing seems suspicious.
Part of #637
ncache.Cache
.cache
package. (Had to get rid of it in this one since some of the changes caused compilation errors in the oldcache
package)ncache
tocache
.