Arvados

20797-rhel8-packages @ e0c95f3349e0a78438a379861d97d6a565ee281b - build-packages-multijob: #3658

I'll need to coordinate with Lucas or someone next week about setting up a new Jenkins target and testing the branch, but everything at least builds and passes tests on my laptop.

Peter Amstutz wrote:

If we want to support the whole 8 series we might need to build on 8.0.

The oldest Docker image Rocky currently publishes is 8.6. I went ahead and built on that since it matches the immediate need. I do not see any published policy about how far back they're willing to publish these images.

Brett Smith wrote in #note-6:

20797-rhel8-packages @ e0c95f3349e0a78438a379861d97d6a565ee281b - build-packages-multijob: #3658

After one bugfix (1cefd714aed3090435b6fe471dbdf3a1ba12179a) and some network trouble downloading Node, a successful build of the new packages on Jenkins: build-packages-rocky8: #5

Note this job does not upload packages yet, because we haven't set up a place for them to go. But this is still progress.

Some comments:

• At arvados/build/package-testing/common-test-rails-server-package.sh Lines 52, 52: Should we use --assumeyes like we do with the other distos?
• The package repo documentation splits CentOS 7 vs the new ones CentOS8/AlmaLinux/etc but it provides the same instructions as CentOS7. Would it make sense to just add the new versions comment to the already existing CentOS7 section?
• PostgreSQL docs for the new distro uses yum, but everywhere else mentions dnf. Not sure which is the more correct, but I think it would be good to maintain consistency.

Lucas Di Pentima wrote in #note-8:

• At arvados/build/package-testing/common-test-rails-server-package.sh Lines 52, 52: Should we use --assumeyes like we do with the other distos?

There's a whole backstory here, see the commit message for 7c3c271a6616e9795581a27f7e5515b03b43a460 explaining why it wasn't in version 1, and why I've gone ahead and added it now.

• The package repo documentation splits CentOS 7 vs the new ones CentOS8/AlmaLinux/etc but it provides the same instructions as CentOS7. Would it make sense to just add the new versions comment to the already existing CentOS7 section?

Done in fca484c3a8160e6671f5372c5d19f34864ce1d4c.

• PostgreSQL docs for the new distro uses yum, but everywhere else mentions dnf. Not sure which is the more correct, but I think it would be good to maintain consistency.

Good catch, fixed in 8ea9c22dd8c5986d1a5eb7a45c223c051a0fc571. Thanks.

This LGTM, thanks!