#1112 Add one script to build them all
Merged by t0xic0der. Opened by darknao.
darknao/fedora-websites ci_buildall  into  master

Download 1112.patch

This is the first step to implement CI for websites.

It introduce a new script tools/build_all.sh which build all active websites in one place (inside the out directory), and can then be served by httpd.
In case one of the site is failing to build, you'll find the build log (build.log) in its own directory, and accessible through the web server.
Additionally, to be able to host each site in a subdirectory, I added the basepath override (variable ${path}) where it was missing.

Thanks to this script, CI can be automatically triggered to give contributors a preview of their changes for each PR.
Preview like this one : https://pr1112-fedora-web-ci.apps.ocp.ci.centos.org/
Build log is available for each site, whatever the build is a success or not. Example for spins : https://pr1112-fedora-web-ci.apps.ocp.ci.centos.org/spins.fedoraproject.org/build.log

Preview is updated every time the corresponding PR is updated or rebased.
Once the PR is merged or closed, the preview site is removed.
At this time, there is also a time limit of 2 weeks, after which it is also deleted.

This PR has already been merged on staging, to make sure nothing break when not using the basepath override.

1 new commit added

  • build_all: can now be used for stg/prod build

Metadata Update from @jflory7:
- Pull-request tagged with: tools

Hi @darknao, you mentioned this was merged on staging already. Is this for the production site builder? Is there any other feedback or testing you want on this Pull Request? Or is it ready to merge?

In addition to CI, this script will also be used for the production (and staging) builder when moved to openshift.
It is ready to be merged right now, if there is no objections.
I'll rebase it in a few.

rebased onto cf91c400b72e84f463714e4cd39798d904dc2fae

@darknao Getting a CI pipeline for this repository to test changes would be super helpful. I have no objections to merging it in, but I also have maybe the least context on how the production build of these sites works as of present.

I'm happy to see this merged so we can continue working on the PR backlog on this repository, and also moving towards a CI pipeline.

+1 from my end too. Merging.

Pull-Request has been merged by t0xic0der

Metadata