README and values.yaml comments don't list options for image.flavor

[jessebot]

Currently image.flavor the README.md shows a default of apache, but no description outside of "image type".

In the image block in values.yaml it explains that if image.tag is unset, it uses image.flavor + .Chart.AppVersion to create image.tag, but doesn't give other options for image.flavor outside of the default apache.

Looking at the nextcloud docker README.md under supported tags, it lists fpm there, so is the only other valid option fpm?

I can submit a pull request if you want to clear up the documentation for newcomers in both files only this is cleared up :)

[TheShadowBanned]

Afaik apache and fpm are the only flavors and its pretty much explained in the nextcloud docs.

tl;dr?

Apache for built in webserver, fpm for custom e.g. nginx.

Or did I get you wrong there?

Greetz

[jessebot]

Thanks for getting back to me :)

Yep, that appears to be the case, but to be more clear, in at least the README, I think it makes sense to just change the description field to have the exact options, to avoid having to then cross reference other areas, since there's a lot of options in the values.yaml and we can easily spend days learning them all:

Parameter	Description	Default
image.repository	nextcloud Image name	nextcloud
image.flavor	nextcloud Image type (options: apache or fpm)	apache