Skip to content

fix(ci): fetch pinned OVN commit directly for release-1.14 - #6892

Merged
zhangzujian merged 1 commit into
kubeovn:release-1.14from
zhangzujian:fix-base-ovn-pinned-fetch-release-1.14
Jun 18, 2026
Merged

fix(ci): fetch pinned OVN commit directly for release-1.14#6892
zhangzujian merged 1 commit into
kubeovn:release-1.14from
zhangzujian:fix-base-ovn-pinned-fetch-release-1.14

Conversation

@zhangzujian

Copy link
Copy Markdown
Member

Summary

  • Fix the base image build on release-1.14 by fetching the pinned OVN commit directly.
  • Replace git clone --depth=50 branch-24.03 && git checkout <sha> with a depth-1 fetch of the exact commit used by the Dockerfile.
  • Avoid depending on the pinned commit remaining within the latest 50 commits of branch-24.03.

Root cause

The scheduled Build Base workflow failed because the Dockerfile cloned branch-24.03 with --depth=50, then checked out ef8f5c1ac63e8094eaf13507013e310991db45a2. That commit is no longer included in the shallow clone window, so checkout failed with:

fatal: reference is not a tree: ef8f5c1ac63e8094eaf13507013e310991db45a2

Verification

  • Reproduced the failing shallow clone/checkout locally.
  • Verified git fetch --depth=1 origin ef8f5c1ac63e8094eaf13507013e310991db45a2 can fetch and check out the pinned commit.
  • Verified all release-1.14 OVN patches apply cleanly on the fetched commit.
  • Started make lint, then interrupted it after the user requested not to fix lint issues; only the Dockerfile change is included.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. ci labels Jun 18, 2026
Signed-off-by: zhangzujian <zhangzujian.7@gmail.com>
@zhangzujian
zhangzujian force-pushed the fix-base-ovn-pinned-fetch-release-1.14 branch from 7fc3197 to e25b371 Compare June 18, 2026 06:07
@zhangzujian
zhangzujian requested a review from Copilot June 18, 2026 06:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zhangzujian
zhangzujian requested a review from oilbeater June 18, 2026 06:13
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jun 18, 2026
@zhangzujian
zhangzujian merged commit 853f538 into kubeovn:release-1.14 Jun 18, 2026
68 of 70 checks passed
@zhangzujian
zhangzujian deleted the fix-base-ovn-pinned-fetch-release-1.14 branch June 18, 2026 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants