One quirk of our infra is that only a specific, more restricted instance has credentials to deploy things, and we use infra.isTrusted() to determine whether it's that instance. So the check for master doesn't suffice. We can probably just keep that part out completely as the trusted.ci instance's crawler job is still freestyle. I'll adjust that myself on merge.
@eroshenkoam Looks pretty good, thanks!
One quirk of our infra is that only a specific, more restricted instance has credentials to deploy things, and we use
infra.isTrusted()
to determine whether it's that instance. So the check formaster
doesn't suffice. We can probably just keep that part out completely as the trusted.ci instance's crawler job is still freestyle. I'll adjust that myself on merge.