Skip to content

agent: Use bucket key constants#2279

Merged
aaronlehmann merged 1 commit intomoby:masterfrom
aaronlehmann:bucket-key-constants
Jun 22, 2017
Merged

agent: Use bucket key constants#2279
aaronlehmann merged 1 commit intomoby:masterfrom
aaronlehmann:bucket-key-constants

Conversation

@aaronlehmann
Copy link
Copy Markdown
Collaborator

Instead of repeating the literal key, use the constants that are already defined.

Instead of repeating the literal key, use the constants that are already
defined.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 21, 2017

Codecov Report

Merging #2279 into master will increase coverage by 0.16%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2279      +/-   ##
==========================================
+ Coverage   60.35%   60.51%   +0.16%     
==========================================
  Files         125      125              
  Lines       20394    20394              
==========================================
+ Hits        12308    12342      +34     
+ Misses       6701     6657      -44     
- Partials     1385     1395      +10

@cyli
Copy link
Copy Markdown
Contributor

cyli commented Jun 21, 2017

LGTM

@aaronlehmann aaronlehmann merged commit 3191998 into moby:master Jun 22, 2017
@aaronlehmann aaronlehmann deleted the bucket-key-constants branch June 22, 2017 00:18
@stevvooe
Copy link
Copy Markdown
Contributor

LGTM

andrewhsu pushed a commit to docker-archive/docker-ce that referenced this pull request Jul 14, 2017
- moby/swarmkit#2266 (support for templating Node.Hostname in docker executor)
- moby/swarmkit#2281 (change restore action on objects to be update, not delete/create)
- moby/swarmkit#2285 (extend watch queue with timeout and size limit)
- moby/swarmkit#2253 (version-aware failure tracking in the scheduler)
- moby/swarmkit#2275 (update containerd and port executor to container client library)
- moby/swarmkit#2292 (rename some generic resources)
- moby/swarmkit#2300 (limit the size of the external CA response)
- moby/swarmkit#2301 (delete global tasks when the node running them is deleted)

Minor cleanups, dependency bumps, and vendoring:
- moby/swarmkit#2271
- moby/swarmkit#2279
- moby/swarmkit#2283
- moby/swarmkit#2282
- moby/swarmkit#2274
- moby/swarmkit#2296 (dependency bump of etcd, go-winio)

Signed-off-by: Ying Li <ying.li@docker.com>
Upstream-commit: 4509a00
Component: engine
silvin-lubecki pushed a commit to silvin-lubecki/engine-extract that referenced this pull request Mar 16, 2020
- moby/swarmkit#2266 (support for templating Node.Hostname in docker executor)
- moby/swarmkit#2281 (change restore action on objects to be update, not delete/create)
- moby/swarmkit#2285 (extend watch queue with timeout and size limit)
- moby/swarmkit#2253 (version-aware failure tracking in the scheduler)
- moby/swarmkit#2275 (update containerd and port executor to container client library)
- moby/swarmkit#2292 (rename some generic resources)
- moby/swarmkit#2300 (limit the size of the external CA response)
- moby/swarmkit#2301 (delete global tasks when the node running them is deleted)

Minor cleanups, dependency bumps, and vendoring:
- moby/swarmkit#2271
- moby/swarmkit#2279
- moby/swarmkit#2283
- moby/swarmkit#2282
- moby/swarmkit#2274
- moby/swarmkit#2296 (dependency bump of etcd, go-winio)

Signed-off-by: Ying Li <ying.li@docker.com>
Upstream-commit: 4509a00
Component: engine
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants