Skip to content

add nodeselector/affinity/tolerations/capabilities to separate r10k deployment#10

Open
ke5C2Fin wants to merge 7 commits intoOpenVoxProject:mainfrom
ke5C2Fin:affinity
Open

add nodeselector/affinity/tolerations/capabilities to separate r10k deployment#10
ke5C2Fin wants to merge 7 commits intoOpenVoxProject:mainfrom
ke5C2Fin:affinity

Conversation

@ke5C2Fin
Copy link
Contributor

@ke5C2Fin ke5C2Fin commented Sep 3, 2025

add nodeselector/affinity/tolerations/capabilities to separate r10k deployment
fix | (pipe) spacing
move puppetserver.masters.affinity to template like puppetserver.compilers.affinity

…ng, move puppetserver.masters to template like puppetserver.compilers
@anthonysomerset
Copy link
Collaborator

this looks ok, please can you update the unit tests as needed, update the change long and bump the version number

if you aren't already in the credits list at the bottom of readme then please do add yourself as a contributor

thanks

@ke5C2Fin
Copy link
Contributor Author

Can you tell me how I can test these tests?
The unit tests appear to be very incomplete but I will try to get them better for what I am affecting.
Maybe you can link me something that explains it?

Copy link
Collaborator

@anthonysomerset anthonysomerset left a comment

Choose a reason for hiding this comment

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

i've effectively reverted your changelog edit as has been superseded by auto changelog generation but otherwise all good so lgtm

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.

2 participants