Add PR deployment preview

[mtrezza]

New Feature / Enhancement Checklist

• I am not disclosing a vulnerability.
• I am not just asking a question.
• I have searched through existing issues.

Current Limitation

Reviewing a dashboard PR requires more effort than PRs from other repos because

• Parse Dashboard is a visual product with UI
• We don't have UI testing as part of the CI

The dashboard PR has to be cloned locally in a development environment to see the effect of most PRs.

Feature / Enhancement Description

Add a preview deployment to easily review and preview PRs online without having to clone the PR locally.

See https://docs.uffizzi.com/references/uffizzi-ci/#2-install-authorize-the-uffizzi-github-app

Example Use Case

n/a

Alternatives / Workarounds

Clone locally.

3rd Party References

n/a

Uffizzi Feedback

• There seems to be an URL issue, see fix: Internal error message on login with missing credential #2370 (comment):

  https://https://app.uffizzi.com/projects/5259/deployments

• It would be nice to have all files in a .uffizzi directory. Currently we have:

  ./docker-compose.uffizzi.yml
  ./nginx-uffizzi/...

• When Uffizzi adds a comment it should probably delete any existing Uffizzi comments, otherwise the thread gets full of comments, and these comments are quite long.

• While deploying a new commit, the preview URL shows an nginx error page; it's unclear whether the deployment / PR has an issue; it would be better to show a "deployment is progress" page
  

[parse-github-assistant]

Thanks for opening this issue!

• 🎉 We are excited about your ideas for improvement!

[mtrezza]

@gadkins, @ShrutiC-git

I've opened this issue to the get Uffizzi integration working. #2364 has been merged.

I've tried existing PR #2370, closed and re-opened it. Uffizzi shows a deployment for that PR, but it didn't post a comment in the PR thread with the deployment URL. Does that only work for new PRs? Uffizzi posted a comment with the URL, so it looks good.

However, after dashboard login, when clicking on the server app, the screen appears blank. Could be a dashboard issue, I'll investigate.

[ShrutiC-git]

thanks for surfacing that, @mtrezza.

I'm sorry you're seeing that issue, indeed seems to be rather weird. So far I can tell, I'm not seeing any obvious issue with the preview environment itself. Do pass along your findings and if we might need to make any modifications, please let me know!

[mtrezza]

(Moved feedback section to top comment)

[mtrezza]

@dblythy If you have a minute, maybe you could take a quick look at this. When you open the preview URL in #2370 (comment) and log in to the dashboard, then a blank screen appears. Debugging the browser it seems to point to #2363. Maybe parsing that specific URL there fails because of the way it's composed.

[mtrezza]

@ShrutiC-git should Uffizzi update the deployment when a new commit happens on a PR? I took a look and there was no sign of re-deployment in the Uffizzi console after commits in #2375.

[ShrutiC-git]

@mtrezza, commented on #2375

I think we all could benefit from getting on a call and going through the process once, and also taking up issues the community is seeing. What do you think?

[mtrezza]

@gadkins you wrote:

Let me clarify Uffizzi behavior:

• You do not need to close and reopen PRs every time you push a new commit
• Uffizzi is receiving webhooks on every commit, so it will rebuild your application each time.
• Each time it rebuilds a new commit, it will preserve the state from the previous build if volumes are mounted in the compose. This tells Uffizzi to preserve this data as long as this PR is open.
• If you close a PR, Uffizzi will destroy the preview and all its state. If you reopen the PR, you will get an fresh environment with empty volumes and a new comment.
• On new commits, Uffizzi will update the existing comment on the PR. It will not post a new comment. The multiple comments you are seeing are caused by opening and closing the PR several times.
• You can see which commit is deployed in the Uffizzi Dashboard, along with the timestamp:

Thanks for clarifying this. I think what we would benefit from is an update of the PR comment by Uffizzi to see when the deployment has finished, without having to go to the Uffizzi dashboard. It seems unintuitive to me that when reopening a PR a new Uffizzi comment is posted, but when making a commit, no new comment is posted and the existing comment isn't updated in any way.

I think what would be more intuitive:

• a) Post 1 permanent comment and update it to indicate which commit has finished deploying
• b) Post 1 comment below the deployed commit and delete any previous Uffizzi comments, which may be more comfortable because one doesn't have to compare any hashes to find out which commit is deployed

I think we all could benefit from getting on a call

Sure, you can reach out to me on Slack and we can schedule a call

[ShrutiC-git]

hey, @mtrezza,

Thanks for clarifying this. I think what we would benefit from is an update of the PR comment by Uffizzi to see when the deployment has finished, without having to go to the Uffizzi dashboard. It seems unintuitive to me that when reopening a PR a new Uffizzi comment is posted, but when making a commit, no new comment is posted and the existing comment isn't updated in any way.

Absolutely, that is the desired behavior. Something might be off if it's not behaving that way for you. I'll write to you on Slack to schedule a call - we'll fix this so it's efficient and intuitive for the community.