trafficcontrol icon indicating copy to clipboard operation
trafficcontrol copied to clipboard

t3c ATS RPM: Enable software collections environment before building OpenSSL

Open zrhoffman opened this issue 3 years ago • 8 comments

This PR gets building the Cache Config Integration Tests's configuration for a CentOS 7 ATS RPM to work again by enabling the newer GCC version before trying to build OpenSSL and using the OS_DISTRO and OS_VERSION values set when the Docker image is built.


Which Traffic Control components are affected by this PR?

  • Traffic Control Cache Config (t3c, formerly ORT) - integration tests ATS RPM build configuration

What is the best way to verify this PR?

Verify that building the ATS RPM for CentOS 7 passes:

export OS_DISTRO=centos OS_VERSION=7
cd cache-config/testing/docker
docker-compose -f docker-compose-ats-build.yml build
docker-compose -f docker-compose-ats-build.yml up

PR submission checklist

zrhoffman avatar Mar 22 '22 15:03 zrhoffman

Should we change the t3c tests to a matrix of the two OSes ? Right now this may fix it, but the default is to use Rocky 8 so the GHAs don't tell me if it's fixed.

ocket8888 avatar Jul 18 '22 18:07 ocket8888

Apache asks projects not to use the matrix execution strategy on PRs, due to the concurrent job limit for the whole Apache org:

My proposal is to create a GitHub Action that will allow to run only a subset of "matrix" test for PRs that are not yet approved by committers.

So we could do matrix on the master branch as long as we don't do it on PRs.

zrhoffman avatar Jul 18 '22 18:07 zrhoffman

Well a matrix would be easiest, but if we shouldn't do that should it at least test them both sequentially?

ocket8888 avatar Jul 18 '22 18:07 ocket8888

That would probably be okay

zrhoffman avatar Jul 18 '22 18:07 zrhoffman

How would you do it sequentially, though, without having the job be twice as long?

zrhoffman avatar Jul 18 '22 18:07 zrhoffman

You wouldn't, which is what I mean about matrix being the easiest way.

ocket8888 avatar Jul 18 '22 19:07 ocket8888

Rocky Linux 8 is already well-tested in the T3C Integration Tests, CDN-in-a-Box CI, and TC Health Client Integration Tests workflows. How about a CentOS 7-only workflow that builds the ATS RPM whenever files in that directory (or the dockerfile) are changed?

Then we wouldn't need matrix.

zrhoffman avatar Jul 18 '22 20:07 zrhoffman

Sure, that's fine by me.

ocket8888 avatar Jul 18 '22 20:07 ocket8888