Skip to content

Conversation

gkoscins
Copy link

@gkoscins gkoscins commented Oct 9, 2025

Tickets:

@gkoscins gkoscins requested a review from a team as a code owner October 9, 2025 07:20
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code category: dockerfiles labels Oct 9, 2025
@gkoscins gkoscins requested a review from akladiev October 10, 2025 11:54
@gkoscins gkoscins requested a review from a team as a code owner October 13, 2025 09:03
@gkoscins gkoscins requested review from mlukasze and removed request for a team October 13, 2025 09:03
image: ${{ fromJSON(needs.docker.outputs.images).ov_test.ubuntu_24_04_x64_code_style }}
volumes:
- /mount:/mount
options: --user 1001 # We use standard github-actions user inside container to avoid permission issues after checkout
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we move it inside the docker image configuration?

Copy link
Author

Choose a reason for hiding this comment

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

I don't think we can, when trying to do so I am getting permission errors, here are the docs that explain it: https://docs.github.com/en/actions/reference/workflows-and-actions/dockerfile-support#user

runs-on: ubuntu-22.04
if: ${{ github.repository_owner == 'openvinotoolkit' }}
needs: Docker
runs-on: aks-linux-4-cores-16gb
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not the medium runner?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CI OpenVINO public CI category: dockerfiles github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants