Skip to content

fix: make error output actually debuggable#258

Open
hfukada wants to merge 1 commit intodatainfrahq:masterfrom
hfukada:output-error-on-reconcile-non-200
Open

fix: make error output actually debuggable#258
hfukada wants to merge 1 commit intodatainfrahq:masterfrom
hfukada:output-error-on-reconcile-non-200

Conversation

@hfukada
Copy link
Copy Markdown

@hfukada hfukada commented Feb 13, 2026

Description

Debugging failures in this operator is pretty impossible without digging into druid itself. On non 200 the reconciler should at least output a log/event. Non 200 error codes just disappear in the current handling.


This PR has:

  • been tested on a real K8S cluster to ensure creation of a brand new Druid cluster works.
  • been tested for backward compatibility on a real K*S cluster by applying the changes introduced here on an existing Druid cluster. If there are any backward incompatible changes then they have been noted in the PR description.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

Key changed/added files in this PR

some logging

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.

1 participant