Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: switch to ubuntu-22.04 image to avoid timeout #2730

Merged
merged 4 commits into from
Jan 23, 2023

Conversation

neteler
Copy link
Member

@neteler neteler commented Jan 4, 2023

Since the older ubuntu-20.04 image based runs commonly time-out, here an attempt with the current ubuntu-22.04 image.

See also: https://github.com/actions/runner-images

Since the older ubuntu-20.04 image based runs commonly time-out, here an attempt with the current ubuntu-22.04 image.
@neteler neteler added the CI Continuous integration label Jan 4, 2023
@neteler neteler added this to the 8.3.0 milestone Jan 4, 2023
@neteler neteler self-assigned this Jan 4, 2023
@neteler
Copy link
Member Author

neteler commented Jan 4, 2023

Mhh, this triggers unrelated other issues:

gcc  -std=gnu11 -fPIC -Wall -Wno-error=maybe-uninitialized -Werror  -I/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include -I/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include   -I/usr/include/gdal -I/usr/include  -DPACKAGE=\""grasslibs"\"     -I/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include -I/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include -DRELDIR=\"lib/ogsf\" -o OBJ.x86_64-pc-linux-gnu/gs2.o -c gs2.c
gs2.c: In function ‘GS_get_distance_alongsurf’:
gs2.c:3296:5: error: ‘gsd_real2surf’ accessing 12 bytes in a region of size 8 [-Werror=stringop-overflow=]
 3296 |     gsd_real2surf(gs, p1);
      |     ^~~~~~~~~~~~~~~~~~~~~
gs2.c:3296:5: note: referencing argument 2 of type ‘float *’
In file included from /home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include/grass/ogsf.h:539,
                 from gs2.c:48:
/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include/grass/defs/ogsf.h:569:6: note: in a call to function ‘gsd_real2surf’
  569 | void gsd_real2surf(geosurf *, Point3);
      |      ^~~~~~~~~~~~~
gs2.c:3297:5: error: ‘gsd_real2surf’ accessing 12 bytes in a region of size 8 [-Werror=stringop-overflow=]
 3297 |     gsd_real2surf(gs, p2);
      |     ^~~~~~~~~~~~~~~~~~~~~
gs2.c:3297:5: note: referencing argument 2 of type ‘float *’
In file included from /home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include/grass/ogsf.h:539,
                 from gs2.c:48:
/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/include/grass/defs/ogsf.h:569:6: note: in a call to function ‘gsd_real2surf’
  569 | void gsd_real2surf(geosurf *, Point3);
      |      ^~~~~~~~~~~~~
cc1: all warnings being treated as errors

@nilason
Copy link
Contributor

nilason commented Jan 4, 2023

Mhh, this triggers unrelated other issues:

Probably caused by newer and stricter version of gcc and/or glibc.

Following warnings are causing error (with -Werror):

@nilason
Copy link
Contributor

nilason commented Jan 5, 2023

Mhh, this triggers unrelated other issues:

Probably caused by newer and stricter version of gcc and/or glibc.

Following warnings are causing error (with -Werror):

-Wstringop-overflow= and -Warray-parameter= warnings will be fixed by #2733.

@neteler
Copy link
Member Author

neteler commented Jan 5, 2023

Thanks so much, @nilason !

@neteler
Copy link
Member Author

neteler commented Jan 8, 2023

Yay, also the two Ubuntu CI runs finished successfully - no timeout!

@neteler neteler changed the title CI: attempt with ubuntu-22.04 image CI: attempt with ubuntu-22.04 image to avoid timeout Jan 8, 2023
@nilason
Copy link
Contributor

nilason commented Jan 13, 2023

Currently the Ubuntu based builds are failing badly, just restarted 'GCC' to see if this PR has the same issue. Indeed it has.

@neteler
Copy link
Member Author

neteler commented Jan 13, 2023

