Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Move isNewService check to TimelockAgent #5555

Merged

Conversation

leonz
Copy link
Contributor

@leonz leonz commented Jul 13, 2021

Goals (and why):
We are looking to introduce a dynamic check for is-new-service which is not based on the static configuration supplied to Timelock. To do this, we intend to override the value supplied by configuration.

So, we need to disable the invariant check that happens on immutable creation since this may not yet be the "final" state.

Implementation Description (bullets):

  • Move the invariant check to the TimelockAgent create function. I believe this is still the critical path.
  • Expose the PaxosInstallConfiguration and TimelockInstallConfiguration builders so that these objects can be re-created externally.

Testing (What was existing testing like? What have you done to improve it?):
Existing unit testing should be sufficient. Unfortunately any of the testing that checks for failure needs to be moved into TimelockAgentTest.

Concerns (what feedback would you like?):

Where should we start reviewing?:

Priority (whenever / two weeks / yesterday):
ASAP

@changelog-app
Copy link

changelog-app bot commented Jul 13, 2021

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Move isNewService check to TimelockAgent

Check the box to generate changelog(s)

  • Generate changelog entry

Copy link
Contributor

@jeremyk-91 jeremyk-91 left a comment

Choose a reason for hiding this comment

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

👍

@bulldozer-bot bulldozer-bot bot merged commit 47f1132 into develop Jul 15, 2021
@bulldozer-bot bulldozer-bot bot deleted the leonz/move-is-new-service-check-into-agent-creation branch July 15, 2021 14:31
@svc-autorelease
Copy link
Collaborator

Released 0.380.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants