gnu-octave / symbolic

A Symbolic Package for Octave using SymPy
https://octave.sourceforge.io/symbolic/
GNU General Public License v3.0
152 stars 36 forks source link

CI: Run tests on Octave 7.2.0. #1200

Closed alexvong243f closed 2 years ago

alexvong243f commented 2 years ago

Fixes #1197.

cbm755 commented 2 years ago

I didn't realize there was no 'latest', I thought that was a builtin feature of Containers...

Perhaps its not worth the complexity?

alexvong243f commented 2 years ago

------- Original Message ------- On Thursday, August 4th, 2022 at 3:44 PM, Colin B. Macdonald @.***> wrote:

I didn't realize there was no 'latest', I thought that was a builtin feature of Containers...

According to https://www.howtogeek.com/devops/understanding-dockers-latest-tag/ "latest" is used as the default tag if no tag is provided. On the other hand, it won't be created automatically if a tag is specified.

Perhaps its not worth the complexity?

I think we have 3 options here. The 1st option is to simply not test the image with the "latest" tag, which is the status quo. The 2nd option is to modify the build process of octave image so that it tags the latest image with the "latest" tag. The 3rd option is to find out the latest image ourselves, which indeed introduces some complexity. WDYT?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

cbm755 commented 2 years ago

I vote for 1 for now. And possibly file an issue upstream if you think 2 is something that should be done.

cbm755 commented 2 years ago

@alexvong1995 this sort of CI failure is relatively common:

octave: X11 DISPLAY environment variable not set
octave: disabling GUI features
error: pkg: could not download file /tmp/generate_html-0.3.3-n6hTZo.tar.gz from URL https://packages.octave.org/download/generate_html-0.3.3.tar.gz
error: called from
    pkg at line 507 column 13
Error: Process completed with exit code 1.

I assume its some kind of rate limiter that gets grouchy when certain IPs download lots of packages (but no evidence).

Anyway, just FYI. I never assume CI should have 100% reliability: we get what we pay for afterall!

alexvong243f commented 2 years ago

The test now passes on main. So probably just a temporary issue.

If the problem reoccurs, we can always use the torsocks -i command from Tor to switch to a fresh IP for each invocation.