HomePhabricator

[mock-chronik-client] Patch issue with aCI deployments of mock-chronik-client

Description

[mock-chronik-client] Patch issue with aCI deployments of mock-chronik-client

Summary:
Missed this when moving repos over to local deps instead of npmjs.

The dockerfile here is copying the full monorepo into the mock-chronik-client workdir, instead of just the mock-chronik-client dir.

This causes publication of the whole monorepo as a module to (thankfully) fail as the npm publish auth is then not at the expected dir level.

Test Plan:
Review and sanity check. Can fully test locally but can't really test this without CI, as the credential is required to publish.

Can confirm we are trying to publish the right thing tho.

Change line 31 to CMD [ "npm", "publish", "--dry-run"]

docker build -f mock-chronik-client.Dockerfile -t mock-chronik-client_local .
docker run mock-chronik-client_local

output:

npm notice === Tarball Details ===
npm notice name:          mock-chronik-client
npm notice version:       1.9.0
npm notice filename:      mock-chronik-client-1.9.0.tgz
npm notice package size:  10.6 kB
npm notice unpacked size: 57.1 kB
npm notice shasum:        918ea19edadde5163538ee324a8022e22a259e23
npm notice integrity:     sha512-jyiv+DDhmv0fv[...]azqrcjcl19gUA==
npm notice total files:   7
npm notice
npm WARN This command requires you to be logged in to https://registry.npmjs.org/ (dry-run)
npm notice Publishing to https://registry.npmjs.org/ with tag latest and default access (dry-run)
+ mock-chronik-client@1.9.0

in line with what is published currently.

compare to CI error unpacked size: 100.7 MB

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D16227

Details

Provenance
bytesofmanAuthored on May 28 2024, 13:20
bytesofmanPushed on May 28 2024, 13:43
Reviewer
Restricted Project
Differential Revision
D16227: [mock-chronik-client] Patch issue with aCI deployments of mock-chronik-client
Parents
rABC9abe1c8ba966: [ecash-herald, mock-chronik-client, alias-server] Migrate ecash-herald to in…
Branches
Unknown
Tags
Unknown