Skip to content

Conversation

@cn-hew
Copy link
Contributor

@cn-hew cn-hew commented Sep 16, 2025

Purpose of the pull request

close #17495

Fix the issue of incorrect mounting paths for Kubernetes 3.3.1 deployment dolphinscheduler-api and statefulset dolphinscheduler-worker resources

Brief change log

Fix the mismatch between the common.properties file and the mount path in the startup script configuration file

Verify this pull request

This pull request is code cleanup without any test coverage.

Pull Request Notice

Pull Request Notice

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

@cn-hew cn-hew requested a review from Gallardot as a code owner September 16, 2025 08:44
@boring-cyborg
Copy link

boring-cyborg bot commented Sep 16, 2025

@cn-hew cn-hew changed the title [Fix-17495-2] Fix configuration file mounting path issue [Fix-17495-2] [Kubernetes]Fix configuration file mounting path issue Sep 16, 2025
@cn-hew cn-hew changed the title [Fix-17495-2] [Kubernetes]Fix configuration file mounting path issue [Fix-17495] [Kubernetes] Fix configuration file mounting path issue Sep 16, 2025
@SbloodyS SbloodyS added this to the 3.3.2 milestone Sep 19, 2025
@SbloodyS SbloodyS added the bug Something isn't working label Sep 19, 2025
@cn-hew
Copy link
Contributor Author

cn-hew commented Sep 19, 2025

@SbloodyS It seems that the download of MySQL by CI has timed out

Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

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

+1

@cn-hew cn-hew requested a review from SbloodyS October 9, 2025 03:28
… deployment dolphinscheduler-api and statefulset dolphinscheduler-worker resources
Copy link
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 9, 2025

@SbloodyS SbloodyS merged commit eea596f into apache:dev Oct 10, 2025
86 of 124 checks passed
@boring-cyborg
Copy link

boring-cyborg bot commented Oct 10, 2025

Awesome work, congrats on your first merged pull request!

@cn-hew cn-hew deleted the fix-17495-2 branch October 11, 2025 02:02
davidzollo pushed a commit to davidzollo/dolphinscheduler that referenced this pull request Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working kubernetes ready-to-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Helm / K8s] 3.3.1 fresh install only works after manually copying storage JARs and changing ConfigMap mount path – is this expected?

3 participants