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 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.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?
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?
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 -~--~~~~--~~--~--~---