-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Use ov in CI + CD #5262
base: main
Are you sure you want to change the base?
Use ov in CI + CD #5262
Conversation
I'm going to come back to this from a Linux machine. |
This is definitely not ready for review, lol. How did that happen? |
Well, at least the error is new and different. |
./ov build | ||
./ov setup-env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have thought this would need to include a call to ov just install
as well; or is the plan to install specifically for each CI step? If the whole ov
container and volume is going to be cached between jobs then installing once at the start would save some time on subsequent jobs 🤔. I don't know if the volume will work that way, though.
Fixes
Fixes #4406 by @sarayourfriend
Description
This PR updates the CI + CD workflow to use
ov
. This should makeov
more resilient to accidental breakages.🚧 WIP:
ov
and its contents.Checklist
Update index.md
).main
) or a parent feature branch.ov just catalog/generate-docs
for catalogPRs) or the media properties generator (
ov just catalog/generate-docs media-props
for the catalog or
ov just api/generate-docs
for the API) where applicable.Developer Certificate of Origin
Developer Certificate of Origin