Skip to content

Update HPP-wffc manifest URL#4042

Merged
kubevirt-bot merged 1 commit into
kubevirt:mainfrom
Davo911:fix-external-provider-hpp-url
Mar 5, 2026
Merged

Update HPP-wffc manifest URL#4042
kubevirt-bot merged 1 commit into
kubevirt:mainfrom
Davo911:fix-external-provider-hpp-url

Conversation

@Davo911
Copy link
Copy Markdown
Member

@Davo911 Davo911 commented Feb 20, 2026

What this PR does / why we need it:
When running cluster-sync on with external provider I was getting:

KUBEVIRT_PROVIDER=external KUBEVIRT_STORAGE=hpp make cluster-sync
+ _kubectl apply -f https://github.com/kubevirt/hostpath-provisioner-operator/releases/download/v0.24.0/storageclass-wffc.yaml
+ kubectl apply -f https://github.com/kubevirt/hostpath-provisioner-operator/releases/download/v0.24.0/storageclass-wffc.yaml
error: unable to read URL "https://github.com/kubevirt/hostpath-provisioner-operator/releases/download/v0.24.0/storageclass-wffc.yaml", server reported 404 Not Found, status code=404
make: *** [Makefile:177: cluster-sync-test-infra] Error 1

They just changed storageclass-wffc.yaml to storageclass-wffc-legacy.yaml

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Release note:

NONE

@kubevirt-bot kubevirt-bot added release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XS labels Feb 20, 2026
Copy link
Copy Markdown
Collaborator

@akalenyu akalenyu left a comment

Choose a reason for hiding this comment

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

Thank you! we could probably just use the CSI version
(check non external provider hpp)

Comment thread cluster-sync/external/provider.sh Outdated
The Manifest name changed for the HPP wffc manifest, resulting in issues cluster-sync with external provider

Signed-off-by: Thomas-David Griedel [email protected]
@Davo911 Davo911 force-pushed the fix-external-provider-hpp-url branch from 7c82c3c to a031e95 Compare March 2, 2026 15:33
@Davo911
Copy link
Copy Markdown
Member Author

Davo911 commented Mar 2, 2026

Updated to csi, thanks @awels and @akalenyu

@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Mar 3, 2026
@awels
Copy link
Copy Markdown
Member

awels commented Mar 3, 2026

/approve

@kubevirt-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: awels

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 3, 2026
@akalenyu
Copy link
Copy Markdown
Collaborator

akalenyu commented Mar 5, 2026

/override pull-containerized-data-importer-e2e-hpp-previous
ci issue

@kubevirt-bot
Copy link
Copy Markdown
Contributor

@akalenyu: Overrode contexts on behalf of akalenyu: pull-containerized-data-importer-e2e-hpp-previous

Details

In response to this:

/override pull-containerized-data-importer-e2e-hpp-previous
ci issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@kubevirt-bot kubevirt-bot merged commit c841de9 into kubevirt:main Mar 5, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants