search

NordicHPC / sonar

Tool to profile usage of HPC resources by regularly probing processes.
GNU General Public License v3.0
8 stars 5 forks source link

github actions user id may contain dash #192

Closed lars-t-hansen closed 1 month ago

lars-t-hansen commented 1 month ago

Seen on #189, #190: spurious test failure in tests/ps-syntax.sh. It failed because the user name that was seen did not conform to the pattern in the test. Turns out that the user name is chosen not from [a-z0-9]+ alone, there can also be -. We should fix this, but also add logging as in #189 to handle future issues.

lars-t-hansen commented 1 month ago

It would have been useful to see this earlier: https://serverfault.com/questions/73084/what-characters-should-i-use-or-not-use-in-usernames-on-linux