<p>Kevin Harwell <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/10992">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  George Joseph: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved; Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">release_summary: Don't update issue fixed version.<br><br>When creating a release summary it should do just that, and<br>not update the fixed version for each issue. Updating is<br>problematic because the process can later be interrupted<br>and the release process stopped - yet the fix version would<br>still have been applied.<br><br>Change-Id: Ic2fe07c008b7db06c4a93efdf91735863ad72b05<br>---<br>M release_summary.py<br>1 file changed, 1 insertion(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/release_summary.py b/release_summary.py</span><br><span>index 9b4a77d..193ac95 100755</span><br><span>--- a/release_summary.py</span><br><span>+++ b/release_summary.py</span><br><span>@@ -15,7 +15,7 @@</span><br><span> from optparse import OptionParser</span><br><span> from progressbar import ProgressBar</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-from digium_jira import get_jira_client, DigiumJiraVersion</span><br><span style="color: hsl(120, 100%, 40%);">+from digium_jira import get_jira_client</span><br><span> from digium_jira_user import AsteriskUser</span><br><span> from digium_git import get_repo, DigiumGitRepo</span><br><span> </span><br><span>@@ -251,12 +251,6 @@</span><br><span>             pbar.maxval = len(self.raw_log_messages)</span><br><span>             pbar.start()</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        # Go ahead and update the fixedVersion on the issue while processing</span><br><span style="color: hsl(0, 100%, 40%);">-        # the release summary. If the version has already been applied it</span><br><span style="color: hsl(0, 100%, 40%);">-        # won't do it again.</span><br><span style="color: hsl(0, 100%, 40%);">-        djv = DigiumJiraVersion(self.options.version,</span><br><span style="color: hsl(0, 100%, 40%);">-                                self.options.project, self.jira)</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>         for i, log_message in enumerate(self.raw_log_messages):</span><br><span> </span><br><span>             if log_message.raw and len(log_message.raw.parents) > 1:</span><br><span>@@ -309,7 +303,6 @@</span><br><span>                     status = str(issue.fields.status).lower()</span><br><span>                     if status == 'closed' or status == 'complete':</span><br><span>                         issue_dict = self.closed_issues</span><br><span style="color: hsl(0, 100%, 40%);">-                        djv.apply_version(issue)</span><br><span>                     else:</span><br><span>                         issue_dict = self.open_issues</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/10992">change 10992</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/10992"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: repotools </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ic2fe07c008b7db06c4a93efdf91735863ad72b05 </div>
<div style="display:none"> Gerrit-Change-Number: 10992 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Joshua C. Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>