Edgewall Software

Opened 3 years ago

Closed 16 months ago

Last modified 16 months ago

#13358 closed enhancement (fixed)

Switch to using Github Actions from Travis CI — at Version 14

Reported by: Jun Omae Owned by: Jun Omae
Priority: normal Milestone: 1.5.4
Component: general Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Restored automatic builds using GitHub Actions.

Description

Builds with OSX on Travis CI often recently timed out. I think we could be going to use Github Actions.

I'm working it.

I noticed one of the downsides which recipient of notifications is not configurable.

Change History (14)

comment:1 by Ryan J Ollos, 3 years ago

I like the idea. I've been using GitHub actions for another Python project and had a good experience so far.

comment:2 by Ryan J Ollos, 3 years ago

I just found out today that GitHub actions can be run locally. I haven't tried it yet, but it would be very helpful if it works.

comment:3 by Jun Omae, 3 years ago

Proposed changes in log:jomae.git@trunk:github-actions.

OS Database Python Dependencies
Ubuntu, Macos, Windows SQLite (in-memory) 3.9 minimum, full (included Subversion 1.14.1)
Ubuntu, Macos, Windows SQLite 3.9 minimum, full (included Subversion 1.14.1)
Ubuntu, Macos, Windows PostgreSQL 3.9 minimum, full (included Subversion 1.14.1)
Ubuntu, Macos, Windows MySQL 3.9 minimum, full (included Subversion 1.14.1)
Ubuntu SQLite (in-memory) 3.8, 3.7, 3.6, 3.5 full (included Subversion 1.14.1)

comment:4 by Ryan J Ollos, 3 years ago

Milestone: 1.61.5.4

comment:5 by Ryan J Ollos, 3 years ago

If we get the deploy working on Windows it would address #13190.

comment:6 by Ryan J Ollos, 3 years ago

Milestone: 1.5.41.5.5

in reply to:  5 ; comment:7 by Jun Omae, 3 years ago

Replying to Ryan J Ollos:

If we get the deploy working on Windows it would address #13190.

bdist_wininst command has been removed in Python 3.10 (PythonBug:42802). I think we should deploy only sdist and wheel.

Updated branch log:jomae.git@trunk:github-actions.

in reply to:  7 comment:8 by Ryan J Ollos, 3 years ago

Replying to Jun Omae:

bdist_wininst command has been removed in Python 3.10 (PythonBug:42802). I think we should deploy only sdist and wheel.

Good point, we did drop the win installers from upload to PyPI in recent releases (comment:2:ticket:13205). They weren't generated for Trac 1.4.3. However, they are still created for TracDownload#Trac14StableRelease. I think it makes sense to drop them.

Last edited 3 years ago by Ryan J Ollos (previous) (diff)

comment:9 by Jun Omae, 17 months ago

Owner: set to Jun Omae
Status: newassigned

We should restore automatic builds using GitHub Actions rather than Travis CI, AppVeyor (long time since automatic builds stopped working)….

in reply to:  9 comment:10 by Ryan J Ollos, 17 months ago

Replying to Jun Omae:

We should restore automatic builds using GitHub Actions rather than Travis CI, AppVeyor (long time since automatic builds stopped working)….

Sounds great!

comment:11 by figaro, 17 months ago

Also very much in favor of that.

comment:12 by Jun Omae, 17 months ago

Proposed changes in:

Branch Changes Results
trunk jomae.git@trunk:github-actions actions/runs/3816838152
1.4-stable jomae.git@1.4-stable:github-actions-1.4 actions/runs/3816783218
1.2-stable jomae.git@1.2-stable:github-actions-1.2 actions/runs/3820240949
1.0-stable jomae.git@1.0-stable:github-actions-1.0 actions/runs/3820236150

Notes:

  • Need to set key and iv of contrib/travis/id_rsa.enc to secrets to upload sdist and wheel files to edgewall.org
  • Functional tests with Trac 1.5.x run only on the following environments because the tests are too slow:
    • SQLite x (ubuntu-22.04, macos-12) x Python 3.11
    • SQLite x windows-2022 x Python 3.10
  • On Python 2.7 on Windows, tests for subversion and mysql are skipped because the bindings is unable to build

comment:13 by Ryan J Ollos, 17 months ago

Need to set key and iv of contrib/travis/id_rsa.enc to secrets to upload sdist and wheel files to edgewall.org

I will be sure to look into doing it this weekend.

comment:14 by Jun Omae, 16 months ago

Internal Changes: modified (diff)
Resolution: fixed
Status: assignedclosed

Committed in [17659-17663].

Note: See TracTickets for help on using tickets.