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

feat: Logs should favor containers over init containers (#4345) #5322

Merged
merged 1 commit into from Jan 26, 2021

Conversation

alexmt
Copy link
Collaborator

@alexmt alexmt commented Jan 26, 2021

Signed-off-by: Alexander Matyushentsev AMatyushentsev@gmail.com

Closes #4345

Pod logs component shows init containers after normal containers.

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@alexmt alexmt requested a review from rbreeze January 26, 2021 22:43
@codecov-io
Copy link

Codecov Report

Merging #5322 (9d2ac9b) into master (7165b45) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5322   +/-   ##
=======================================
  Coverage   41.06%   41.06%           
=======================================
  Files         136      136           
  Lines       18384    18384           
=======================================
  Hits         7549     7549           
  Misses       9755     9755           
  Partials     1080     1080           

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 7165b45...9d2ac9b. Read the comment docs.

Copy link
Member

@rbreeze rbreeze left a comment

Choose a reason for hiding this comment

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

LGTM. Glad to see the Observable Argo UI fix going through too

@alexmt alexmt merged commit 00f53b5 into argoproj:master Jan 26, 2021
@alexmt alexmt deleted the pod-logs-viewer-container branch January 26, 2021 23:26
shubhamagarwal19 pushed a commit to shubhamagarwal19/argo-cd that referenced this pull request Apr 15, 2021
… (argoproj#5322)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
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.

Logs should favor containers over init containers
3 participants