x/deb-python-tempest-lib
Code Issues Proposed changes

Remove file path from migration commit msg

Browse Source 
migrate_from_tempest.sh creates a commit for migrating code from
Tempest, but the commit message becomes long and redundant because
of containing file paths. Then now developers need to remove them
by hands.
This patch makes the script remove file paths.
In addition, this patch makes the script add the blueprint msg
"Partially implements blueprint migrate-service-clients-to-tempest-lib"
for managing these tasks on gerrit.

Change-Id: If8fc115dea67f779a86d33d26ac56cc8540a8c0f
This commit is contained in:
Ken'ichi Ohmichi
2015-09-25 04:39:57 +00:00
parent 6757c130b9
commit 59408c9253
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tools/migrate_from_tempest.sh
View File

@@ -53,7 +53,8 @@ cd $tmpdir
for file in $files; do
# Get the latest change-id for each file
change_id=`git log -n1 --grep "Change-Id: " -- $file | grep "Change-Id: " | awk '{print $2}'`
CHANGE_LIST=`echo -e "$CHANGE_LIST\n * $file: $change_id"`
filename=`basename $file`
CHANGE_LIST=`echo -e "$CHANGE_LIST\n * $filename: $change_id"`
done
# Move files and commit
@@ -102,4 +103,9 @@ rm -rf $tmpdir
commit_message="Migrated $file_list from tempest"
pre_list=$"This migrates the above files from tempest. This includes tempest commits:"
post_list=$"to see the commit history for these files refer to the above Change-Ids in the tempest repository"
git commit -m "$commit_message" -m "$pre_list" -m "$CHANGE_LIST" -m "$post_list"
if [ $service_client -eq 1 ]; then
bp_msg="Partially implements blueprint migrate-service-clients-to-tempest-lib"
else
bp_msg=""
fi
git commit -m "$commit_message" -m "$pre_list" -m "$CHANGE_LIST" -m "$post_list" -m "$bp_msg"