Open Bug 1853877 Opened 8 months ago Updated 2 months ago

mach try perf selects wrong base revision if branch is up to date with secondary remote

Categories

(Testing :: Performance, defect, P2)

defect

Tracking

(Not tracked)

People

(Reporter: jnicol, Unassigned)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [fxp])

I'm using git, and have two remotes: mozilla-unified via git-cinnabar, and my own github repo.

mach try perf selects the base revision here. This works okay as long as my local patches have not been pushed to any remote. However, if I have already pushed by patches to my github repo, then git rev-list HEAD --topo-order --boundary --not --remotes will return empty, and we end up using the current revision as the base revision.

I'm not sure exactly what the right solution is here, but here are a few thoughts:

  • We should probably error, or at least warn the user if the calculated base revision is the same as the current revision. It took me quite a while to figure out that the wrong revision was being pushed, let alone why. I doubt anyone wants to push the same revision twice, and --single-run exists for a single revision.
  • We could specify a remote to work around this, eg --remotes=origin. However, we cannot guarantee what the name of the correct remote to use is. Mine is actually called mozilla rather than origin.
  • Perhaps we could figure the remote out from the current branch's upstream, though that of course only works if the user has configured that properly
Whiteboard: [fxp]
Severity: -- → S3
Priority: -- → P2
You need to log in before you can comment on or make changes to this bug.