adoptium / aqa-test-tools

Home of Test Results Summary Service (TRSS) and PerfNext. These tools are designed to improve our ability to monitor and triage tests at the Adoptium project. The code is generic enough that it is extensible for use by any project that needs to monitor multiple CI servers and aggregate their results.
Apache License 2.0
28 stars 79 forks source link

Update TRSS sync script #856

Closed smlambert closed 1 week ago

smlambert commented 3 months ago

From the work that took place under https://github.com/adoptium/aqa-test-tools/issues/404 and the shift to running TRSS & its components inside of containers, we need to update the TRSS synch script / job.

The script is here and the ci.adoptium.net job is here.

We can update the script to stop docker, run the git pull / update and then start docker again (see instructions here).

sxa commented 3 months ago

Noting that, at present, the new TRSS machine is not connected to jenkins. If you need me to assist with that let me know (I only realised that today as I disconnected the old one which isn't there any more after seeing errors in the jenkins system log)

smlambert commented 3 months ago

Run locally and see that sometimes npm run docker-down doesn't always shut everything down. Think it may be good to update the package.json file L9 to:

"docker-down": "docker compose down --remove-orphans",

sxa commented 3 months ago

Run locally and see that sometimes npm run docker-down doesn't always shut everything down.

Yeah I've noted that in the message you get when you log into the Adoptium TRSS. I've definitely had to issue manual docker stop operations before.

smlambert commented 3 months ago

I have updated the package.json file in https://github.com/adoptium/aqa-test-tools/pull/868, so now orphans will get cleaned up, so will not expect future issues with things hanging open after docker-down. That sync script has been run manually on the machine. Once trss.adoptium.net is added to Jenkins, I will update the sync job in Jenkins to run weekly.