nginxinc / nginx-otel

Apache License 2.0
168 stars 19 forks source link

[RFC] Added NGINX Buildsystem github action test. #28

Open thresheek opened 11 months ago

thresheek commented 11 months ago

Proposed changes

This adds a CI pipeline to test if the usual way of building nginx and modules work. This functionality is now not covered in the CI tests, and may be further expanded if/when https://github.com/nginxinc/nginx-otel/pull/27 is merged.

I've chosen Alpine as it has most of dependencies already built, which will drastically lower the time needed for checks to succeed or fail. Also, this adds a test on the submitted PR.

Currently this fails due to https://github.com/nginxinc/nginx-otel/issues/16 and it will fail later on the actual test step - the perl tests seem to have protoc paths hardcoded to whatever is used by cmake-based build system.

Checklist

Before creating a PR, run through this checklist and mark each as complete.