salt/tools/changelog.py
Pedro Algarvio 4548b57365 On dev versions, choose the latest available release notes file
Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
2023-02-28 07:40:38 +00:00

372 lines
12 KiB
Python

"""
These commands are used manage Salt's changelog.
"""
# pylint: disable=resource-leakage,broad-except
from __future__ import annotations
import datetime
import logging
import os
import pathlib
import re
import sys
import textwrap
from ptscripts import Context, command_group
from tools.utils import REPO_ROOT, Version
CHANGELOG_LIKE_RE = re.compile(r"([\d]+)\.([a-z]+)$")
CHANGELOG_TYPES = (
"removed",
"deprecated",
"changed",
"fixed",
"added",
"security",
)
CHANGELOG_ENTRY_RE = re.compile(
r"([\d]+|(CVE|cve)-[\d]{{4}}-[\d]+)\.({})(\.md)?$".format("|".join(CHANGELOG_TYPES))
)
log = logging.getLogger(__name__)
# Define the command group
changelog = command_group(
name="changelog",
help="Changelog tools",
description=__doc__,
venv_config={
"requirements_files": [
REPO_ROOT
/ "requirements"
/ "static"
/ "ci"
/ "py{}.{}".format(*sys.version_info)
/ "changelog.txt"
],
},
)
@changelog.command(
name="pre-commit-checks",
arguments={
"files": {
"nargs": "*",
}
},
)
def check_changelog_entries(ctx: Context, files: list[pathlib.Path]):
"""
Run pre-commit checks on changelog snippets.
"""
docs_path = REPO_ROOT / "doc"
tests_integration_files_path = REPO_ROOT / "tests" / "integration" / "files"
changelog_entries_path = REPO_ROOT / "changelog"
exitcode = 0
for entry in files:
path = pathlib.Path(entry).resolve()
# Is it under changelog/
try:
path.relative_to(changelog_entries_path)
if path.name in (".keep", ".template.jinja"):
# This is the file we use so git doesn't delete the changelog/ directory
continue
# Is it named properly
if not CHANGELOG_ENTRY_RE.match(path.name):
ctx.error(
"The changelog entry '{}' should have one of the following extensions: {}.".format(
path.relative_to(REPO_ROOT),
", ".join(f"{ext}.md" for ext in CHANGELOG_TYPES),
),
)
exitcode = 1
continue
if path.suffix != ".md":
ctx.error(
f"Please rename '{path.relative_to(REPO_ROOT)}' to "
f"'{path.relative_to(REPO_ROOT)}.md'"
)
exitcode = 1
continue
except ValueError:
# No, carry on
pass
# Does it look like a changelog entry
if CHANGELOG_LIKE_RE.match(path.name) and not CHANGELOG_ENTRY_RE.match(
path.name
):
try:
# Is this under doc/
path.relative_to(docs_path)
# Yes, carry on
continue
except ValueError:
# No, resume the check
pass
try:
# Is this under tests/integration/files
path.relative_to(tests_integration_files_path)
# Yes, carry on
continue
except ValueError:
# No, resume the check
pass
ctx.error(
"The changelog entry '{}' should have one of the following extensions: {}.".format(
path.relative_to(REPO_ROOT),
", ".join(f"{ext}.md" for ext in CHANGELOG_TYPES),
)
)
exitcode = 1
continue
# Is it a changelog entry
if not CHANGELOG_ENTRY_RE.match(path.name):
# No? Carry on
continue
# Is the changelog entry in the right path?
try:
path.relative_to(changelog_entries_path)
except ValueError:
exitcode = 1
ctx.error(
"The changelog entry '{}' should be placed under '{}/', not '{}'".format(
path.name,
changelog_entries_path.relative_to(REPO_ROOT),
path.relative_to(REPO_ROOT).parent,
)
)
if path.suffix != ".md":
ctx.error(
f"Please rename '{path.relative_to(REPO_ROOT)}' to "
f"'{path.relative_to(REPO_ROOT)}.md'"
)
exitcode = 1
ctx.exit(exitcode)
def _get_changelog_contents(ctx: Context, version: Version):
"""
Return the full changelog generated by towncrier.
"""
ret = ctx.run(
"towncrier",
"build",
"--draft",
f"--version={version}",
capture=True,
check=False,
)
if ret.returncode:
ctx.error(ret.stderr.decode())
ctx.exit(1)
return ret.stdout.decode()
def _get_pkg_changelog_contents(ctx: Context, version: Version):
"""
Return a version of the changelog entries suitable for packaged changelogs.
"""
changes = _get_changelog_contents(ctx, version)
changes = "\n".join(changes.split("\n")[2:])
changes = changes.replace("### ", "# ").replace("\n\n\n", "\n\n")
return changes
def _get_salt_version(ctx):
ret = ctx.run("python3", "salt/version.py", capture=True, check=False)
if ret.returncode:
ctx.error(ret.stderr.decode())
ctx.exit(1)
return Version(ret.stdout.decode().strip())
@changelog.command(
name="update-rpm",
arguments={
"salt_version": {
"help": (
"The salt package version. If not passed "
"it will be discovered by running 'python3 salt/version.py'."
),
"nargs": "?",
"default": None,
},
"draft": {
"help": "Do not make any changes, instead output what would be changed.",
},
},
)
def update_rpm(ctx: Context, salt_version: Version, draft: bool = False):
if salt_version is None:
salt_version = _get_salt_version(ctx)
changes = _get_pkg_changelog_contents(ctx, salt_version)
ctx.info(f"Salt version is {salt_version}")
orig = ctx.run(
"sed",
f"s/Version: .*/Version: {salt_version}/g",
"pkg/rpm/salt.spec",
capture=True,
check=True,
).stdout.decode()
dt = datetime.datetime.utcnow()
date = dt.strftime("%a %b %d %Y")
header = f"* {date} Salt Project Packaging <saltproject-packaging@vmware.com> - {salt_version}\n"
parts = orig.split("%changelog")
tmpspec = "pkg/rpm/salt.spec.1"
with open(tmpspec, "w") as wfp:
wfp.write(parts[0])
wfp.write("%changelog\n")
wfp.write(header)
wfp.write(changes)
wfp.write(parts[1])
try:
with open(tmpspec) as rfp:
if draft:
ctx.info(rfp.read())
else:
with open("pkg/rpm/salt.spec", "w") as wfp:
wfp.write(rfp.read())
finally:
os.remove(tmpspec)
@changelog.command(
name="update-deb",
arguments={
"salt_version": {
"help": (
"The salt package version. If not passed "
"it will be discovered by running 'python3 salt/version.py'."
),
"nargs": "?",
"default": None,
},
"draft": {
"help": "Do not make any changes, instead output what would be changed.",
},
},
)
def update_deb(ctx: Context, salt_version: Version, draft: bool = False):
if salt_version is None:
salt_version = _get_salt_version(ctx)
changes = _get_pkg_changelog_contents(ctx, salt_version)
formated = "\n".join([f" {_.replace('-', '*', 1)}" for _ in changes.split("\n")])
dt = datetime.datetime.utcnow()
date = dt.strftime("%a, %d %b %Y %H:%M:%S +0000")
tmpchanges = "pkg/rpm/salt.spec.1"
debian_changelog_path = "pkg/debian/changelog"
tmp_debian_changelog_path = f"{debian_changelog_path}.1"
with open(tmp_debian_changelog_path, "w") as wfp:
wfp.write(f"salt ({salt_version}) stable; urgency=medium\n\n")
wfp.write(formated)
wfp.write(
f"\n -- Salt Project Packaging <saltproject-packaging@vmware.com> {date}\n\n"
)
with open(debian_changelog_path) as rfp:
wfp.write(rfp.read())
try:
with open(tmp_debian_changelog_path) as rfp:
if draft:
ctx.info(rfp.read())
else:
with open(debian_changelog_path, "w") as wfp:
wfp.write(rfp.read())
finally:
os.remove(tmp_debian_changelog_path)
@changelog.command(
name="update-release-notes",
arguments={
"salt_version": {
"help": (
"The salt version used to generate the release notes. If not passed "
"it will be discovered by running 'python3 salt/version.py'."
),
"nargs": "?",
"default": None,
},
"draft": {
"help": "Do not make any changes, instead output what would be changed.",
},
"release": {
"help": "Update for an actual release and not just a temporary CI build.",
},
},
)
def update_release_notes(
ctx: Context, salt_version: Version, draft: bool = False, release: bool = False
):
if salt_version is None:
salt_version = _get_salt_version(ctx)
changes = _get_changelog_contents(ctx, salt_version)
changes = "\n".join(changes.split("\n")[2:])
if salt_version.local:
# This is a dev release, let's pick up the latest changelog file
versions = {}
for fpath in pathlib.Path("doc/topics/releases").glob("*.md"):
versions[(Version(fpath.stem))] = fpath
release_notes_path = versions[sorted(versions)[-1]]
else:
release_notes_path = pathlib.Path("doc/topics/releases") / "{}.md".format(
".".join(str(part) for part in salt_version.release)
)
if not release_notes_path.exists():
release_notes_path.write_text(
textwrap.dedent(
f"""\
(release-{salt_version})=
# Salt {salt_version} release notes - UNRELEASED
"""
)
)
ctx.run("git", "add", str(release_notes_path))
ctx.info(f"Created bare {release_notes_path} release notes file")
existing = release_notes_path.read_text()
if release is True:
existing = existing.replace(" - UNRELEASED", "")
tmp_release_notes_path = (
release_notes_path.parent / f"{release_notes_path.name}.tmp"
)
tmp_release_notes_path.write_text(f"{existing}\n## Changelog\n{changes}")
try:
contents = tmp_release_notes_path.read_text().strip()
if draft:
ctx.print(contents, soft_wrap=True)
else:
release_notes_path.write_text(contents)
finally:
os.remove(tmp_release_notes_path)
@changelog.command(
name="update-changelog-md",
arguments={
"salt_version": {
"help": (
"The salt version to use in the changelog. If not passed "
"it will be discovered by running 'python3 salt/version.py'."
),
"nargs": "?",
"default": None,
},
"draft": {
"help": "Do not make any changes, instead output what would be changed.",
},
},
)
def generate_changelog_md(ctx: Context, salt_version: Version, draft: bool = False):
if salt_version is None:
salt_version = _get_salt_version(ctx)
cmd = ["towncrier", "build", f"--version={salt_version}"]
if draft:
cmd += ["--draft"]
else:
cmd += ["--yes"]
ctx.run(*cmd, check=True)
ctx.run("git", "restore", "--staged", "CHANGELOG.md", "changelog/", check=True)