Project

General

Profile

action #90989

containers: Configurable workdir location in docker-compose

Added by ilausuch 4 months ago. Updated 3 months ago.

Status:
Workable
Priority:
Normal
Assignee:
-
Category:
Feature requests
Target version:
Start date:
2021-04-12
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

Motivation

Currently, the workdir is created in the containers/webui directory with two problems:

  • This clutters the git directory. (The path is already in .gitignore, though.)
  • The directory is owned by root causing the tidy script to fail on permission errors.

Acceptance criteria

  • AC1: Use a different default directory or at least prevent the interference with the tidy script.
  • AC2: Allow to configure the workdir path, e.g. via an environment variable.
  • AC3: Document where files are stored by default and AC2.

Related issues

Related to openQA Project - action #89755: container: Fix missing shared directories and its permissionsResolved2021-03-09

History

#1 Updated by okurz 4 months ago

thank you for recording the idea in a ticket. But please find proper acceptance criteria. What you stated are good suggestions

#2 Updated by okurz 4 months ago

  • Target version set to future

#3 Updated by mkittler 4 months ago

  • Description updated (diff)

#4 Updated by ilausuch 3 months ago

  • Related to action #89755: container: Fix missing shared directories and its permissions added

#5 Updated by cdywan 3 months ago

  • Subject changed from containers: Fix workdir location in docker-compose to containers: Configurable workdir location in docker-compose
  • Description updated (diff)
  • Status changed from New to Workable

I was asking ilausuch in chat, but repeating my point here for the record. AC2: Allow to configure the workdir path sounds like a feature request to me. And I'm not clear that the workdir as it is used now breaks anything. Hence I think the title of the ticket should reflect this better.

#6 Updated by ilausuch 3 months ago

Yes, it's a feature

Also available in: Atom PDF