Skip to content

Use Github Releases for hosting node binaries#6030

Merged
danpat merged 2 commits intoProject-OSRM:masterfrom
mjjbell:mbell/node_github_releases
May 17, 2021
Merged

Use Github Releases for hosting node binaries#6030
danpat merged 2 commits intoProject-OSRM:masterfrom
mjjbell:mbell/node_github_releases

Conversation

@mjjbell
Copy link
Copy Markdown
Member

@mjjbell mjjbell commented May 8, 2021

Issue

Replaces S3 hosting of node binaries with Github Releases.
node-pre-gyp publish works exclusively with S3, so upload step is now performed by the Travis deployment provider.

The behaviour for the package user should not change.
When building a new version tag, Travis will create a release for the tag if it does not already exist.

The secure token is my own personal token that has the public_repo Github OAuth permission.
This is not ideal - Travis recommends to create a machine user for deployments. Someone with owner or admin permissions for Project-OSRM would need to do this.

Testing

I tested this on my own fork (see the test release from this branch as an example).

Tasklist

Requirements / Relations

Fixes #6013

Replaces S3 hosting of node binaries with Github Releases.
`node-pre-gyp publish` works exclusively with S3, so upload step
is now performed by the Travis deployment provider.

The behaviour for the package user should not change.
When building a new version tag, Travis will create a release for the
tag if it does not already exist.
@mjjbell mjjbell changed the title Use Github Releases for hosting node_osrm binaries Use Github Releases for hosting node binaries May 8, 2021
@mjjbell mjjbell force-pushed the mbell/node_github_releases branch from 99ba3f2 to 24ad672 Compare May 8, 2021 17:11
@danpat
Copy link
Copy Markdown
Member

danpat commented May 17, 2021

@mjjbell I'd like to cut a new release to get #6020 out into the public.

I'll figure out how to create a standalone user to manage the auth token and update this PR.

@danpat danpat merged commit baca7b7 into Project-OSRM:master May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need new home for publishing binary modules

2 participants