Re: Report application name to Perforce using -zprog?

2009-06-13 Thread Christian Hammond
So this would allow for reporting progress on, say, file fetches from
Perforce? Something like that would be interesting for displaying progresses
on rendering diffs, but would be a lot of work. We'd need to find a good use
case for it and implement things in a way where our code could take
advantage of this information without requiring (for all the other SCMs).

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com


On Thu, Apr 9, 2009 at 4:35 PM, Paul Scott psc...@vmware.com wrote:


 Hey Review Boarders,

 My team uses the -zprog global option with a few major internal
 scripts/applications to help better track usage patterns. (If you're
 unfamiliar with -zprog, you can read about it here:
 http://kb.perforce.com/AdminTasks/SuperuserTasks/UsingTheZpro..erProcesses).
 I was thinking about adding this to our Review Board and post-review.
 How/where I implement this will be a little bit different based on whether
 this is a customization or something I intend to submit as a patch, so I
 wanted to see if this is something folks would like Review Board to do, or
 not like Review Board to do, or what.

 Thanks,
 Paul Scott

 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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
-~--~~~~--~~--~--~---



RE: Report application name to Perforce using -zprog?

2009-06-13 Thread Paul Scott

Hmm... I don't know about that. I mean, if you really wanted to try and stick 
unique identifiers in your p4 commands so you could track which ones were 
currently pending, I guess you could, but I assume your Perforce administrators 
would get cranky at you. ;) Also, I don't really know how feasible that would 
be for showing progress... I mean, all you get from monitor is what processes 
are still alive, and how long they've been running. Maybe I misunderstood where 
you were going with that.

The main purpose of throwing in a -zprog is just to make things a little 
prettier on the Perforce side. It lets you identify your application to the 
server, so logs and monitor output will display that instead of a generic p4 
(for post-review) or whatever the p4 python api reports (for Review Board). 
Mostly this is just a way of playing nice in the general Perforce environment. 
Though this might also allow RB admins who also have access to Perforce server 
logs to get a better sense of how they are affecting or being affected by 
Perforce performance. (There may be other ways of identifying Review Board in 
logs depending on your configuration--user, RB server IP--though probably not 
for post-review).

This wouldn't be a huge change (just tack that on to p4 commands, and set 
something in the p4python connection object) and I would think the only major 
reason not to do it is if people didn't really want Review Board reported as 
reviewboard or similar in their Perforce logs.

-- Paul


From: reviewboard@googlegroups.com [reviewbo...@googlegroups.com] On Behalf Of 
Christian Hammond [chip...@chipx86.com]
Sent: Saturday, June 13, 2009 4:32 PM
To: reviewboard@googlegroups.com
Subject: Re: Report application name to Perforce using -zprog?

So this would allow for reporting progress on, say, file fetches from Perforce? 
Something like that would be interesting for displaying progresses on rendering 
diffs, but would be a lot of work. We'd need to find a good use case for it and 
implement things in a way where our code could take advantage of this 
information without requiring (for all the other SCMs).

Christian

--
Christian Hammond - chip...@chipx86.commailto:chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com


On Thu, Apr 9, 2009 at 4:35 PM, Paul Scott 
psc...@vmware.commailto:psc...@vmware.com wrote:

Hey Review Boarders,

My team uses the -zprog global option with a few major internal 
scripts/applications to help better track usage patterns. (If you're unfamiliar 
with -zprog, you can read about it here: 
http://kb.perforce.com/AdminTasks/SuperuserTasks/UsingTheZpro..erProcesses). I 
was thinking about adding this to our Review Board and post-review. How/where I 
implement this will be a little bit different based on whether this is a 
customization or something I intend to submit as a patch, so I wanted to see if 
this is something folks would like Review Board to do, or not like Review Board 
to do, or what.

Thanks,
Paul Scott






--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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
-~--~~~~--~~--~--~---



Re: Report application name to Perforce using -zprog?

2009-06-13 Thread David Trowbridge

I don't see any reason why we shouldn't do this. Accountability is a good thing.

-David

On Sat, Jun 13, 2009 at 8:41 PM, Paul Scottpsc...@vmware.com wrote:

 Hmm... I don't know about that. I mean, if you really wanted to try and stick 
 unique identifiers in your p4 commands so you could track which ones were 
 currently pending, I guess you could, but I assume your Perforce 
 administrators would get cranky at you. ;) Also, I don't really know how 
 feasible that would be for showing progress... I mean, all you get from 
 monitor is what processes are still alive, and how long they've been running. 
 Maybe I misunderstood where you were going with that.

 The main purpose of throwing in a -zprog is just to make things a little 
 prettier on the Perforce side. It lets you identify your application to the 
 server, so logs and monitor output will display that instead of a generic 
 p4 (for post-review) or whatever the p4 python api reports (for Review 
 Board). Mostly this is just a way of playing nice in the general Perforce 
 environment. Though this might also allow RB admins who also have access to 
 Perforce server logs to get a better sense of how they are affecting or being 
 affected by Perforce performance. (There may be other ways of identifying 
 Review Board in logs depending on your configuration--user, RB server 
 IP--though probably not for post-review).

 This wouldn't be a huge change (just tack that on to p4 commands, and set 
 something in the p4python connection object) and I would think the only major 
 reason not to do it is if people didn't really want Review Board reported as 
 reviewboard or similar in their Perforce logs.

 -- Paul

 
 From: reviewboard@googlegroups.com [reviewbo...@googlegroups.com] On Behalf 
 Of Christian Hammond [chip...@chipx86.com]
 Sent: Saturday, June 13, 2009 4:32 PM
 To: reviewboard@googlegroups.com
 Subject: Re: Report application name to Perforce using -zprog?

 So this would allow for reporting progress on, say, file fetches from 
 Perforce? Something like that would be interesting for displaying progresses 
 on rendering diffs, but would be a lot of work. We'd need to find a good use 
 case for it and implement things in a way where our code could take advantage 
 of this information without requiring (for all the other SCMs).

 Christian

 --
 Christian Hammond - chip...@chipx86.commailto:chip...@chipx86.com
 Review Board - http://www.review-board.org
 VMware, Inc. - http://www.vmware.com


 On Thu, Apr 9, 2009 at 4:35 PM, Paul Scott 
 psc...@vmware.commailto:psc...@vmware.com wrote:

 Hey Review Boarders,

 My team uses the -zprog global option with a few major internal 
 scripts/applications to help better track usage patterns. (If you're 
 unfamiliar with -zprog, you can read about it here: 
 http://kb.perforce.com/AdminTasks/SuperuserTasks/UsingTheZpro..erProcesses). 
 I was thinking about adding this to our Review Board and post-review. 
 How/where I implement this will be a little bit different based on whether 
 this is a customization or something I intend to submit as a patch, so I 
 wanted to see if this is something folks would like Review Board to do, or 
 not like Review Board to do, or what.

 Thanks,
 Paul Scott






 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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
-~--~~~~--~~--~--~---



Report application name to Perforce using -zprog?

2009-04-09 Thread Paul Scott

Hey Review Boarders,

My team uses the -zprog global option with a few major internal 
scripts/applications to help better track usage patterns. (If you're unfamiliar 
with -zprog, you can read about it here: 
http://kb.perforce.com/AdminTasks/SuperuserTasks/UsingTheZpro..erProcesses). I 
was thinking about adding this to our Review Board and post-review. How/where I 
implement this will be a little bit different based on whether this is a 
customization or something I intend to submit as a patch, so I wanted to see if 
this is something folks would like Review Board to do, or not like Review Board 
to do, or what.

Thanks,
Paul Scott

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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
-~--~~~~--~~--~--~---