-
Notifications
You must be signed in to change notification settings - Fork 179
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
Working directory 'C:/Program Files/Filebeat' does not exist #224
Comments
@pcfens I do not use chocolatey and install 7zip prior to this step. if that's the case maybe a boolean of use_chocolatey or use_7zip that would allow for https://github.com/pcfens/puppet-filebeat/blob/master/manifests/install/windows.pp#L40-L47 |
If it's possible to resolve the dependency issue without requiring a specific piece of software then I think that's a good thing, but since I'm not a Windows person at all I'm not sure what's reasonable on that front and what isn't. I don't know if @edestecd is still around, but his comments might be helpful since he's the original author of this (and included a comment about it). I read the comment as though any unzip package will work with the current implementation, but the archive module requires 7zip. By not using the archive module's built in feature users are free to use whichever unzip tool they like. |
This should have nothing to do with 7zip or chocolatey as far as i can see. Neither is used here. |
All the proper requires are used here. I have experienced some "lag" in windows file manipulation in the past. This is my guess. I don't really know how to fix this as it seems like an underlying os or puppet issue. |
@edestecd not sure. It constantly is failing for my users trying to apply configuration that the parent directory of C:\Program Files\Filebeat doesn’t exist. (install_dir) |
I can confirm this. it looks like this is weird filebeat installer should also create the directory and the config should wait any way. Personally, Filebeat should be putting conf files in the ProgramsData location anyway. maybe on windows decouple the conf from the |
I've seen a similar issue with the metricbeat module. I've seen the note in the install module about the archive module requiring 7-zip to be installed but this isn't strictly true (at least not any more). The archive resource will happily unzip a Today I submitted a PR to the metricbeat module which allows the behaviour to be configurable - feel free to take it if it's possible that this may help with this issue (since it means that the archive resource will unzip and cleanup in one fell swoop). |
module version: 3.4.0
This is definitely an ordering issue because the second run works fine. I'm just trying to track down which resource is the culprit.
The text was updated successfully, but these errors were encountered: