gridcf / gct

Grid Community Toolkit
Apache License 2.0
46 stars 30 forks source link

CI changes #199

Closed fscheiner closed 1 year ago

fscheiner commented 1 year ago

This PR implements the following changes:

  1. Switchover to new source repo hosted by EGI
  2. Use Ubuntu Focal (20.04) as build environment for Travis CI instead of the out-of-date but still default Ubuntu Xenial (16.04), clean up the configuration according to config validation at Travis CI and enable builds on ARM64, too.
  3. Add Rocky Linux 9 as build target for GHA and update secret name for uploads to the source repo
  4. Build source tarballs and SRPMs for all OS cases for GHA

no.4 unveiled the following issues:

fscheiner commented 1 year ago

Not sure anyone would still be interested in the myproxy-oauth. In principle https://github.com/ncsa/OA4MP is what's supported, but it still could be nice if it's not too difficult to get to work.

What's the difference between myproxy-oauth and OA4MP anyhow?

msalle commented 1 year ago

Not sure anyone would still be interested in the myproxy-oauth. In principle https://github.com/ncsa/OA4MP is what's supported, but it still could be nice if it's not too difficult to get to work.

What's the difference between myproxy-oauth and OA4MP anyhow?

See #200 let's just remove the myproxy-oauth code and close #200.

fscheiner commented 1 year ago

Ok, I agree and I'll look into that and come up with another PR.

ellert commented 1 year ago
  • "globus-xio-udt-driver" can't be built (also for SRPM) on CentOS Stream 9 and Rocky Linux 9 because "libnice(-devel)" is not (yet) available for these OSes, so (S)RPMs are not built for *9 OS versions, too. Mattias has already opened a bug about branching and building libnice in EPEL 9.

Now buildable in EPEL 9. The RHEL 9.1 release added some missing dependencies needed to build the missing dependencies in EPEL.

globus-xio-udt-driver package update for EPEL 9: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-cd6c9c9c8b

fscheiner commented 1 year ago

Thanks for getting this going Mattias (@ellert), I saw it in the morning and will make the required changes.