Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove hidden plugin command #833

Merged
merged 1 commit into from
Aug 12, 2019

Conversation

zubron
Copy link
Contributor

@zubron zubron commented Aug 12, 2019

What this PR does / why we need it:
This change removes the hidden plugin command. This command was
intended for debugging the generation of plugin Job and DaemonSet
manifests and is not in active use. There is currently other work in
progress to remove the use of templates when creating plugin resources
which means that a command to render the template is not required. If we
decided to have a debug command there would be further work required to
ensure that the same flow is used as is used in the plugin aggreator.

Signed-off-by: Bridget McErlean [email protected]

Release note:

NONE

This change removes the hidden `plugin` command. This command was
intended for debugging the generation of plugin Job and DaemonSet
manifests and is not in active use. There is currently other work in
progress to remove the use of templates when creating plugin resources
which means that a command to render the template is not required. If we
decided to have a debug command there would be further work required to
ensure that the same flow is used as is used in the plugin aggreator.

Signed-off-by: Bridget McErlean <[email protected]>
@zubron zubron changed the title Remove hiddenplugin command Remove hidden plugin command Aug 12, 2019
@zubron zubron requested a review from johnSchnake August 12, 2019 18:55
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #833      +/-   ##
==========================================
+ Coverage   44.55%   44.73%   +0.18%     
==========================================
  Files          76       75       -1     
  Lines        4624     4565      -59     
==========================================
- Hits         2060     2042      -18     
+ Misses       2431     2390      -41     
  Partials      133      133
Impacted Files Coverage Δ
cmd/sonobuoy/app/root.go 92.85% <ø> (-0.48%) ⬇️

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 838ea5a...4d26c09. Read the comment docs.

@johnSchnake johnSchnake merged commit f68cb44 into vmware-tanzu:master Aug 12, 2019
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