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
Add support to custom xlogdir parameter #256
Conversation
Can one of the admins verify this patch? |
@mnecia I think the code itself looks good, but I would add an rspec-system test to ensure things still work when the xlog dir is provided. Its just too easy for this stuff not to work or be fragile esp. on older OS distros, so its not worth leaving it out of coverage. |
Also, this lacks any update to the README.md, can you ensure the parameter is documented there @mnencia? |
There isn't any test nor documentation also for the datadir parameter. I'll add both. |
@mnencia hey perfect, yes no coverage for datadir - adding that would be awesome. We do need to get better at adding those tests, my fault this time as I had a chance to do it - but yeah - we should be harder on ourselves. For example: I bet you a dollar putting in a test for xlogdir and/or datadir brings something to light :-). |
@mnencia hows it going? |
I'm sorry, but in the last two weeks I've been ultra-busy, and so I'll be until the end of PgConf.EU (1st November). |
@mnencia there is another user trying to solve the datadir problem. If you want, just try and solve the xlogdir problem for now so at least we can get this part solved - you don't have to boil the ocean. We can always get the custom datadir solved later. |
Sent @mnencia a PR here (https://github.com/mnencia/puppetlabs-postgresql/pull/1). Let me know if that isn't everything that we need and I'll get it added. |
Having a custom xlogdir location is desiderable for performances in many production environments
I've jut modified the system rspec. It should now work correctly. |
It's failing in Travis CI, but it passes both spec and spec:system target on my local system. I suspect that the failure is not related to this feature. |
Add support to custom xlogdir parameter
Add support to custom xlogdir parameter
Having a custom xlogdir location is desiderable for performances in many production environments