import propose-final-releases command from release_tools
Change-Id: I628edccf8b9327d8dcebb3044a6b181728ba0d95 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
This commit is contained in:
parent
2010a24353
commit
4447e00585
13
README.rst
13
README.rst
@ -490,3 +490,16 @@ To set the pre-release group membership:
|
||||
::
|
||||
|
||||
tox -e aclmanager -- groups pre_release ttx
|
||||
|
||||
propose-final-releases
|
||||
----------------------
|
||||
|
||||
Command to edit the deliverable files in a releases repository to
|
||||
propose final releases. The command modifies files in an existing copy
|
||||
of the repository and does not invoke git at all, so you need to
|
||||
create a branch before running it then review the output, commit the
|
||||
changes, and push the patch to gerrit.
|
||||
|
||||
::
|
||||
|
||||
tox -e venv -- propose-final-releases newton ocata
|
||||
|
209
openstack_releases/cmds/propose_final_releases.py
Normal file
209
openstack_releases/cmds/propose_final_releases.py
Normal file
@ -0,0 +1,209 @@
|
||||
# All Rights Reserved.
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
||||
# not use this file except in compliance with the License. You may obtain
|
||||
# a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
||||
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
||||
# License for the specific language governing permissions and limitations
|
||||
# under the License.
|
||||
|
||||
from __future__ import print_function
|
||||
|
||||
import argparse
|
||||
import atexit
|
||||
import glob
|
||||
import os.path
|
||||
import re
|
||||
import shutil
|
||||
import tempfile
|
||||
|
||||
import yaml
|
||||
|
||||
import openstack_releases
|
||||
from openstack_releases import gitutils
|
||||
from openstack_releases import governance
|
||||
|
||||
PROJECT_TEMPLATE = '''\
|
||||
- repo: {repo}
|
||||
hash: {hash}'''
|
||||
|
||||
VERSION_TEMPLATE = '''\
|
||||
- version: {version}
|
||||
{diff_start_comment}diff-start: {diff_start}
|
||||
projects:
|
||||
{projects}
|
||||
'''
|
||||
|
||||
PRE_RELEASE = re.compile('(a|b|rc)')
|
||||
|
||||
|
||||
def get_prior_branch_point(workdir, repo, branch):
|
||||
"""Return the tag of the base of the branch.
|
||||
|
||||
The diff-start is the old version is the tag on the commit where
|
||||
we created the branch. To determine that, we need to clone the
|
||||
repo and look at the branch.
|
||||
|
||||
See
|
||||
http://lists.openstack.org/pipermail/openstack-dev/2016-October/104901.html
|
||||
for a better description of what the desired tag info is.
|
||||
|
||||
"""
|
||||
gitutils.clone_repo(workdir, repo)
|
||||
branch_base = gitutils.get_branch_base(
|
||||
workdir, repo, branch,
|
||||
)
|
||||
if branch_base:
|
||||
return gitutils.get_latest_tag(
|
||||
workdir, repo, branch_base,
|
||||
)
|
||||
# Work backwards from the most recent commit looking for the first
|
||||
# version that is not a pre-release, and assume that is the
|
||||
# previous release on a non-branching repository like for the
|
||||
# os-*-config tools.
|
||||
start = None
|
||||
while True:
|
||||
print(' looking for version before {}'.format(start))
|
||||
version = gitutils.get_latest_tag(workdir, repo, start)
|
||||
if not version:
|
||||
return None
|
||||
if not PRE_RELEASE.search(version):
|
||||
return version
|
||||
start = '{}^'.format(version)
|
||||
return version
|
||||
|
||||
|
||||
def main():
|
||||
parser = argparse.ArgumentParser()
|
||||
parser.add_argument(
|
||||
'--no-cleanup',
|
||||
dest='cleanup',
|
||||
default=True,
|
||||
action='store_false',
|
||||
help='do not remove temporary files',
|
||||
)
|
||||
parser.add_argument(
|
||||
'--all',
|
||||
default=False,
|
||||
action='store_true',
|
||||
help='process all deliverables, including release:cycle-trailing',
|
||||
)
|
||||
parser.add_argument(
|
||||
'--verbose', '-v',
|
||||
action='store_true',
|
||||
default=False,
|
||||
help='produce detailed output',
|
||||
)
|
||||
parser.add_argument(
|
||||
'--deliverables-dir',
|
||||
default=openstack_releases.deliverable_dir,
|
||||
help='location of deliverable files',
|
||||
)
|
||||
parser.add_argument(
|
||||
'prior_series',
|
||||
help='the name of the previous series',
|
||||
)
|
||||
parser.add_argument(
|
||||
'series',
|
||||
help='the name of the release series to work on'
|
||||
)
|
||||
args = parser.parse_args()
|
||||
|
||||
if args.verbose:
|
||||
def verbose(msg):
|
||||
print(msg)
|
||||
else:
|
||||
def verbose(msg):
|
||||
pass
|
||||
|
||||
deliverables_dir = args.deliverables_dir
|
||||
|
||||
team_data = governance.get_team_data()
|
||||
teams = [
|
||||
governance.Team(n, i)
|
||||
for n, i in team_data.items()
|
||||
]
|
||||
deliverables = {
|
||||
d.name: d
|
||||
for t in teams
|
||||
for d in t.deliverables.values()
|
||||
}
|
||||
|
||||
workdir = tempfile.mkdtemp(prefix='releases-')
|
||||
print('creating temporary files in %s' % workdir)
|
||||
|
||||
def cleanup_workdir():
|
||||
if args.cleanup:
|
||||
try:
|
||||
shutil.rmtree(workdir)
|
||||
except Exception:
|
||||
pass
|
||||
else:
|
||||
print('not cleaning up %s' % workdir)
|
||||
atexit.register(cleanup_workdir)
|
||||
|
||||
pattern = os.path.join(deliverables_dir,
|
||||
args.series, '*.yaml')
|
||||
verbose('Scanning {}'.format(pattern))
|
||||
deliverable_files = sorted(glob.glob(pattern))
|
||||
|
||||
for filename in deliverable_files:
|
||||
verbose('\n{}'.format(filename))
|
||||
deliverable_name = os.path.basename(filename)[:-5]
|
||||
with open(filename, 'r') as f:
|
||||
deliverable_data = yaml.safe_load(f)
|
||||
releases = deliverable_data.get('releases')
|
||||
if not releases:
|
||||
verbose('# no releases')
|
||||
continue
|
||||
latest_release = releases[-1]
|
||||
projects = latest_release.get('projects')
|
||||
if not projects:
|
||||
verbose('# no projects')
|
||||
continue
|
||||
for pre_rel in ['a', 'b', 'rc']:
|
||||
if pre_rel in str(latest_release['version']):
|
||||
break
|
||||
else: # we did not find any pre_rel
|
||||
verbose('# not a release candidate')
|
||||
continue
|
||||
deliverable = deliverables.get(deliverable_name)
|
||||
if deliverable and 'release:cycle-trailing' in deliverable.tags:
|
||||
verbose('# {} is a cycle-trailing project'.format(deliverable_name))
|
||||
if not args.all:
|
||||
continue
|
||||
# The new version is the same as the latest release version
|
||||
# without the pre-release component at the end. Make sure it
|
||||
# has 3 sets of digits.
|
||||
new_version = '.'.join(
|
||||
(latest_release['version'].split('.')[:-1] + ['0'])[:3]
|
||||
)
|
||||
branch = 'stable/{}'.format(args.prior_series)
|
||||
diff_start = get_prior_branch_point(
|
||||
workdir, projects[0]['repo'], branch,
|
||||
)
|
||||
deliverable_data['releases'].append({
|
||||
'version': new_version,
|
||||
'diff_start': diff_start,
|
||||
'projects': latest_release['projects'],
|
||||
})
|
||||
print('new version for {}: {}'.format(os.path.basename(filename),
|
||||
new_version))
|
||||
|
||||
# NOTE(dhellmann): PyYAML doesn't preserve layout when you
|
||||
# write the data back out, so do the formatting ourselves.
|
||||
projects = '\n'.join(PROJECT_TEMPLATE.format(**p)
|
||||
for p in latest_release['projects'])
|
||||
new_block = VERSION_TEMPLATE.format(
|
||||
version=new_version,
|
||||
diff_start=diff_start,
|
||||
diff_start_comment=('# ' if diff_start is None else ''),
|
||||
projects=projects,
|
||||
).rstrip() + '\n'
|
||||
with open(filename, 'a') as f:
|
||||
f.write(new_block)
|
@ -34,6 +34,7 @@ console_scripts =
|
||||
init-series = openstack_releases.cmds.init_series:main
|
||||
list-liaisons = openstack_releases.wiki:main
|
||||
get-deliverable-owner = openstack_releases.cmds.get_deliverable_owner:main
|
||||
propose-final-releases = openstack_releases.cmds.propose_final_releases:main
|
||||
|
||||
[extras]
|
||||
sphinxext =
|
||||
|
Loading…
Reference in New Issue
Block a user