Skip to content

Conversation

@ahal
Copy link
Collaborator

@ahal ahal commented Nov 5, 2025

This flag will setup the current local checkout as a volume to the task's checkout and avoid all VCS operations.

This allows developers to make changes to the source and immediately see it in action in the task.

@ahal ahal self-assigned this Nov 5, 2025
@ahal ahal force-pushed the ahal/push-qkyoxpoqxlnt branch 2 times, most recently from e0b3f33 to b511048 Compare November 6, 2025 20:18
@ahal ahal marked this pull request as ready for review November 6, 2025 20:38
@ahal ahal requested a review from a team as a code owner November 6, 2025 20:38
@ahal ahal requested a review from abhishekmadan30 November 6, 2025 20:38
@ahal ahal force-pushed the ahal/push-qkyoxpoqxlnt branch 2 times, most recently from 527cfc2 to cf5c6e2 Compare November 7, 2025 21:15
@jcristau
Copy link
Contributor

I worry a bit that tasks might treat the checkout as throw-away, and the risk of losing data; but the warning in the doc might be good enough, and the only thing I know for sure can trash the repo is robustcheckout, and AIUI that is bypassed here.

One other thing that might cause issues is permissions, since the host and in-docker UIDs can easily not match.

@ahal
Copy link
Collaborator Author

ahal commented Nov 14, 2025

I worry a bit that tasks might treat the checkout as throw-away, and the risk of losing data; but the warning in the doc might be good enough, and the only thing I know for sure can trash the repo is robustcheckout, and AIUI that is bypassed here.

One other thing that might cause issues is permissions, since the host and in-docker UIDs can easily not match.

Yeah, anecdotally I've hit permission issues already. I'm not sure there's really a good solution for this. Regarding dataloss potential, yeah that's a bit riskier. We could maybe put up a warning prompt the user has to by-pass before proceeding? Though part of my motivation for this was to get Claude to fix task failures in the background, so I'd like to make sure it can still be used without any interaction.

@ahal ahal force-pushed the ahal/push-qkyoxpoqxlnt branch from cf5c6e2 to bb931b5 Compare November 14, 2025 15:19
ahal added 2 commits November 14, 2025 10:19
This flag will setup the current local checkout as a volume to the
task's checkout and avoid all VCS operations.

This allows developers to make changes to the source and immediately
see it in action in the task.
@ahal ahal force-pushed the ahal/push-qkyoxpoqxlnt branch from bb931b5 to 71f6cf7 Compare November 14, 2025 15:19
@ahal
Copy link
Collaborator Author

ahal commented Nov 14, 2025

Added a warning prompt the user needs to bypass. Can be disabled by setting an env.

@ahal ahal enabled auto-merge (rebase) November 14, 2025 15:20
@ahal ahal merged commit cc76aac into taskcluster:main Nov 14, 2025
16 checks passed
@ahal ahal deleted the ahal/push-qkyoxpoqxlnt branch November 14, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants