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

Allow Puppetfile to be refreshed #304

Closed
cmd-ntrf opened this issue May 8, 2024 · 0 comments · Fixed by #306
Closed

Allow Puppetfile to be refreshed #304

cmd-ntrf opened this issue May 8, 2024 · 0 comments · Fixed by #306
Assignees

Comments

@cmd-ntrf
Copy link
Member

cmd-ntrf commented May 8, 2024

In puppet.yaml, we currently have this:

%{ if puppetfile != "" ~}
%{ if strcontains(puppetfile, "forge") ~}
  - cat /etc/puppetlabs/Puppetfile  > /etc/puppetlabs/code/environments/production/Puppetfile
%{ else }
  - cat /etc/puppetlabs/Puppetfile >> /etc/puppetlabs/code/environments/production/Puppetfile
%{ endif ~}
  - rm /etc/puppetlabs/Puppetfile
%{ endif ~}

We could instead, keep the Puppetfile, and make a second call to r10k by specifying the Puppetfile path and the moduledir path.

USAGE
    r10k puppetfile install

OPTIONS
       --force                             Force locally changed files to be
                                           overwritten
       --module-exclude-regex=<value>      A regex to exclude modules from
                                           installation. Helpful in CI
                                           environments.
       --moduledir=<value>                 Path to install modules to
       --puppetfile=<value>                Path to puppetfile

Puppetfile could be included in the puppetserver_etc zip file and the r10k command could be called by the provision module.

@cmd-ntrf cmd-ntrf self-assigned this May 8, 2024
@cmd-ntrf cmd-ntrf linked a pull request May 13, 2024 that will close this issue
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 a pull request may close this issue.

1 participant