Re: Issue 3173 in reviewboard: cannot compare several couple of clearcase file revision using --revision-range

2013-12-26 Thread reviewboard


Comment #6 on issue 3173 by nano...@gmail.com: cannot compare several  
couple of clearcase file revision using --revision-range

http://code.google.com/p/reviewboard/issues/detail?id=3173

For close term, the documentation should be fixed, as proposed Jan in #5158  
but example should include several files.

For long term, I have proposed some comments on your hackpad page

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Issue 3176 in reviewboard: post-review requests to a server with thousand of repo is very slow

2013-12-26 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 3176 by nano...@gmail.com: post-review requests to a server with  
thousand of repo is very slow

http://code.google.com/p/reviewboard/issues/detail?id=3176

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
rejected.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** Do not post confidential information in this bug report!


What version are you running?
rbtools-0.5.1
reviewboard server 1.7.6

What's the URL of the page containing the problem?
api/repositories

What steps will reproduce the problem?
1. reviewboard server with more than 1000 repo in Europe
2. postreview client in China
3. before posting a review, list of repo is requested to match repository  
id/path, by default max-results is 250, so a lot of slow requests


What is the expected output? What do you see instead?
increase default max-results for repository request ? could be all repo or  
been configurable in ~/.reviewboardrc



What operating system are you using? What browser?
linux ubuntu/rhel

Please provide any additional information below.
many git repo and many clearcase ones

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3176 in reviewboard: post-review requests to a server with thousand of repo is very slow

2013-12-26 Thread reviewboard


Comment #1 on issue 3176 by nano...@gmail.com: post-review requests to a  
server with thousand of repo is very slow

http://code.google.com/p/reviewboard/issues/detail?id=3176

on postreview.py, this workaround has been done, is there a better way to  
prevent hardcoding max-results? or set it to all?
 rbtools/postreview.py  


index fe86b71..17a6b94 100755
@@ -514,8 +514,9 @@ class ReviewBoardServer(object):
 rsp = self.api_get('api/json/repositories/')
 repositories = rsp['repositories']
 else:
+# TCH: add max-results=1000 to reduce HTTP queries count
 rsp = self.api_get(
-self.root_resource['links']['repositories']['href'])
+self.root_resource['links']['repositories']['href']  
+ '?max-results=1000')

 repositories = rsp['repositories']

 while 'next' in rsp['links']:

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Issue 3177 in reviewboard: matching clearcase repo

2013-12-26 Thread reviewboard

Status: New
Owner: 
Labels: Type-Enhancement Priority-Medium

New issue 3177 by nano...@gmail.com: matching clearcase repo
http://code.google.com/p/reviewboard/issues/detail?id=3177

*** For customer support, please post to reviewbo...@googlegroups.com
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** Do not post confidential information in this enhancement request!


What version are you running?
rbtools-0.5.1
reviewboard server 1.7.6

What's the URL of the page this enhancement relates to, if any?
api/repositories

Describe the enhancement and the motivation for it.
when using clearcase scm, post-review identifies repository in several  
steps:

1) request repositories
2) iterate on clearcase repository only
3) get repository info to compare clearcase vob uuid with current working  
vob.


It requires to get repository info for each clearcase repository.  
Unfortunately if there are several clearcase vobs before the one, this  
operation will send many http request and can spend long time on slow  
connections.


What operating system are you using? What browser?
linux ubuntu/rhel

Please provide any additional information below.
Instead of requesting clearcase repositories info, we could compare first   
repository name and then compare vob uid if vobtag match. If no vobtag  
match then we fallback to previous loop

PS: Our clearcase repository are called by their common vobtag

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Issue 3178 in reviewboard: rbtools show extra changes when a clearcase directory has changed

2013-12-26 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 3178 by nano...@gmail.com: rbtools show extra changes when a  
clearcase directory has changed

http://code.google.com/p/reviewboard/issues/detail?id=3178

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
rejected.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** Do not post confidential information in this bug report!


What version are you running?
rbtools-0.5.1
reviewboard server 1.7.6


What's the URL of the page containing the problem?
NA

What steps will reproduce the problem?
1. under clearcase checkout a directory
2. add a private file to this directory
3. checkin this directory
4. send a review including this directory change

What is the expected output? What do you see instead?
directory listing is done with ls instead of cleartool, so all private  
files are included in directory changeset
directory must contain only clearcase object and does not display private  
files



What operating system are you using? What browser?
linux ubuntu/rhel

Please provide any additional information below.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3178 in reviewboard: rbtools show extra changes when a clearcase directory has changed

2013-12-26 Thread reviewboard


Comment #1 on issue 3178 by nano...@gmail.com: rbtools show extra changes  
when a clearcase directory has changed

http://code.google.com/p/reviewboard/issues/detail?id=3178

please find a proposal to solve such issue
https://reviews.reviewboard.org/r/5167

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Issue 3179 in reviewboard: add support for UCM clearcase activity

2013-12-26 Thread reviewboard

Status: New
Owner: 
Labels: Type-Enhancement Priority-Medium

New issue 3179 by nano...@gmail.com: add support for UCM clearcase activity
http://code.google.com/p/reviewboard/issues/detail?id=3179

*** For customer support, please post to reviewbo...@googlegroups.com
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** Do not post confidential information in this enhancement request!


What version are you running?
rbtools-0.5.1
reviewboard server 1.7.6

What's the URL of the page this enhancement relates to, if any?
NA


Describe the enhancement and the motivation for it.
create a review request of changeset made during a ucm clearcase activity


What operating system are you using? What browser?
linux ubuntu/rhel


Please provide any additional information below.
UCM clearcase activity changeset contains list of all files checked in  
during this activity.


For example:
file1@@/main/stream/1
file2@@/main/stream/2
file2@@/main/stream/3
means during this activity changeset will be something like that

file@@/main/stream/min_id-1:file@@/main/stream/max_id-1

ie
file1@@/main/stream/0:file1@@/main/stream/1
+
file2@@/main/stream/1:file2@@/main/stream/3

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3179 in reviewboard: add support for UCM clearcase activity

2013-12-26 Thread reviewboard


Comment #1 on issue 3179 by nano...@gmail.com: add support for UCM  
clearcase activity

http://code.google.com/p/reviewboard/issues/detail?id=3179

please find a proposal here
https://reviews.reviewboard.org/r/5168

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 2604 in reviewboard: Why do we find opened files ('Affected files') by using pattern matching instead of 'p4 opened -c CHANGESET'?

2013-12-26 Thread reviewboard

Updates:
Status: PendingReview
Owner: trowb...@gmail.com

Comment #2 on issue 2604 by trowb...@gmail.com: Why do we find opened files  
('Affected files') by using pattern matching instead of 'p4 opened -c  
CHANGESET'?

http://code.google.com/p/reviewboard/issues/detail?id=2604

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 2972 in reviewboard: post-review fails with Perforce depot paths containing spaces in the default change list

2013-12-26 Thread reviewboard

Updates:
Status: PendingReview
Owner: trowb...@gmail.com

Comment #2 on issue 2972 by trowb...@gmail.com: post-review fails with  
Perforce depot paths containing spaces in the default change list

http://code.google.com/p/reviewboard/issues/detail?id=2972

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
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 reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.