Skip to content

Commit 39e392a

Browse files
committed
fix(controller): Cannot routing:disable
1 parent 7d0d97d commit 39e392a

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

rootfs/api/models/app.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -209,10 +209,10 @@ def _refresh_ingress(self, hosts, tls_map, ssl_redirect):
209209
except KubeException as e:
210210
raise ServiceUnavailable('Could not create Ingress in Kubernetes') from e
211211

212-
def refresh(self):
212+
def refresh(self, app_settings=None):
213213
if not getattr(self, 'refresh_enabled', True):
214214
return
215-
app_settings = self.appsettings_set.latest()
215+
app_settings = app_settings if not app_settings else self.appsettings_set.latest()
216216
if not app_settings.routable:
217217
return
218218
tls = self.tls_set.latest()

rootfs/api/models/appsettings.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def _update_routable(self, previous_settings):
5252
old = getattr(previous_settings, 'routable', None)
5353
new = getattr(self, 'routable', None)
5454
# If no previous settings then assume it is the first record and default to true
55-
if not previous_settings:
55+
if previous_settings is None:
5656
setattr(self, 'routable', True)
5757
self.app.routable(True)
5858
# if nothing changed copy the settings from previous
@@ -182,5 +182,5 @@ def save(self, *args, **kwargs):
182182
try:
183183
return super(AppSettings, self).save(**kwargs)
184184
finally:
185-
self.app.refresh()
185+
self.app.refresh(app_settings=self)
186186
self.app.log('summary of app setting changes: {}'.format(summary), logging.DEBUG)

0 commit comments

Comments
 (0)