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

Add nodeSelector, tolerations and affinity to metrics deployment #534

Conversation

SwitzerChees
Copy link
Contributor

Signed-off-by: Patrick Michel [email protected]

@SwitzerChees SwitzerChees force-pushed the add-metrics-affinity-toleration-nodeselector branch from 3cb13cf to cf8a36f Compare February 10, 2024 17:02
@SwitzerChees
Copy link
Contributor Author

Just added the options nodeSelector, tolerations and affinity to the metrics template. For our use case it is important to also be able to control the placement of this pod based on these rules. And probably other people will also be happy about that in the future.

Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, maybe @wrenix you also want to take a look? :)

@provokateurin
Copy link
Member

You need to bump the chart version btw.

@SwitzerChees
Copy link
Contributor Author

You need to bump the chart version btw.

Done just a patch version.
Or is it a minor version do you follow the semantic versioning concept? ;-)

Copy link
Collaborator

@wrenix wrenix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code part looks really nice.

For my opionion, it is such a small new feature, that i would use a patch level.
But yes, you are right that is a feature so in semver it would be a minor.

charts/nextcloud/values.yaml Outdated Show resolved Hide resolved
@SwitzerChees SwitzerChees force-pushed the add-metrics-affinity-toleration-nodeselector branch from 9efec16 to 735bd6d Compare June 27, 2024 15:42
@provokateurin
Copy link
Member

Something looks off with all these commits, please rebase onto master to drop all of them.

… deployment

Signed-off-by: Patrick Michel <[email protected]>
Signed-off-by: switzerchees <[email protected]>
Signed-off-by: WrenIX <[email protected]>
@wrenix wrenix force-pushed the add-metrics-affinity-toleration-nodeselector branch from 00b3824 to 230e4e9 Compare September 12, 2024 18:42
@wrenix wrenix merged commit 5381a62 into nextcloud:main Sep 12, 2024
9 checks passed
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

Successfully merging this pull request may close these issues.

4 participants