Skip to content

Bug: using the default processor middleware in the processor template causes an error #111

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

Closed
maha-hajja opened this issue Jan 23, 2025 · 1 comment · Fixed by ConduitIO/conduit-processor-template#22
Assignees
Labels
bug Something isn't working

Comments

@maha-hajja
Copy link
Contributor

Bug description

if you write your processor using the processor template, it uses sdk.ProcessorWithMiddleware(&Processor{}, sdk.DefaultProcessorMiddleware()...) in the NewProcessor method.
after building the processor using make build and adding it to a processors directory, and running conduit, you get this:

2025-01-23T13:10:50+00:00 WRN processor teardown failed error="processor plugin stopped while trying to send command *processorv1.CommandRequest_Teardown: plugin is not running\nmodule closed with exit_code(2)" component=plugin.processor.standalone.Registry stack=null
2025-01-23T13:10:50+00:00 WRN could not load standalone processor plugin error="processor plugin stopped while waiting for response to command *processorv1.CommandRequest_Specify: plugin is not running" component=plugin.processor.standalone.Registry plugin_path=/Users/mahahajja/repos/conduit/processors/conduit-processor-hl7.wasm stack=[{"file":"/Users/mahahajja/repos/conduit/pkg/plugin/processor/standalone/processor.go","func":"github.com/conduitio/conduit/pkg/plugin/processor/standalone.(*wasmProcessor).executeCommand","line":347},{"file":"/Users/mahahajja/repos/conduit/pkg/plugin/errors.go","func":"github.com/conduitio/conduit/pkg/plugin.init","line":23}]

Steps to reproduce

  1. use the processor template repo
  2. write a filler processor
  3. run make build
  4. add the wasm processor to a processors directory
  5. run conduit

Version

v0.4.0

@maha-hajja maha-hajja added the bug Something isn't working label Jan 23, 2025
@raulb raulb removed the status in Conduit Main Feb 8, 2025
@maha-hajja maha-hajja moved this to Triage in Conduit Main Mar 26, 2025
@lovromazgon lovromazgon moved this from Triage to Todo in Conduit Main Apr 2, 2025
@hariso hariso moved this from Todo to In Progress in Conduit Main Apr 7, 2025
@hariso hariso self-assigned this Apr 7, 2025
@hariso
Copy link
Contributor

hariso commented Apr 9, 2025

This PR fixes the issue: ConduitIO/conduit-processor-template#22. And here are 2 PRs that improve logging so that we can more easily troubleshoot this in the future: #131 and ConduitIO/conduit#2241.

@hariso hariso moved this from In Progress to Ready for review in Conduit Main Apr 9, 2025
@github-project-automation github-project-automation bot moved this from Ready for review to Done in Conduit Main Apr 10, 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
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants