Skip to content

Error setting SSA resource version on Creation #2033

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
csviri opened this issue Aug 25, 2023 · 4 comments
Closed

Error setting SSA resource version on Creation #2033

csviri opened this issue Aug 25, 2023 · 4 comments

Comments

@csviri
Copy link
Collaborator

csviri commented Aug 25, 2023

The following error produced by Kubernetes: resourceVersion should not be set on objects to be created

Note that we did not see this before just happened in this CI.

From CI Definition:

  MINIKUBE_VERSION: v1.28.0
  KUBERNETES_VERSION: v1.22.3

Logs:

https://github.com/keycloak/keycloak/actions/runs/5957796945/job/16161289266

WARN: Keycloak failed to become ready "example-kc" ---
2023-08-24T00:50:35.5896006Z conditions:
2023-08-24T00:50:35.5896322Z - type: "Ready"
2023-08-24T00:50:35.5896553Z   status: "Unknown"
2023-08-24T00:50:35.5896803Z - type: "HasErrors"
2023-08-24T00:50:35.5897189Z   message: "Error performing operations:\nException(s) during workflow execution.\
2023-08-24T00:50:35.5897939Z     \ Details:\n - org.keycloak.operator.controllers.KeycloakServiceDependentResource\
2023-08-24T00:50:35.5898669Z     \ -> io.fabric8.kubernetes.client.KubernetesClientException: Failure executing:\
2023-08-24T00:50:35.5899567Z     \ PATCH at: https://10.96.0.1:443/api/v1/namespaces/keycloak-test-89c23f9e-1627-48ed-b7e1-73871f2dc708/services/example-kc-service?fieldManager=keycloakcontroller&force=true.\
2023-08-24T00:50:35.5900345Z     \ Message: resourceVersion should not be set on objects to be created. Received\
2023-08-24T00:50:35.5900821Z     \ status: Status(apiVersion=v1, code=500, details=null, kind=Status, message=resourceVersion\
2023-08-24T00:50:35.5901712Z     \ should not be set on objects to be created, metadata=ListMeta(_continue=null,\
2023-08-24T00:50:35.5902197Z     \ remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}),\
2023-08-24T00:50:35.5902962Z     \ reason=null, status=Failure, additionalProperties={}).\n\tat io.fabric8.kubernetes.client.KubernetesClientException.copyAsCause(KubernetesClientException.java:238)\n\
2023-08-24T00:50:35.5903909Z     \tat io.fabric8.kubernetes.client.dsl.internal.OperationSupport.waitForResult(OperationSupport.java:518)\n\
2023-08-24T00:50:35.5904698Z     \tat io.fabric8.kubernetes.client.dsl.internal.OperationSupport.handleResponse(OperationSupport.java:535)\n\
2023-08-24T00:50:35.5905486Z     \tat io.fabric8.kubernetes.client.dsl.internal.OperationSupport.handlePatch(OperationSupport.java:430)\n\
2023-08-24T00:50:35.5906316Z     \tat io.fabric8.kubernetes.client.dsl.internal.OperationSupport.handlePatch(OperationSupport.java:408)\n\
2023-08-24T00:50:35.5907066Z     \tat io.fabric8.kubernetes.client.dsl.internal.BaseOperation.handlePatch(BaseOperation.java:713)\n\
2023-08-24T00:50:35.5907836Z     \tat io.fabric8.kubernetes.client.dsl.internal.HasMetadataOperation.lambda$patch$2(HasMetadataOperation.java:232)\n\
2023-08-24T00:50:35.5908634Z     \tat io.fabric8.kubernetes.client.dsl.internal.HasMetadataOperation.patch(HasMetadataOperation.java:237)\n\
2023-08-24T00:50:35.5909582Z     \tat io.fabric8.kubernetes.client.dsl.internal.HasMetadataOperation.patch(HasMetadataOperation.java:252)\n\
2023-08-24T00:50:35.5910335Z     \tat io.fabric8.kubernetes.client.dsl.internal.BaseOperation.serverSideApply(BaseOperation.java:1132)\n\
2023-08-24T00:50:35.5911097Z     \tat io.fabric8.kubernetes.client.dsl.internal.BaseOperation.serverSideApply(BaseOperation.java:92)\n\
2023-08-24T00:50:35.5912046Z     \tat io.javaoperatorsdk.operator.processing.dependent.kubernetes.KubernetesDependentResource.update(KubernetesDependentResource.java:145)\n\
2023-08-24T00:50:35.5913217Z     \tat io.javaoperatorsdk.operator.processing.dependent.kubernetes.CRUDKubernetesDependentResource.update(CRUDKubernetesDependentResource.java:16)\n\
2023-08-24T00:50:35.5914329Z     \tat io.javaoperatorsdk.operator.processing.dependent.AbstractDependentResource.handleUpdate(AbstractDependentResource.java:144)\n\
2023-08-24T00:50:35.5915650Z     \tat io.javaoperatorsdk.operator.processing.dependent.kubernetes.KubernetesDependentResource.handleUpdate(KubernetesDependentResource.java:122)\n\
2023-08-24T00:50:35.5916859Z     \tat io.javaoperatorsdk.operator.processing.dependent.kubernetes.KubernetesDependentResource.handleUpdate(KubernetesDependentResource.java:32)\n\
2023-08-24T00:50:35.5917958Z     \tat io.javaoperatorsdk.operator.processing.dependent.AbstractDependentResource.reconcile(AbstractDependentResource.java:72)\n\
2023-08-24T00:50:35.5919029Z     \tat io.javaoperatorsdk.operator.processing.dependent.SingleDependentResourceReconciler.reconcile(SingleDependentResourceReconciler.java:19)\n\
2023-08-24T00:50:35.5920084Z     \tat io.javaoperatorsdk.operator.processing.dependent.AbstractDependentResource.reconcile(AbstractDependentResource.java:52)\n\
2023-08-24T00:50:35.5921128Z     \tat io.javaoperatorsdk.operator.processing.dependent.workflow.WorkflowReconcileExecutor$NodeReconcileExecutor.doRun(WorkflowReconcileExecutor.java:115)\n\
2023-08-24T00:50:35.5922072Z     \tat io.javaoperatorsdk.operator.processing.dependent.workflow.NodeExecutor.run(NodeExecutor.java:22)\n\

@metacosm
Copy link
Collaborator

We might want to update #1657 and merge it after all as we're definitely behind on which kube versions we're testing with…

@shawkins
Copy link
Collaborator

Another oddity to note here is this is happening on an update - so the cache state thinks there's an existing entry, but the server thinks it's a creation.

@csviri
Copy link
Collaborator Author

csviri commented Aug 30, 2023

@vmuzikar confirmed that this happens only on k8s version < 1.23.

@csviri
Copy link
Collaborator Author

csviri commented Sep 4, 2023

Since this is on a quite old version of Kubernetes will close it (if no objections), since we don't support that now.

@csviri csviri closed this as completed Sep 4, 2023
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

No branches or pull requests

3 participants