Page MenuHomePhabricator

Get kartotherian/kartotherian@master working with its dependencies (kartotherian/*) @master
Closed, ResolvedPublic

Description

At the time of writing this, kartotherian/kartotherian depends on old-ish versions of its dependencies.

In order to be able to develop and deploy a fix or a feature to one of the dependency, it has to be in working condition and contain no surprise changes that were not integrated with the main repo.

packageprod versioncommits after prod versionnotes
kartotherian/core0.0.238Updated to 0.1.1 (commit)
kartotherian/server0.0.175Updated to 0.0.22 (commit)
kartotherian/autogen0.0.100
kartotherian/babel0.0.78Will be handled as part of the i18n work
kartotherian/cassandra0.0.126Updated to 0.0.13 (commit)
kartotherian/postgres0.0.101Updated to 0.0.11 (commit)
kartotherian/demultiplexer0.0.101Not used, removed from config, removed from package.json
kartotherian/layermixer0.0.80
kartotherian/overzoom0.0.105PR was merged. Updated to use latest (commit)
kartotherian/substantial0.0.85Updated to use latest (commit)
kartotherian/tilelive-http0.12.10
kartotherian/osm-bright-source0.0.50
kartotherian/osm-bright-style2.1.40
kartotherian/geoshapes0.0.130
kartotherian/maki0.0.100
kartotherian/snapshot0.3.41Updated to use latest (commit)
kartotherian/input-validator0.0.60

Event Timeline

Mattflaschen-WMF renamed this task from Get kartotherian/server master working with kartotherian/kartotherian update-snapshot-dev to Get kartotherian/server master branch working with kartotherian/kartotherian update-snapshot-dev branch.Feb 15 2018, 3:21 AM

Ideally, kartotherian/kartotherian@master should work with kartotherian/*@master

All update-snapshot-dev does is reference kartotherian/snapshot@master instead of some older tag version. And that PR appears to be on hold for now.

For reference, this shrinkwrap file lists what's currently deployed to production.

kartotherian/core/lib/sources.js Sources.prototype.init was updated to accept a single 'conf' argument instead of 'variables' and 'sources' (commit from April 18, 2017).

Consumer of this code, kartotherian/kartotherian/routes/kartotherian.js, was not updated to pass a single argument.

SBisson renamed this task from Get kartotherian/server master branch working with kartotherian/kartotherian update-snapshot-dev branch to Get kartotherian/kartotherian@master working with its dependencies (kartotherian/*) @master.Feb 16 2018, 8:53 PM
SBisson updated the task description. (Show Details)
This comment was removed by SBisson.

Change 413381 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[maps/kartotherian/deploy@master] Specify modules in config file

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

Change 413429 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[maps/kartotherian/deploy@master] Specify requestHandlers in config

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

Change 413381 merged by jenkins-bot:
[maps/kartotherian/deploy@master] Specify modules in config file

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

Change 415350 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[maps/kartotherian/deploy@master] Remove unused demultiplexer from modules

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

Change 415848 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[maps/kartotherian/package@master] Package the 'update-deps2' branch

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

Change 415350 merged by Gehel:
[maps/kartotherian/deploy@master] Remove unused demultiplexer from modules

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

Change 413429 merged by Gehel:
[maps/kartotherian/deploy@master] Specify requestHandlers in config

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

Change 415848 merged by Gehel:
[maps/kartotherian/package@master] Package the 'update-deps2' branch

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

The package and deploy repos have been updated to be able to deploy the 'update-deps2' branch, which should be done on Monday (March 5th).

Once that branch is successfully tested on maps-test, a PR will be created to get it merged into kartotherian/kartotherian.

The 'update-deps2' branch has been deployed to maps-test* servers and tested successfully.

The PR to bring it to master has been merged. Next step is to deploy it to production.

Change 417237 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[maps/kartotherian/package@master] Package v0.0.34 for deployment

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

Change 417237 merged by Gehel:
[maps/kartotherian/package@master] Package v0.0.34 for deployment

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

Change 417345 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[maps/kartotherian/deploy@master] Put requestHandlers at the right place in config

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

Change 417345 merged by jenkins-bot:
[maps/kartotherian/deploy@master] Put requestHandlers at the right place in config

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