-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Implement cache from #4310
Implement cache from #4310
Conversation
Signed-off-by: Thomas Schaaf <[email protected]>
@@ -594,6 +594,20 @@ def test_build_with_build_args(self): | |||
service.build() | |||
assert service.image() | |||
|
|||
@v2_only() |
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.
Is this correct?
@@ -56,9 +56,10 @@ | |||
{ | |||
"type": "object", | |||
"properties": { | |||
"args": {"$ref": "#/definitions/list_or_dict"}, | |||
"cachefrom": {"$ref": "#/definitions/list_of_strings"}, |
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.
Am I allowed to add it to the 2.0 schema?
Since the docker-py issue was merged in, this can be retested and reviewed, no? |
3ce31aa
to
ed3a331
Compare
Signed-off-by: Thomas Schaaf <[email protected]>
1e3a41b
to
811c1f1
Compare
Closing via #4514 - Thanks! |
Fixes: #4309
Depends on: docker/docker-py#1383
Signed-off-by: Thomas Schaaf [email protected]