Skip to content

Bug 2019411 - Move update_verify and beetmover_transform from firefox-main to mozilla-taskgraph#174

Open
dandarnell wants to merge 2 commits intomozilla-releng:mainfrom
dandarnell:new-transforms
Open

Bug 2019411 - Move update_verify and beetmover_transform from firefox-main to mozilla-taskgraph#174
dandarnell wants to merge 2 commits intomozilla-releng:mainfrom
dandarnell:new-transforms

Conversation

@dandarnell
Copy link

Both transforms are used by both Firefox and Thunderbird, so they should be good project-agnostic candidates for migration from firefox-main -> mozilla-taskgraph.

I've also created the relevant patches to point Firefox's and Thunderbird's taskgraphs at mozilla-taskgraph for update_verify and beetmover_apt:

Firefox:
update_verify
beetmover_apt

Thunderbird:
update_verify
beetmover_apt

@dandarnell dandarnell requested a review from a team as a code owner March 8, 2026 23:34
@dandarnell dandarnell changed the title This adds two new transforms from firefox-main to mozilla-taskgraph Move update_verify and beetmover_transform from firefox-main to mozilla-taskgraph Mar 8, 2026
@dandarnell dandarnell changed the title Move update_verify and beetmover_transform from firefox-main to mozilla-taskgraph Bug 2019411 - Move update_verify and beetmover_transform from firefox-main to mozilla-taskgraph Mar 8, 2026
generate_artifact_registry_gcs_sources,
get_beetmover_apt_repo_scope,
get_beetmover_repo_action_scope,
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Sadly these imports won't work :)

We'll need to port these over as well or else re-write the transform to be a more focused on just the parts that aren't dependent on other stuff in gecko_taskgraph.

from taskgraph.util.copy import deepcopy
from taskgraph.util.treeherder import add_suffix, inherit_treeherder_from_dep

from gecko_taskgraph.util.attributes import task_name
Copy link
Contributor

Choose a reason for hiding this comment

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

This one needs to be ported as well.. But I think it's a lot simpler.

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.

2 participants