Skip to content

Migrate timeout configuration from server to debug settings#54

Merged
fsul7o merged 4 commits intomasterfrom
fix-timeout
May 21, 2025
Merged

Migrate timeout configuration from server to debug settings#54
fsul7o merged 4 commits intomasterfrom
fix-timeout

Conversation

@fsul7o
Copy link
Copy Markdown
Contributor

@fsul7o fsul7o commented May 19, 2025

Description

Since the timeout setting value was only applied to the debug server, I moved the timeout setting value from the server to the debug.

before:

server:
  timeout: 10s
  debug:
    ...

after:

server:
  debug:
    timeout: 10s

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes, no api changes)
  • Non-code changes (update documentation, pipeline, etc.)

Flags

  • Breaks backward compatibility
  • Requires a documentation update
  • Has untestable code

Related issue/PR

Delete this section if there are no issues or pull requests that relate to this pull request.

  • Fixes #issue
  • Closes #PR

Checklist

  • Followed the guidelines in the CONTRIBUTING document
  • Added prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip] in the PR title if necessary
  • Tested and linted the code
  • Commented the code
  • Made corresponding changes to the documentation
  • Passed all pipeline checking

Checklist for maintainer

  • Use Squash and merge
  • Double-confirm the merge message has prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip]
  • Delete the branch after merge

Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
@fsul7o fsul7o changed the title Timeout configuration from server to debug settings Migrate timeout configuration from server to debug settings May 19, 2025
@fsul7o fsul7o marked this pull request as ready for review May 19, 2025 02:06
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
fsul7o and others added 2 commits May 21, 2025 10:17
Co-authored-by: y-myajima <137997379+y-myajima@users.noreply.github.com>
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
@fsul7o fsul7o merged commit fb39bf8 into master May 21, 2025
2 checks passed
@fsul7o fsul7o deleted the fix-timeout branch May 21, 2025 04:36
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