Last modified: 2014-07-07 15:40:12 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68087, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66087 - Path to recurrent report (from dashboard) should have better semantics
Path to recurrent report (from dashboard) should have better semantics
Status: RESOLVED FIXED
Product: Analytics
Classification: Unclassified
Wikimetrics (Other open bugs)
unspecified
All All
: High normal
: ---
Assigned To: Nobody - You can work on this!
u=WikimetricsUser c=Wikimetrics p=3 s...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-03 18:53 UTC by nuria
Modified: 2014-07-07 15:40 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description nuria 2014-06-03 18:53:19 UTC
Path to reports from wikimetrics UI look like:
http://localhost:5000/static/public/1/full_report.json

Instead they should be like:
http://localhost:5000/static/public/wiki/newly-register-users/full_report.json

http://localhost:5000/static/public/<metric>/<cohort>/full_report.json
Comment 1 nuria 2014-06-12 16:14:17 UTC
To be clear, this can be implemented with a symlink tree that abides to our conevention.
Comment 2 Dan Andreescu 2014-06-25 18:01:50 UTC
Note: Symlinking the reports is good enough
Note: We do not need to change the report generating code.
Note: It's only about the CronUsers recurrent public reports

current status:
domain/static/public/124124/full_report.json

desired (only for WikimetricsBot):
domain/static/public/WikimetricsBot/<metric>/<cohort>.json
Comment 3 Gerrit Notification Bot 2014-06-30 15:41:03 UTC
Change 143040 had a related patch set uploaded by Milimetric:
Add pretty symlink for WikimetricsBot

https://gerrit.wikimedia.org/r/143040
Comment 4 Gerrit Notification Bot 2014-07-04 08:13:24 UTC
Change 143040 merged by jenkins-bot:
Add pretty symlink for WikimetricsBot

https://gerrit.wikimedia.org/r/143040
Comment 5 christian 2014-07-06 10:16:13 UTC
As we kicked symlink handling down the road for the backup script, and
this card is now adding symlinks, this card needs to make the backup
work with symlinks. Hence, reopening the bug.

I had a short look, but it seems that wikimetrics generates absolute
symlink targets, starting in /srv/wikimetrics/[...] instead of
relative links.

This use of absolute targets for symlinks unnecessarily binds the
directory of public files to the full path where wikimetrics gets
deployed to.

Also, since the backup is currently grabbing the public files from
/var/lib/wikimetrics, having the symlinks' absolute targets start in
/srv/wikimetrics would make it necessary that the backup grabs files
outside of its public directory. So it would make it hard for the
backup to determine which targets of symbolic links are fine, and
which are not.

Could we use relative targets for links instead?

That way, we could keep wikimetric's internal logic bound to the
wikimetrics repository and would not need to duplicate this internal
logic to the external backup script.
And also, it would make the public files directory more portable.
Comment 6 Dan Andreescu 2014-07-06 12:16:16 UTC
Yes, we can change it to relative symlinks.
Comment 7 Gerrit Notification Bot 2014-07-07 10:56:47 UTC
Change 144439 had a related patch set uploaded by Nuria:
Making pretty report symlinks relative

https://gerrit.wikimedia.org/r/144439
Comment 8 Gerrit Notification Bot 2014-07-07 12:17:54 UTC
Change 144439 merged by jenkins-bot:
Making pretty report symlinks relative

https://gerrit.wikimedia.org/r/144439

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links