Remove pulls trigger #450
Remove pulls trigger #450
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #450 +/- ##
==========================================
- Coverage 90.58% 89.97% -0.61%
==========================================
Files 401 324 -77
Lines 12508 9199 -3309
Branches 2103 1632 -471
==========================================
- Hits 11330 8277 -3053
+ Misses 1069 859 -210
+ Partials 109 63 -46
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
nora-codecov
force-pushed
the
nora/1029
branch
from
39e18e1 to
9f5c4fd
Compare
shared/django_apps/core/models.py
Outdated
| if self.state != PullStates.OPEN.value: | ||
| # while a pull is OPEN, we check whether to write_to_storage | ||
| # when a pull is no longer OPEN, we no longer want the flare in storage. | ||
| # The nightly cron job cleans up the value in storage (see FlareCleanupTask in worker) | ||
| return False |
There was a problem hiding this comment.
this would rather mean that the flare is being stored in the JSON field of the table.
I’m unsure what would happen if you change this value based on the state. will is copy over data from storage to the database? or will that only happen if you save a new value in the field?
There was a problem hiding this comment.
good call - I think I was thinking that if this came back False it wouldn't be saved anywhere, but you're right, this would just force it onto the db field. Removing.
nora-codecov
force-pushed
the
nora/1029
branch
from
2e54905 to
96d0b0c
Compare
| @@ -147,7 +147,7 @@ def delete_file(self, bucket_name, path): | |||
| except ClientError: | |||
| raise | |||
|
|
|||
| def delete_files(self, bucket_name, paths=[]): | |||
There was a problem hiding this comment.
This default value is dangerous and unnecessary. fixed the occurrences I found across the project.
nora-codecov
force-pushed
the
nora/1029
branch
3 times, most recently
from
710ef8b to
741151b
Compare
…rchiveService, remove default value from paths
nora-codecov
force-pushed
the
nora/1029
branch
from
741151b to
2331c4a
Compare
Part of the quest to reduce locks by eliminating triggers https://github.com/codecov/internal-issues/issues/1029
The trigger is deleted by these changes. The behavior is replaced in a cron job on worker codecov/worker#947.
What the trigger did: every time a pull was updated, if it was transitioning from
OPENto any otherstate, it set theflarefield tonull.flareis an Archive field, so the value is either in_flareon the object or in our Archive storage.Replacement behavior: see
FlareCleanupTaskcodecov/worker#947Once the trigger is removed, when the pull transitions from
OPENto any otherstate, leaveflarealone in order to reduce locks and wait time.The cron job runs overnight, gets all pulls that are non-
OPENand have flare (either in our db or in Archive storage), clears it (either in our db or in Archive storage) soflareisnull.more about
flare:should_write_to_storage