------------------------------------------------------------------------------
To reply, visit https://hellosplat.com/s/beanbag/tickets/4853/
------------------------------------------------------------------------------

New ticket #4853 by marcin.wolcendorf
For Beanbag, Inc. > RBTools

Status: New
Tags: Priority:Medium, Type:Defect


------------------------------------------------------------------------------
rbt fails to post (not diff) from svn project
==============================================================================

# What version are you running?
RBTools 1.0.2

# What steps will reproduce the problem?
1. 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.

------------------------------------------------------------------------------

-- 
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard-issues/20191217085335.5029.31687%40ip-10-1-54-209.ec2.internal.

Reply via email to