Skip to content

bug: Update reservation_tool lockfile#197

Merged
pdettori merged 1 commit intokagenti:mainfrom
esnible:reservation-tool-lockfile
Mar 23, 2026
Merged

bug: Update reservation_tool lockfile#197
pdettori merged 1 commit intokagenti:mainfrom
esnible:reservation-tool-lockfile

Conversation

@esnible
Copy link
Copy Markdown
Contributor

@esnible esnible commented Mar 23, 2026

Summary

The reservation tool cannot be used because it expects a uv.lock that matches, but the file doesn't match.

Local build attempts with Docker fail with "The lockfile at uv.lock needs to be updated, but --locked was provided. To update the lockfile, run uv lock."

This PR was generated by running uv lock.

Related issue(s)

#196

This causes our release tagging to fail. The most recent failure is alpha.4: https://github.com/kagenti/agent-examples/actions/runs/23249686675

(Optional) Testing Instructions

I tested this using Podman and time docker build --load --tag ghcr.io/kagenti/agent-examples/reservation_tool:ecs-debug . from the
agent-examples/mcp/reservation_tool directory.

Fixes #

#196

Signed-off-by: Ed Snible <snible@us.ibm.com>
Copy link
Copy Markdown
Contributor

@pdettori pdettori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@pdettori pdettori merged commit c915ffc into kagenti:main Mar 23, 2026
9 checks passed
@esnible esnible deleted the reservation-tool-lockfile branch March 23, 2026 22:04
Copy link
Copy Markdown
Collaborator

@cwiklik cwiklik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lockfile sync fix — removes stale pyjwt dependency from uv.lock that was blocking Docker builds. LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants