mirror of
				https://github.com/zulip/zulip.git
				synced 2025-11-04 05:53:43 +00:00 
			
		
		
		
	During the upgrade process of a postgresql-only Zulip installation, (`puppet_classes = zulip::profile::postgresql` in `/etc/zulip/zulip.conf`) either `scripts/start-server` or `scripts/stop-server` fail because they try to handle supervisor services that are not available (e.g. Tornado) since only `/etc/supervisor/conf.d/zulip/zulip_db.conf` is present and not `/etc/supervisor/conf.d/zulip/zulip.conf`. While this wasn't previously supported, it's a pretty reasonable thing to do, and can be readily supported by just adding a few conditionals.
		
			
				
	
	
		
			49 lines
		
	
	
		
			1.6 KiB
		
	
	
	
		
			Python
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			49 lines
		
	
	
		
			1.6 KiB
		
	
	
	
		
			Python
		
	
	
		
			Executable File
		
	
	
	
	
#!/usr/bin/env python3
 | 
						|
import logging
 | 
						|
import os
 | 
						|
import pwd
 | 
						|
import subprocess
 | 
						|
import sys
 | 
						|
import time
 | 
						|
 | 
						|
sys.path.append(os.path.join(os.path.dirname(__file__), ".."))
 | 
						|
from scripts.lib.zulip_tools import ENDC, OKGREEN, WARNING, has_application_server
 | 
						|
 | 
						|
deploy_path = os.path.realpath(os.path.join(os.path.dirname(__file__), ".."))
 | 
						|
os.chdir(deploy_path)
 | 
						|
 | 
						|
if pwd.getpwuid(os.getuid()).pw_name != "zulip":
 | 
						|
    logging.error("Must be run as user 'zulip'.")
 | 
						|
    sys.exit(1)
 | 
						|
 | 
						|
logging.Formatter.converter = time.gmtime
 | 
						|
logging.basicConfig(format="%(asctime)s stop-server: %(message)s", level=logging.INFO)
 | 
						|
 | 
						|
services = []
 | 
						|
 | 
						|
# Start with the least-critical services:
 | 
						|
if os.path.exists("/etc/supervisor/conf.d/zulip/zulip_db.conf"):
 | 
						|
    services.append("process-fts-updates")
 | 
						|
 | 
						|
if has_application_server():
 | 
						|
    # Contrary to the order in (re)start-server, we stop django before the
 | 
						|
    # workers, to increase the chance that we finish processing any work
 | 
						|
    # that may have been enqueued by the Django, leaving the final state
 | 
						|
    # closer to "empty."  We stop Django before Tornado so it doesn't try
 | 
						|
    # to make requests to make queues with a down'd Tornado.
 | 
						|
    services.append("zulip-django")
 | 
						|
    services.extend(["zulip-tornado", "zulip-tornado:*"])
 | 
						|
    services.append("zulip-workers:*")
 | 
						|
 | 
						|
subprocess.check_call(["supervisorctl", "stop", *services])
 | 
						|
 | 
						|
print()
 | 
						|
print(OKGREEN + "Zulip stopped successfully!" + ENDC)
 | 
						|
 | 
						|
using_sso = subprocess.check_output(["./scripts/get-django-setting", "USING_APACHE_SSO"])
 | 
						|
if using_sso.strip() == b"True":
 | 
						|
    print()
 | 
						|
    print(WARNING + "Apache2 needs to be shut down; as root, run:" + ENDC)
 | 
						|
    print("    service apache2 stop")
 | 
						|
    print()
 |