4562: Review board 22.214.171.124 does not work with diffs from Bazaar 2.7.0dev1
- Review Board
What version are you running?
What's the URL of the page containing the problem?
What steps will reproduce the problem?
- I create a diff in bazaar as bzr diff --old=xxx > diff.file
- I try to upload it to review board but it stays in "loading" status
- Still loading and loading
What is the expected output? What do you see instead?
To upload the diff to review board
What operating system are you using? What browser?
Please provide any additional information below.
So far, we were using review board 2.0.18 and everything worked without problems. Strangely, some diffs can still be uploaded, but some cannot. The differences are minimal. I am appending a diff that does not get uploaded (but worked in 2.0.18).
Same issue for me. Current production configuration server is [Ubuntu 10.0.4 / Bazaar 2.5.1 / ReviewBoard 2.0.14].
New test server is built with [Ubuntu 16.04.3 / Bazaar 2.7.0 / ReviewBoard 2.5.15] and the existing ReviewBoard has been migrated and upgraded. I am seeing the same issue where the browser stays in the "loading" status. I attempted to create reviews using existing diff files that worked as expected in the production system.
I set the ReviewBoard log level to DEBUG, no errors reported, the only output generated just prior to entering the "loading" state is as follows:
2017-09-07 03:41:50,302 - DEBUG - - root - DiffParser.parse: Beginning parse of diff, size = 1658
2017-09-07 03:41:50,303 - DEBUG - - root - DiffParser.parse: Finished parsing diff.
I also confirmed that:
- I can raise non-bzr reviews by creating a simple review with a file attachment only
- Existing BZR-based reviews appear OK: I can open them and view the comments and the code diff
Attached for ref is one of the diff files that causes the issue.
As always, forgot the attachment (facepalm)
Not sure if it is related, with DEBUG loggin turned on the following warning is generated to the log file every now and then:
2017-09-14 05:10:42,666 - WARNING - - py.warnings - /usr/local/lib/python2.7/dist-packages/feedparser.py:345: DeprecationWarning: To avoid breaking existing software while fixing issue 310, a temporary mapping has been created from
updated_parseddoesn't exist. This fallback will be removed in a future version of feedparser.
(there are no other warnings in the log)