Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ jobs:
with:
context: ./perry
load: true
tags: workspace:latest
tags: perry:latest
cache-from: type=registry,ref=${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:buildcache-test
cache-to: ${{ steps.docker-changes.outputs.changed == 'true' && format('type=registry,ref={0}/{1}:buildcache-test,mode=max', env.REGISTRY, env.IMAGE_NAME) || '' }}

Expand Down
2 changes: 1 addition & 1 deletion docs/docs/troubleshooting.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ Common issues and their solutions when working with Perry.
docker ps
```

### "Image 'workspace:latest' not found"
### "Image 'perry:latest' not found"

**Symptoms:** Workspace creation fails with missing image error.

Expand Down
2 changes: 1 addition & 1 deletion src/shared/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ export const DEFAULT_AGENT_PORT = 7391;
export const SSH_PORT_RANGE_START = 2200;
export const SSH_PORT_RANGE_END = 2400;

export const WORKSPACE_IMAGE_LOCAL = 'workspace:latest';
export const WORKSPACE_IMAGE_LOCAL = 'perry:latest';
export const WORKSPACE_IMAGE_REGISTRY = 'ghcr.io/gricha/perry';

export const VOLUME_PREFIX = 'workspace-';
Expand Down
8 changes: 4 additions & 4 deletions test/setup/global.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ async function cleanupOrphanedResources() {

function imageExists() {
try {
execSync('docker image inspect workspace:latest', { stdio: 'ignore' });
execSync('docker image inspect perry:latest', { stdio: 'ignore' });
return true;
} catch {
return false;
Expand All @@ -38,20 +38,20 @@ function imageExists() {

async function buildImage() {
if (process.env.SKIP_DOCKER_BUILD === 'true' && imageExists()) {
console.log('\n✅ Using existing workspace:latest image (SKIP_DOCKER_BUILD=true)\n');
console.log('\n✅ Using existing perry:latest image (SKIP_DOCKER_BUILD=true)\n');
return;
}

if (imageExists() && !process.env.FORCE_DOCKER_BUILD) {
console.log('\n✅ Using existing workspace:latest image\n');
console.log('\n✅ Using existing perry:latest image\n');
return;
}

return new Promise((resolve, reject) => {
console.log('\n🏗️ Building workspace Docker image once for all tests...\n');

const buildContext = path.join(process.cwd(), 'perry');
const proc = spawn('docker', ['build', '-t', 'workspace:latest', buildContext], {
const proc = spawn('docker', ['build', '-t', 'perry:latest', buildContext], {
stdio: 'inherit',
});

Expand Down