mathjazz / pontoon

In-place localization tool
https://pontoon.mozilla.org/
BSD 3-Clause "New" or "Revised" License
3 stars 1 forks source link

Daily sync errors for android-l10n project #662

Closed mathjazz closed 3 years ago

mathjazz commented 3 years ago

This issue was created automatically with bugzilla2github.

Bug 1726957

Bug Reporter: @flodolo CC: @mathjazz

Starting August 18, we constantly get Sync errors from android-l10n

! [rejected]            HEAD -> master (fetch first)
error: failed to push some refs to 'git@github.com:mozilla-l10n/android-l10n.git'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.

My guess is that Pontoon doesn't know how to deal gracefully with two projects working on the same repository, given only Pontoon writes to it these days.

mathjazz commented 3 years ago

Comment Author: @mathjazz

Correct, we have the same problem in l10n-central repos, which is why we exclude those errors from email alerts. We should do the same for the Android errors, but I'm not sure Papertrail search syntax supports that.

The HG error message is verbose enough in the first line to filter out, which is not the case for the GitHub message, and I'm not sure we can filter out messages based on surrounding lines.

--

HG: Aug 21 09:28:26 mozilla-pontoon app/worker.1 Failed to sync locale el for project seamonkey due to commit error: abort: push creates new remote head ce5b5f67d119!

GitHub: Aug 21 15:18:12 mozilla-pontoon app/worker.1 Failed to sync locale hu for project focus-for-android due to commit error: To github.com:mozilla-l10n/android-l10n.git

--

We'll likely need to intercept the error message before we log it and modify it accordingly to be able to catch it.