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

Fix flag name for systemd-logs image #1066

Merged
merged 1 commit into from
Jan 8, 2020

Conversation

zubron
Copy link
Contributor

@zubron zubron commented Jan 7, 2020

What this PR does / why we need it:
The flag was mistakenly named systemd-logs-flag when it should have
been systemd-logs-image. The existing flag has been left in place but
has been marked as hidden and deprecated. It will continue to work but
will not appear in the help/usage.

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

Release note:

The flag for specifying the image for `systemd-logs` plugin has been renamed from `systemd-logs-flag` to `systemd-logs-image`. The previous flag, `systemd-logs-flag`, is now deprecated and will be removed in a future release.

The flag was mistakenly named `systemd-logs-flag` when it should have
been `systemd-logs-image`. The existing flag has been left in place but
has been marked as hidden and deprecated. It will continue to work but
will not appear in the help/usage.

Signed-off-by: Bridget McErlean <[email protected]>
@zubron zubron requested a review from johnSchnake January 7, 2020 22:33
@codecov-io
Copy link

Codecov Report

Merging #1066 into master will increase coverage by 0.13%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1066      +/-   ##
==========================================
+ Coverage   50.61%   50.75%   +0.13%     
==========================================
  Files          79       79              
  Lines        5745     5751       +6     
==========================================
+ Hits         2908     2919      +11     
+ Misses       2662     2659       -3     
+ Partials      175      173       -2
Impacted Files Coverage Δ
cmd/sonobuoy/app/args.go 91.32% <100%> (+0.27%) ⬆️
pkg/plugin/aggregation/aggregator.go 75.77% <0%> (+3.1%) ⬆️

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 efb9d68...83c13a7. Read the comment docs.

Copy link
Contributor

@johnSchnake johnSchnake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just make a ticket to remove the deprecated flag in v0.18 (seems like plenty of time for this; its a new flag so no one is depending on it yet either)

@zubron zubron merged commit 7d04c2f into vmware-tanzu:master Jan 8, 2020
@zubron zubron deleted the fix-system-logs-image-flag branch January 8, 2020 19:47
zubron added a commit to zubron/sonobuoy that referenced this pull request Mar 30, 2020
In vmware-tanzu#1066, we deprecated this flag with the intention of removing it in
the 0.18.0 release. Given that we are about to release this version,
this flag is now being removed.

Signed-off-by: Bridget McErlean <[email protected]>
zubron added a commit that referenced this pull request Mar 30, 2020
In #1066, we deprecated this flag with the intention of removing it in
the 0.18.0 release. Given that we are about to release this version,
this flag is now being removed.

Signed-off-by: Bridget McErlean <[email protected]>
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