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

[v0.16.0] Default Admission Controllers #1857

Merged
merged 2 commits into from May 27, 2020

Conversation

dominicgunn
Copy link
Contributor

Changes

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 26, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign dominicgunn
You can assign the PR to them by writing /assign @dominicgunn in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 26, 2020
@dominicgunn dominicgunn added this to the v0.16.0 milestone May 26, 2020
@dominicgunn dominicgunn changed the title [v0.16.x] Default Admission Controllers [v0.16.0] Default Admission Controllers May 26, 2020
@codecov-commenter
Copy link

codecov-commenter commented May 26, 2020

Codecov Report

Merging #1857 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1857      +/-   ##
==========================================
+ Coverage   24.40%   24.42%   +0.01%     
==========================================
  Files          98       98              
  Lines        5117     5114       -3     
==========================================
  Hits         1249     1249              
+ Misses       3728     3725       -3     
  Partials      140      140              
Impacted Files Coverage Δ
pkg/api/cluster.go 0.00% <ø> (ø)
pkg/api/types.go 0.00% <ø> (ø)

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 bdc0325...9d5048a. Read the comment docs.

@dominicgunn
Copy link
Contributor Author

/lgtm

@k8s-ci-robot
Copy link
Contributor

@dominicgunn: you cannot LGTM your own PR.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@dominicgunn dominicgunn merged commit a773ae3 into master May 27, 2020
@mumoshu mumoshu deleted the feature/v0.16.0-default-admission-controllers branch May 28, 2020 11:11
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. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants