Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add working-dir option to service definition #158

Closed
benhoyt opened this issue Oct 18, 2022 · 0 comments · Fixed by #245
Closed

Add working-dir option to service definition #158

benhoyt opened this issue Oct 18, 2022 · 0 comments · Fixed by #245

Comments

@benhoyt
Copy link
Contributor

benhoyt commented Oct 18, 2022

Both pebble exec (one-shot commands) and exec-type health checks have an optional working-dir argument, which allows you to specify the working directory the command runs in. Services, oddly, don't have this. We should add working-dir to the plan.Service struct here.

Then once --context is implemented (for one-shot commands and exec health checks) that would also be included in the context. See #125.
This was referenced Oct 18, 2022
Closed
Closed
@cjdcordeiro cjdcordeiro mentioned this issue Jun 23, 2023
Merged
benhoyt added a commit to benhoyt/pebble that referenced this issue Jun 26, 2023
@benhoyt
Add support for service's to configure working directory
987e5d2 
Fixes canonical#158 (in preparation for canonical#159)
@benhoyt benhoyt mentioned this issue Jun 26, 2023
Merged
benhoyt added a commit that referenced this issue Jun 26, 2023
@benhoyt
Add support for service's to configure working directory (#245)
a807026 
This adds support for a "working-dir" option in the layered service
configuration, for starting a service in a specified working directory
(the default is still to start it in Pebble's working directory).

Fixes #158
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for services to configure working directory benhoyt/pebble
1 participant
@benhoyt