Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

OP-312 telemetry module #479

Merged
merged 28 commits into from
Jul 4, 2022
Merged

OP-312 telemetry module #479

merged 28 commits into from
Jul 4, 2022

Conversation

goto-eof
Copy link
Contributor

@goto-eof goto-eof commented Aug 26, 2021 •
edited by mwithi
Loading

goto-eof and others added 16 commits August 18, 2021 00:29
@goto-eof
enable sql file execution that starts with 7*
8c84df2
@goto-eof
added telemetry module + refactoring
8bae421
@mwithi
Chore: PatientExamination code cleaning
d362e48
@dbmalkovsky
Chore: use parameterized logging instead of concatenation (#469)
88d64d3
@dbmalkovsky
Chore: remove unused imports (#470)
33a2941
@dbmalkovsky
Chore: collapse duplicate/similar/subtype catches into single stateme…
3108f29 
…nt (#471)
@dbmalkovsky
Chore: remove unnecessary boxing/unboxing (#472)
8262fdd
@dbmalkovsky
Chore: missorted modifiers (#473)
c8bab15
@dbmalkovsky
Chore: nullcheck of movement when variable previously dereferenced (#474
ed8d8c3 
)
@dbmalkovsky
Chore: month values are zero based; so 11 not 12 (#475)
ea3b5be
@dbmalkovsky
OP-628 update maven wrapper version (#468)
3baf986
@mwithi
Using full version number
aa3e229
@mwithi
OP-392 Fix: not zero cost check only when charging
cf8afbd
@dbmalkovsky
Chore: add missing Override method annotation (#478)
81255ba
@goto-eof
Telemetry updates
17f1556 
added Telemetry daemon + configuration
renamed SmsSender class to SmsDaemon
@goto-eof
resolved conflict
072befc
@mwithi mwithi changed the title Op 312 telemetry module draft OP-312 telemetry module Sep 1, 2021
@mwithi mwithi marked this pull request as draft September 1, 2021 09:37
@goto-eof
implemented data collectors
6e28143 
- implemented most of data collectors
- removed useless class BooleanUtils
- corrected indentation
@goto-eof
completed last two data collectors
8685194 
- completed last two data collectors
- refactoring
- added exception management
- added freegeoip remote service interface in
org.isf.org.isf.remote.iptools.geoip.freegeoip (we need a global remote
package?)

not tested yet!!!
@goto-eof
corrections based on Alessandro's suggestions
a411627
@goto-eof
fixed HQL queries (tested with HQL validator only)
a80f470
@goto-eof
Seems that it is working....Needs further tests and small corrections
860e33f 
Fixed Single Connection class (see database.properties.sample in gui),
GeoIP response wrapper, Opd query
@goto-eof
fixed telemetry daemon running bug
cb4aba6
@goto-eof
implementation telemetry module completed
efc182b 
+ added also simulated message sending (when DEBUG=yes)
+ tested daemon
+ fixed TimeDataCollector -> if no record found sends current date
@goto-eof goto-eof marked this pull request as ready for review September 8, 2021 00:34
@mwithi mwithi mentioned this pull request Sep 13, 2021
Merged
@mwithi mwithi marked this pull request as draft May 27, 2022 13:58
@mwithi mwithi reopened this Jul 4, 2022
@mwithi mwithi changed the base branch from develop to OP-312-telemetry-module July 4, 2022 16:27
@mwithi mwithi marked this pull request as ready for review July 4, 2022 16:29
@mwithi mwithi merged commit 70e49c8 into informatici:OP-312-telemetry-module Jul 4, 2022
mwithi added a commit that referenced this pull request Sep 16, 2022
@AndreiDodu @mwithi @dbmalkovsky
Recovery OP-312 telemetry module (#479)
848f020 
* enable sql file execution that starts with 7*

* added telemetry module + refactoring

* Chore: PatientExamination code cleaning

* Chore: use parameterized logging instead of concatenation (#469)

* Chore: remove unused imports (#470)

* Chore: collapse duplicate/similar/subtype catches into single statement (#471)

* Chore: remove unnecessary boxing/unboxing (#472)

* Chore: missorted modifiers (#473)

* Chore: nullcheck of movement when variable previously dereferenced (#474)

* Chore: month values are zero based; so 11 not 12 (#475)

* OP-628 update maven wrapper version (#468)

* Using full version number

* OP-392 Fix: not zero cost check only when charging

* Chore: add missing Override method annotation (#478)

* Telemetry updates
added Telemetry daemon + configuration
renamed SmsSender class to SmsDaemon

* resolved conflict

* implemented data collectors

- implemented most of data collectors
- removed useless class BooleanUtils
- corrected indentation

* - added todo

* - added header license

* completed last two data collectors

- completed last two data collectors
- refactoring
- added exception management
- added freegeoip remote service interface in
org.isf.org.isf.remote.iptools.geoip.freegeoip (we need a global remote
package?)

not tested yet!!!

* corrections based on Alessandro's suggestions

* fixed HQL queries (tested with HQL validator only)

* Seems that it is working....Needs further tests and small corrections

Fixed Single Connection class (see database.properties.sample in gui),
GeoIP response wrapper, Opd query

* fixed telemetry daemon running bug

* implementation telemetry module completed

+ added also simulated message sending (when DEBUG=yes)
+ tested daemon
+ fixed TimeDataCollector -> if no record found sends current date

* Added Telemetry menu item in the Settings menu

* refactoring dbSingleConnection + bug fix TelemetryDaemon

Co-authored-by: Alessandro Domanico <[email protected]>
Co-authored-by: David B Malkovsky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbmalkovsky dbmalkovsky dbmalkovsky left review comments

@mwithi mwithi mwithi left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@goto-eof @dbmalkovsky @mwithi @AndreiDodu