[tempo-distributed] Define default affinity for compactor

[froblesmartin]

As it is not even defined as null as an example, I guess it was just totally forgotten. 😄 But it is available to be used at

helm-charts/charts/tempo-distributed/templates/compactor/deployment-compactor.yaml

Lines 114 to 116 in 51db949

	affinity:
	{{- tpl . $ | nindent 8 }}
	{{- end }}

I am defining the same affinities as everywhere else, also the same as in loki-distributed for the compactor.

Please, let me know if everything is alright or if I should change something.

[CLAassistant]

All committers have signed the CLA.

[froblesmartin]

Hi @mapno ! :)

I have pushed an update with the README.md updated as well as bumping the patch version. In this case, this change defines a new default value, which could be considered a breaking change 🤔 but in this case, if someone had defined something then it will not affect them, and if they didn't, this is a good to have. Still, someone with a single node could see pods after the 1st one not being scheduled.

Let me know what you think 😉

Thanks!

[mapno]

Hi @mapno ! :)

I have pushed an update with the README.md updated as well as bumping the patch version. In this case, this change defines a new default value, which could be considered a breaking change 🤔 but in this case, if someone had defined something then it will not affect them, and if they didn't, this is a good to have. Still, someone with a single node could see pods after the 1st one not being scheduled.

Let me know what you think 😉

Thanks!

Hi @froblesmartin. I think putting a disclaimer in the README should be enough for this change. A minor version update seems ok. Thanks for the PR!

[froblesmartin]

Hi @froblesmartin. I think putting a disclaimer in the README should be enough for this change. A minor version update seems ok. Thanks for the PR!

@mapno done 😉 bumped a minor version and added some lines in the README 👍

[mapno]

LGTM, thanks!