sneak/upaas
1
0
Fork 0
Code Issues 21 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Use app name and deployment ID in build directory structure

Browse Source 
Change build directory from builds/<app-id>-<random> to
builds/<appname>/<deployment-id>-<random> for better organization
and easier debugging.
This commit is contained in:
Jeffrey Paul 2025-12-30 11:57:02 +07:00
parent 2ed23912a9
commit 4ece7431af
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
internal/service/deploy/deploy.go
View File

@ -205,17 +205,18 @@ func (svc *Service) cloneRepository(
deployment *models.Deployment, deployment *models.Deployment,
) (string, func(), error) { ) (string, func(), error) {
// Use a subdirectory of DataDir for builds since it's mounted from the host // Use a subdirectory of DataDir for builds since it's mounted from the host
// and accessible to Docker for bind mounts (unlike /tmp inside the container) // and accessible to Docker for bind mounts (unlike /tmp inside the container).
buildsDir := filepath.Join(svc.config.DataDir, "builds") // Structure: builds/<appname>/<deployment-id>-<random>
appBuildsDir := filepath.Join(svc.config.DataDir, "builds", app.Name)
err := os.MkdirAll(buildsDir, buildsDirPermissions) err := os.MkdirAll(appBuildsDir, buildsDirPermissions)
if err != nil { if err != nil {
svc.failDeployment(ctx, app, deployment, fmt.Errorf("failed to create builds dir: %w", err)) svc.failDeployment(ctx, app, deployment, fmt.Errorf("failed to create builds dir: %w", err))
return "", nil, fmt.Errorf("failed to create builds dir: %w", err) return "", nil, fmt.Errorf("failed to create builds dir: %w", err)
} }
tempDir, err := os.MkdirTemp(buildsDir, app.ID+"-*") tempDir, err := os.MkdirTemp(appBuildsDir, fmt.Sprintf("%d-*", deployment.ID))
if err != nil { if err != nil {
svc.failDeployment(ctx, app, deployment, fmt.Errorf("failed to create temp dir: %w", err)) svc.failDeployment(ctx, app, deployment, fmt.Errorf("failed to create temp dir: %w", err))