4853: rbt fails to post (not diff) from svn project

Reported by:	marcin.wolcendorf 4 years, 4 months
Unassigned

What version are you running?

RBTools 1.0.2

What steps will reproduce the problem?

rbt post --disable-ssl-verification "29591:29620"

What is the expected output? What do you see instead?

I would expect the successful posting of a review request. Instead all I get is:
CRITICAL: sequence item 4: expected string or Unicode, NoneType found

What operating system are you using?

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 16.04.6 LTS
Release: 16.04
Codename: xenial

$ uname -srvmpio
Linux 4.4.27-rt38-rc1 #1 SMP PREEMPT RT Fri Nov 11 12:33:15 CET 2016 x86_64 x86_64 x86_64 GNU/Linux

Attach the debug out from the command.

$ rbt post --debug --disable-ssl-verification "29591:29620" &> rbt_debug.log

RBTools 1.0.2
Python 2.7.12 (default, Oct 8 2019, 14:14:10)
[GCC 5.4.0 20160609]
Running on Linux-4.4.27-rt38-rc1-x86_64-with-Ubuntu-16.04-xenial
Home = /home/y1mwolce
Current directory = /home/y1mwolce/work/test/imcr_smt.test
Command line: rbt post --debug --disable-ssl-verification 29591:29620
Running: tf vc help
Checking for a Subversion repository...
Running: svn --non-interactive info
Running: diff --version
Running: svn --non-interactive --version -q
Repository info: Path: https://srv1.intra.org/svn/54_IMCR, Base path: /smt, Supports changesets: False
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/repositories/?tool=Subversion
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/repositories/?tool=Subversion expired and was modified
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/repositories/1/info/
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/repositories/1/info/ expired and was modified
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/repositories/2/info/
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/repositories/2/info/ expired and was modified
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/repositories/3/info/
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/repositories/3/info/ expired and was modified
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/repositories/4/info/
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/repositories/4/info/ expired and was modified
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/repositories/5/info/
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/repositories/5/info/ expired and was modified
Repository info: Path: https://srv1.intra.org/svn/54_IMCR, Base path: /smt, Supports changesets: False
Running: svn --non-interactive log --xml -r 29591 -l 1
Running: svn --non-interactive log --xml -r 29620 -l 1
Running: svn --non-interactive diff --diff-cmd=diff --notice-ancestry -r 29591:29620
Running: svn --non-interactive info trunk/release_notes.txt
Running: svn --non-interactive info trunk
Running: svn --non-interactive info trunk/tests/PISA.py
Running: svn --non-interactive info trunk/tests
Running: svn --non-interactive info trunk/tools/get_machine.py
Running: svn --non-interactive info trunk/tools
Running: svn --non-interactive info trunk/tools/process_gd2.py
Running: svn --non-interactive info trunk/tools/pycharm_remote_python_wrapper.sh
Running: svn --non-interactive info trunk/tools/gd2_parser.py
Running: svn --non-interactive info trunk/tools/generate_filelist.py
Running: svn --non-interactive info tags/R0.0/release_notes.txt
Running: svn --non-interactive info tags/R0.0
Running: svn --non-interactive info tags
Running: svn --non-interactive info tags/R0.0/tests/PISA.py
Running: svn --non-interactive info tags/R0.0/tests
Running: svn --non-interactive info tags/R0.0/tools/get_machine.py
Running: svn --non-interactive info tags/R0.0/tools
Running: svn --non-interactive info tags/R0.0/tools/process_gd2.py
Running: svn --non-interactive info tags/R0.0/tools/pycharm_remote_python_wrapper.sh
Running: svn --non-interactive info tags/R0.0/tools/gd2_parser.py
Running: svn --non-interactive info tags/R0.0/tools/generate_filelist.py
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/HDM/MC_HDM_4_5.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/HDM
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp
Running: svn --non-interactive info branches/y1kemi_pycharm
Running: svn --non-interactive info branches
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/HDM/MC_HDM_6_0.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/HDM/MC_HDM_6_3.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/sHDM/MC_sHDM_4_5.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/sHDM
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/sHDM/MC_sHDM_6_0.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/MC/sHDM/MC_sHDM_6_3.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM/TC_PISS_HDM_4_5.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM/TC_PISS_HDM_6_0.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM/TC_PISS_HDM_6_3.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM/TC_PIZP_HDM_4_5.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM/TC_PIZP_HDM_6_0.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/HDM/TC_PIZP_HDM_6_3.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM/TC_PISS_sHDM_4_5.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM/TC_PISS_sHDM_6_0.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM/TC_PISS_sHDM_6_3.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM/TC_PIZP_sHDM_4_5.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM/TC_PIZP_sHDM_6_0.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tmp/Test_sHDM_Bipod_Clip_files/TC/sHDM/TC_PIZP_sHDM_6_3.clip
Running: svn --non-interactive info branches/y1kemi_pycharm/tools/gd2_parser.py
Running: svn --non-interactive info branches/y1kemi_pycharm/tools
Running: svn --non-interactive info branches/y1kemi_pycharm/tools/generate.py
Running: svn --non-interactive info branches/y1kemi_pycharm/tools/get_machine.py
Running: svn --non-interactive info branches/y1kemi_pycharm/tools/process_gd2.py
Running: svn --non-interactive info branches/y1kemi_pycharm/tools/pycharm_remote_python_wrapper.sh
Running: svn --non-interactive info branches/y1kemi_pycharm/tests/PISA.py
Running: svn --non-interactive info branches/y1kemi_pycharm/tests
Running: svn --non-interactive diff --diff-cmd=diff --notice-ancestry -r 29591:29620 --no-diff-deleted
Generated diff size: 101562 bytes
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/validation/diffs/
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/validation/diffs/ expired and was modified
Making HTTP POST request to https://srv2.intra.org/reviewboard/api/validation/diffs/
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/review-requests/?only-links=create&only-fields=
Cached response for HTTP GET https://srv2.intra.org/reviewboard/api/review-requests/?only-links=create&only-fields= expired and was modified
Making HTTP POST request to https://srv2.intra.org/reviewboard/api/review-requests/
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/review-requests/53/diffs/?only-fields=
Added cache entry for HTTP GET request to https://srv2.intra.org/reviewboard/api/review-requests/53/diffs/?only-fields=
Making HTTP POST request to https://srv2.intra.org/reviewboard/api/review-requests/53/diffs/
Making HTTP GET request to https://srv2.intra.org/reviewboard/api/review-requests/53/draft/?only-fields=commit_id
Added cache entry for HTTP GET request to https://srv2.intra.org/reviewboard/api/review-requests/53/draft/?only-fields=commit_id
Running: svn --non-interactive log --xml -r 29591:29620
Traceback (most recent call last):
File "/usr/local/bin/rbt", line 10, in <module>
sys.exit(main())
File "/usr/local/lib/python2.7/dist-packages/rbtools/commands/main.py", line 120, in main
command.run_from_argv([RB_MAIN, command_name] + args)
File "/usr/local/lib/python2.7/dist-packages/rbtools/commands/init.py", line 725, in run_from_argv
exit_code = self.main(*args) or 0
File "/usr/local/lib/python2.7/dist-packages/rbtools/commands/post.py", line 928, in main
base_dir=base_dir)
File "/usr/local/lib/python2.7/dist-packages/rbtools/commands/post.py", line 609, in post_request
self.check_guess_fields()
File "/usr/local/lib/python2.7/dist-packages/rbtools/commands/post.py", line 709, in check_guess_fields
commit_message = self.tool.get_commit_message(self.revisions)
File "/usr/local/lib/python2.7/dist-packages/rbtools/clients/init.py", line 481, in get_commit_message
commit_message = self.get_raw_commit_message(revisions)
File "/usr/local/lib/python2.7/dist-packages/rbtools/clients/svn.py", line 376, in get_raw_commit_message
return '\n\n'.join(message.text for message in messages)
TypeError: sequence item 4: expected string or Unicode, NoneType found

Please provide any additional information below

The names of the servers have been modified by mgmt. request.

Welcome to the Splat alpha!