Skip to content

WIP: Fix artifact upload - #7739

Merged
tacaswell merged 1 commit into
matplotlib:masterfrom
jkseppan:artifact-upload
Oct 21, 2017
Merged

tacaswell merged 1 commit into
matplotlib:masterfrom
jkseppan:artifact-upload

Conversation

@jkseppan

@jkseppan jkseppan commented Jan 4, 2017

Copy link
Copy Markdown
Member

I noticed that the artifact upload isn't working: https://travis-ci.org/matplotlib/matplotlib/jobs/188282794#L1227

"gem install travis-artifacts" no longer works, now you have to use the artifacts addon and set some variables. The encrypted variables may need to be updated too. See documentation at https://docs.travis-ci.com/user/uploading-artifacts/

This worked when I tested with my own setup: https://travis-ci.org/jkseppan/matplotlib/builds/188857981

@mdboom, @tacaswell: do you have access to the matplotlib AWS organization and can you ensure that the encrypted Travis variables here are correct? The variable names should be ARTIFACTS_KEY and ARTIFACTS_SECRET.

@tacaswell tacaswell added this to the 2.1 (next point release) milestone Jan 4, 2017
@tacaswell
tacaswell requested a review from mdboom January 4, 2017 16:30
@tacaswell tacaswell modified the milestones: 2.2 (next next feature release), 2.1 (next point release), 2.1.1 (next bug fix release) Aug 29, 2017
"gem install travis-artifacts" no longer works, now you have to use
the artifacts addon and set some variables. The encrypted variables
may need to be updated too, the variable names should be ARTIFACTS_KEY
and ARTIFACTS_SECRET.

See documentation at https://docs.travis-ci.com/user/uploading-artifacts/
@tacaswell

Copy link
Copy Markdown
Member

@jkseppan I took the liberty of rebasing and force-pushing

@tacaswell tacaswell modified the milestones: v2.1.1, v2.2 Oct 21, 2017
@tacaswell

Copy link
Copy Markdown
Member

Going to merge this to see if it works (as it will only work on branch builds).

@tacaswell
tacaswell merged commit bd38f40 into matplotlib:master Oct 21, 2017
@matplotlib matplotlib deleted a comment from codecov-io Oct 21, 2017
@Kojoley
Kojoley removed the request for review from mdboom October 21, 2017 23:00
@QuLogic QuLogic modified the milestones: needs sorting, v2.2.0 Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants