Skip to content
This repository was archived by the owner on Sep 1, 2026. It is now read-only.

fix: make error output actually debuggable - #258

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

hfukada wants to merge 1 commit into
datainfrahq:masterfrom
hfukada:output-error-on-reconcile-non-200

Conversation

@hfukada

@hfukada hfukada commented Feb 13, 2026

Copy link
Copy Markdown

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant