2182: ReviewBoard cannot cope with Perforce proxies

ch***@newbo****** (Google Code) (Is this you? Claim this profile.)
Feb. 1, 2014 
What version are you running?
1.5.5

We have a enterprise-wide Perforce environment that includes multiple, redundant servers. These servers are behind Perforce proxies that direct clients to the appropriate server based on geographic and/or server status.

While clients can connect to the proxies, direct connections to the server(s) are explicitly disallowed.

The problem with ReviewBoard is that:

(1) post-review insists that the server address in the repository
    configuration match that found in the output of 'p4 info'; 
    'p4 info' returns the identity of the _actual_ Perforce server,
    not the proxy.

(2) Configuring the ReviewBoard repository with the server name
    from 'p4 info' allows post-review to work, but all Perforce 
    operations from the server fail as they attempt to contact
    the server directly rather than through the proxy.

(3) There does not appear to be any configurable logging to
    see the Perforce commands that ReviewBoard is issuing or any
    resulting errors--making this issue very hard to diagnose. 
    (Perhaps I've overlooked something here, though...)

This is effectively a show-stopper for us.
#1 ch***@newbo****** (Google Code) (Is this you? Claim this profile.) 
It turns out that ReviewBoard does support Perforce environments with proxies, though configuration is somewhat obscure. We now have this (sort of) working in our environment.

One remaining issue is that the configuration is not robust to changes in the name of the actual Perforce server on the far side of the proxy. This occurs, for example, when there is a fail-over from one server to another.

See further discussion here: http://groups.google.com/group/reviewboard/browse_thread/thread/87c39f8c7e05066e
david
#2 david
  • +Component-SCMTools
david
#3 david 
This has been much improved over the past few months. There may still be a server name mismatch, but that can be handled with the REPOSITORY setting in .reviewboardrc.
  • +Fixed