delta/docker.git/container, branch master github.com: dotcloud/docker.git container: split security options to a SecurityOptions struct 2023-04-28T22:03:37+00:00 Sebastiaan van Stijn github@gone.nl 2023-04-14T07:27:20+00:00 3eebf4d1622a0d8fc60f30694ee99d2130db1f4b - Split these options to a separate struct, so that we can handle them in isolation. - Change some tests to use subtests, and improve coverage Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
- Split these options to a separate struct, so that we can handle them in isolation.
- Change some tests to use subtests, and improve coverage

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Resolve and store manifest when creating container 2023-03-06T14:13:36+00:00 Laura Brehm laurabrehm@hey.com 2023-02-25T17:18:05+00:00 a34060cdb42f6c4b1452cec27b715fa4c0a13e4f This addresses the previous issue with the containerd store where, after a container is created, we can't deterministically resolve which image variant was used to run it (since we also don't store what platform the image was fetched for). This is required for things like `docker commit`, and computing the containers layer size later, since we need to resolve the specific image variant. Signed-off-by: Laura Brehm <laurabrehm@hey.com> 
This addresses the previous issue with the containerd store where, after a container is created, we can't deterministically resolve which image variant was used to run it (since we also don't store what platform the image was fetched for).

This is required for things like `docker commit`, and computing the containers layer size later, since we need to resolve the specific image variant.

Signed-off-by: Laura Brehm <laurabrehm@hey.com>
Merge pull request #42264 from thaJeztah/update_the_update 2023-01-10T10:40:36+00:00 Sebastiaan van Stijn thaJeztah@users.noreply.github.com 2023-01-10T10:40:36+00:00 d73aba2500a04d78939ea81669c0740152f9fecb restartmanager: Remove RestartManager interface, and unused error return 
restartmanager:  Remove RestartManager interface, and unused error return
 Merge pull request #43787 from thaJeztah/memdb_nits 2023-01-06T16:09:06+00:00 Brian Goff cpuguy83@gmail.com 2023-01-06T16:09:06+00:00 73e09ddecf03477c690b2016a613b06156b54969 container: ViewDB: cleanup error-types 
container: ViewDB: cleanup error-types
 restartmanager: remove RestartManager interface 2022-12-28T08:36:58+00:00 Sebastiaan van Stijn github@gone.nl 2022-02-10T16:27:38+00:00 c5d4b6b311645d23ebd5e0bd173708d5dd4a6c85 It only had a single implementation, so we may as well remove the added complexity of defining it as an interface. Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
It only had a single implementation, so we may as well remove the added
complexity of defining it as an interface.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
restartmanager: add SetPolicy() to the RestartManager interface 2022-12-28T08:36:58+00:00 Sebastiaan van Stijn github@gone.nl 2021-04-07T11:40:15+00:00 efb97da0dacb630bc91e12be0f26cbc21528990c Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
pkg/ioutils: TempDir: move to pkg/longpath 2022-12-20T22:24:12+00:00 Sebastiaan van Stijn github@gone.nl 2022-11-23T16:28:13+00:00 c63ea32a1780bfd063c2a6bc5dcaefe4026f27e9 This utility wasn't very related to all other utilities in pkg/ioutils. Moving it to longpath to also make it more clear what it does. It looks like there's only a single (public) external consumer of this utility, and only used in a test, and it's not 100% clear if it was intentional to use our package, of if it was a case of "I actually meant `io/ioutil.MkdirTemp`" so we could consider skipping the alias. While moving the package, I also renamed `TempDir` to `MkdirTemp`, which is the signature it matches in "os" from stdlib. Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
This utility wasn't very related to all other utilities in pkg/ioutils.
Moving it to longpath to also make it more clear what it does.

It looks like there's only a single (public) external consumer of this
utility, and only used in a test, and it's not 100% clear if it was
intentional to use our package, of if it was a case of "I actually meant
`io/ioutil.MkdirTemp`" so we could consider skipping the alias.

While moving the package, I also renamed `TempDir` to `MkdirTemp`, which
is the signature it matches in "os" from stdlib.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
container: ViewDB: use logrus.WithError() 2022-12-08T13:36:22+00:00 Sebastiaan van Stijn github@gone.nl 2022-12-08T13:35:13+00:00 28382c58ec2078d17ae3a3d6a2f262733a1e779d Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
container: ViewDB: return typed system errors 2022-12-08T13:33:57+00:00 Sebastiaan van Stijn github@gone.nl 2022-07-08T11:29:56+00:00 6549a270e962dba0e24033367ec328b586444c92 Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
container: ViewDB: GetByPrefix() return typed errors 2022-12-08T13:33:57+00:00 Sebastiaan van Stijn github@gone.nl 2022-07-08T11:23:01+00:00 94dea2018e6e9b4836af610b68293f81bb5adb62 Signed-off-by: Sebastiaan van Stijn <github@gone.nl> 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>