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

[Dialogue] Part 12: Fix Misdeclared Product Dependencies #4830

Merged
merged 3 commits into from
Jun 11, 2020

Conversation

jeremyk-91
Copy link
Contributor

Goals (and why):

  • @jackwickham notes that the product dependencies were not set up correctly for AtlasDB users of internal TimeLock. The main problem is that these were applied to the lock-conjure-api project, not its subprojects. The version was also inconveniently two versions too early.

Implementation Description (bullets):

  • Declare the product dependencies as pertaining to the scope of subprojects.

Testing (What was existing testing like? What have you done to improve it?):

  • I built the artifacts and read the manifest of the jars. No automated test written here.

Concerns (what feedback would you like?):

  • Should more aggressive corrective action be taken internally? I doubt it given that in spite of potentially incorrect dependencies, all monitored internal stacks are past the true minimum version needed.

Where should we start reviewing?: build.gradle

Priority (whenever / two weeks / yesterday): this week please

@jeremyk-91 jeremyk-91 requested a review from sudiksha27 June 10, 2020 16:16
@changelog-app
Copy link

changelog-app bot commented Jun 10, 2020

Generate changelog in changelog/@unreleased

Type

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

Description

The Conjure version of the V1 lock service now correctly depends on a version of TimeLock server that exposes these endpoints. Previously this wasn't the case, and a service which attempted to use the Conjure version would run into 404s when attempting to acquire or release locks.

Check the box to generate changelog(s)

  • Generate changelog entry

Copy link
Contributor

@sudiksha27 sudiksha27 left a comment

Choose a reason for hiding this comment

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

👍

@jeremyk-91 jeremyk-91 merged commit 1e25eb2 into develop Jun 11, 2020
@delete-merged-branch delete-merged-branch bot deleted the jkong/dialogue-prod branch June 11, 2020 08:54
@svc-autorelease
Copy link
Collaborator

Released 0.227.5

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.

3 participants