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

(WIP) add psql task #929

Closed
wants to merge 2 commits into from
Closed

Conversation

spacepants
Copy link

This adds a task to execute arbitrary SQL via psql using a Ruby script and open3.

I've stubbed out tests based on examples from other Puppetlabs modules, but if there's a preferred testing method, I'm more than happy to refactor. Thanks!

@tphoney
Copy link
Contributor

tphoney commented Oct 17, 2017

@spacepants can you look at #930 I had this ready for puppet conf but didnt get to release it. Is there anything i am missing ?

@spacepants
Copy link
Author

Oh cool, beaker-task_helper cleans up a lot of this. From the travis logs, it's bombing out because it's not finding bolt at /opt/puppetlabs/puppet/bin/bolt, but install_bolt_on() should be installing it in the right place.

I'll see if I can replicate the issue.

@tphoney
Copy link
Contributor

tphoney commented Oct 18, 2017

@spacepants so i fixed up beaker-task_helper bolt installs successfully. Still no dice. I will hopefully get to look tomorrow, any ideas are welcome

@tphoney
Copy link
Contributor

tphoney commented Dec 1, 2017

Apologies @spacepants, i have been AFK for a few weeks. #930 Is merged. Is there anything i missed.

Thanks for the work !

@david22swan
Copy link
Member

@spacepants Is there any movement on this PR?

@spacepants
Copy link
Author

Sorry, yeah this was accomplished with #930.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants