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.

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

Mount Workspace #4483

Merged
merged 2 commits into from
Oct 24, 2023
Merged

Conversation

DenisRumyantsev
Copy link
Contributor

@DenisRumyantsev DenisRumyantsev commented Oct 20, 2023

Description: With these changes, docker create will mount the pipeline workspace instead of the working directory.

Related issue

Steps to reproduce:

  • Run this pipeline twice and check the source folder after each run —
trigger: none

pool: <Self-Hosted Agent Pool>

variables:
  system.debug: true

resources:
  containers:
  - container: <container>
    image: <image>

jobs:
- job:
  steps:
  - checkout: self
    path: source/project/repo

  - bash: |
      cd ../..
      touch ./$(build.buildid).label
    target: <container>
  • Check the Initialize containers step —

    • The first time docker create runs with -v "<agent>\_work\1":"/__w/1"

    • The second time docker create runs with -v "<agent>\_work\1\source\project":"/__w/1/source/project"

Tip: Select Hide whitespace during the review —

image

@DenisRumyantsev DenisRumyantsev marked this pull request as ready for review October 21, 2023 04:16
@DenisRumyantsev DenisRumyantsev requested review from a team as code owners October 21, 2023 04:16
@DenisRumyantsev DenisRumyantsev force-pushed the users/DenisRumyantsev/mount-workspace branch 4 times, most recently from 460529c to b2f5920 Compare October 23, 2023 09:29
@DenisRumyantsev DenisRumyantsev force-pushed the users/DenisRumyantsev/mount-workspace branch from b2f5920 to b893bd3 Compare October 23, 2023 09:31
@DenisRumyantsev DenisRumyantsev merged commit b453d51 into master Oct 24, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants