Skip to content

Commit

Permalink
step output is very noisy as logs and cost us a lot in logging (#627)
Browse files Browse the repository at this point in the history
  • Loading branch information
wintonzheng authored Aug 9, 2024
1 parent ea21f9e commit 9342dfb
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions skyvern/forge/agent.py
Original file line number Diff line number Diff line change
Expand Up @@ -1408,7 +1408,7 @@ async def update_step(
update_comparison = {
key: {"old": getattr(step, key), "new": value}
for key, value in updates.items()
if getattr(step, key) != value
if getattr(step, key) != value and key != "output"
}
LOG.info(
"Updating step in db",
Expand Down Expand Up @@ -1542,7 +1542,6 @@ async def handle_completed_step(
step_id=step.step_id,
step_order=step.order,
step_retry=step.retry_index,
output=step.output,
max_steps=max_steps_per_run,
)
last_step = await self.update_step(step, is_last=True)
Expand All @@ -1559,7 +1558,6 @@ async def handle_completed_step(
step_id=step.step_id,
step_order=step.order,
step_retry=step.retry_index,
output=step.output,
)
next_step = await app.DATABASE.create_step(
task_id=task.task_id,
Expand Down

0 comments on commit 9342dfb

Please sign in to comment.