-
Notifications
You must be signed in to change notification settings - Fork 3
refactor(core): add binaries, remove unnecessary, change build images #895
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
Merged
universal-itengineer
merged 81 commits into
main
from
chore/core/cleanup-unnecessary-binaries
Apr 11, 2025
Merged
Changes from 78 commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
229a34e
chore(core): remove unnecessary binaries
universal-itengineer aead539
cdi-artifact: rm cloner_startup.sh
universal-itengineer 97baf5b
change import
universal-itengineer 0ef11fe
cleanup cdi-importer
universal-itengineer b8a5fd1
cdi-importer: remove sh, qemu-io,sqlite3 pkgs, add lib libsqlite3
universal-itengineer afab131
cloner, dvcr-artifact, rm sh and other
universal-itengineer 5f40951
virt-handler: rm bins bash,sh and etc
universal-itengineer 230ab7e
virt-artifact, refactor bins
universal-itengineer ad4ea84
virt-artifact: add source repo var, virt-launcher: rm some binaries
universal-itengineer aa50330
add echo in Taskfile for show wich dir will be delete
universal-itengineer db3d97b
virt-laucnher: rm bash sh and run util sleep directly
universal-itengineer 9622bb8
refactor node-labeller patch, run without sh
universal-itengineer d947983
resolve conflict
universal-itengineer 3899257
virt-launcher: remove unnecessary commented bins
universal-itengineer 9b9c0d9
virt-handler: remove unnecessary commented bins
universal-itengineer 06f7cae
virt-handler: remove procps
universal-itengineer f80bd43
virt-laucnher: add cp util
universal-itengineer b1b779b
virt-launcher: rm sysctl
universal-itengineer 1652345
virt-launcher: comment utils
universal-itengineer 4a935a4
virt-launcher: add static bin for RenderHotplugAttachmentTriggerPodTe…
universal-itengineer e6c4432
cdi-cloner remove comented mount,umount
universal-itengineer acd47df
remove echo, bash,cp from virt-controller template.go by patch
universal-itengineer 082ed4e
back cp util to virt-launcher
universal-itengineer 3dc5a70
temp_pod
universal-itengineer 98a20fe
dvcr-artifact: back file
universal-itengineer 9285a14
dvcr-artifact: remove commented bins
universal-itengineer 1dfb63d
virt-handler: remove commented bins, like pgrep and etc
universal-itengineer 3a40f50
virt-handler: rm chacl
universal-itengineer 33db83e
fix patch formatting
universal-itengineer 5cd95e6
virt-launcher: fix build and copy libs
universal-itengineer b85b3a0
tmp, add build_version
universal-itengineer ca932a5
test cache ver
universal-itengineer c4716c0
add dbg msgs
universal-itengineer e153491
copy libs via for
universal-itengineer 32ba827
copy with use * libs
universal-itengineer ff721b2
test via relocate_binaries LIBS and FILES
universal-itengineer db14543
dbg json libs
universal-itengineer 4cb5bfe
copy libjson-c.so.5
universal-itengineer 3ce1a54
try copy via cp jsonlib
universal-itengineer a0b6a40
try to reloc json to dir reloc
universal-itengineer 462dcce
change order for copy libs
universal-itengineer 2f61474
1042025.4
universal-itengineer c92ad10
copy libs via cp
universal-itengineer d9707f9
reloc libs
universal-itengineer 0e0eca6
add caps utils, set caps for virt-launcher-monitor
universal-itengineer e9fbf37
add libs
universal-itengineer db20f58
refactor+cleanup
universal-itengineer c5aaf89
add info about patches
universal-itengineer c51d11d
refactor virt-laucnher,handler
universal-itengineer 1c45aa3
add bin caps to vl
universal-itengineer c2e1602
fix caps cap_net_bind_service=+ep
universal-itengineer db6f775
back caps to 'cap_net_bind_service=+eip'
universal-itengineer e22ab2b
remove utils caps
universal-itengineer 08e7354
coment binaries tst1
universal-itengineer db85d01
rm ip,bridge sysctl, libffi, semenage
universal-itengineer 70cfef7
rm blockdev
universal-itengineer 946388c
rm passt
universal-itengineer 1cf8a1b
add mount
universal-itengineer 4e101a0
rm mount
universal-itengineer dce2b70
install packages to separate stage
universal-itengineer 514f812
remove commented bins for import to virtlauncher
universal-itengineer 9d6c108
add cache version
universal-itengineer 0c281a2
add cache version vl
universal-itengineer e52a6c3
remove virtctl from virt-operator
universal-itengineer aadd448
remove cacheVersion
universal-itengineer 7c5cbc9
rm qemu-io from cdi-importer
universal-itengineer ac88dee
remove virsh bin
universal-itengineer 7061626
fix dvcr binaries images
universal-itengineer 4c61dfd
dvcr: change stage to 'before: setup'
universal-itengineer d2ed88a
virt-handler: add sysctl
universal-itengineer e757f6e
virt-handler: replace sysctl by 'os.ReadFile(/proc/sys/kernel/sched_r…
universal-itengineer 8dd0aba
virt-handler: rm sysctl bins
universal-itengineer 9e4a1ee
add patch readme
universal-itengineer 3841a54
Update images/bounder/werf.inc.yaml
universal-itengineer 2c8fe59
Update images/virt-artifact/patches/README.md
universal-itengineer dc9a3be
Update images/virt-launcher/werf.inc.yaml
universal-itengineer 3026d20
add space Readme
universal-itengineer 58ec6f4
remove commented sysctl bins
universal-itengineer c634b60
remove unicode symbols
universal-itengineer 9570918
refactor patches virt-artifact
universal-itengineer 56ba805
rename 047 patch
universal-itengineer File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| #include <stdio.h> | ||
|
|
||
| int main() { | ||
| printf("Hello, my name is Bounder. I’m here to fulfill my destiny… to mount PVCs and vanish into the void.\n"); | ||
| return 0; | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
15 changes: 15 additions & 0 deletions
15
images/virt-artifact/patches/043-virt-launcher-image-holder-command-sleep.patch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| diff --git a/pkg/virt-operator/resource/generate/components/daemonsets.go b/pkg/virt-operator/resource/generate/components/daemonsets.go | ||
| index 93c55c1619..ebc1e790bf 100644 | ||
| --- a/pkg/virt-operator/resource/generate/components/daemonsets.go | ||
| +++ b/pkg/virt-operator/resource/generate/components/daemonsets.go | ||
| @@ -145,8 +145,8 @@ func NewHandlerDaemonSet(namespace, repository, imagePrefix, version, launcherVe | ||
| Name: "virt-launcher-image-holder", | ||
| Image: launcherImage, | ||
| ImagePullPolicy: corev1.PullIfNotPresent, | ||
| - Command: []string{"/bin/sh", "-c"}, | ||
| - Args: []string{"sleep infinity"}, | ||
| + Command: []string{"sleep"}, | ||
| + Args: []string{"infinity"}, | ||
| Resources: corev1.ResourceRequirements{ | ||
| Limits: map[corev1.ResourceName]resource.Quantity{ | ||
| corev1.ResourceCPU: resource.MustParse("100m"), |
60 changes: 60 additions & 0 deletions
60
images/virt-artifact/patches/044-hotplug-attachment-trigger-pod-remove-bash.patch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,60 @@ | ||
| diff --git a/pkg/virt-controller/services/template.go b/pkg/virt-controller/services/template.go | ||
| index 1221448946..7e2519d129 100644 | ||
| --- a/pkg/virt-controller/services/template.go | ||
| +++ b/pkg/virt-controller/services/template.go | ||
| @@ -382,9 +382,7 @@ func (t *templateService) renderLaunchManifest(vmi *v1.VirtualMachineInstance, i | ||
| if tempPod { | ||
| logger := log.DefaultLogger() | ||
| logger.Infof("RUNNING doppleganger pod for %s", vmi.Name) | ||
| - command = []string{"/bin/bash", | ||
| - "-c", | ||
| - "echo", "bound PVCs"} | ||
| + command = []string{"temp_pod"} | ||
| } else { | ||
| command = []string{"/usr/bin/virt-launcher-monitor", | ||
| "--qemu-timeout", generateQemuTimeoutWithJitter(t.launcherQemuTimeout), | ||
| @@ -921,7 +919,7 @@ func (t *templateService) RenderHotplugAttachmentPodTemplate(volumes []*v1.Volum | ||
| zero := int64(0) | ||
| runUser := int64(util.NonRootUID) | ||
| sharedMount := k8sv1.MountPropagationHostToContainer | ||
| - command := []string{"/bin/sh", "-c", "/usr/bin/container-disk --copy-path /path/hp"} | ||
| + command := []string{"/usr/bin/container-disk", "--copy-path", "/path/hp"} | ||
|
|
||
| tmpTolerations := make([]k8sv1.Toleration, len(ownerPod.Spec.Tolerations)) | ||
| copy(tmpTolerations, ownerPod.Spec.Tolerations) | ||
| @@ -1076,11 +1074,9 @@ func (t *templateService) RenderHotplugAttachmentTriggerPodTemplate(volume *v1.V | ||
| sharedMount := k8sv1.MountPropagationHostToContainer | ||
| var command []string | ||
| if tempPod { | ||
| - command = []string{"/bin/bash", | ||
| - "-c", | ||
| - "exit", "0"} | ||
| + command = []string{"temp_pod"} | ||
| } else { | ||
| - command = []string{"/bin/sh", "-c", "/usr/bin/container-disk --copy-path /path/hp"} | ||
| + command = []string{"/usr/bin/container-disk", "--copy-path", "/path/hp"} | ||
universal-itengineer marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| } | ||
|
|
||
| annotationsList := make(map[string]string) | ||
| diff --git a/pkg/virt-controller/watch/vmi_test.go b/pkg/virt-controller/watch/vmi_test.go | ||
| index b4b76bd73e..9fb2c56118 100644 | ||
| --- a/pkg/virt-controller/watch/vmi_test.go | ||
| +++ b/pkg/virt-controller/watch/vmi_test.go | ||
| @@ -400,7 +400,7 @@ var _ = Describe("VirtualMachineInstance watcher", func() { | ||
|
|
||
| return "" | ||
| }, | ||
| - Equal("/bin/bash -c echo bound PVCs"), | ||
| + Equal("temp_pod"), | ||
| ) | ||
|
|
||
| controller.Execute() | ||
| @@ -676,7 +676,7 @@ var _ = Describe("VirtualMachineInstance watcher", func() { | ||
|
|
||
| return "" | ||
| }, | ||
| - Equal("/bin/bash -c echo bound PVCs")) | ||
| + Equal("temp_pod")) | ||
| expectMatchingPodCreation(vmi, IsPodWithoutVmPayload) | ||
| expectVMIWithMatcherConditions(vmi.Namespace, vmi.Name, ContainElement(MatchFields(IgnoreExtras, | ||
| Fields{"Type": Equal(virtv1.VirtualMachineInstanceProvisioning)})), | ||
28 changes: 28 additions & 0 deletions
28
images/virt-artifact/patches/045-node-labeller-rereplace-sysctl-readfile.patch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| diff --git a/pkg/virt-handler/node-labeller/node_labeller.go b/pkg/virt-handler/node-labeller/node_labeller.go | ||
| index e5ddaf3784..8de36d72c9 100644 | ||
| --- a/pkg/virt-handler/node-labeller/node_labeller.go | ||
| +++ b/pkg/virt-handler/node-labeller/node_labeller.go | ||
| @@ -23,7 +23,7 @@ import ( | ||
| "context" | ||
| "encoding/json" | ||
| "fmt" | ||
| - "os/exec" | ||
| + "os" | ||
| "runtime" | ||
| "strings" | ||
| "time" | ||
| @@ -327,11 +327,12 @@ const kernelSchedRealtimeRuntimeInMicrosecods = "kernel.sched_rt_runtime_us" | ||
| // workloads at peak performance. | ||
|
|
||
| func isNodeRealtimeCapable() (bool, error) { | ||
| - ret, err := exec.Command("sysctl", kernelSchedRealtimeRuntimeInMicrosecods).CombinedOutput() | ||
| + ret, err := os.ReadFile("/proc/sys/kernel/sched_rt_runtime_us") | ||
| if err != nil { | ||
| return false, err | ||
| } | ||
| - st := strings.Trim(string(ret), "\n") | ||
| + sched_rt_runtime_us := strings.Trim(string(ret), "\n") | ||
| + st := fmt.Sprintf("%s = %s", kernelSchedRealtimeRuntimeInMicrosecods, sched_rt_runtime_us) | ||
| return fmt.Sprintf("%s = -1", kernelSchedRealtimeRuntimeInMicrosecods) == st, nil | ||
| } | ||
|
|
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.