Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Standalone overhaul: Implements a set of standard 2 column width emojis for logging #2420

Conversation

jpmcb
Copy link
Contributor

@jpmcb jpmcb commented Nov 1, 2021

All changes related to the standalone cluster overhaul

What this PR does / why we need it

  • Implements a set of standard 2 column width emojis that can be used with the Event and Eventf logger methods.
  • Adds breaks to each "event" as it's logged
  • Removes logging for the list operation (which is more in line with what kubectl get pods might do)

Describe testing done for PR

Take a look!

Screen Shot 2021-11-01 at 4 25 06 PM

@jpmcb jpmcb requested a review from a team as a code owner November 1, 2021 21:47
@jpmcb jpmcb marked this pull request as draft November 1, 2021 21:54
@jpmcb jpmcb force-pushed the standalone-hide-completions branch 2 times, most recently from 92f91d4 to 28c8d33 Compare November 1, 2021 22:20
Signed-off-by: John McBride <jmcbride@vmware.com>
@jpmcb jpmcb force-pushed the standalone-hide-completions branch from 28c8d33 to c1e057c Compare November 1, 2021 22:25
@jpmcb jpmcb marked this pull request as ready for review November 1, 2021 22:26
@joshrosso
Copy link
Contributor

Tried it locally, looks super good.

@joshrosso joshrosso merged commit 4fcf4b1 into vmware-tanzu:standalone-overhaul Nov 2, 2021
joshrosso pushed a commit that referenced this pull request Jan 18, 2022
Signed-off-by: John McBride <jmcbride@vmware.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants