Ticket #10559 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

New algorithm that sends json document to statistics service

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: blocker Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By: #10549, #10558, #10566, #10573, #10712, #10730
Blocking: #10770, #10771, #10772, #10810 Tester: Steven Hahn

Description

This is an algorithm that does the work described in the "Usage Statistics" section of MeasureUsageStatistics

Change History

comment:1 Changed 6 years ago by Peter Peterson

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Peter Peterson

  • Status changed from assigned to inprogress

comment:3 Changed 6 years ago by Peter Peterson

  • Blocked By 10566 added

comment:4 Changed 6 years ago by Peter Peterson

  • Blocked By 8924 added

Adding #8924 since Nick is putting together some useful utilities.

comment:5 Changed 6 years ago by Pete Peterson

Re #10559. Adding stubs from class maker

Changeset: 0e58e9d802829b2c9a35e13de3d1c2cf41113bde

comment:6 Changed 6 years ago by Pete Peterson

Re #10559. Put together the json document.

Changeset: 7f6a3303dcd5fd53d277c566bb6a7baef63e601f

comment:7 Changed 6 years ago by Pete Peterson

Re #10559. Caching common info in static variable.

Changeset: e7df2f7dee6345844515ee7a8dc4011f527d39d6

comment:8 Changed 6 years ago by Nick Draper

  • Blocked By 10549, 10573 added; 8924 removed

comment:9 Changed 6 years ago by Nick Draper

Changed the blocking from 8924 to 10549, which adds the Kernel/InternetHelper class. This may need to be extended to support forms if they are required.

comment:10 Changed 6 years ago by Peter Peterson

  • Priority changed from major to blocker

comment:11 Changed 6 years ago by Peter Peterson

  • Blocked By 10712 added

comment:12 Changed 6 years ago by Pete Peterson

Re #10559. Hashing user and host info.

Changeset: 09b0e22041f73c13aad6e7dacc607ba678888749

comment:13 Changed 6 years ago by Martyn Gigg

  • Blocked By 10730 added

comment:14 Changed 6 years ago by Pete Peterson

Re #10559. Modified InternetHelper to do POST requests too.

Changeset: e5fc620fb19603698b94e092d647a0c8de753874

comment:15 Changed 6 years ago by Pete Peterson

Re #10559. Filling in the details to post json to a server.

Changeset: fe8cd571fb75fb7e423c1ef150aa9acf88e77968

comment:16 Changed 6 years ago by Pete Peterson

Re #10559. Send current time every time.

Changeset: 6cb1560ce73bead7d54bd407274fd6f6953b9bc9

comment:17 Changed 6 years ago by Pete Peterson

Re #10559. Removing debug print.

Changeset: 9fc257584b87720fa592334c47bad8b54a49e5e0

comment:18 Changed 6 years ago by Pete Peterson

Re #10559. Removing debug print.

Changeset: 47328709c860eeed1fac7fd7efb4e5a9f4468cdf

comment:19 Changed 6 years ago by Pete Peterson

Re #10559. Refactored code to consolidate work in InternetHelper.

Changeset: 95a76a27a3527d94a699d76c20c1b03a7cbee42e

comment:20 Changed 6 years ago by Pete Peterson

Re #10559. Only look for created during post.

Changeset: 5b40184dd140314bc86be0b45b49b10226c9a8a0

comment:21 Changed 6 years ago by Pete Peterson

Re #10559. Pointing at the real url endpoint.

Changeset: 4bb8b88822e51fb4b28a2505548b0220fffd697c

comment:22 Changed 6 years ago by Pete Peterson

Re #10559. Small rework to implement real test.

Changeset: 7475b26f1ad5cb7064e6bdfbc24e161ee527e5d6

comment:23 Changed 6 years ago by Peter Peterson

  • Blocking 10770 added

comment:24 Changed 6 years ago by Peter Peterson

  • Blocking 10771 added

comment:25 Changed 6 years ago by Pete Peterson

  • Status changed from inprogress to verify
  • Resolution set to fixed

This is being verified as pull request #104.

comment:26 Changed 6 years ago by Peter Peterson

  • Blocking 10772 added

comment:27 Changed 6 years ago by Ross Whitfield

I tested this on Ubuntu 14.10 and it works using the openshift server ("uid": "65e3df6194b3eddcb92405aede40c649"). But it should probably also be tested by someone using Windows and OSX.

comment:28 Changed 6 years ago by Pete Peterson

please retest this

comment:29 Changed 6 years ago by Pete Peterson

retest this please

comment:30 Changed 6 years ago by Pete Peterson

Re #10559. Fixing doxygen warnings.

Changeset: 3c37e2634b25783928bb43324b3c58b97bd36fb4

comment:31 Changed 6 years ago by Pete Peterson

retest this please

comment:32 Changed 6 years ago by Martyn Gigg

This seems fine on Windows behind a proxy. Perhaps @OwenArnold could check Mac?

comment:33 Changed 6 years ago by Pete Peterson

It is DownloadInstrumentTest that is intermittently failing on windoze.

comment:34 Changed 6 years ago by Pete Peterson

Re #10559. Fixing usage tests.

Changeset: 93712d3eb54678d47161884daafad71092de6626

comment:35 Changed 6 years ago by Pete Peterson

Can @quantumsteve try this on mac?

comment:36 Changed 6 years ago by Steven Hahn

  • Status changed from verify to verifying
  • Tester set to Steven Hahn

comment:37 Changed 6 years ago by Peter Peterson

  • Blocking 10810 added

comment:38 Changed 6 years ago by Steven Hahn

Also seems fine on Mac.

comment:39 Changed 6 years ago by Pete Peterson

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: 69ba3238e3c5cdf41c81a9862b75fc229369b162

comment:40 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: 172baacca6d54fa1537d77eca7a751df9f42b0b1

comment:41 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: e9423bdb34b07213a69caa90913e40307c17c6cc

comment:42 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: 978f0faba260ace3478f21c0f31b35dd70ce73f4

comment:43 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: 0307cae04d7ea2a81cf29d0cb39726afad8bbace

comment:44 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: a70324907bbdcc829c94dbac5aed26eeffa8316a

comment:45 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: 5b74bcc0844ddc63a77b624fe047a3fdc122022e

comment:46 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: a71f37d91884ba4658a1c64a6e84c427272bdcdd

comment:47 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/master' into feature/10559_stat_service

Full changeset: 1751650bd3e4f578583ca1f5a0a03d403293998b

comment:48 Changed 6 years ago by Steven Hahn

Merge pull request #104 from mantidproject/feature/10559_stat_service

New algorithm that sends json document to statistics service

Full changeset: fd37a14fb0c0d19730eaf15b8a485ddeb977a8be

comment:49 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11401

Note: See TracTickets for help on using tickets.