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

(RK-360) Allow overriding the cachedir in deploy module #1020

Merged
merged 1 commit into from
Jan 31, 2020

Conversation

Magisus
Copy link
Collaborator

@Magisus Magisus commented Jan 31, 2020

Previously, we added the ability to configure the cachedir from the CLI
for the deploy environment command, since Code Manager configures a
separate cachedir for each of its r10k workers. Now that we are also
using the deploy module command in Code Manager, it needs to respect
these special cachedirs the same way. The --cachedir flag can now be
used with the deploy module command.

Previously, we added the ability to configure the cachedir from the CLI
for the `deploy environment` command, since Code Manager configures a
separate cachedir for each of its r10k workers. Now that we are also
using the `deploy module` command in Code Manager, it needs to respect
these special cachedirs the same way. The `--cachedir` flag can now be
used with the `deploy module` command.
@Magisus Magisus requested a review from a team January 31, 2020 17:24
Copy link
Member

@justinstoller justinstoller left a comment

Choose a reason for hiding this comment

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

Seems eminently sensible.

I'm curious how many of the actions would/could use cachedir, but it looks like Base has very little in it. We can add auditing options to the "things we'd like to clean up in this repo" list.

@Magisus
Copy link
Collaborator Author

Magisus commented Jan 31, 2020

Yeah it was a little difficult to follow the thread of exactly where this gets plumed through. It winds up kicking in here mostly, I think.

@mwaggett mwaggett merged commit 3cc381f into puppetlabs:master Jan 31, 2020
@Magisus Magisus mentioned this pull request Apr 22, 2020
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.

3 participants