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.

Sign up for GitHub

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

Jump to bottom

[Bug] [deploy on k8s] add default config of quartz and zookeeper in common configmap && pick #7562 fix to worker #7639 #7641

Merged
merged 15 commits into from
Dec 28, 2021
Merged

[Bug] [deploy on k8s] add default config of quartz and zookeeper in common configmap && pick #7562 fix to worker #7639 #7641

merged 15 commits into from
Dec 28, 2021

Conversation

zwZjut
Copy link
Contributor

@zwZjut zwZjut commented Dec 27, 2021

Purpose of the pull request

#7639

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:

zwZjut and others added 15 commits December 13, 2021 10:24
@zwZjut
[Feature][dolphinscheduler-api] parse traceId in http header for Cros…
2f26f5f 
…s system delivery to #7237 (#7238)

* to #7237

* rerun test

Co-authored-by: honghuo.zw <[email protected]>
@zwZjut
chery-pick 05aef27 and handle conflicts
4a90027
@zwZjut
to #7065: fix ExecutorService and schedulerService (#7072)
9c45a4f 
Co-authored-by: honghuo.zw <[email protected]>
@zwZjut
[Feature][dolphinscheduler-api] access control of taskDefinition and …
6897517 
…taskInstance in project to #7081  (#7082)

* to #7081

* fix #7081

* to #7081

Co-authored-by: honghuo.zw <[email protected]>
@zwZjut
chery-pick 8ebe060 and handle conflicts
6f491e4
@zwZjut
cherry-pick 1f18444 and handle conflicts
1bcde64
@zwZjut @CalvinKirs
fix #6807: dolphinscheduler.zookeeper.env_vars - > dolphinscheduler.r…
9cddad9 
…egistry.env_vars (#6808)

Co-authored-by: honghuo.zw <[email protected]>
Co-authored-by: Kirs <[email protected]>
@zwZjut
add default constructor (#6780)
70ec96a 
Co-authored-by: honghuo.zw <[email protected]>
@zwZjut
to #7108 (#7109)
c79acd1
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
6c29e89 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
3367f56 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
9737335 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
8842fe6 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
ada2a71 
…duler into 2.0.2-prepare
add conf && pick #7562 to worker
b10ecd9
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (2.0.2-prepare@744577f). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##             2.0.2-prepare    #7641   +/-   ##
================================================
  Coverage                 ?   31.30%           
  Complexity               ?     1554           
================================================
  Files                    ?      433           
  Lines                    ?    14769           
  Branches                 ?     1476           
================================================
  Hits                     ?     4623           
  Misses                   ?     9694           
  Partials                 ?      452

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 744577f...b10ecd9. Read the comment docs.

@sonarcloud
Copy link

sonarcloud bot commented Dec 27, 2021

SonarCloud Quality Gate failed.    Quality Gate failed

Bug E 24 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 319 Code Smells

16.3% 16.3% Coverage
0.9% 0.9% Duplication

caishunfeng
caishunfeng approved these changes Dec 28, 2021
View reviewed changes
Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

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

+1

@caishunfeng caishunfeng merged commit df40cdc into apache:2.0.2-prepare Dec 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@caishunfeng caishunfeng caishunfeng approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zwZjut @codecov-commenter @caishunfeng