Skip to content

Conversation

ahal
Copy link
Collaborator

@ahal ahal commented Sep 5, 2025

This will make it easier to test out images (e.g, the Decision image) in other repos before we merge them in Taskgraph.

@ahal ahal self-assigned this Sep 5, 2025
@ahal ahal changed the title Implement blobless / shallow clone in run-task ci: index docker-images by pull request number Sep 5, 2025
@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch from e5b343f to 8728ebb Compare September 5, 2025 18:12
@ahal
Copy link
Collaborator Author

ahal commented Sep 5, 2025

Example of how I'm using this:
mozilla-releng/staging-firefox@8f8872b

(I guess we don't even need to land this PR for my testing purposes, but might as well)

@ahal ahal added the CI Issues affecting Taskgraph's own CI label Sep 5, 2025
@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch 4 times, most recently from 4c73607 to 20c2b32 Compare September 11, 2025 14:28
@ahal ahal marked this pull request as ready for review September 11, 2025 14:29
@ahal ahal requested a review from a team as a code owner September 11, 2025 14:29
@ahal ahal requested a review from Eijebong September 11, 2025 14:29
@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch 2 times, most recently from 7364316 to 9b4feb1 Compare September 11, 2025 14:30
Copy link
Contributor

@jcristau jcristau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I'm sold on this being something we need/want more generally.. so -0 from me.

@ahal
Copy link
Collaborator Author

ahal commented Sep 11, 2025

Fair enough, I don't really care too much.

Though keep in mind that in the future Gecko will be using the decision image rather than the gecko_decision image, so testing changes to the decision image cross repo will become more common.

@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch 4 times, most recently from 4f23f88 to ac2a53c Compare October 7, 2025 17:38
@ahal
Copy link
Collaborator Author

ahal commented Oct 7, 2025

I think I would like to get this landed as I keep alternating between rebasing onto this stack to test, and then rebasing off of it to get review. I've accidentally forgotten to rebase onto it a few times, resulting in me wasting time due to the wrong image getting used ><

@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch 2 times, most recently from fe9f34e to e9fa779 Compare October 7, 2025 17:53
@ahal
Copy link
Collaborator Author

ahal commented Oct 7, 2025

Latest version of the patch makes this only apply to Taskgraph CI rather than everywhere.

@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch from e9fa779 to bc05f73 Compare October 7, 2025 17:53
ahal added 2 commits October 7, 2025 13:54
This will allow us to index tasks by pull request which can help testing
changes in external repos.
@ahal ahal force-pushed the ahal/push-tkkuzuptxytm branch from bc05f73 to b2831c2 Compare October 7, 2025 17:54
@ahal ahal requested review from jcristau and a team October 7, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Issues affecting Taskgraph's own CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants