KusionStack / karpor

Intelligence for Kubernetes. World's most promising Kubernetes Visualization Tool for Developer and Platform Engineering teams.
https://karpor-demo.kusionstack.io
Apache License 2.0
434 stars 44 forks source link

FailedAttachVolume #550

Open danielstankw opened 1 month ago

danielstankw commented 1 month ago

What happened?

There is an issue with the helm chart deployment and Volume. The volume 'vol-03e1c30abe533984b' does not exist. - seems like a misconfiguration of volume name on the helm site

When deploying the helm chart the following happens:

elasticsearch-5654f4f8c5-8xjkj   1/1     Running             0              16m
etcd-0                           0/1     ContainerCreating   0              13m
karpor-server-d9d477955-9gb4t    0/1     CrashLoopBackOff    7 (3m ago)     16m
karpor-syncer-64d449f4c-fbzjv    0/1     Error               8 (5m7s ago)   16m

This is due: k describe pod etcd-0

  Warning  FailedAttachVolume  12m   attachdetach-controller  AttachVolume.Attach failed for volume "pvc-f28692d1-1d3e-4b8a-b2fc-1980dc2c1eee" : InvalidVolume.NotFound: The volume 'vol-03e1c30abe533984b' does not exist.
           status code: 400, request id: ef0543fb-03dd-4a9c-82ab-06f2cca0faa2

k get pvc

NAME          STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
data-etcd-0   Bound    pvc-f28692d1-1d3e-4b8a-b2fc-1980dc2c1eee   10Gi       RWO            gp2            3h59m

What did you expect to happen?

....

How can we reproduce it (as minimally and precisely as possible)?

Run the default helm chart with unchanged values.

Anything else we need to know?

No response

Karpor version

```console $ karpor -V # paste output here ```

OS version

```console # On Linux: $ cat /etc/os-release # paste output here $ uname -a # paste output here # On Windows: C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture # paste output here ```

Install tools

elliotxx commented 1 month ago

Ohh, I'm not in front of the computer right now. I'll reproduce it locally later.

Does this mistake exist all the time, or will it just wait a while?

danielstankw commented 1 month ago

@elliotxx For me its been there all the time. It seems like its trying to use volume that doesnt exist.

elliotxx commented 1 month ago

Strange, we didn't encounter this case during the test. Let me see what's going on.

danielstankw commented 1 month ago

@elliotxx any update on this?

ruquanzhao commented 1 month ago

Hi, @danielstankw, welcome to the Community!

It seems you are using a k8s cluster with AWS. Can you please check your Persistent Volume? If you don't have an available Persistent Volume, you might need to either configure a StorageClass or manually create a 10GB storage volume.

danielstankw commented 1 month ago

@ruquanzhao thanks for reply, It seems like issue is happening due to the difference in AWS setup. Would it be possible for you to add fields for defining pvc in the helm chart? - that would solve the issue

ruquanzhao commented 1 month ago

@ruquanzhao thanks for reply, It seems like issue is happening due to the difference in AWS setup. Would it be possible for you to add fields for defining pvc in the helm chart? - that would solve the issue

Yes, you are right, the stroageclass can be configured through Helm chart. Let's discuss more for details! WDYT, @elliotxx ?

elliotxx commented 1 month ago

@ruquanzhao @danielstankw Yeah, PVC has been mentioned multiple times (#347 #522 ...), and we will support specifying it in helm this week!

rajeswarr commented 1 month ago

@ruquanzhao thanks for reply, It seems like issue is happening due to the difference in AWS setup. Would it be possible for you to add fields for defining pvc in the helm chart? - that would solve the issue

Yes, you are right, the stroageclass can be configured through Helm chart. Let's discuss more for details! WDYT, @elliotxx ?

is this implemented, still I am getting the issue?

elliotxx commented 1 month ago

@ruquanzhao thanks for reply, It seems like issue is happening due to the difference in AWS setup. Would it be possible for you to add fields for defining pvc in the helm chart? - that would solve the issue

Yes, you are right, the stroageclass can be configured through Helm chart. Let's discuss more for details! WDYT, @elliotxx ?

is this implemented, still I am getting the issue?

@rajeswarr Thank you feedback. Have you encountered this problem too? We will provide the ability to customize StorageClass this week.

rajeswarr commented 1 month ago

@ruquanzhao thanks for reply, It seems like issue is happening due to the difference in AWS setup. Would it be possible for you to add fields for defining pvc in the helm chart? - that would solve the issue

Yes, you are right, the stroageclass can be configured through Helm chart. Let's discuss more for details! WDYT, @elliotxx ?

is this implemented, still I am getting the issue?

@rajeswarr Thank you feedback. Have you encountered this problem too? We will provide the ability to customize StorageClass this week.

@elliotxx , I got this issue on-premises cluster.

danielstankw commented 1 month ago

@elliotxx if you want I can modify the helm chart and create pull request, unless you started on this topic?

elliotxx commented 1 month ago

@elliotxx if you want I can modify the helm chart and create pull request, unless you started on this topic?

@danielstankw Very welcome! As I replied in another issue, I can only start optimizing the chart a few days later. If you can start directly, that would be great.

danielstankw commented 1 month ago

@elliotxx sounds good, will do

rajeswarr commented 1 month ago

@elliotxx sounds good, will do

Thank you @elliotxx and @danielstankw. Please let me know once changes are done.

elliotxx commented 1 month ago

@elliotxx sounds good, will do

Thank you @elliotxx and @danielstankw. Please let me know once changes are done.

@rajeswarr Of course 👌

wenxuanwu commented 1 week ago

I also encountered a similar problem and can temporarily solve it by creating a local pv

elliotxx commented 1 week ago

I also encountered a similar problem and can temporarily solve it by creating a local pv

Yeah, We are following this problem