mirror of
https://github.com/fabro-sh/fabro.git
synced 2026-09-08 22:21:45 +00:00
parent
4e67c0cb27
commit
1a11ea37f2
5 changed files with 79 additions and 0 deletions
57
checkpoint.json
Normal file
57
checkpoint.json
Normal file
|
|
@ -0,0 +1,57 @@
|
|||
{
|
||||
"timestamp": "2026-03-16T11:59:55.745655Z",
|
||||
"current_node": "setup",
|
||||
"completed_nodes": [
|
||||
"start",
|
||||
"setup"
|
||||
],
|
||||
"node_retries": {
|
||||
"start": 1,
|
||||
"setup": 1
|
||||
},
|
||||
"context_values": {
|
||||
"internal.fidelity": "compact",
|
||||
"thread.start.current_node": "setup",
|
||||
"failure_signature": "setup|deterministic|script failed with exit code: <n> ## stdout fatal: destination path '.' already exists and is not an empty directory.",
|
||||
"current.preamble": "Goal: delete() on instances of models without any dependencies doesn't clear PKs.\nDescription\n\t\nDeleting any model with no dependencies not updates the PK on the model. It should be set to None after .delete() call.\nSee Django.db.models.deletion:276-281. Should update the model line 280.\n\n\n\n## Additional Context\n\nReproduced at 1ffddfc233e2d5139cc6ec31a4ec6ef70b10f87f. Regression in bc7dd8490b882b2cefdc7faf431dc64c532b79c9. Thanks for the report.\nRegression test.\nI have attached a simple fix which mimics what https://github.com/django/django/blob/master/django/db/models/deletion.py#L324-L326 does for multiple objects. I am not sure if we need https://github.com/django/django/blob/master/django/db/models/deletion.py#L320-L323 (the block above) because I think field_updates is only ever filled if the objects are not fast-deletable -- ie https://github.com/django/django/blob/master/django/db/models/deletion.py#L224 is not called due to the can_fast_delete check at the beginning of the collect function. That said, if we want to be extra \"safe\" we can just move lines 320 - 326 into an extra function and call that from the old and new location (though I do not think it is needed).\n",
|
||||
"current_node": "setup",
|
||||
"failure_class": "deterministic",
|
||||
"graph.goal": "delete() on instances of models without any dependencies doesn't clear PKs.\nDescription\n\t\nDeleting any model with no dependencies not updates the PK on the model. It should be set to None after .delete() call.\nSee Django.db.models.deletion:276-281. Should update the model line 280.\n\n\n\n## Additional Context\n\nReproduced at 1ffddfc233e2d5139cc6ec31a4ec6ef70b10f87f. Regression in bc7dd8490b882b2cefdc7faf431dc64c532b79c9. Thanks for the report.\nRegression test.\nI have attached a simple fix which mimics what https://github.com/django/django/blob/master/django/db/models/deletion.py#L324-L326 does for multiple objects. I am not sure if we need https://github.com/django/django/blob/master/django/db/models/deletion.py#L320-L323 (the block above) because I think field_updates is only ever filled if the objects are not fast-deletable -- ie https://github.com/django/django/blob/master/django/db/models/deletion.py#L224 is not called due to the can_fast_delete check at the beginning of the collect function. That said, if we want to be extra \"safe\" we can just move lines 320 - 326 into an extra function and call that from the old and new location (though I do not think it is needed).",
|
||||
"internal.retry_count.start": 1,
|
||||
"internal.thread_id": "start",
|
||||
"internal.node_visit_count": 1,
|
||||
"internal.run_id": "01KKV88HD8QZR2PJTPE9EA7C6W",
|
||||
"outcome": "fail",
|
||||
"internal.retry_count.setup": 1,
|
||||
"command.stderr": "",
|
||||
"graph.rankdir": "LR",
|
||||
"command.output": "fatal: destination path '.' already exists and is not an empty directory.\n"
|
||||
},
|
||||
"logs": [],
|
||||
"node_outcomes": {
|
||||
"start": {
|
||||
"status": "success",
|
||||
"duration_ms": 0
|
||||
},
|
||||
"setup": {
|
||||
"status": "fail",
|
||||
"context_updates": {
|
||||
"command.stderr": "",
|
||||
"command.output": "fatal: destination path '.' already exists and is not an empty directory.\n"
|
||||
},
|
||||
"failure": {
|
||||
"message": "Script failed with exit code: 128\n\n## stdout\nfatal: destination path '.' already exists and is not an empty directory.\n",
|
||||
"failure_class": "deterministic"
|
||||
},
|
||||
"duration_ms": 38
|
||||
}
|
||||
},
|
||||
"next_node_id": "solve",
|
||||
"loop_failure_signatures": {
|
||||
"setup|deterministic|script failed with exit code: <n> ## stdout fatal: destination path '.' already exists and is not an empty directory.": 1
|
||||
},
|
||||
"node_visits": {
|
||||
"setup": 1,
|
||||
"start": 1
|
||||
}
|
||||
}
|
||||
5
nodes/setup/script_invocation.json
Normal file
5
nodes/setup/script_invocation.json
Normal file
|
|
@ -0,0 +1,5 @@
|
|||
{
|
||||
"command": "git clone https://github.com/django/django.git . && git checkout 19fc6376ce67d01ca37a91ef2f55ef769f50513a && python -m pip install -e .",
|
||||
"language": "shell",
|
||||
"timeout_ms": null
|
||||
}
|
||||
5
nodes/setup/script_timing.json
Normal file
5
nodes/setup/script_timing.json
Normal file
|
|
@ -0,0 +1,5 @@
|
|||
{
|
||||
"duration_ms": 38,
|
||||
"exit_code": 128,
|
||||
"timed_out": false
|
||||
}
|
||||
6
nodes/setup/status.json
Normal file
6
nodes/setup/status.json
Normal file
|
|
@ -0,0 +1,6 @@
|
|||
{
|
||||
"status": "fail",
|
||||
"notes": null,
|
||||
"failure_reason": "Script failed with exit code: 128\n\n## stdout\nfatal: destination path '.' already exists and is not an empty directory.\n",
|
||||
"timestamp": "2026-03-16T11:59:55.745477+00:00"
|
||||
}
|
||||
6
nodes/start/status.json
Normal file
6
nodes/start/status.json
Normal file
|
|
@ -0,0 +1,6 @@
|
|||
{
|
||||
"status": "success",
|
||||
"notes": null,
|
||||
"failure_reason": null,
|
||||
"timestamp": "2026-03-16T11:59:55.705725+00:00"
|
||||
}
|
||||
Loading…
Add table
Reference in a new issue