@@ -1081,9 +1081,10 @@ def _log_config_updated(**kwargs):
10811081
10821082
10831083def _log_domain_added (** kwargs ):
1084- domain = kwargs ['instance' ]
1085- msg = "domain {} added" .format (domain )
1086- log_event (domain .app , msg )
1084+ if kwargs .get ('created' ):
1085+ domain = kwargs ['instance' ]
1086+ msg = "domain {} added" .format (domain )
1087+ log_event (domain .app , msg )
10871088
10881089
10891090def _log_domain_removed (** kwargs ):
@@ -1093,8 +1094,9 @@ def _log_domain_removed(**kwargs):
10931094
10941095
10951096def _log_cert_added (** kwargs ):
1096- cert = kwargs ['instance' ]
1097- logger .info ("cert {} added" .format (cert ))
1097+ if kwargs .get ('created' ):
1098+ cert = kwargs ['instance' ]
1099+ logger .info ("cert {} added" .format (cert ))
10981100
10991101
11001102def _log_cert_removed (** kwargs ):
@@ -1127,10 +1129,13 @@ def _etcd_purge_user(**kwargs):
11271129 pass
11281130
11291131
1130- def _etcd_create_app (** kwargs ):
1132+ def _etcd_publish_app (** kwargs ):
11311133 appname = kwargs ['instance' ]
1132- if kwargs [ 'created' ] :
1134+ try :
11331135 _etcd_client .write ('/deis/services/{}' .format (appname ), None , dir = True )
1136+ except KeyError :
1137+ # Ignore error when the directory already exists.
1138+ pass
11341139
11351140
11361141def _etcd_purge_app (** kwargs ):
@@ -1143,9 +1148,8 @@ def _etcd_purge_app(**kwargs):
11431148
11441149def _etcd_publish_cert (** kwargs ):
11451150 cert = kwargs ['instance' ]
1146- if kwargs ['created' ]:
1147- _etcd_client .write ('/deis/certs/{}/cert' .format (cert ), cert .certificate )
1148- _etcd_client .write ('/deis/certs/{}/key' .format (cert ), cert .key )
1151+ _etcd_client .write ('/deis/certs/{}/cert' .format (cert ), cert .certificate )
1152+ _etcd_client .write ('/deis/certs/{}/key' .format (cert ), cert .key )
11491153
11501154
11511155def _etcd_purge_cert (** kwargs ):
@@ -1167,13 +1171,12 @@ def _etcd_publish_config(**kwargs):
11671171 prevExist = True , dir = True , recursive = True )
11681172 except KeyError :
11691173 pass
1170- if kwargs ['created' ]:
1171- for k , v in config .values .iteritems ():
1172- _etcd_client .write (
1173- '/deis/config/{}/{}' .format (
1174- config .app ,
1175- unicode (k ).encode ('utf-8' ).lower ()),
1176- unicode (v ).encode ('utf-8' ))
1174+ for k , v in config .values .iteritems ():
1175+ _etcd_client .write (
1176+ '/deis/config/{}/{}' .format (
1177+ config .app ,
1178+ unicode (k ).encode ('utf-8' ).lower ()),
1179+ unicode (v ).encode ('utf-8' ))
11771180
11781181
11791182def _etcd_purge_config (** kwargs ):
@@ -1187,8 +1190,7 @@ def _etcd_purge_config(**kwargs):
11871190
11881191def _etcd_publish_domains (** kwargs ):
11891192 domain = kwargs ['instance' ]
1190- if kwargs ['created' ]:
1191- _etcd_client .write ('/deis/domains/{}' .format (domain ), domain .app )
1193+ _etcd_client .write ('/deis/domains/{}' .format (domain ), domain .app )
11921194
11931195
11941196def _etcd_purge_domains (** kwargs ):
@@ -1226,7 +1228,7 @@ def create_auth_token(sender, instance=None, created=False, **kwargs):
12261228 post_delete .connect (_etcd_purge_user , sender = get_user_model (), dispatch_uid = 'api.models' )
12271229 post_save .connect (_etcd_publish_domains , sender = Domain , dispatch_uid = 'api.models' )
12281230 post_delete .connect (_etcd_purge_domains , sender = Domain , dispatch_uid = 'api.models' )
1229- post_save .connect (_etcd_create_app , sender = App , dispatch_uid = 'api.models' )
1231+ post_save .connect (_etcd_publish_app , sender = App , dispatch_uid = 'api.models' )
12301232 post_delete .connect (_etcd_purge_app , sender = App , dispatch_uid = 'api.models' )
12311233 post_save .connect (_etcd_publish_cert , sender = Certificate , dispatch_uid = 'api.models' )
12321234 post_delete .connect (_etcd_purge_cert , sender = Certificate , dispatch_uid = 'api.models' )
0 commit comments