deployments icon indicating copy to clipboard operation
deployments copied to clipboard

New Delete Env Outside Deployments Check

Open brolewis opened this issue 2 years ago • 6 comments

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

brolewis avatar Jan 23 '23 18:01 brolewis

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.

schorfES avatar Jan 23 '23 18:01 schorfES

@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 avatar Jan 23 '23 18:01 brolewis

@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).

schorfES avatar Jan 24 '23 09:01 schorfES

cc @buckett could you take a look at this?

bobheadxi avatar Jan 30 '23 21:01 bobheadxi

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

gugaiz avatar Feb 03 '23 15:02 gugaiz

Is there a workaround for this?

dstockhammer avatar Jul 21 '23 08:07 dstockhammer