Merge lp:~mbruzek/charms/precise/tomcat/trunk into lp:charms/tomcat

[Matt Bruzek]

A user suggested a different relation name that makes more sense.

This involved updating the metadata.yaml and renaming the relation file.

I also updated the tests to run with our latest best practices.

[Cory Johns (johnsca)]

LGTM

review: Approve

[Matt Bruzek (mbruzek)]

Related bug: https://bugs.launchpad.net/charms/+source/tomcat/+bug/1538715

[Kevin W Monroe (kwmonroe)]

LGTM, merged.

review: Approve

=== added file 'hooks/webapp-container-relation-joined'
--- hooks/webapp-container-relation-joined	1970-01-01 00:00:00 +0000
+++ hooks/webapp-container-relation-joined	2016-01-27 22:05:19 +0000
@@ -0,0 +1,6 @@
+#!/bin/bash
+
+set -ex
+
+VERSION=`config-get tomcat_version`
+relation-set webapp-path /var/lib/$VERSION/webapps

=== removed file 'hooks/webapp-relation-joined'
--- hooks/webapp-relation-joined	2015-01-22 23:19:37 +0000
+++ hooks/webapp-relation-joined	1970-01-01 00:00:00 +0000
@@ -1,6 +0,0 @@
-#!/bin/bash
-
-set -ex
-
-VERSION=`config-get tomcat_version`
-relation-set webapp-path /var/lib/$VERSION/webapps

=== modified file 'metadata.yaml'
--- metadata.yaml	2015-01-22 23:19:37 +0000
+++ metadata.yaml	2016-01-27 22:05:19 +0000
@@ -6,11 +6,15 @@
   JavaServer Pages (JSP) specifications, and provides a "pure Java" HTTP
   web server environment for Java web applications to run.
 
-categories: ["app-servers"]
+tags:
+  - application_development
+  - java
+  - webapp
+
 provides:
   website:
     interface: http
-  webapp:
+  webapp-container:
     interface: java-webapp
     scope: container
 requires:

=== modified file 'tests/10-configured-deploy.py'
--- tests/10-configured-deploy.py	2014-10-10 08:56:18 +0000
+++ tests/10-configured-deploy.py	2016-01-27 22:05:19 +0000
@@ -55,7 +55,7 @@
     'cluster_enabled': True
 }
 
-d = amulet.Deployment()
+d = amulet.Deployment(series='precise')
 # Add the tomcat charm to the deployment.
 d.add('tomcat', units=scale)
 d.add('memcached')
@@ -83,18 +83,18 @@
     raise
 
 # Get the sentry unit for tomcat0.
-tomcat_unit = d.sentry.unit['tomcat/0']
+tomcat_unit = d.sentry['tomcat'][0]
 # Get the public address for tomcat0.
 tomcat0_address = tomcat_unit.info['public-address']
 # Get the terracotta unit.
-terracotta_unit = d.sentry.unit['terracotta/0']
+terracotta_unit = d.sentry['terracotta'][0]
 # Get the relation to terracotta.
 terracotta_relation = terracotta_unit.relation('dso', 'tomcat:jndi-terracotta')
 # Get the private address of terracotta to ensure the relation has that data.
 terracotta_private_address = terracotta_relation['private-address']
 print("terracotta private address " + terracotta_private_address)
 # Get the memcached unit.
-memcached_unit = d.sentry.unit['memcached/0']
+memcached_unit = d.sentry['memcached'][0]
 # Get the relation to memcached.
 memcached_relation = memcached_unit.relation('cache', 'tomcat:jndi-memcached')
 # Get the private address of memcached to ensure the relation has that data.
@@ -118,7 +118,7 @@
 tomcat1_address = address.decode().rstrip('\n')
 
 ###############################################################################
-## Read the /etc/tomcatX/default file to verifiy configuration settings.
+## Read the /etc/tomcatX/default file to verify configuration settings.
 ###############################################################################
 # Read in the default file from the tomcat unit.
 default_tomcat = tomcat_unit.file_contents('/etc/default/{0}'.format(
@@ -169,7 +169,7 @@
 static_member = server_xml[index:].split('\n')[0]
 print(static_member)
 print(tomcat1_address)
-# Search for tomcat1 address in the tomcat0 server.xml to verifiy clustered.
+# Search for tomcat1 address in the tomcat0 server.xml to verify clustered.
 static_member.index(tomcat1_address)
 
 ###############################################################################