Skip to content

Conversation

@fuchanghai
Copy link
Member

Purpose of the pull request

plz close : #14447

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2023

Codecov Report

Merging #14448 (a1c4402) into dev (9cbb262) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head a1c4402 differs from pull request most recent head 92032c1. Consider uploading reports for the commit 92032c1 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14448      +/-   ##
============================================
- Coverage     38.52%   38.52%   -0.01%     
+ Complexity     4555     4554       -1     
============================================
  Files          1236     1236              
  Lines         43486    43486              
  Branches       4817     4817              
============================================
- Hits          16755    16751       -4     
- Misses        24875    24878       +3     
- Partials       1856     1857       +1     
Impacted Files Coverage Δ
...lphinscheduler/plugin/task/api/utils/K8sUtils.java 3.50% <0.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@zhongjiajie zhongjiajie requested a review from Radeity July 7, 2023 01:08
@zhongjiajie
Copy link
Member

PTAL @Radeity

@Radeity Radeity added bug Something isn't working 3.1.x for 3.1.x version labels Jul 7, 2023
@Radeity Radeity added this to the 3.1.8 milestone Jul 7, 2023
@Radeity Radeity merged commit 742a2dc into apache:dev Jul 7, 2023
IT-Kwj pushed a commit to IT-Kwj/dolphinscheduler that referenced this pull request Jul 14, 2023
@fuchanghai fuchanghai deleted the fix-#14447 branch July 17, 2023 01:22
zhongjiajie pushed a commit that referenced this pull request Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.1.x for 3.1.x version backend bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [K8S-Task] The judgment in k8sUtils#getPodLog is not accurate

5 participants