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

Agents-to-masters service is identical to master service #124

Open
skoef opened this issue Jun 30, 2022 · 0 comments
Open

Agents-to-masters service is identical to master service #124

skoef opened this issue Jun 30, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@skoef
Copy link
Contributor

skoef commented Jun 30, 2022

Describe the Bug

In the chart, once you enable the use of separate compilers, the service described in templates/puppetserver-service-agents-to-masters.yaml (canonically called agents-to-puppet) is created. This service is, except for the name, 100% identical to the service describe in templates/puppetserver-service-masters.yaml (called puppet). Also, still with compilers enabled, the service described in templates/puppetserver-service-compilers.yaml (called puppet-compilers) is created.

I don't understand why there should be two identical services, both with the master pods as endpoints.

Expected Behavior

In my setup, I point my agents to the puppet service as their ca_server and to the puppet-compilers service as their server. This works fine.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Install the chart with --set puppetserver.compilers.enabled=true
  2. See how identical the puppet and agents-to-puppet services are and how they have identical endpoints
  3. See how the puppet-compilers service is the only relevant service to reach the compilers considered both puppet-compilers and agents-to-puppet.

Environment

  • Version: chart 6.3.0
  • Platform: kubernetes/helm

Additional Context

I think the agents-to-puppet service can be dropped altogether. Unless I don't understand the reasoning behind it. Please, enlighten me! :)

@skoef skoef added the bug Something isn't working label Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant