Merge lp:~james-w/python-oops-tools/prod-deploy into lp:python-oops-tools

[James Westby]

Add django_openid_auth.

Our production deployment hacks things such that django_openid_auth is used.
This adds it for real.

In addition it adds the common pattern of allowing a local_settings.py
that overrides values in settings.py so that prod can configure things like
the SSO server and the team mappings.

Hi,

This is a step towards cleaning up the prod deploy.

Currently prod runs non-trunk code as it changes a couple of files
to add openid support.

This branch adds the basic support for openid, and allows for configuration
in prod without having to change versioned files. The commit message
has more rationale about the individual changes.

Thanks,

James

[Martin Albisetti (beuno)]

review: Approve

[Launchpad PQM Bot (launchpad-pqm)]

The attempt to merge lp:~james-w/python-oops-tools/prod-deploy into lp:python-oops-tools failed.Below is the output from the failed tests.

find . -type f -name '*.py[co]' -exec rm -f {} \;
rm -f bin/buildout
rm -f src/oopstools/settings.py
bzr clean-tree --unknown --force
bzr up download-cache
python bootstrap.py \
  --setup-source=download-cache/ez_setup.py \
  --download-base=download-cache/dist --eggs=eggs
Download error: [Errno 111] Connection refused -- Some packages may not be found!
Download error: [Errno 111] Connection refused -- Some packages may not be found!
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/buildout'.
bin/buildout configuration:db-port=5433
Develop: '/home/pqm/pqm-workdir/oops-tools/.'
Couldn't find index page for 'django-openid-auth' (maybe misspelled?)
Getting distribution for 'django-openid-auth==0.4'.

W: line 4 [buildbot-staging]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
W: line 2 [pqm]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
W: line 3 [pqm-oops-tools]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
W: line 1 [precise]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
I: [pqm-oops-tools chroot] Running command: "cd /home/pqm/pqm-workdir/oops-tools && make check"
Nothing to delete.
Tree is up to date at revision 533 of branch http://bazaar.launchpad.net/~launchpad/lp-source-dependencies/trunk
While:
  Installing.
  Getting section filetemplates.
  Initializing part filetemplates.
  Getting distribution for 'django-openid-auth==0.4'.
Error: Couldn't find a distribution for 'django-openid-auth==0.4'.
make: *** [bin/buildout] Error 1

[James Westby (james-w)]

Oh yeah, I can't land this until we fix the filename of django-openid-auth in the download-cache.

Thanks,

James

[Launchpad PQM Bot (launchpad-pqm)]

The attempt to merge lp:~james-w/python-oops-tools/prod-deploy into lp:python-oops-tools failed.Below is the output from the failed tests.

find . -type f -name '*.py[co]' -exec rm -f {} \;
rm -f bin/buildout
rm -f src/oopstools/settings.py
bzr clean-tree --unknown --force
bzr up download-cache
python bootstrap.py \
  --setup-source=download-cache/ez_setup.py \
  --download-base=download-cache/dist --eggs=eggs
Download error: [Errno 111] Connection refused -- Some packages may not be found!
Download error: [Errno 111] Connection refused -- Some packages may not be found!
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/buildout'.
bin/buildout configuration:db-port=5433
Develop: '/home/pqm/pqm-workdir/oops-tools/.'
Getting distribution for 'django-openid-auth==0.4'.
zip_safe flag not set; analyzing archive contents...
Got django-openid-auth 0.4.
Uninstalling filetemplates.
Updating scripts.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/load_sample_data'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/prune'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/analyse_error_reports'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/dir_finder'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/amqp2disk'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/update_db'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/report'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/graph_report'.
Generated interpreter '/home/pqm/pqm-workdir/oops-tools/bin/py'.
Installing filetemplates.
Updating django.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/django'.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/test'.
Updating docs.
Generated script '/home/pqm/pqm-workdir/oops-tools/bin/docs'.
Updating tags.
bin/test
Creating test database for alias 'default'...
Destroying test database for alias 'default'...

