Page MenuHomePhabricator

fundraising_code_update could handle bad submodule shas better
Closed, ResolvedPublic

Description

If you accidentally merge a submodule commit pointing to an unavailable SHA, then run fundraising_code_update, it barrels ahead with the cleanup phase and asks you to delete everything. It should stop and yell at you.

Details

Related Changes in Gerrit:

Event Timeline

Change 397883 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/dash@deployment] Bad submodule pointer commit

https://gerrit.wikimedia.org/r/397883

Change 397883 abandoned by Ejegg:
Bad submodule pointer commit

https://gerrit.wikimedia.org/r/397883

This should be fixed, the new behavior is that a fatal git error causes the script to exit. A bad submodule hash shows up during the preview/lint stage, and exiting at this stage prevents git changes beyond fetch.

Also the state file updates immediately after project changes, so fatal errors in one project shouldn't interfere with state tracking for other projects.