From 79ea12259092ff131be224175e8b7ebe67712334 Mon Sep 17 00:00:00 2001
From: Doug Hellmann <doug@doughellmann.com>
Date: Thu, 1 Oct 2015 19:40:15 +0000
Subject: [PATCH] show more verbose git log output

Include the graph log output for the range merging in so we can try to
verify that there are no merges from a bad branch.

Include the full log output for the commits so it is easier to tell
which ones might be features.

Change-Id: I74e34ce78abef63a38b41d6ff1d2d275317fb209
---
 openstack_releases/cmds/list_changes.py | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/openstack_releases/cmds/list_changes.py b/openstack_releases/cmds/list_changes.py
index d5f02c279c..959d6f0f0a 100644
--- a/openstack_releases/cmds/list_changes.py
+++ b/openstack_releases/cmds/list_changes.py
@@ -31,14 +31,14 @@ from openstack_releases import defaults
 from openstack_releases import gitutils
 
 
-def git_log(workdir, repo, title, git_range):
+def git_log(workdir, repo, title, git_range, extra_args=[]):
     header = '%s %s' % (title, git_range)
     print('\n%s' % header)
     print('-' * len(header))
-    subprocess.check_call(['git', 'log', '--no-color',
-                           '--format=%h %ci %s', '--no-merges',
-                           git_range],
-                          cwd=os.path.join(workdir, repo))
+    cmd = ['git', 'log', '--no-color']
+    cmd.extend(extra_args)
+    cmd.append(git_range)
+    subprocess.check_call(cmd, cwd=os.path.join(workdir, repo))
     print()
 
 
@@ -136,10 +136,17 @@ def main():
             else:
                 git_range = project['hash']
 
-            # Show the changes since the last release.
+            # Show the changes since the last release, first as a
+            # graph view so we can check for bad merges, and then with
+            # more detail.
             git_log(workdir, project['repo'],
                     'Release %s will include' % new_release['version'],
-                    git_range)
+                    git_range,
+                    extra_args=['--graph', '--oneline', '--decorate'])
+            git_log(workdir, project['repo'],
+                    'Details Contents',
+                    git_range,
+                    extra_args=['--no-merges'])
 
             # If the sha for HEAD and the requested release don't
             # match, show any unreleased changes on the branch. We ask
@@ -156,7 +163,8 @@ def main():
                 print('Request releases from HEAD on %s' % branch)
             else:
                 git_log(workdir, project['repo'], 'Release will NOT include',
-                        '%s..%s' % (requested_sha, head_sha))
+                        '%s..%s' % (requested_sha, head_sha),
+                        extra_args=['--format=%h %ci %s'])
 
             # Show more details about the commit being tagged.
             print()