W: line 4 [buildbot-staging]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
W: line 2 [pqm]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
W: line 3 [pqm-oops-tools]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
W: line 1 [precise]: Deprecated key 'location' used
I: This option will be removed in the future; please update your configuration
I: [pqm-oops-tools chroot] Running command: "cd /home/pqm/pqm-workdir/oops-tools && make check"
Nothing to delete.
R dist/django-openid-auth_0.4.tar.gz => dist/django-openid-auth-0.4.tar.gz
All changes applied successfully.
Updated to revision 534 of branch http://bazaar.launchpad.net/~launchpad/lp-source-dependencies/trunk
F......F.....FFF.....................EEEEE
======================================================================
ERROR: test_inactive_report (test_report.ReportTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/pqm/pqm-workdir/oops-tools/src/oopstools/oops/test/test_report.py", line 39, in test_inactive_report
    resp = self.client.get('/reports/%s/' % (report_name...

[lp:~james-w/python-oops-tools/prod-deploy]

54. By James Westby on 2012-12-06

Add python-openid as django_openid_auth doesn't depend on it.

=== modified file 'buildout-templates/src/oopstools/settings.py.in'
--- buildout-templates/src/oopstools/settings.py.in	2012-09-23 00:52:28 +0000
+++ buildout-templates/src/oopstools/settings.py.in	2012-12-06 22:14:36 +0000
@@ -111,8 +111,9 @@
     'django.contrib.sessions',
     'django.contrib.sites',
     'django.contrib.admin',
+    'django_openid_auth',
     'oopstools.oops',
-    'south'
+    'south',
 )
 
 OOPS_WEB_AMQP_HOST = '${configuration:oops-web-amqp-host}'
@@ -121,3 +122,15 @@
 OOPS_WEB_AMQP_VHOST = '${configuration:oops-web-amqp-vhost}'
 OOPS_WEB_AMQP_EXCHANGE = '${configuration:oops-web-amqp-exchange}'
 OOPS_WEB_AMQP_ROUTING = '${configuration:oops-web-amqp-routing}'
+
+AUTHENTICATION_BACKENDS = (
+    'django.contrib.auth.backends.ModelBackend',
+    'django_openid_auth.auth.OpenIDBackend',
+)
+LOGIN_URL = '/openid/login/'
+LOGIN_REDIRECT_URL = '/admin/'
+
+try:
+    from oopstools.local_settings import *
+except ImportError:
+    pass
 
=== renamed file 'production.cfg' => 'production.cfg.example'
--- production.cfg	2011-10-13 20:18:51 +0000
+++ production.cfg.example	2012-12-06 22:14:36 +0000
@@ -1,4 +1,5 @@
-# Buildout configuration for the production deployment
+# Example Buildout configuration for a production deployment.
+# Rename the file to production.cfg and fill in the details.
 [buildout]
 extends = buildout.cfg
 
 
=== modified file 'setup.py'
--- setup.py	2012-09-19 03:44:41 +0000
+++ setup.py	2012-12-06 22:14:36 +0000
@@ -51,6 +51,7 @@
     install_requires=[
         'BeautifulSoup',
         'django',
+        'django_openid_auth',
         'fixtures',
         'launchpadlib',
         'lazr.config',
@@ -60,6 +61,7 @@
         'oops-wsgi',
         'oops-timeline',
         'psycopg2',
+        'python-openid',
         'pytz',
         'setuptools',
         'South',
 
=== modified file 'src/oopstools/urls.py'
--- src/oopstools/urls.py	2011-10-13 20:18:51 +0000
+++ src/oopstools/urls.py	2012-12-06 22:14:36 +0000
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU Affero General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-from django.conf.urls.defaults import *
+from django.conf.urls.defaults import patterns, include, url
 from django.conf import settings
 
 # Uncomment the next two lines to enable the admin:
@@ -24,7 +24,7 @@
     # Example:
     # (r'^oopstools/', include('oopstools.foo.urls')),
 
-    # Uncomment the admin/doc line below and add 'django.contrib.admindocs' 
+    # Uncomment the admin/doc line below and add 'django.contrib.admindocs'
     # to INSTALLED_APPS to enable admin documentation:
     # (r'^admin/doc/', include('django.contrib.admindocs.urls')),
 
@@ -39,4 +39,5 @@
     (r'^oops[.py]*/meta$', 'oopstools.oops.views.meta'),
     (r'^oops/static/(?P<path>.*)$', 'django.views.static.serve',
         {'document_root': settings.STATIC_DOC_ROOT}),
+    url(r'^auth/', include('django_openid_auth.urls')),
 )
 
=== modified file 'versions.cfg'
--- versions.cfg	2012-12-06 21:01:15 +0000
+++ versions.cfg	2012-12-06 22:14:36 +0000
@@ -14,6 +14,7 @@
 Sphinx = 1.0.7
 sqlparse = 0.1.3
 django = 1.3.3
+django-openid-auth = 0.4
 djangorecipe = 1.3
 distribute = 0.6.12
 docutils = 0.6