David Caro has uploaded a new change for review.

Change subject: Fixed issue with login+fetch bug
......................................................................

Fixed issue with login+fetch bug

It was not getting the external info from the bug when using bg.login from
shell script.

Also typo on warn_if_not_merged.

Change-Id: Iffbe0eb60a53a2ceb917120f1834f77b810909a4
Signed-off-by: David Caro <[email protected]>
---
M hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch
M hooks/lib/bz.sh
2 files changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/gerrit-admin refs/changes/93/29493/1

diff --git 
a/hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch 
b/hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch
index 5d3b3c8..905c52d 100755
--- a/hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch
+++ b/hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch
@@ -48,7 +48,7 @@
         long_msg.append(
             "WARNING: This patch does not exist yet on all the newer branches"
             "\n  Missing on : %s" % ', '.join(no_patch_branches))
-        msg.append("* Check mrged to previous::WARN, Still missing on branches 
%s"
+        msg.append("* Check merged to previous::WARN, Still missing on 
branches %s"
                    % ', '.join(no_patch_branches))
     print '-1\n0\n' + '\n'.join(msg)
     logging.error('\n'.join(long_msg))
diff --git a/hooks/lib/bz.sh b/hooks/lib/bz.sh
index 7fb497d..0beb42d 100644
--- a/hooks/lib/bz.sh
+++ b/hooks/lib/bz.sh
@@ -170,6 +170,7 @@
         --bz-user "$bz_user" \
         --bz-pass "$bz_password" \
         Bug.get "ids=[$bug_id]" \
+        "extra_fields=[\"flags\", \"external_bugs\"]" \
     )"
     [[ "$res" =~ ^LOGIN\ ERROR.* ]] \
     && {


-- 
To view, visit http://gerrit.ovirt.org/29493
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iffbe0eb60a53a2ceb917120f1834f77b810909a4
Gerrit-PatchSet: 1
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: David Caro <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to