Skip to content
This repository has been archived by the owner on Apr 29, 2020. It is now read-only.

move preparer hook installation and create preparer init hook type #1100

Merged

Conversation

esuen
Copy link
Contributor

@esuen esuen commented Dec 13, 2018

move when hooks are installed for p2-preparer from after preparer init to within preparer init
create new hook type PreparerInit to be ran after hooks are installed in preparer initialization

mpuncel and others added 2 commits December 10, 2018 17:06
* square/master:
  add feature to mark platform hooks as requred in preparer config
  Modified test for err == nil when reading invalid bool
  Adding readbool method to config
  grpc scheduler: add configurable request timeout
@esuen esuen force-pushed the suen.reorder-p2-preparer-hooks-installation branch from 64fa057 to 1a1eeda Compare December 13, 2018 23:27
…t to within preparer init.

Create new hook type PreparerInit to be ran after hooks are installed in preparer initialization.
Properly error in ExitUnlessEvent if unrecognized hook type is passed.
@esuen esuen force-pushed the suen.reorder-p2-preparer-hooks-installation branch from d6e586e to 6dc5de5 Compare December 17, 2018 23:20
@esuen esuen merged commit 79a5b83 into square:master Dec 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants