Skip to content

Conversation

Tom-Willemsen
Copy link
Member

@Tom-Willemsen Tom-Willemsen commented Oct 1, 2025

Description of work

Stream bluesky documents to kafka. This enables remote, out-of-process callbacks to trigger from bluesky scans or data-points.

See https://reduce.isis.cclrc.ac.uk/redpanda-console/topics/NDW2922_bluesky?p=-1&s=50&o=-1#messages for an example of msgpack-encoded documents which have been streamed from my dev machine.

This is using the upstream bluesky-kafka integration. It looks like that implementation is somewhat used by one or two facilities, but much less than core bluesky. If the upstream implementation causes a problem later, it would be pretty trivial to write our own (direct) wrapper that forwards messages to confluent-kafka via msgpack directly.

Ticket

None

Labels

  • 'bluesky-Semver-Minor' - New feature

Acceptance criteria

  • Pull request title is understandable for a user (e.g. scientist) reading the release notes. The PR title should be a short description of the change from a user perspective.
  • Pull request has appropriate labels for automatic release-notes generation

List the acceptance criteria for the PR. The aim is provide information to help the reviewer

Documentation

Kafka docs added in 'developer' section.

Not expecting this to be user-facing, so no user-facing docs added.

@Tom-Willemsen Tom-Willemsen added the bluesky-Semver-Minor New functionality / back-compatible changes label Oct 1, 2025
Don't kill a scan if kafka fails...
@Tom-Willemsen Tom-Willemsen requested a review from rerpha October 1, 2025 17:22
Copy link
Contributor

@rerpha rerpha left a comment

Choose a reason for hiding this comment

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

worth an ADR? maybe not yet as it's really a "because we can" atm?

@pull-request-size pull-request-size bot added size/L and removed size/M labels Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bluesky-Semver-Minor New functionality / back-compatible changes size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants