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.

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

[Bug]: bal run --watch does not terminate when there is a panic, or when the entire service is removed from the source. #43309

Open
gayaldassanayake opened this issue Aug 22, 2024 · 0 comments
Labels
Area/CLI-BuildTools needTriage The issue has to be inspected and labeled manually Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) Type/Bug userCategory/Compilation

Comments

@gayaldassanayake
Copy link
Contributor

Description

bal run --watch watches service projects and reruns the services based on the changes to the source files. However, it was observed that

  • when there is a panic from the service,
  • when the service code (the listener) is entirely removed,
    the program doesn't exist. This is not desirable. The panic leaves the service in a hanging state which is not responsive.

This worked fine before adding the debouncing logic. Might need to checkout to a commit before that and see what went wrong.

Steps to Reproduce

  • Case 1
    • Do bal run --watch on a service. Then once the service is started, remove the listener code from the source file.
  • Case 2
    • Add a panic in an endpoint of the service. Do bal run --watch and once the service is up, hit that endpoint.

Affected Version(s)

2201.10.0

OS, DB, other environment details and versions

No response

Related area

-> Compilation

Related issue(s) (optional)

No response

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response

@gayaldassanayake gayaldassanayake added Type/Bug Area/CLI-BuildTools Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) labels Aug 22, 2024
@ballerina-bot ballerina-bot added needTriage The issue has to be inspected and labeled manually userCategory/Compilation labels Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/CLI-BuildTools needTriage The issue has to be inspected and labeled manually Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) Type/Bug userCategory/Compilation
Projects
None yet
Development

No branches or pull requests

2 participants