Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Add some additional cleanup to rkt containers #1008

Merged

Conversation

dvdthms
Copy link
Contributor

@dvdthms dvdthms commented Nov 14, 2017

Most of the services that use rkt containers had a command to perform some cleanup.

I've added the corresponding command to a service that was missing it, and also added some additional syntax to pre-existing commands in order to keep them consistent.

@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Nov 14, 2017
@codecov-io
Copy link

Codecov Report

Merging #1008 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1008   +/-   ##
=======================================
  Coverage   34.88%   34.88%           
=======================================
  Files          59       59           
  Lines        4133     4133           
=======================================
  Hits         1442     1442           
  Misses       2532     2532           
  Partials      159      159

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6585cb9...27adf32. Read the comment docs.

@mumoshu
Copy link
Contributor

mumoshu commented Nov 20, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 20, 2017
@mumoshu
Copy link
Contributor

mumoshu commented Nov 20, 2017

@dvdthms Thanks for your contribution as always!

@mumoshu mumoshu merged commit 6c0eb03 into kubernetes-retired:master Nov 20, 2017
@mumoshu mumoshu added this to the v0.9.9-rc.3 milestone Nov 20, 2017
@mumoshu mumoshu removed their assignment Nov 22, 2017
kylehodgetts pushed a commit to HotelsDotCom/kube-aws that referenced this pull request Mar 27, 2018
…kt-container-cleanup

Add some additional cleanup to rkt containers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. improvement lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants