operator-framework / operator-registry

Operator Registry runs in a Kubernetes or OpenShift cluster to provide operator catalog data to Operator Lifecycle Manager.
Apache License 2.0
212 stars 248 forks source link

Fix upstream-builder image build #1467

Closed varad-ahirwadkar closed 1 month ago

varad-ahirwadkar commented 1 month ago

Description of the change: This PR adds the installation of the linux-headers package in the upstream-builder image to fix the build error-linux/limits.h: No such file or directory

Motivation for the change:

This PR fixes #1466

Reviewer Checklist

openshift-ci[bot] commented 1 month ago

Hi @varad-ahirwadkar. Thanks for your PR.

I'm waiting for a operator-framework member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
perdasilva commented 1 month ago

/ok-to-test

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 48.61%. Comparing base (c8307ca) to head (287c2db). Report is 8 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1467 +/- ## ======================================= Coverage 48.61% 48.61% ======================================= Files 135 135 Lines 12793 12793 ======================================= Hits 6219 6219 Misses 5533 5533 Partials 1041 1041 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

openshift-ci[bot] commented 1 month ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: perdasilva, varad-ahirwadkar

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/operator-framework/operator-registry/blob/master/OWNERS)~~ [perdasilva] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment