Missing download for generating statistics

Bug #1793513 reported by Hachmann
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape-docs
Undecided
Unassigned

Bug Description

Running 'make' in the updated Inkscape docs repo, it works until it gets to the statistics generation part.

That ends with

### Making '' in 'statistics'
make[1]: Verzeichnis „/.../Branches/ink-documentation/statistics“ wird betreten
Makefile:7: 'INKBRANCH' is not set, assuming 'master'
# downloading updated Inkscape po directory from CI
rm -rf inkscape.po translations.zip
wget --output-document=translations.zip --no-verbose \
        https://gitlab.com/inkscape/inkscape/-/jobs/artifacts/master/download?job=translations
https://gitlab.com/inkscape/inkscape/-/jobs/artifacts/master/download?job=translations:
2018-09-20 13:42:04 FEHLER 404: Not Found.
Makefile:39: die Regel für Ziel „po-inkscape“ scheiterte
make[1]: *** [po-inkscape] Fehler 8
make[1]: Verzeichnis „/.../Branches/ink-documentation/statistics“ wird verlassen
Makefile:5: die Regel für Ziel „default“ scheiterte
make: *** [default] Fehler 2

I could find something related at
https://gitlab.com/inkscape/inkscape/-/jobs/99261648/artifacts/download
which is for the translations job for a specific commit.

However, I couldn't find a more generic url that would lead to a similar page. Would be quite annoying if gitlab's urls changed like this, so it doesn't accept url parameters for job names anymore :-/

According to gitlab documentation, the link looks correct.
(https://docs.gitlab.com/ee/user/project/pipelines/job_artifacts.html#downloading-the-latest-artifacts)

It also doesn't seem to be a permissions issue, as I can access the zip from the download buttons on the page.

Don't know what could be wrong...

Revision history for this message
Patrick Storz (ede123) wrote :

Hmm, can you open the link manually from within a browser?

Opening
  https://gitlab.com/inkscape/inkscape/-/jobs/artifacts/master/download?job=translations
works just fine for me right now. Also the latest CI run succeeded [1].

The idea behind this URL is that it should always yield the latest build artifact making it unnecessary to specify a job id (as that would indeed be annoying ;-) )

It could also have been a temporary outage of GitLab.

[1] https://gitlab.com/inkscape/inkscape-docs/documentation/-/jobs/99451352

Revision history for this message
Hachmann (marenhachmann) wrote :

Yes, seems to work again in the browser (it didn't previously) - now I'm running make, to try out your fixes for the other issue :)

Revision history for this message
Hachmann (marenhachmann) wrote :

Mmh. Getting merge conflicts in inkscape-web-i18n... meh.

Revision history for this message
Hachmann (marenhachmann) wrote :

Deleted the old website-i18n repo, and ran make again (thanks for cloning the git repos automatically now!) - worked!

Revision history for this message
Patrick Storz (ede123) wrote :

> Getting merge conflicts in inkscape-web-i18n

Should be fixed in
  https://gitlab.com/inkscape/inkscape-docs/documentation/commit/f037f4a05887c3ea6858ccdf498e756eb19693aa

I'll close this bug until we'll be able to reproduce the download issue (hopefully we won't).

Changed in inkscape-docs:
status: New → Invalid
status: Invalid → Confirmed
Revision history for this message
Patrick Storz (ede123) wrote :

OK, reproduced (I only tought of re-testing after closing...)

Revision history for this message
Patrick Storz (ede123) wrote :
Revision history for this message
Hachmann (marenhachmann) wrote :

Thanks for investigating and also fixing the depth, Patrick!

Revision history for this message
Patrick Storz (ede123) wrote :

As https://gitlab.com/gitlab-org/gitlab-ce/issues/45697 is not really our issue and I wouldn't know how to work around it I'll close this for now.

I've set the schedule for creation of the source documentation to weekly (it was daily), so this issue will hopefully be less frequent, but obviously might still occur occasionally (Sundays at 4:00am to be precise ;-) ).

Changed in inkscape-docs:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.