...
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/universe/p/proj/proj-bin_8.2.1-1_amd64.deb  Unable to connect to azure.archive.ubuntu.com:http: [IP: 52.252.75.106 80]
...

:-(

@nilason
Copy link
Contributor

nilason commented Jan 13, 2023

actions/runner-images#6911 looks like being related to this.

@nilason
Copy link
Contributor

nilason commented Jan 13, 2023

@nilason
Copy link
Contributor

nilason commented Jan 13, 2023

Or this actions/runner-images#6913

@nilason
Copy link
Contributor

nilason commented Jan 13, 2023

By the look of it, it works again.

@neteler neteler changed the title CI: attempt with ubuntu-22.04 image to avoid timeout CI: switch to ubuntu-22.04 image to avoid timeout Jan 23, 2023
@neteler
Copy link
Member Author

neteler commented Jan 23, 2023

Any objections to merge this PR?

@nilason
Copy link
Contributor

nilason commented Jan 23, 2023

I have no objections, we have the CentOS runner for testing on older systems, no?. If this indeed solves the timeout issues -- even better.

@neteler
Copy link
Member Author

neteler commented Jan 23, 2023

we have the CentOS runner for testing on older systems, no?

Yes and no:

However, we will check if the code compiles on the older Centos runner while - with this PR - running the tests on the recent Ubuntu runner which doesn't timeout. And keeping the old Ubuntu runner which mostly time-outs doesn't help much.

@nilason
Copy link
Contributor

nilason commented Jan 23, 2023

Fair enough. Maybe we can try this for some time, to see if it consistently works on a newer system and revert to the 20.04 if it doesn't.

@neteler
Copy link
Member Author

neteler commented Jan 23, 2023

So, let's try! Merging...

@neteler neteler merged commit 4a68d5a into OSGeo:main Jan 23, 2023
@neteler neteler deleted the ci_ubuntu-22.04_image branch January 23, 2023 22:17
neteler added a commit to OSGeo/grass-addons that referenced this pull request Jan 24, 2023
* GitHub actions: update to current versions

The GHA `ci.yml` and others show the warning

_Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions/checkout@v2, actions/setup-python@v2, r-lib/actions/setup-r@v1, actions/upload-artifact@v2. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/._

Additionally:
- update the various actions
- updates ubuntu-20.04 to ubuntu-22.04 (see also OSGeo/grass#2730)
- sync Python to v3.10
- updates to super-linter:v4
- super-linter: enable markdown linting
- use the real super-linter action instead of a container
- super-linter.yml: do not mix VALIDATE=true and VALIDATE=false

Co-authored-by: Nicklas Larsson <n_larsson@yahoo.com>
Co-authored-by: Edouard Choinière <27212526+echoix@users.noreply.github.com>
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
Since the older ubuntu-20.04 image based runs commonly time-out:
* update to the current ubuntu-22.04 image
* rename scripts to reflect usage of ubuntu-22.04 image
cwhite911 pushed a commit to cwhite911/grass-addons that referenced this pull request Sep 19, 2023
* GitHub actions: update to current versions

The GHA `ci.yml` and others show the warning

_Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions/checkout@v2, actions/setup-python@v2, r-lib/actions/setup-r@v1, actions/upload-artifact@v2. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/._

Additionally:
- update the various actions
- updates ubuntu-20.04 to ubuntu-22.04 (see also OSGeo/grass#2730)
- sync Python to v3.10
- updates to super-linter:v4
- super-linter: enable markdown linting
- use the real super-linter action instead of a container
- super-linter.yml: do not mix VALIDATE=true and VALIDATE=false

Co-authored-by: Nicklas Larsson <n_larsson@yahoo.com>
Co-authored-by: Edouard Choinière <27212526+echoix@users.noreply.github.com>
neteler added a commit to nilason/grass that referenced this pull request Nov 7, 2023
Since the older ubuntu-20.04 image based runs commonly time-out:
* update to the current ubuntu-22.04 image
* rename scripts to reflect usage of ubuntu-22.04 image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants