feat(ci): Add comments for the preview deploy
Problem
Extend the preview deployment flow with some comments to give feedback to the user.
Changes
How did you test this code?
đ Stay up-to-date with PostHog coding conventions for a smoother review.
Changelog: (features only) Is this feature complete?
Hey @langesven! đ This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.
đĻ Preview instance
â Preview deployment ready
đ Access the instance
URL: https://do-ci-hobby-pr-43126.posthog.cc
SSH: ssh [email protected]
IP: 143.198.110.74
Mode: đ Preview (persistent)
Commit: 5c1d587
Workflow run: #4508
Full instance details
Droplet ID: 536042018
Droplet IP: 143.198.110.74
DNS Record ID: 1800232776
DNS Record Name: do-ci-hobby-pr-43126
SSH: ssh [email protected]
URL: https://do-ci-hobby-pr-43126.posthog.cc
Deployment output
đ Preview mode enabled - checking for existing droplet for PR #43126
âšī¸ No existing droplet found - creating new one
Creating preview droplet for PR #43126
Branch: sl/hobby-deploy-with-pr-comments
SHA: 5c1d587
Droplet name: do-ci-hobby-pr-43126
Droplet not booted yet - waiting a bit
Droplet not booted yet - waiting a bit
Droplet not booted yet - waiting a bit
Droplet not booted yet - waiting a bit
Droplet not booted yet - waiting a bit
completed
Public IP found: 143.198.110.74
Exporting the droplet ID: 536042018 and DNS record ID: 1800232776 for name do-ci-hobby-pr-43126
Preview instance has started. You will be able to access it here after PostHog boots (~15 minutes):
đ URL: https://do-ci-hobby-pr-43126.posthog.cc
Can I merge this to the main PR?
Can I merge this to the main PR?
yeah I think so, the label logic definitely works, the comments are created/updated for all relevant cases.
It might need some tweaks still based on the greptile comment though style: labeled trigger may cause workflow to run when any label is added, not just hobby-preview., but I think it'd then just go into the "normal" smoke-test flow so it should be okay
I see. Will definitely test more but wanted to consolidate now to get things in order (last working week of the year and so)