mirror of https://github.com/zulip/zulip.git
tools: Avoid upgrading stripe automatically.
Upgrades to the stripe library can sometimes break semantics for our billing system, and so we should make sure to use our documented testing process before doing them.
This commit is contained in:
parent
56259c7acd
commit
e7507a6eb1
|
@ -62,6 +62,7 @@ def do_upgrade(requirement_file: str, locked_requirement_files: List[str]) -> No
|
|||
pkg_name = package["name"]
|
||||
version = package["version"]
|
||||
latest_version = package["latest_version"]
|
||||
log_file = "{}/{}".format(log_dir, pkg_name)
|
||||
|
||||
if pkg_name in unupgradable_packages:
|
||||
continue
|
||||
|
@ -76,20 +77,26 @@ def do_upgrade(requirement_file: str, locked_requirement_files: List[str]) -> No
|
|||
file_path = "requirements/{}".format(requirement_file)
|
||||
subprocess.check_call(["sed", "-i", sed_string, file_path])
|
||||
|
||||
diff = subprocess.check_output(["git", "diff", file_path])
|
||||
if diff:
|
||||
files_updated = True
|
||||
files_updated = subprocess.check_output(["git", "diff", file_path])
|
||||
|
||||
if not files_updated:
|
||||
continue
|
||||
|
||||
if pkg_name == "stripe":
|
||||
with open(log_file, "w") as f:
|
||||
url = ("https://zulip.readthedocs.io/en/stable/subsystems/billing.html"
|
||||
"#upgrading-stripe-api-versions")
|
||||
f.write("Stripe upgrade available. Upgrade manually. Make sure to update fixtures as well.")
|
||||
f.write(url)
|
||||
continue
|
||||
|
||||
if files_updated:
|
||||
try:
|
||||
print("Trying to upgrade {}".format(pkg_name))
|
||||
prepare_for_commit()
|
||||
commit_msg = "requirements: Upgrade {} from {} to {}.".format(pkg_name, version,
|
||||
latest_version)
|
||||
commit_msg = "requirements: Upgrade {} from {} to {}.".format(pkg_name, version, latest_version)
|
||||
commit_and_push(commit_msg)
|
||||
except subprocess.CalledProcessError as e:
|
||||
print("{} upgrade failed".format(pkg_name))
|
||||
log_file = "{}/{}".format(log_dir, pkg_name)
|
||||
with open(log_file, "w") as f:
|
||||
f.write(e.output.decode("utf-8"))
|
||||
|
||||
|
|
Loading…
Reference in New Issue