search

cloudsoft / brooklyn-social-apps

Brooklyn templates for social web site applications
Other
0 stars 6 forks source link

Wordpress #6

Closed aledsage closed 11 years ago

aledsage commented 11 years ago

ready for review, and hopefully merging

Adds a Wordpress entity, and an example of clustering that with an nginx load balancer.

Also converts Drupal to run with brooklyn 0.5.0-M2

buildhive commented 11 years ago

Cloudsoft » brooklyn-social-apps #30 SUCCESS This pull request looks good (what's this?)

pveentjer commented 11 years ago

What should be done with this pull request? (It clearly indicates that it shouldn't be merged yet).

buildhive commented 11 years ago

Cloudsoft » brooklyn-social-apps #33 SUCCESS This pull request looks good (what's this?)

aledsage commented 11 years ago

The code needs to be reviewed, and good if someone can confirm that it works for them as well. I need to further investigate some additional config required to get nginx to work with a wordpress cluster - the WordpressClusterEc2LiveTest fails because of this - see comment at top of that class).

If we can get a 0.5.0-M2 released soon (it's so overdue!) then we can merge this pull request soon as well.

buildhive commented 11 years ago

Cloudsoft » brooklyn-social-apps #37 FAILURE Looks like there's a problem with this pull request (what's this?)

buildhive commented 11 years ago

Cloudsoft » brooklyn-social-apps #40 SUCCESS This pull request looks good (what's this?)

buildhive commented 11 years ago

Cloudsoft » brooklyn-social-apps #42 SUCCESS This pull request looks good (what's this?)

aledsage commented 11 years ago

Ready for review, and hopefully merging. @pveentjer can you take a look when you get a chance.

pveentjer commented 11 years ago

Hi Aled, do all the integration tests run again?

aledsage commented 11 years ago

The DrupalLiveTest ran (previously named DrupalTest), and the ClusteredWordpressEc2LiveTest worked too. I tested the wordpress apps, but not the drupal app.