Made pgd cookbook openstack compatible

osuosl-cookbooks / pgd

Apache License 2.0

1 stars 0 forks source link

Made pgd cookbook openstack compatible #14

Closed pop closed 9 years ago

pop commented 9 years ago

Added .kitchen.cloud.yml file

Modified attributes/default.rb The pgd user, group, pgd_path, virtualenv_path have been changed to work with the centos user and not the vagrant user. This currently breaks running kitchen converge locally.

refs #3

pop commented 9 years ago

I'm not sure what I did to make this the kind of merge that you can't just click but when it's accepted I'll deal with the actual merging into master.

jordane commented 9 years ago

@ElijahCaine We typically fix this by rebasing against master.

mathuin commented 9 years ago

I checked out this branch and tried to run it, and it failed.

There were two errors: the dsspcmbi binary is apparently not being installed, and the database superuser isn't being created. I thought the superuser thing was due to a missing equals sign but it looks deeper than that -- the settings file also isn't being found -- so I will talk with @ElijahCaine more about it tomorrow!

ramereth commented 9 years ago

Just a point of clarification after talking about this PR with @mathuin. This PR should be merged before #12 and then #12 should be rebased to match this code base. I'm taking a look at this right now.

ramereth commented 9 years ago

It seems this is broken with the newer mysql cookbook. We're still using 3.0.x on prod so the Berksfile should be updated to have cookbook 'mysql', '~> 3.0.0'. It also seems that you're deploying the mysql server for testing and the chef run fails with the following:

[2015-01-29T19:15:10+00:00] FATAL: You must set node['mysql']['server_debian_password'], node['mysql']['server_root_password'], node['mysql']['server_repl_password'] in chef-solo mode. For more information, see https://github.com/opscode-cookbooks/mysql#chef-solo-note

We typically add those attributes to the yml files. I'm surprised this is even working on the non-openstack yml.

pop commented 9 years ago

I just tested this and didn't find any errors. As mentioned this won't be merge in until #12 is merged in, just wanted to give a status update. @ramereth if you're having problems with converging this on open stack let me know and I'll try to resolve the issue.

jordane commented 9 years ago

@ElijahCaine Please add a LICENSE file with the Apache v2 license, as well as appropriate headers in files.

jordane commented 9 years ago

I'm +1 now.

@ramereth A re-review from you would be nice.