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

ci: Mark the current git directory as safe #386

Merged
merged 1 commit into from
Mar 21, 2023
Merged

Conversation

mz-pdm
Copy link
Member

@mz-pdm mz-pdm commented Mar 21, 2023

This is necessary to prevent

fatal: detected dubious ownership in repository at '/__w/vdsm/vdsm'

messages in the CentOS Stream 9 container and, more importantly, making wrong rpm release version there when building rpm’s. Without this, the determined release version on CI is always ‘1’, which results in not installing the required set of rpm’s in OST and an OST failure.

This is necessary to prevent

  fatal: detected dubious ownership in repository at '/__w/vdsm/vdsm'

messages in the CentOS Stream 9 container and, more importantly,
making wrong rpm release version there when building rpm’s.  Without
this, the determined release version on CI is always ‘1’, which
results in not installing the required set of rpm’s in OST and an OST
failure.

Signed-off-by: Milan Zamazal <mzamazal@redhat.com>
@michalskrivanek
Copy link
Member

"--define="vdsm_release 202303211349.gitebda4d6d9"

yep, works better

@mz-pdm mz-pdm merged commit 2ad3e22 into oVirt:master Mar 21, 2023
@mz-pdm mz-pdm deleted the fix-ost# branch March 21, 2023 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants