WIP: Collect uptime data in daily heartbeat from connect-ng

chajain commented 6 months ago

Description

Uptime data is included in the data sent to SCC via RMT https://jira.suse.com/browse/CSE-504 Fixes # (issue)

Change Type

Please select the correct option.

[ ] Bug Fix (a non-breaking change which fixes an issue)
[ ] New Feature (a non-breaking change which adds new functionality)
[ ] Documentation Update (a change which only updates documentation)

Checklist

Please check off each item if the requirement is met.

[X] I have verified that my code follows RMT's coding standards with rubocop.
[ ] I have reviewed my own code and believe that it's ready for an external review.
[ ] I have provided comments for any hard-to-understand code.
[ ] I have documented the MANUAL.md file with any changes to the user experience.
[ ] RMT's test coverage remains at 100%.
[ ] If my changes are non-trivial, I have added a changelog entry to notify users at package/obs/rmt-server.changes.

Other Notes

Please use this space to provide notes or thoughts to the team, such as tips on how to review/demo your changes.

How to test this PR

On the client system clone https://github.com/guangyee/connect-ng/tree/uptime_tracking_for_msps ( this branch has the connect-ng changes for uptime tracking feature )
Follow the instructions in https://github.com/guangyee/connect-ng/commit/f6ea992cb3b5fe7dda06f1a2b93abbb5ab4dc578
On the RMT system, clone this code
Execute rails db:migrate to create the system_uptimes table
Start the server bin/rails server -b 0.0.0.0
Register the system on client system using command suseconnect --url <rmt_system_ip> , it should populate the system_uptimes table with the entries from /etc/zypp/suse-uptime.logs on client system

felixsch commented 5 months ago

We have the rule of describing the at least a little bit in the PR description text what are you doing + having instructions how to test this changed behavior. Can you add these two missing items in the PR description?

chajain commented 5 months ago

Created new PR https://github.com/SUSE/rmt/pull/1090 , so closing this

SUSE / rmt