Skip to content

Backport(v1.16): winsvc: Fix bug where service accidentally stops after starting (#4954) #4955

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

Merged
merged 1 commit into from
May 14, 2025

Conversation

daipom
Copy link
Contributor

@daipom daipom commented May 14, 2025

Which issue(s) this PR fixes:
Backport #4954.

What this PR does / why we need it:
After 53bcd3c (#4909), the service accidentally stops after starting, without stopping the supervisor and workers.

Docs Changes:
Not needed.

Release Note:
The same as the title.

@daipom daipom added this to the v1.16.9 milestone May 14, 2025
@daipom daipom force-pushed the backport-pr4954 branch from db303ca to 1d24a46 Compare May 14, 2025 06:42
@daipom
Copy link
Contributor Author

daipom commented May 14, 2025

Currently, Ruby on Windows dose not work correctly...

@daipom daipom requested review from kenhys, ashie and Watson1978 May 14, 2025 07:06
**Which issue(s) this PR fixes**:
This fixes a bug of #4909.

**What this PR does / why we need it**:
After 53bcd3c (#4909), the service
accidentally stops after starting, without stopping the supervisor and
workers.

**Docs Changes**:
Not needed.

**Release Note**:
The same as the title.

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Co-authored-by: Takuro Ashie <ashie@clear-code.com>
Co-authored-by: Kentaro Hayashi <hayashi@clear-code.com>
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
@daipom daipom force-pushed the backport-pr4954 branch from 1d24a46 to 8c67b3c Compare May 14, 2025 07:20
@daipom daipom merged commit 51b7da0 into v1.16 May 14, 2025
13 of 24 checks passed
@daipom daipom deleted the backport-pr4954 branch May 14, 2025 07:28
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.

2 participants