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 revision isn't actually deployed #23

Closed
samos123 opened this issue Jan 14, 2021 · 4 comments
Closed

New revision isn't actually deployed #23

samos123 opened this issue Jan 14, 2021 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@samos123
Copy link

The github action reports success however when I go to cloud run revision console it shows the latest deployed revision was from a week ago even though the github action ran today. The only way I can get the GH action to work again is to manually deploy a new revision from Cloud Run UI and then run the GH action again. That will work. However then any subsequent GH action won't result in new revision being deployment.

I highly suspect the logic with revision naming is causing issues. This might be the code that's causing the behaviour: https://github.com/google-github-actions/deploy-cloudrun/blob/main/src/cloudRun.ts#L194

This is the GH action example where it reported success but didn't actually deploy a new revision: https://github.com/websu-io/websu/runs/1693522408?check_suite_focus=true

Happy to show it through screenshare as well

@samos123 samos123 added the bug Something isn't working label Jan 14, 2021
@averikitsch
Copy link
Contributor

Thanks for submitting, this looks similar to issue #12.

@samos123
Copy link
Author

How can I easily try out your pull request to verify that it fixes the issue?

@averikitsch
Copy link
Contributor

How can I easily try out your pull request to verify that it fixes the issue?

That can be a little complicated with the repo setup. The PR doesn't include the compiled code - that gets built on PR merge. I did add tests for the use case to ensure revisions are being created.

However, I pushed a branch with the compiled code which can be referenced in your workflow:

uses: google-github-actions/deploy-cloudrun@revision-test

Let me know if you have any issues.

@averikitsch
Copy link
Contributor

Now available in v0.3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

2 participants