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

Choice between g10k and r10k #190

Open
ldaneliukas opened this issue Nov 17, 2023 · 1 comment
Open

Choice between g10k and r10k #190

ldaneliukas opened this issue Nov 17, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@ldaneliukas
Copy link
Collaborator

ldaneliukas commented Nov 17, 2023

Use Case

I'd like to use g10k instead of r10k. The way that the chart is made right now, does not allow you to not use r10k. Changing the r10k image to g10k will not work due to configuration differences. Utilizing g10k with webhooks is also not possible since you need an ingress exposed, leaving you to deploy g10k alongside your puppetserver as an independent deployment.

The issue with the latter is that r10k can't be disabled and you need to change the command it uses to basically do nothing in an infinite loop.

r10k is usually fine, but in setups with a control repository that is used by a lot of different people it simply does not cut it as it is really slow.

Describe the Solution You Would Like

Ideally: A choice for whether you want to use g10k or r10k and the possibility to utilize webhooks (with g10k) by exposing an ingress as this is by far the fastest way to deploy environments.

Okey: A choice for whether you want to use g10k or r10k.

At least: Possibility to disable r10k entirely if the user opts to manage his code/environments independently.

Describe Alternatives You've Considered

Currently overcoming this limitation by deploying g10k independently and setting the following values for r10k:

  r10k:
    code:
      # There is no option to disable r10k. Sleep until interrputed (helm delete/upgrade/etc)
      command: ["/bin/sh"]
      args: ["-c", "trap : TERM INT; sleep 9999999999d & wait"]
      cronJob:
        enabled: false
      readinessProbe:
        - /bin/true
    hiera:
      cronJob:
        enabled: false

Additional Context

I can create a PR for this.

@ldaneliukas ldaneliukas added the enhancement New feature or request label Nov 17, 2023
@ldaneliukas ldaneliukas changed the title Using g10k instead of r10k Choice between g10k and r10k Nov 17, 2023
@Xtigyro
Copy link
Contributor

Xtigyro commented Nov 27, 2023

Yeah - please create a PR. Sounds reasonable to me. Maybe @davidphay can comment here too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants