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

New Delete Env Outside Deployments Check #135

Open
brolewis opened this issue Jan 23, 2023 · 6 comments
Open

New Delete Env Outside Deployments Check #135

brolewis opened this issue Jan 23, 2023 · 6 comments

Comments

@brolewis
Copy link

It looks like the new call to deleteAnEnvironment is outside the check for deployments. Seeing errors when trying to run delete-env in a workflow where that specific run didn't have an environment.

https://github.com/bobheadxi/deployments/blob/main/src/lib/delete.ts#L39-L43

@schorfES
Copy link

Same here, receiving the following error when deleting an environment:

unexpected error encountered: HttpError: Resource not accessible by integration
Error: unexpected error encountered: HttpError: Resource not accessible by integration - see logs for more information

I assume this problem is related to the described issue by @brolewis.

@brolewis
Copy link
Author

@schorfES Maybe, but I just realized that for our workflows I thought we were using a PAT token, but we in fact were not, so that can cause problems according to the docs for this action as well, so in my case, it might be a user error.

@schorfES
Copy link

@schorfES Maybe, but I just realized that for our workflows I thought we were using a PAT token, but we in fact were not, so that can cause problems according to the docs for this action as well, so in my case, it might be a user error.

@brolewis thank you for that hint. I will investigate in that direction. I was wondering because it worked as expected for the last ~5 month and is failing since yesterday morning. Yes, probably a token is expired but we currently do not pass them explicitly to the job (following the docs).

@bobheadxi
Copy link
Owner

cc @buckett could you take a look at this?

@gugaiz
Copy link

gugaiz commented Feb 3, 2023

I am explicitly passing the PAT token and the env. It has been failing for the last 2 weeks with the same error posted here

@dstockhammer
Copy link

Is there a workaround for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants