-
-
Notifications
You must be signed in to change notification settings - Fork 217
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
[CEP] Deploy History Report #27839
Comments
+1 Would it make sense to stop writing a git tag on each deploy? It seems unnecessary if we have a record of deploy commits in the deploy history report. That would reduce the tag clutter in github. |
When I saw that message in my email, I thought for a sec you were quoting me, because I wrote that almost verbatim in the "open questions" section above, but later deleted it. I somewhat agree, especially for the Since as a rule, we don't force-push to master, perhaps a better state would be a single As an aside, I'm also mulling over the idea of adding a search field to the deploy history page that accepts a git ref and shows the first deploy to contain that ref. |
I like this idea. The only tag we care about 99% of the time is the most recent one (for each env). I've also noticed that the performance of some git tools is negatively affected by the presence of many tags.
That sounds useful. |
+1 |
Abstract
3rd party environments do not currently have access to commit information in deploy history. This CEP proposes to make this information available on all environments. Additionally, it proposes moving the deploy history report to a standalone page.
Motivation
To better keep track of when changes are deployed on third party environments.
Specification
For reference, here's what the deploy report currently looks like:
This is at the bottom of the system info page, which is mostly broken and has a lot of other stuff going on.
HQ only stores the diff URL, and then not for third-party repos, this actually looks very easy to address:
last_commit_sha
has a sufficient fallback when tags are unavailable, meaning that this clause should be unnecessary.However, it would be preferable to also correct this to store the commit directly, rather than parsing it from the URL:
HqDeploy
modelcommit
field to store the git commit directly.diff_url
field when possible (note that these utils do related stuff already)record_deploy_success
to also accept thiscommit
field and pass it when instantiating a newHqDeploy
.commcare-cloud
will need the following:record_successful_deploy
to pass the current commit to the mgmt command to be recorded.And finally, that deploy history table should be moved to a standalone, paginated report. This should fall under the existing "System Health" header.
UserListReport
Impact on users
None
Impact on hosting
Should make useful information more visible.
Backwards compatibility
Release Timeline
Open questions and issues
The text was updated successfully, but these errors were encountered: