Skip to content

Conversation

@thaJeztah
Copy link
Member

This code was using swarmkit's genericresource package as intermediate; add a local copy of that code that skips the protobufs as intermediate.

- Human readable description for the release notes

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

@thaJeztah thaJeztah added this to the 29.0.0 milestone Nov 5, 2025
@thaJeztah thaJeztah added status/2-code-review area/swarm kind/refactor PR's that refactor, or clean-up code labels Nov 5, 2025
@codecov-commenter
Copy link

codecov-commenter commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 88.88889% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...mmand/service/internal/genericresource/validate.go 64.28% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah force-pushed the local_parsegeneric_resource branch from fffe0e4 to 8f14bc8 Compare November 5, 2025 10:54
@thaJeztah thaJeztah changed the title cli/command/service: parse generic resources without proto-bufs cli/command/service: parse generic resources without protobufs Nov 5, 2025
This code was using swarmkit's genericresource package as intermediate;
add a local copy of that code that skips the protobufs as intermediate.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the local_parsegeneric_resource branch from 8f14bc8 to 774f1d6 Compare November 5, 2025 11:26
@thaJeztah thaJeztah merged commit 050aca8 into docker:master Nov 5, 2025
106 of 107 checks passed
@thaJeztah thaJeztah deleted the local_parsegeneric_resource branch November 5, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/swarm kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants