openstack/releases
Code Issues Proposed changes

Do not list em and eol tags on releases site

Browse Source 
We want these tags in git for developers and those working directly with
the source code, but we do not want to have these listed on the releases
site. Consumers of the site are just looking for actual official
releases they can use, so we should avoid confusion by not including
these "internal" tags.

Change-Id: I6a3fcc11efed6254ea6fb95579eafe5288508661
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
This commit is contained in:
Sean McGinnis
2019-12-04 11:18:02 -06:00
parent d051863fd9
commit fdbed3a126
1 changed files with 19 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
doc/source/_exts/deliverables.py
View File

@@ -227,10 +227,18 @@ class DeliverableDirectiveBase(rst.Directive):
recent_version = earliest_version = deliv.earliest_release
# Determine the most recent release that is not an EOL
# tag.
has_non_eol_em = False
for r in reversed(deliv.releases):
if not (r.is_eol or r.is_em):
recent_version = r.version
has_non_eol_em = True
break
# Even though we have this check later for specific
# deliverables, we need to do this at a couple points to make
# sure we don't start adding something that doesn't actually
# end up having any individual release data to show
if not has_non_eol_em:
continue
ref = ':ref:`%s-%s`' % (series, deliv.name)
release_notes = deliv.release_notes
if not release_notes:
@@ -264,6 +272,13 @@ class DeliverableDirectiveBase(rst.Directive):
# Show the detailed history of the deliverables within the series.
for deliv in deliverables:
has_non_eol_em = False
for r in deliv.releases:
if not (r.is_eol or r.is_em):
has_non_eol_em = True
break
if not has_non_eol_em:
continue
# These closures need to be redefined in each iteration of
# the loop because they use the deliverable name.
@@ -309,7 +324,8 @@ class DeliverableDirectiveBase(rst.Directive):
deliv),
p.repo, p.hash)
for r in reversed(deliv.releases)
for p in r.projects)
for p in r.projects
if not (r.is_eol or r.is_em))
columns = [10, 10, 40, 50]
else:
headers = ['Version', 'Repo', 'Git Commit']
@@ -317,7 +333,8 @@ class DeliverableDirectiveBase(rst.Directive):
deliv),
p.repo, p.hash)
for r in reversed(deliv.releases)
for p in r.projects)
for p in r.projects
if not (r.is_eol or r.is_em))
columns = [10, 40, 50]
_list_table(
_add,