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

refactor(cdi): update to v1.60.3 #382

Draft
wants to merge 35 commits into
base: main
Choose a base branch
from
Draft

Conversation

diafour
Copy link
Member

@diafour diafour commented Sep 17, 2024

  • Remove 006-customizer.patch: it is already in the upstream.
  • Remove redundant imports reformats.
  • Optimize 011-change-storage-class-for-scratch-pvc.patch: one line change is required.
  • Optimize 013-converting-images-on-filesystem-to-qcow2.patch: move new code in a separate file to increase chances to apply without problems in the future.

Description

Why do we need it, and what problem does it solve?

What is the expected result?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Comment on lines +5 to 11
@@ -50,6 +50,9 @@ var CapabilitiesByProvisionerKey = map[string][]StorageCapabilities{
"openshift-storage.cephfs.csi.ceph.com": {{rwx, file}},
// LINSTOR
"linstor.csi.linbit.com": createAllButRWXFileCapabilities(),
+ // Deckhouse
+ "replicated.csi.storage.deckhouse.io": createLinstorCapabilities(),
+ "replicated.csi.storage.deckhouse.io": createAllButRWXFileCapabilities(),
+ "local.csi.storage.deckhouse.io": createTopoLVMCapabilities(),
Copy link
Member Author

Choose a reason for hiding this comment

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

Our block is moved to be closer to LINSTOR to quicker understand what's changed in upstream in the future.

@diafour diafour force-pushed the refactor/cdi/update-to-1-60 branch 2 times, most recently from bb6b134 to 5b391eb Compare September 19, 2024 12:04
@diafour diafour force-pushed the refactor/kubevirt/update-to-1.3.1 branch 2 times, most recently from 14e4380 to a320fd9 Compare September 24, 2024 19:43
Base automatically changed from refactor/kubevirt/update-to-1.3.1 to main September 26, 2024 14:34
An error occurred while trying to automatically change base from refactor/kubevirt/update-to-1.3.1 to main September 26, 2024 14:34
- Remove 006-customizer.patch: it is already in the upstream.
- Remove redundant imports reformats.
- Optimize 011-change-storage-class-for-scratch-pvc.patch: one line change is required.
- Optimize 013-converting-images-on-filesystem-to-qcow2.patch: move new code in a separate file to increase chances to apply without problems in the future.

Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
++ remove rpmtrees from bazel builds

Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
…Ignore embedded CRDs for prettier.

Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
…d binaries, no bundling into images

- Update virtualization-artifact dependencies: kubevirt-api and cdi-api

Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
- Update kubevirt.io/api to 1.3.1
- Ignore prettier for crds/embedded

++ revert p11 for cdi-apiserver

Signed-off-by: Ivan Mikheykin <[email protected]>
++ add build without bazel

Signed-off-by: Ivan Mikheykin <[email protected]>
++ add build without bazel
++ add custom-build dir to git

Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
@universal-itengineer universal-itengineer added the build/self-hosted/virt-runners Run jobs on virtualization git hub runners label Sep 27, 2024
- Small refinements in dvcr artifact.
- Use Scratch for static binaries: cdi-operator, cdi-apiserver, cdi-controller, cdi-uploadproxy.
- Update cdi API Go module in virtualization-artifact.

Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build/self-hosted/virt-runners Run jobs on virtualization git hub runners
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants