Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added build on RedHat system to build & test RPM packages #20134

Merged
merged 12 commits into from
Oct 1, 2023

Conversation

ilya-lavrenov
Copy link
Contributor

@ilya-lavrenov ilya-lavrenov commented Sep 28, 2023

Details:

  • Added building and testing for RPM packages
  • Properly fixed issue with RPM / Debian packages rebuild (previously, Build Debian step built OpenVINO almost from scratch)

@ilya-lavrenov ilya-lavrenov added this to the 2023.2 milestone Sep 28, 2023
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Sep 28, 2023
@ilya-lavrenov ilya-lavrenov force-pushed the rpm-packages branch 2 times, most recently from 946e24e to 274a1cb Compare September 28, 2023 20:41
@github-actions github-actions bot added the category: build OpenVINO cmake script / infra label Sep 28, 2023
@ilya-lavrenov ilya-lavrenov changed the title Rpm packages Added build on RedHat system to build & test RPM packages Sep 28, 2023
@ilya-lavrenov ilya-lavrenov marked this pull request as ready for review September 28, 2023 21:19
@ilya-lavrenov ilya-lavrenov requested review from a team as code owners September 28, 2023 21:19
@github-actions github-actions bot added the category: inference OpenVINO Runtime library - Inference label Sep 29, 2023
shell: bash
runs-on: aks-linux-16-cores
container:
image: fedora:33
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that we need a mirror image on the private Hub as there are limitations for pulling from Docker Hub on the aks runners and we already use some images from there.

.github/workflows/fedara.yml Outdated Show resolved Hide resolved
.github/workflows/fedara.yml Outdated Show resolved Hide resolved
.github/workflows/fedara.yml Outdated Show resolved Hide resolved
shell: bash
runs-on: ubuntu-20.04
container:
image: fedora:33
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same about the image.

(I believe that we need a mirror image on the private Hub as there are limitations for pulling from Docker Hub on the aks runners and we already use some images from there.)

.github/workflows/linux.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@akashchi akashchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GHA-related LGTM. Please consider the possibility of too many pulls of the Docker Hub image/use mirror in the private Hub.

.github/workflows/fedora.yml Show resolved Hide resolved
@ilya-lavrenov ilya-lavrenov requested a review from a team as a code owner September 30, 2023 23:31
@github-actions github-actions bot added the category: Core OpenVINO Core (aka ngraph) label Sep 30, 2023
@github-actions github-actions bot added the category: Python API OpenVINO Python bindings label Sep 30, 2023
@ilya-lavrenov ilya-lavrenov requested review from a team as code owners October 1, 2023 13:05
@github-actions github-actions bot added category: MO Model Optimizer category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin category: PDPD FE OpenVINO PaddlePaddle FrontEnd category: TF FE OpenVINO TensorFlow FrontEnd category: TFL FE OpenVINO TensorFlow Lite FrontEnd labels Oct 1, 2023
@ilya-lavrenov ilya-lavrenov merged commit 95e3096 into openvinotoolkit:master Oct 1, 2023
34 of 36 checks passed
@ilya-lavrenov ilya-lavrenov deleted the rpm-packages branch October 1, 2023 19:23
alvoron pushed a commit to alvoron/openvino that referenced this pull request Nov 6, 2023
…olkit#20134)

* Added GHA workflow for RPM packages

* Avoid rebuild for RPM / Debian packages

* Removed conditional include headers

* try only post-build

* Beautification

* Fixed testdata generation for mulit-config generators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra category: CI OpenVINO public CI category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: GPU OpenVINO GPU plugin category: inference OpenVINO Runtime library - Inference category: MO Model Optimizer category: PDPD FE OpenVINO PaddlePaddle FrontEnd category: Python API OpenVINO Python bindings category: TF FE OpenVINO TensorFlow FrontEnd category: TFL FE OpenVINO TensorFlow Lite FrontEnd github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants