Skip to content

[Feature-13354][deploy] Add configurable busybox image in values.yaml… #13560

New issue

Have a question about this project? No Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “No 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? No Sign in to your account

Merged
merged 1 commit into from
Feb 14, 2023

Conversation

qindongliang
Copy link
Contributor

@qindongliang qindongliang commented Feb 14, 2023

this pr closes: #13554

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@SbloodyS SbloodyS added first time contributor First-time contributor backend 3.2.0 for 3.2.0 version labels Feb 14, 2023
@SbloodyS SbloodyS added this to the 3.2.0 milestone Feb 14, 2023
@SbloodyS SbloodyS added the feature new feature label Feb 14, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 14, 2023

Codecov Report

Merging #13560 (50f90bc) into dev (2bd65fb) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head 50f90bc differs from pull request most recent head 8e26771. Consider uploading reports for the commit 8e26771 to get more accurate results

@@             Coverage Diff              @@
##                dev   #13560      +/-   ##
============================================
- Coverage     39.47%   39.45%   -0.03%     
+ Complexity     4382     4377       -5     
============================================
  Files          1102     1102              
  Lines         41437    41437              
  Branches       4733     4736       +3     
============================================
- Hits          16359    16348      -11     
- Misses        23262    23272      +10     
- Partials       1816     1817       +1     
Impacted Files Coverage Δ
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 43.54% <0.00%> (-7.26%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 52.08% <0.00%> (-0.70%) ⬇️
...pache/dolphinscheduler/common/utils/DateUtils.java 68.30% <0.00%> (-0.55%) ⬇️

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

Copy link
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

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

Thanks

@sonarqubecloud
Copy link

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@kezhenxu94 kezhenxu94 merged commit 6a0f48e into apache:dev Feb 14, 2023
@qindongliang
Copy link
Contributor Author

Thanks for such an efficient merge

No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Labels
3.2.0 for 3.2.0 version backend feature new feature first time contributor First-time contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature][deploy] The configurable busybox image address is in values.yaml
4 participants