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

Add WSGIApplicationGroup and WSGIImportScript directives #606

Merged
merged 2 commits into from
Feb 11, 2014

Conversation

bodgit
Copy link
Contributor

@bodgit bodgit commented Feb 4, 2014

These were the two WSGI options I was missing, might go part way to chipping at #476.

@blkperl
Copy link
Contributor

blkperl commented Feb 4, 2014

Can you add some spec tests? Thanks.

@blkperl
Copy link
Contributor

blkperl commented Feb 6, 2014

This no longer merges cleanly, please rebase this branch against master. Thanks.

@bodgit
Copy link
Contributor Author

bodgit commented Feb 7, 2014

Done. The tests do pass, the error bombed on a couple of the Puppet/Ruby permutations for taking longer than 50 mins to complete, but a couple of them succeeded.

@igalic
Copy link
Contributor

igalic commented Feb 7, 2014

We're still missing spec/acceptance tests…

@bodgit
Copy link
Contributor Author

bodgit commented Feb 11, 2014

Anything else needed here? The Travis CI build errors are again because some permutations time out after 50 minutes.

@igalic
Copy link
Contributor

igalic commented Feb 11, 2014

The only thing left is aesthetics. Can you please squash the "stray comma" commit?

@bodgit
Copy link
Contributor Author

bodgit commented Feb 11, 2014

Done

igalic added a commit that referenced this pull request Feb 11, 2014
Add WSGIApplicationGroup and WSGIImportScript directives
@igalic igalic merged commit 0c737e1 into puppetlabs:master Feb 11, 2014
@bodgit bodgit deleted the wsgi branch February 12, 2014 11:13
@bodgit bodgit restored the wsgi branch April 2, 2014 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants