Skip to content

Za/central tdms summariser - #744

Merged
amrc-za merged 11 commits into
mainfrom
za/central-tdms-summariser
Sep 15, 2026
Merged

amrc-za merged 11 commits into
mainfrom
za/central-tdms-summariser

Conversation

@amrc-za

@amrc-za amrc-za commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

New acs-file-summariser service that watches for specific file types (currently only implemented for TDMS) via ConfigDB NotifyAPI. Upon receiving a notification about new file upload the service creates a summary of the TDMS file and saves it to InfluxDB.

This is a new central service which watches for File uploads through ConfigDB NotifyAPI and creates a summary. Currently it only supports TDMS file type.
@amrc-za
amrc-za requested a review from djnewbould September 7, 2026 12:23

@djnewbould djnewbould left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and well documented! I've just left a couple of comments. I think it also needs adding to the github build action

Comment thread deploy/values.yaml Outdated
Comment thread acs-file-summariser/Dockerfile
Comment thread acs-file-summariser/lib/influx.js
@amrc-za
amrc-za requested a review from djnewbould September 9, 2026 10:28
@amrc-za

amrc-za commented Sep 15, 2026

Copy link
Copy Markdown
Contributor Author

I have also added optional device_uuid to be recorded as part of Files metadata in ConfigDB.
File Summariser reads the device_uuid and stores it as topLevelInstance tag in influxdb.

@amrc-za
amrc-za merged commit 9d94c77 into main Sep 15, 2026
1 check passed
@amrc-za
amrc-za deleted the za/central-tdms-summariser branch September 15, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants