Hi, Please try using 'rbt post' instead of post-review, and see if that works better. It has different logic for this than post-review does, and will be replacing post-review completely in a future release. post-review at this point is deprecated.
Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Tue, Apr 23, 2013 at 2:45 AM, chuck j <cjerry2...@gmail.com> wrote: > Its seems for ssl enabled and configured list present in the reviewboard > is not handled in the post-review tool. > > if isinstance(self.info.path, list): > repositories = self.get_repositories() > > debug("Repositories on Server: %s" % repositories) > debug("Server Aliases: %s" % self.info.path) > > for repository in repositories: > if repository['path'] in self.info.path: > self.info.path = repository['path'] > break > > Reviewboard Admin guide says to register ssl enable repository use ssl: > prefix, when it gets registered with name ssl:perforce:1234, but > self.info.path will give you list of repositories without ssl: prefix hence > it die reporting all these messages. > > Just to let you know that "p4 info" output gives you Server Address > without ssl prefix i.e perforce:1234. > > Do you guys think the logic to handle multiple repository having ssl > enable perforce repository configured needs changes, please let me know if > i am wrong. > > -Chuck > > > On Tuesday, April 23, 2013 1:01:40 PM UTC+5:30, chuck j wrote: >> >> Hi All, >> >> posting of review request shows "Unknown repository paths found" >> >> but review board server configured with two repositories. >> >> List of configured repositories: >> pf-sa.company.com:1666 >> ssl:pf-sa-svr1:19773 >> Unknown repository paths found: >> p4-area:19773 >> p4-area.company.com:19773 >> pf-sa-svr1:19773 >> pf-sa-svr1.company.com:19773 >> >> All the above unknown reported by post-review tool are Server Aliases and >> are one which is configured with ssl. >> >> Following argument passed with post-review tool. >> >> post-review --bugs-closed="TAI2343" --branch="MAIN" --server= >> https://svrrb1.na.**company.com/svrrb<https://svrrb1.na.company.com/svrrb>--submit-as=chuck >> --description-file="/tmp/ >> **tmpVwAGQY" 191269 --p4-port="ssl:pf-sa-svr1:**19773" >> --repository-url="ssl:pf-sa-**svr1:19773" >> >> Why post-review unknown repository even when they are configured >> properly. What is missing? >> >> -Chuck >> >> >> >> >> -- > Want to help the Review Board project? Donate today at > http://www.reviewboard.org/donate/ > Happy user? Let us know at http://www.reviewboard.org/users/ > -~----------~----~----~----~------~----~------~--~--- > To unsubscribe from this group, send email to > reviewboard+unsubscr...@googlegroups.com > For more options, visit this group at > http://groups.google.com/group/reviewboard?hl=en > --- > You received this message because you are subscribed to the Google Groups > "reviewboard" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to reviewboard+unsubscr...@googlegroups.com. > For more options, visit https://groups.google.com/groups/opt_out. > > > -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~----------~----~----~----~------~----~------~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
