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(ui): Correct pod log viewer to support short log lines (#14402) #14543

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

terrytangyuan
Copy link
Member

Fixes #14402

…j#14402

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
@crenshaw-dev
Copy link
Collaborator

@terrytangyuan how far back do we need to cherry-pick this? 2.7 I think?

@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.01 ⚠️

Comparison is base (4bd8b07) 49.79% compared to head (58456e4) 49.79%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #14543      +/-   ##
==========================================
- Coverage   49.79%   49.79%   -0.01%     
==========================================
  Files         261      261              
  Lines       44751    44755       +4     
==========================================
+ Hits        22283    22284       +1     
- Misses      20274    20276       +2     
- Partials     2194     2195       +1     
Impacted Files Coverage Δ
...cationset/controllers/applicationset_controller.go 62.64% <100.00%> (+0.10%) ⬆️
util/argo/argo.go 66.49% <100.00%> (+0.04%) ⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@terrytangyuan
Copy link
Member Author

terrytangyuan commented Jul 17, 2023

v2.7 seems reasonable (I am using v2.7.6). Could you also help test the change locally to make sure nothing else breaks?

@terrytangyuan terrytangyuan changed the title fix: Correct pod log viewer to support short log lines. Fixes #14402 fix(ui): Correct pod log viewer to support short log lines. Fixes #14402 Jul 17, 2023
@crenshaw-dev crenshaw-dev changed the title fix(ui): Correct pod log viewer to support short log lines. Fixes #14402 fix(ui): Correct pod log viewer to support short log lines (#14402) Jul 17, 2023
@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.8

@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.7

@crenshaw-dev
Copy link
Collaborator

@rbreeze have you had a chance to test locally? If so, I'll smash merge.

@terrytangyuan terrytangyuan marked this pull request as draft July 17, 2023 20:04
@terrytangyuan
Copy link
Member Author

terrytangyuan commented Jul 17, 2023

Let me perform more testing before merging this since this will be cherry-picked.

@crenshaw-dev
Copy link
Collaborator

I was about to test, but kind is misbehaving. :-P

@terrytangyuan terrytangyuan marked this pull request as ready for review July 18, 2023 01:18
@terrytangyuan
Copy link
Member Author

@crenshaw-dev Verified that this looks good locally.

@crenshaw-dev crenshaw-dev merged commit 462dffc into argoproj:master Jul 18, 2023
26 checks passed
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 18, 2023
…14543)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 18, 2023
…14543)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
@terrytangyuan terrytangyuan deleted the fix-pod-viewer-width branch July 18, 2023 01:58
crenshaw-dev pushed a commit that referenced this pull request Jul 18, 2023
…14543) (#14561)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
Co-authored-by: Yuan Tang <terrytangyuan@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 18, 2023
…14543) (#14562)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
Co-authored-by: Yuan Tang <terrytangyuan@gmail.com>
Jneville0815 pushed a commit to radiusmethod/argo-cd that referenced this pull request Jul 18, 2023
…j#14402 (argoproj#14543)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
Signed-off-by: Jimmy Neville <jimmyeneville@gmail.com>
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…j#14402 (argoproj#14543) (argoproj#14562)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
Co-authored-by: Yuan Tang <terrytangyuan@gmail.com>
Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
yyzxw pushed a commit to yyzxw/argo-cd that referenced this pull request Aug 9, 2023
@suzaku suzaku mentioned this pull request Aug 15, 2023
tesla59 pushed a commit to tesla59/argo-cd that referenced this pull request Dec 16, 2023
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.

Pod logs are displayed as line numbers
3 participants