-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove CI-specific UID/GID to appease GH's breaking change #50176
Conversation
Signed-off-by: Fred Heinecke <[email protected]>
Signed-off-by: Fred Heinecke <[email protected]>
@fheinecke See the table below for backport results.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this might've broken TestRootX11ForwardPermissions
on v15. It failed for me twice in a row.
sshserver_test.go:1098:
Error Trace: /__w/teleport/teleport/lib/srv/regular/sshserver_test.go:1098
Error: Received unexpected error:
user: unknown userid 1000
Test: TestRootX11ForwardPermissions
https://github.com/gravitational/teleport/actions/runs/12315623784/job/34374219868#step:6:1942
https://github.com/gravitational/teleport/actions/runs/12315086928/job/34372541318#step:6:1943
#49992
GitHub unexpectedly made a breaking change to the
runner
user used in GHA pipelines. This works around the issue by removing the hard-coded uid/gid values in our makefile.There is a reasonable chance that this will break some workflow, but it should get our Go security release out the door.
Passing build: https://github.com/gravitational/teleport.e/actions/runs/12304943632/job/34343525931 (failures are due to apple notarization service)
Refs: