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

[task] move data to /var/cache #60

Closed
GeoffWilliams opened this issue Sep 26, 2018 · 2 comments · Fixed by #63
Closed

[task] move data to /var/cache #60

GeoffWilliams opened this issue Sep 26, 2018 · 2 comments · Fixed by #63
Assignees
Labels
enhancement New feature or request

Comments

@GeoffWilliams
Copy link
Contributor

There are some instances of state being stored in /etc/os_patching. This isn't recommend as the directory mixes auto-generated state with puppet-written config files. Linux FHS System state should be under /var/lib somewhere [1]

Affected files:

  • /etc/os_patching/security_package_updates
  • /etc/os_patching/package_updates

Nitpicking?

@GeoffWilliams GeoffWilliams changed the title [bug?] stores stated under /etc/ [bug?] stores state under /etc/ Sep 26, 2018
@GeoffWilliams
Copy link
Contributor Author

There are no user-servicable parts in /etc/os_patching, therefore the correct location is /var/cache/os_patching:

/var/cache
Application cache data. Such data are locally generated as a result of time-consuming I/O or calculation. The application must be able to regenerate or restore the data. The cached files can be deleted without loss of data

@GeoffWilliams GeoffWilliams changed the title [bug?] stores state under /etc/ [task] move data to /var/cache Sep 26, 2018
@GeoffWilliams
Copy link
Contributor Author

Proposed C:\programdata\os_patching for windows

@albatrossflavour albatrossflavour self-assigned this Sep 27, 2018
@albatrossflavour albatrossflavour added the enhancement New feature or request label Sep 27, 2018
@albatrossflavour albatrossflavour added this to In progress in Test project Sep 28, 2018
Test project automation moved this from In progress to Done Sep 28, 2018
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
No open projects
Test project
  
Done
Development

Successfully merging a pull request may close this issue.

2 participants