Setting task_accepted_time before serialization#2963
Setting task_accepted_time before serialization#2963centosinfra-prod-github-app[bot] merged 1 commit intopackit:mainfrom
Conversation
|
✔️ pre-commit SUCCESS in 2m 23s |
4d41e54 to
0529306
Compare
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 44s |
lbarcziova
left a comment
There was a problem hiding this comment.
just one question, overall this approach LGTM, thanks!
0529306 to
b7638f6
Compare
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 46s |
b7638f6 to
5630c78
Compare
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 47s |
Signed-off-by: Jiri Podivin <jpodivin@redhat.com>
5630c78 to
df9c41f
Compare
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 47s |
|
Build succeeded (gate pipeline). ✔️ pre-commit SUCCESS in 1m 45s |
If we set
task_accepted_timeon the Event object before it proceeds to serialization, the value will correctly propagate to the database, andcopr_build_finished_timeshould start getting reported again.Since this code runs in a loop, and new times are getting appended, the last element of the list is used to set the value.