Skip to content
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

github-secrets: use keyless #1170

Merged
merged 4 commits into from
Jun 20, 2024
Merged

github-secrets: use keyless #1170

merged 4 commits into from
Jun 20, 2024

Conversation

v1v
Copy link
Member

@v1v v1v commented Jun 19, 2024

Use Keyless access.

Test -> https://github.com/elastic/apm-agent-php/actions/runs/9585060144

image

There were some failures but all of them have been addressed now:

Expand to view

Created credentials file at "/home/runner/work/apm-agent-php/apm-agent-php/gha-creds-c42c301e3f0c0e16.json"
Error: google-github-actions/auth failed with: failed to generate Google Cloud federated token for //iam.googleapis.com/projects/911195782929/locations/global/workloadIdentityPools/github/providers/repo-28ae8e252eba0f7b95257e8cb2c: {"error":"invalid_target","error_description":"The target service indicated by the \"audience\" parameters is invalid. This might either be because the pool or provider is disabled or deleted or because it doesn't exist."}
  Error: google-github-actions/upload-cloud-storage failed with: {"code":412,"message":"The type of authentication token used for this request requires that Uniform Bucket Level Access be enabled.","errors":[{"message":"The type of authentication token used for this request requires that Uniform Bucket Level Access be enabled.","domain":"global","reason":"conditionNotMet","locationType":"header","location":"If-Match"}]}
$ gsutil uniformbucketlevelaccess set on gs://apm-agent-php
Enabling Uniform bucket-level access for gs://apm-agent-php...

@v1v v1v self-assigned this Jun 19, 2024
@v1v v1v marked this pull request as ready for review June 19, 2024 16:07
@v1v v1v requested review from a team June 19, 2024 16:07
@v1v v1v merged commit d0b3530 into elastic:main Jun 20, 2024
84 of 170 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants