Drop in-built support for bitnami's memcached

[relusc]

What this PR does:

Removes all references to bitnami images and subcharts (namely memcached). The plan would be to not be able to install memcached by default since cortex can run without it. The deployment of memcached is up to the cortex user and is not in the responsibility of the chart anymore.

Which issue(s) this PR fixes:
It doesn't really fix an issue, but it relates to #548

Checklist

• CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX], [DEPENDENCY]

[nschad]

Thanks just some tiny nit

[kd7lxl]

docs/guides/configure_memcached.markdown will need to be updated as well.

[relusc]

@kd7lxl @nschad I updated the guide and also readded some of the values. I figured that I still want to be able to reference a memcached installation for the components, but I don't want the chart to install them for me

[nschad]

@kd7lxl @nschad I updated the guide and also readded some of the values. I figured that I still want to be able to reference a memcached installation for the components, but I don't want the chart to install them for me

You can also add CLI Flags via the already existing extraArgs directive. I don't know if we want to keep the helpers, don't really have a strong opinion about it.

[relusc]

i am fine with both, however for me as a user i'd rather have these extra opts. but that's just my personal opinion

[relusc]

hi guys, any updates here ? :)

[relusc]

chart tests should be fixed now: ae17361

[relusc]

hi, once again i'm bumping here, is anything still missing or what is planned here ? :)

[nschad]

hi, once again i'm bumping here, is anything still missing or what is planned here ? :)

Personally I still don't really like the half helpers.tpl for memcached? I would honestly remove it

WDYT @kd7lxl

[kd7lxl]

My opinion is that the built-in memcached support adds value and is not too onerous to maintain. We should keep it. It's also easy to disable for users who want to provide their own.

I would accept a guide that documents how to disable the built-in memcached with values and provide your own.

[relusc]

@kd7lxl I added a section how to set the memcached deployments without using the dedicated section in the values. let me know if this is not what you wanted, as I was not completely sure myself what kind of documentation you asked for 😅