-
Notifications
You must be signed in to change notification settings - Fork 10
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.ActionsEnvironmentSecret
gives 404 Not Found
#248
Comments
This seems related, but not the same as #246 |
github.get_actions_public_key(repository)
returns 404 Not Found
githubActionsEnvironmentSecret
gives 404 Not Found
githubActionsEnvironmentSecret
gives 404 Not Found
github.ActionsEnvironmentSecret
gives 404 Not Found
Possibly related to integrations/terraform-provider-github#667 |
Could this be related to integrations/terraform-provider-github#578? integrations/terraform-provider-github#578 (comment) suggests setting the owner field of the Provider configuration as a workaround. |
I'm also having a related problem here. When I run pulumi up locally, it does work and create the GitHub secret. But, when running with GitHub actions, it seems to duplicate the GITHUB_OWNER in the URL:
Github Action: When I try removing the GITHUB_OWNER, the instead of duplicating GITHUB_OWNER. It adds my github username to the url:
Maybe I'm missing something here, I'm passing the exact same env variables that I use locally to the GitHub actions. |
I'm also having a similar (but not the same) issue:
It looks like, somehow, there's an extra slash being interpolated for me. I tried to use the workaround above and it didn't fix the issue. Any suggestions? Edit: managed to work around it by adding |
This reverts commit fa089a1.
This reverts commit fa089a1.
* Revert "I think this should finally fix the issues..." This reverts commit a2651d0. * Revert "see if removing GITHUB_OWNER will make pulumi / tf happier" This reverts commit eb9696f. * Revert "try one last time to see if I can meddle with the repo name to get the github integration to work" This reverts commit f35a857. * Revert "- provide GITHUB_TOKEN to Staging (oops)" This reverts commit 1f6ab05. * Revert "remove presubmit requirement for staging" This reverts commit d7e3dc3. * Revert "Remove the insanely expensive NAT gateway and test for missing" This reverts commit 5d86ec7. * Revert "re introduce ../ to see if we can make the bastard Actions secret.'" This reverts commit 2647abf. * Revert "follow aws provided example better https://github.com/aws-samples/container-patterns/blob/ea823c0c43a0818e1d1e73ea0b646846af504563/pattern/pulumi-ecs-service-in-vpc/files/service-in-vpc.ts\#L17" This reverts commit 352a3f0. * Revert "instruct the Fargate service to wait on the internet gateway and nat gateways on a hunch" This reverts commit 10a0dc0. * Revert "move listener out of awsx/alb -- couldnt work out how to correctly specify ARN." This reverts commit 565587a. * Revert "fixes" This reverts commit 0184110. * Revert "fix missed parent" This reverts commit 47e9309. * Revert "fix alb name" This reverts commit 91461b2. * Revert "progress" This reverts commit e05fc98. * Revert "progress" This reverts commit d8b3bbf. * Revert "remove code that didnt do anything" This reverts commit 7d6e9fd. * Revert "fix build in ts/pulumi/..." This reverts commit 67826bf. * Revert "Manually add transient peer @aws-sdk/region-config-resolver" This reverts commit 81d1615. * Revert "upgrade aws-sdk in an attempt to get @aws-sdk/region-config-resolver available" This reverts commit 54465eb. * Revert "- Add @aws-sdk/region-config-resolver (pulumi was complaining" This reverts commit f67876f. * Revert "Attempt to fix Pulumi dependency issue by upgrading deps." This reverts commit 379626e. * Revert "Change error to warning for Pulumi testing." This reverts commit 8759be5. * Revert "Try to test if the GitHub credentials are incorrect." This reverts commit 14636e7. * Revert "Try to see if we can use the correct repo name again." This reverts commit a00b58c. * Revert "try to automagically get a gateway set up. also i need to fix pulumi which was interrupted in a broken state apparently" This reverts commit 8e28892. * Revert "currently there is a problem where the fargate compute units cannot access the outside internet. perhaps this will fix it..." This reverts commit 0814c09. * Revert "I hope this counts as progress" This reverts commit 5c331db. * Revert "many changes" This reverts commit a1906bc. * Revert "giving up for the night. i might have to implement this awsx feature myself, because i dont know why its trying to proxy :https" This reverts commit 5650545. * Revert "try to fix 443 error..." This reverts commit f89f7c8. * Revert "try to correct ports" This reverts commit 14d412e. * Revert "correct ports" This reverts commit 9127801. * Revert "public IP" This reverts commit adfd3ad. * Revert "write the dockerfile instead of just trying to send its text" This reverts commit bf58a88. * Revert "expose ALB directly to the internet" This reverts commit 3774738. * Revert "just a minor push to force a redeploy" This reverts commit 127f167. * Revert "fix formatting" This reverts commit 68a892b. * Revert "oops" This reverts commit c7cc113. * Revert "add new access token and create staging secret" This reverts commit f95b9cd. * Revert "crazy hack that just might work to fix https://github.com/pulumi/pulumi-github/issues/248\#issuecomment-1742415825" This reverts commit cd94510. * Revert "fix for pulumi/pulumi-github#248" This reverts commit fa089a1. * Revert "the api is annoying. so we are doing this instead" This reverts commit b09d530. * Revert "use fullName with actions secret" This reverts commit ec218df. * Revert "set GITHUB_TOKEN for pulumi access" This reverts commit 6a52c00. * Revert "update bazel_rce code to github org monorepo" This reverts commit d231257. * Revert "Bazel remote caching" This reverts commit 9929e93. * re-add access to GITHUB_TOKEN so old infra can be torn down. * add dummy bootstrap_bazel_remote_cache.sh * chmod
This comment has been minimized.
This comment has been minimized.
The extra slash indeed seems to be related. Removing it resulted in a 403, requiring admin access:
|
Hello everyone. I've resolved this by following the instructions here https://www.pulumi.com/registry/packages/github/installation-configuration/#configuring-credentials. |
Hi @smith558 - thank you for reporting here! I'm going to close this issue as resolved for now - but do feel free to file a follow-up issue if you believe we can improve the experience. |
What happened?
Hi there,
TL;DR I believe
github.ActionsEnvironmentSecret
may be hitting an out of date GitHub endpoint.I'm trying to create a new
github.Repository
via Pulumi and also create 3 GitHub actions environments with several secrets in each.This forum answer made me realize you have to encrypt strings before you can send them to the GitHub API via the pulumi resource.
The
github.get_actions_public_key(repository)
recommended in the forum answer calls this endpoint (GitHub docs on the endpoint here) to fetch the "GitHub environment public key".Here is the error message I get on
pulumi up
when I try to do this:Note the repository URL is
543829938
in this example, and the environment name isSandbox
. The repo name issample-repo
.I was able to reproduce the error by hitting the GitHub API myself like so:
I was able to solve it by removing the
$REPO_NAME:
portion of the request path:Can the underlying provider be modified so that the correct URL endpoint is used?
Steps to reproduce
Expected Behavior
A public key is fetched.
Actual Behavior
A public key is not found, instead a
404
error is returned.Output of
pulumi about
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).
The text was updated successfully, but these errors were encountered: