[bitnami/flink] Flink global.defaultStorageClass is not working

[wklee610]

Name and Version

bitnami/flink 1.4.5

What architecture are you using?

None

What steps will reproduce the bug?

If only specify defaultStorageClass in an existing Helm chart and run "helm upgrade --install", PV and PVC won't be created.

Are you using any custom parameters or values?

global.defaultStorageClass

What is the expected behavior?

Create PV, PVC

What do you see instead?

nothing

Additional information

No response

[javsalgar]

Hi,

Does the issue happen in a fresh instance? Or does it only happen when upgrading?

[wklee610]

It happens in a fresh instance.
In Charts, seems volumeClaimTemplates doesn't exist.

[fmulero]

Hi @wklee610 thanks for using bitnami/charts.

Sorry but I am a little bit confused, Flink chart doesn't have any PV, PVC or PersistentVolumeClaimTemplate in its code. Do you think they are missing? Would you like to contribute by creating a PR adding them? The Bitnami team will be happy to review it and provide feedback. Here you can find the contributing guidelines.

[wklee610]

First, I'm sorry for confusing you. I thought it was supported because there was defaultStorageClass in this code.
After looking into it, it seems that PV and PVC are not provided by default in Flink.
Normally, people use it in a way like S3 or HDFS or upload a JAR file in the Flink Web UI but in my case, I kept running "kubectl cp" while uploading a JAR file on k8s for testing purposes, so, it seemed more convenient to have PV/PVC in Flink (If I was using it correctly).

Of course, I would like to contribute by creating a PR (Adding volumeClaimTemplates or delete defaultStorageClass in global).

I don't know whether I should delete it or add it, so if you make any decision, I'll try to make a PR.

Thank you so much :)

[wklee610]

Hello, @fmulero
I've submitted a PR for adding PVC support based on your suggestion. I’d appreciate it if you could take a look when you have time. Thanks a lot!

#32828