Skip to content
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

Allow passing LokiStack storageClassName #260

Open
mffiedler opened this issue Sep 13, 2022 · 4 comments
Open

Allow passing LokiStack storageClassName #260

mffiedler opened this issue Sep 13, 2022 · 4 comments

Comments

@mffiedler
Copy link
Contributor

To run the netobserv-perf automation on non-AWS clouds (or on 4.12 where the gp2 storageClass no longer exists), we should let the user pass the storageClassName for the LokiStack CRD. It can default to gp2.

cc: @nathan-weinberg

@nathan-weinberg
Copy link
Contributor

cc: @memodi

@memodi
Copy link

memodi commented Sep 13, 2022

@mffiedler - there seems to be default storageclass that cluster uses:

4.12

$ oc get storageclass -o=jsonpath='{.items[?(@.metadata.annotations.storageclass\.kubernetes\.io/is-default-class=="true")].metadata.name}'
gp3-csi

4.11:

$ oc get storageclass -o=jsonpath='{.items[?(@.metadata.annotations.storageclass\.kubernetes\.io/is-default-class=="true")].metadata.name}'
gp2

do you think a better approach would be to use the default cluster storageClass derived from above method?

As for user to be able to pass storageClassName, for our use case, I am trying to think what could be scenarios in which we could need other storage class? given that Loki is uploading flow logs to cloud providers object storage platform anyway.

For pipeline to work for other cloud providers, I created this task: https://issues.redhat.com/browse/NETOBSERV-582

@mffiedler
Copy link
Contributor Author

Default would be great. Will the LokiStack use the default sc if you leave storageClassName unspecified? or if there is better way to take the default, that would be OK by me

@memodi
Copy link

memodi commented Sep 13, 2022

Will the LokiStack use the default sc if you leave storageClassName unspecified?

Just checked, users must specify storageClass while creating LokiStack, no default values. We'll change our LokiStack manifests to use cluster default SC.

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