Bugzilla – Full Text Bug Listing
| Summary: | [website] automatically update bug count on contributors page | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Frank Becker <eclipse> | ||||||||||
| Component: | Mylyn | Assignee: | Frank Becker <eclipse> | ||||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||||
| Severity: | enhancement | ||||||||||||
| Priority: | P3 | CC: | sam.davis, steffen.pingel, tomasz.zarna | ||||||||||
| Version: | unspecified | ||||||||||||
| Target Milestone: | 3.14 | ||||||||||||
| Hardware: | All | ||||||||||||
| OS: | All | ||||||||||||
| URL: | http://www.eclipse.org/mylyn/contributors/ | ||||||||||||
| Whiteboard: | |||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Frank Becker
Created attachment 176608 [details]
patch V1
This patch is not commited in HEAD, please verify before commit.
Created attachment 176609 [details]
mylyn/context/zip
Makes sense. I have commit the patch but did not change the bug counts. Should we a) remove the bug counts b) update the bug counts the following url show all Assignee for Mylyn bugs https://bugs.eclipse.org/bugs/report.cgi?x_axis_field=&y_axis_field=assigned_to&z_axis_field=&query_format=report-table&short_desc_type=allwordssubstr&short_desc=&classification=Tools&product=Mylyn&longdesc_type=allwordssubstr&longdesc=&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&bug_status=RESOLVED&bug_status=VERIFIED&bug_status=CLOSED&emailtype1=substring&email1=&emailtype2=substring&email2=&bugidtype=include&bug_id=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=&format=table&action=wrap&field0-0-0=noop&type0-0-0=noop&value0-0-0= You wee see that there are some contributers not in the web mylyn/contributors/ +1 for removing the bug counts. I'll add that as an item for next weeks meeting to get input from other committers. Updated table: https://bugs.eclipse.org/bugs/report.cgi?x_axis_field=assigned_to_realname&y_axis_field=&z_axis_field=&query_format=report-table&short_desc_type=allwordssubstr&short_desc=&classification=Mylyn&bug_status=RESOLVED&bug_status=VERIFIED&bug_status=CLOSED&resolution=FIXED&longdesc_type=allwordssubstr&longdesc=&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&bug_id=&bug_id_type=anyexact&votes=&votes_type=greaterthaneq&emailtype1=substring&email1=&emailtype2=substring&email2=&emailtype3=substring&email3=&chfieldvalue=&chfieldfrom=&chfieldto=Now&j_top=AND&f1=assigned_to&o1=notequals&v1=frank%40Frank-Becker.de&f2=assigned_to&o2=notequals&v2=steffen.pingel%40tasktop.com&f3=assigned_to&o3=notequals&v3=david.green%40tasktop.com&f4=assigned_to&o4=notequals&v4=mik.kersten%40tasktop.com&f5=assigned_to&o5=notequals&v5=robert.elves%40tasktop.com&f6=assigned_to&o6=notequals&v6=shawn.minto%40tasktop.com&f7=assigned_to&o7=notsubstring&v7=mylyn-inbox&f8=assigned_to&o8=notequals&v8=r4e-triaged%40eclipse.org&f9=noop&o9=noop&v9=&format=table&action=wrap How do I embed it within the contributors page? Any tips on filtering out contributors with just one bug fixed. Because I did not know how to test website updates I implement this using the vagrant setup. Review https://git.eclipse.org/r/32200 show how we can do this. The source from contributer.php should be part of /contributers/index.php You can test this using the following url http://localhost:2080/cgi-bin/contributer.php Created attachment 246282 [details]
mylyn/context/zip
I think we maybe need an updated Query. if we want to count the contributions of a committer to an other sub level project without committer status we need the following https://bugs.eclipse.org/bugs/report.cgi?x_axis_field=assigned_to_realname&y_axis_field=&z_axis_field=&no_redirect=1&query_format=report-table&short_desc_type=allwordssubstr&short_desc=&classification=Mylyn&bug_status=RESOLVED&bug_status=VERIFIED&bug_status=CLOSED&longdesc_type=allwordssubstr&longdesc=&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&bug_id=&bug_id_type=anyexact&votes=&votes_type=greaterthaneq&emailtype1=substring&email1=&emailtype2=substring&email2=&emailtype3=substring&email3=&chfieldvalue=&chfieldfrom=&chfieldto=Now&j_top=AND&n10=1&f10=OP&j10=AND&f11=product&o11=equals&v11=Mylyn+Tasks&f13=OP&j13=OR&f14=assigned_to&o14=equals&v14=Steffen.Pingel%40tasktop.com&f15=assigned_to&o15=equals&v15=Eclipse%40Frank-Becker.de&f16=assigned_to&o16=equals&v16=David.Green%40tasktop.com&f17=assigned_to&o17=equals&v17=Mik.Kersten%40tasktop.com&f18=assigned_to&o18=equals&v18=robert.elves%40tasktop.com&f19=assigned_to&o19=equals&v19=shawn.minto%40tasktop.com&f20=assigned_to&o20=equals&v20=tomasz.zarna%40tasktop.com&f21=assigned_to&o21=equals&v21=sam.davis%40tasktop.com&f23=CP&f25=CP&n26=1&f26=OP&j26=AND&f27=product&o27=equals&v27=Mylyn+Builds&f28=OP&j28=OR&f29=assigned_to&o29=equals&v29=Steffen.Pingel%40tasktop.com&f30=assigned_to&o30=equals&v30=benjamin.muskalla%40tasktop.com&f31=assigned_to&o31=equals&v31=stepper%40esc-net.de&f32=assigned_to&o32=equals&v32=torkildr%40gmail.com&f33=noop&o33=noop&v33=&format=table&action=wrap this is only for the Build and the Task sub projects as an example But maybe we should keep this simple and only exclude all committer of any sub level project. Thoughts? Frank, the website is hosted in Git: ssh://git.eclipse.org:29418/www.eclipse.org/mylyn. I think if you push changes to the staging branch they will be published at http://staging.eclipse.org/mylyn/ (after several minutes). Changes pushed to master get published to http://eclipse.org/mylyn/ We do actual not have the following sections. * Committers * Active Contributors * Past Contributors Need to do some more work because the list of the committers is not actual. Hope that I can do this over the weekend. Thoughts / Comments? I don't think we should hardcode the list of committers because it will be too hard to maintain. The committers are listed on pages like https://projects.eclipse.org/projects/mylyn.tasks/who - if there is somewhere that that data is easily available to use we could use it, otherwise I think we should just list all contributors including committers in one list. Now you can see in http://staging.eclipse.org/mylyn/contributors/index.php how the new page looks. But the time to build the page is very long. Should I create a copy of the index.php without the list of committer and only the keyword contributed? I wonder if we could use a hudson job to periodically build the page offline and publish it. (In reply to Sam Davis from comment #16) > I wonder if we could use a hudson job to periodically build the page offline > and publish it. I try this in the next days with the localhost environment. I have changed the php (http://localhost:2080/cgi-bin/contributer.php) to use curl to load the content of contributer.inc into a string. Is the module php5-curl is installed on the eclipse webserver? Now I try to setup an hudson job to build the contributer.inc file in the workspace. Created attachment 246931 [details]
mylyn/context/zip
(In reply to comment #18) > I have changed the php (http://localhost:2080/cgi-bin/contributer.php) to use > curl to load the content of contributer.inc into a string. > > Is the module php5-curl is installed on the eclipse webserver? I don't know. You could try pushing something to staging that uses curl and see if it works. > > Now I try to setup an hudson job to build the contributer.inc file in the > workspace. Cool! (In reply to Sam Davis from comment #20) > (In reply to comment #18) > > I have changed the php (http://localhost:2080/cgi-bin/contributer.php) to use > > curl to load the content of contributer.inc into a string. > > > > Is the module php5-curl is installed on the eclipse webserver? > > I don't know. You could try pushing something to staging that uses curl and > see if it works. Here is the review: https://git.eclipse.org/r/#/c/33270/1 Maybe instead of keeping the php code to generate the contributors list in Hudson, we could keep it in the website Git repository and have Hudson check it out from git://git.eclipse.org/gitroot/www.eclipse.org/mylyn and run it. That way it will be part of version control. create a new review https://git.eclipse.org/r/#/c/33392/ because for some reason the code which work in localhost did not work on staging. So I try an other way. Sam, Steffen, I do not see that the changes in the staging branch are in http://staging.eclipse.org/mylyn/contributors. Do you know how long it takes between merge of staging and access ower the web? No. It is supposed to take only a few minutes, I think, but it seems to be longer. (In reply to comment #25) > No. It is supposed to take only a few minutes, I think, but it seems to be > longer. Now it is there (look for the " Test!!!"). So we have the content of the page on https://hudson.eclipse.org/mylyn/job/mylyn-update-contributors/lastSuccessfulBuild/artifact/contributor.inc Can you give feedback if we should use this in real. Frank this is really great! I think we should definitely use this! I pushed a few small changes to https://git.eclipse.org/r/#/c/34069/, can you take a look? To summarize how this is working, "this":https://hudson.eclipse.org/mylyn/job/mylyn-update-contributors/ build job runs a php script from the website Git which parses the https://projects.eclipse.org/projects/*/who pages to find out who the committers are and queries bugzilla to create a page which lists the committers and contributers separately, sorted by number of contributions. The contributors page then loads the content from the latest run of the build job. Frank, the page at http://staging.eclipse.org/mylyn/contributors/index.php still has "Test!!!" even though I removed that in https://git.eclipse.org/r/#/c/34079/. I guess it is just very slow to update? Should we merge the staging branch into master? (In reply to Sam Davis from comment #28) > Frank, the page at http://staging.eclipse.org/mylyn/contributors/index.php > still has "Test!!!" even though I removed that in > https://git.eclipse.org/r/#/c/34079/. I guess it is just very slow to update? > > Should we merge the staging branch into master? Yes, I think it is save to merge this. I've merged staging into master: https://www.eclipse.org/mylyn/contributors/index.php. I think we can consider this done. Nice work Frank! |