Andrew Bogott has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/280578

Change subject: Proxy panel:  More . juggling
......................................................................

Proxy panel:  More . juggling

Now that the proxy api stores domains without a terminal '.'
we need to add it back on when reading the domains in
and before passing them off to designate.

Change-Id: I1c85e3d2870e63b4cc393b2babf7a544c50b54a8
---
M modules/openstack/files/liberty/horizon/proxy/views.py
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/78/280578/1

diff --git a/modules/openstack/files/liberty/horizon/proxy/views.py 
b/modules/openstack/files/liberty/horizon/proxy/views.py
index 92d2b21..2614c7e 100644
--- a/modules/openstack/files/liberty/horizon/proxy/views.py
+++ b/modules/openstack/files/liberty/horizon/proxy/views.py
@@ -65,6 +65,8 @@
     def delete(self, request, obj_id):
         record = obj_id[:obj_id.find('.')]
         domain = obj_id[obj_id.find('.') + 1:]
+        if not domain.endswith('.'):
+            domain += '.'
 
         # First let's make sure that this proxy is really ours to delete.
         fqdn = "%s.%s" % (record, domain)

-- 
To view, visit https://gerrit.wikimedia.org/r/280578
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c85e3d2870e63b4cc393b2babf7a544c50b54a8
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to