Update deprecated ghcr.io image reference to public.ecr.aws - #40
Update deprecated ghcr.io image reference to public.ecr.aws#40luisina-santos wants to merge 1 commit into
Conversation
README.md referenced this connector's image via ghcr.io/conductorone; container images are now published to public.ecr.aws/conductorone instead. Only this connector's own image reference was updated. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
| ``` | ||
| docker run --rm -v $(pwd):/out -e BATON_CONNECTION_STRING="user=baton password=baton host=localhost port=5432 dbname=hammerhead_production" ghcr.io/conductorone/baton-retool:latest -f "/out/sync.c1z" | ||
| docker run --rm -v $(pwd):/out -e BATON_CONNECTION_STRING="user=baton password=baton host=localhost port=5432 dbname=hammerhead_production" public.ecr.aws/conductorone/baton-retool:latest -f "/out/sync.c1z" | ||
| docker run --rm -v $(pwd):/out ghcr.io/conductorone/baton:latest -f "/out/sync.c1z" resources |
There was a problem hiding this comment.
🟡 Suggestion: The next line in the same code block still points at ghcr.io/conductorone/baton:latest. That's the baton CLI image rather than this connector's image, so it's outside the stated scope of this PR — but if ghcr.io is deprecated for all ConductorOne images, this reference will break for anyone copy-pasting the docker section. Worth confirming whether public.ecr.aws/conductorone/baton:latest exists and updating it here too.
Connector PR Review: Update deprecated ghcr.io image reference to public.ecr.awsBlocking Issues: 0 | Suggestions: 1 | Threads Resolved: 0 Review SummaryThe full PR diff was scanned for security and correctness: it is a single-line documentation change in Security IssuesNone found. Correctness IssuesNone found. Suggestions
Note: the machine-readable review-state marker could not be written by this run because the sandbox command guard rejects it; the next review will run in full mode. Reviewed head SHA 2dc332a against base 778d209. Prompt for AI agents |
Summary
ghcr.io/conductorone/baton-retool:latest; container images are now published topublic.ecr.aws/conductorone/baton-retool:latestinstead.Test plan
public.ecr.aws/conductorone/baton-retool:latestresolves (manifest exists) before opening this PR.🤖 Generated with Claude Code