From 50a5af421e5fbc91018892cccd3ea7c7bf9eb1b8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rados=C5=82aw=20Piliszek?= <radoslaw.piliszek@gmail.com>
Date: Sun, 20 Feb 2022 19:19:25 +0100
Subject: [PATCH] Remove references to TC tags

The TC tags framework is discontinued [1], so all references to
TC tags will be invalidated once TC passes the change.
This patch removes these references to avoid breaking the releases
repo.

[1] http://lists.openstack.org/pipermail/openstack-discuss/2022-February/027214.html

Change-Id: Idf5cddc7d1a535e1a9bcc2c2922cf23c3a8d539b
---
 openstack_releases/cmds/list_changes.py      |  5 +---
 openstack_releases/cmds/list_deliverables.py | 27 --------------------
 openstack_releases/deliverable.py            | 13 ----------
 3 files changed, 1 insertion(+), 44 deletions(-)

diff --git a/openstack_releases/cmds/list_changes.py b/openstack_releases/cmds/list_changes.py
index 7bb55b9aaa..c21e0c55c1 100644
--- a/openstack_releases/cmds/list_changes.py
+++ b/openstack_releases/cmds/list_changes.py
@@ -289,10 +289,7 @@ def main():
                 if team_deliv:
                     print('found deliverable %s' % deliv.name)
                     for rn, repo in sorted(team_deliv.repositories.items()):
-                        print('\nrepo %s\ntags:' % repo.name)
-                        for t in repo.tags:
-                            print('  %s' % t)
-                        print('')
+                        print('\nrepo %s' % repo.name)
                         if stable_branch:
                             print('Deliverable is on stable branch')
                 else:
diff --git a/openstack_releases/cmds/list_deliverables.py b/openstack_releases/cmds/list_deliverables.py
index 413ecb6c31..0c2ae0ff0c 100644
--- a/openstack_releases/cmds/list_deliverables.py
+++ b/openstack_releases/cmds/list_deliverables.py
@@ -74,12 +74,6 @@ def main():
         default=False,
         help='Show last release date (in verbose mode)',
     )
-    parser.add_argument(
-        '--show-tags',
-        action='store_true',
-        default=False,
-        help='Show tags associated with deliverable (in verbose mode)',
-    )
     model = parser.add_mutually_exclusive_group()
     model.add_argument(
         '--model',
@@ -106,12 +100,6 @@ def main():
         choices=sorted(deliverable_schema.release_types),
         help='deliverable type, such as "library" or "service"',
     )
-    parser.add_argument(
-        '--tag',
-        default=[],
-        action='append',
-        help='look for one more more tags on the deliverable or team',
-    )
     parser.add_argument(
         '--deliverables-dir',
         default=openstack_releases.deliverable_dir,
@@ -185,8 +173,6 @@ def main():
         verbose_template += ' {type:15}'
     if not args.model:
         verbose_template += ' {model:15}'
-    if args.show_tags:
-        verbose_template += ' {tags}'
 
     if args.forced:
         args.all_releases = True
@@ -250,17 +236,6 @@ def main():
                 continue
         if args.missing_rc and deliv.is_released and 'rc' in deliv.latest_release:
             continue
-        if args.tag:
-            tags = deliv.tags
-            ignore = False
-            for t in args.tag:
-                if t not in tags:
-                    ignore = True
-                    break
-            if ignore:
-                continue
-
-        tag_str = '(' + ', '.join(deliv.tags) + ')'
 
         if args.missing_final and deliv.latest_release:
             if not ('rc' in deliv.latest_release or
@@ -321,7 +296,6 @@ def main():
                     team=deliv.team,
                     type=deliv.type,
                     model=deliv.model,
-                    tags=tag_str,
                 ))
         elif args.verbose:
             print(verbose_template.format(
@@ -331,7 +305,6 @@ def main():
                 team=deliv.team,
                 type=deliv.type,
                 model=deliv.model,
-                tags=tag_str,
             ))
         elif args.repos:
             if args.group_key and cur_group != deliv_group:
diff --git a/openstack_releases/deliverable.py b/openstack_releases/deliverable.py
index 0c7201ee4d..1a16a7a91c 100644
--- a/openstack_releases/deliverable.py
+++ b/openstack_releases/deliverable.py
@@ -21,7 +21,6 @@ import os
 import os.path
 import weakref
 
-from openstack_governance import governance
 import pbr.version
 
 from openstack_releases import series_status
@@ -573,18 +572,6 @@ class Deliverable(object):
                 return b['location']
         return None
 
-    @property
-    def tags(self):
-        if self._gov_data is None:
-            Deliverable._gov_data = governance.Governance.from_remote_repo()
-        try:
-            team = self._gov_data.get_team(self.team)
-            deliv = team.deliverables[self.name]
-        except (KeyError, ValueError):
-            # The deliverable is no longer listed under governance.
-            return []
-        return deliv.tags
-
     @property
     def filename(self):
         return os.path.join(self.series, self.name + '.yaml')