Re: Report application name to Perforce using -zprog?
I'll check with our CM people on monday, we'll see if they have any problem with this. 2009/6/13 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 Scott 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.com<mailto: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 > > > > > > > > > > > > > > > > > > > > > -- Jeff O|||O --~--~-~--~~~---~--~~ 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?
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 Scott 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.com<mailto: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 > mailto: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?
Oh, I completely misunderstood this. I'm fine with adding it. We just need to be sure that it doesn't break existing setups. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Sat, Jun 13, 2009 at 8:41 PM, Paul Scott 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.com<mailto: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?
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.com<mailto: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 mailto: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?
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 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?
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 -~--~~~~--~~--~--~---