janhoy commented on issue #602:
URL: https://github.com/apache/solr-operator/issues/602#issuecomment-6768803491

   ## Reproduction
   
   Reproduced, and it is worse than "not supported": the change is accepted 
into the CR and then silently never happens.
   
   `CopyStatefulSetFields` skips `volumeClaimTemplates` 
(`controllers/util/common.go` — the block is commented out, since Kubernetes 
does not allow modifying them) but still copies the pod template. So the 
operator builds an update that drops the `data` emptyDir while being unable to 
add the `data` volumeClaimTemplate, and the API server rejects it outright:
   
   ```
   spec.template.spec.containers[0].volumeMounts[0].name: Not found: "data"
   ```
   
   Adding the volumeClaimTemplate on its own is forbidden too, so the switch 
cannot be made by updating the existing StatefulSet at all. The effect is that 
your pods keep running on ephemeral storage, the SolrCloud still looks healthy, 
and the operator retries the same rejected update on every reconcile — no 
event, no status change, so the only trace is the operator log. That silence is 
the bug.
   
   ## Proposal
   
   Document that `dataStorage` cannot be switched after creation, and reject 
the change at apply time so you get a clear error instead of nothing.
   
   Migrating an existing cloud between storage types is a new feature reuest 
and is out of scope for this bug issue.
   
   🤖 AI Assisted


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to