Skip to content

Conversation

@tdgroot
Copy link
Member

@tdgroot tdgroot commented Jan 3, 2023

In the future, when we reiterate on Hypernode Deploy for clusters, we'll see what we really need and what's best to do. For now, it doesn't make sense to keep this in here.

@tdgroot tdgroot changed the title This was causing task with a certain selector always running Remove roles selector from builtin tasks Jan 3, 2023
@tdgroot tdgroot force-pushed the remove_roles_selector_from_builtin_tasks branch 2 times, most recently from b5a9e8e to 3d1a88d Compare January 3, 2023 16:42
@tdgroot tdgroot force-pushed the remove_roles_selector_from_builtin_tasks branch from 3d1a88d to b1380b8 Compare January 3, 2023 17:02
This was causing task with a certain selector always running.

In the future, when we reiterate on Hypernode Deploy for clusters, we'll see what we really need and what's best to do. For now, it doesn't make sense to keep this in here.
@tdgroot tdgroot force-pushed the remove_roles_selector_from_builtin_tasks branch from b1380b8 to 729d278 Compare January 4, 2023 08:05
Copy link
Collaborator

@martijneichelsheim martijneichelsheim left a comment

Choose a reason for hiding this comment

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

Lgtm!

@AlexanderGrooff AlexanderGrooff merged commit d9d4dc6 into master Jan 4, 2023
@AlexanderGrooff AlexanderGrooff deleted the remove_roles_selector_from_builtin_tasks branch January 4, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants