I think it's okay, but I have one uncertainty: the example in the gitlab docs that uses grouping (?:v.+|main|release.*) uses a non-capturing group ?:. I'm not sure why, maybe gitlab uses the capturing groups internally? Doesn't sound very logical that it would though. Given that we don't use the capturing group you introduce with the braces, it wouldn't hurt to add the non-capturing ?: to it, except I have the feeling we would be spreading some weird hoax about it being necessary.
I tried checking the git blame of the documentation to see whether it was discussed, but the lines were moved without using git mv, so it's hard to find the original MR in the git history. Maybe we should I will add it just to be sure...
Not sure if the cleanup has been triggered at all, it sayd "clean up will happen in 4 days", so maybe the 7 day timer got reset after I last changed the cleanup policy.
I set it to run the clean up every day now, so we can check on Monday what it has done.
The clean-up seems to have cleaned up around 18 images. There were still a lot of non-versioned images lying around, because I set it to "keep the 25 most recent images". I've set that number to 5 now, so I expect the dashboard registry to contain less than 43 images tomorrow.