Skip to content

api: deprecate BuildCache.Parent, add BuildCache.Parents in API >= v1.42#43631

Merged
thaJeztah merged 6 commits intomoby:masterfrom
thaJeztah:deprecate_buildcache_parent
Aug 4, 2022
Merged

api: deprecate BuildCache.Parent, add BuildCache.Parents in API >= v1.42#43631
thaJeztah merged 6 commits intomoby:masterfrom
thaJeztah:deprecate_buildcache_parent

Conversation

@thaJeztah
Copy link
Member

First two commits are documenting the fields

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah
Copy link
Member Author

@crazy-max @tonistiigi PTAL

@thaJeztah thaJeztah force-pushed the deprecate_buildcache_parent branch 2 times, most recently from 90dbc2f to 6576d5b Compare May 23, 2022 14:00
@thaJeztah thaJeztah requested a review from tonistiigi as a code owner May 23, 2022 14:00
@thaJeztah
Copy link
Member Author

Having some discussion on Slack; looks like the replacement is a Parents (plural) field, so we may have to add that one as well

@thaJeztah thaJeztah force-pushed the deprecate_buildcache_parent branch from 6576d5b to c62be91 Compare August 2, 2022 07:51
@thaJeztah thaJeztah changed the title api: deprecate BuildCache.Parent in API >= v1.42 api: deprecate BuildCache.Parent, add BuildCache.Parents in API >= v1.42 Aug 2, 2022
@thaJeztah thaJeztah force-pushed the deprecate_buildcache_parent branch from c62be91 to 3010aa4 Compare August 2, 2022 08:23
@thaJeztah
Copy link
Member Author

Having some discussion on Slack; looks like the replacement is a Parents (plural) field, so we may have to add that one as well

I added the Parents field, and updated the documentation

@tianon @tonistiigi @crazy-max PTAL

@crazy-max
Copy link
Member

SGTM thanks!

* The `BuildCache.Parent` field, as returned by `GET /system/df` is deprecated
and is now omitted. API versions before v1.42 continue to include this field.
* `GET /system/df` now includes a new `Parents` field, for "build-cache" records,
which contains a list of parent ID's for the build-cache record.
Copy link
Member

Choose a reason for hiding this comment

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

I believe it is IDs

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I think you're right; must've been in "dutch" mode while I wrote this 🇳🇱 😁

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed 👍

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This field has been deprecated in BuildKit, so this follows the deprecation
in the Engine API.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This field was added to replace the deprecated "Parent" field.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

Failure on Windwos 2022 is unrelated, but let me post this as I've seen this error pop-up a few times (could be a platform issue)

=== FAIL: github.com/docker/docker/integration/container TestWaitNonBlocked/wait-nonblocking-exit-0 (1.49s)
    wait_test.go:44: assertion failed: error is not nil: Error response from daemon: hcs::CreateComputeSystem 715ba8aab407c4fc98495e33f58ada3af44a9b2431d9ebd3fd1a60ece43db89d: Element not found.
    --- FAIL: TestWaitNonBlocked/wait-nonblocking-exit-0 (1.49s)

@thaJeztah
Copy link
Member Author

All green now 👍

@thaJeztah thaJeztah merged commit a263241 into moby:master Aug 4, 2022
@thaJeztah thaJeztah deleted the deprecate_buildcache_parent branch August 4, 2022 11:19
@thaJeztah
Copy link
Member Author

@dgiebert opened #43913

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants