{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":117756215,"defaultBranch":"main","name":"containerized-data-importer","ownerLogin":"kubevirt","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2018-01-16T23:35:48.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/18700703?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1713551543.0","currentOid":""},"activityList":{"items":[{"before":"76f9c0a4233ece912cde58d7666af866270a3d6c","after":"5c08db5e468b5ae3da5711805c6518a04fd5d2bb","ref":"refs/heads/main","pushedAt":"2024-06-07T12:12:13.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Document environment variables in Makefile (#3297)\n\n* Document environment variables in Makefile\n\nSigned-off-by: Edu Gómez Escandell \n\n* Updated environment variables in docs\n\nSigned-off-by: Edu Gómez Escandell \n\n* Fix typos\n\nSigned-off-by: Edu Gómez Escandell \n\n---------\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Document environment variables in Makefile (#3297)"}},{"before":"09b1689bb783f0d788bca16a796cd9507fe5dd83","after":"76f9c0a4233ece912cde58d7666af866270a3d6c","ref":"refs/heads/main","pushedAt":"2024-06-07T00:39:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Add virt default storage class mention in onboarding doc (#3272)\n\nSigned-off-by: Alex Kalenyuk ","shortMessageHtmlLink":"Add virt default storage class mention in onboarding doc (#3272)"}},{"before":"e662d51bda2a3db0ff8fae2bdeb5fa6ab3d73e09","after":"09b1689bb783f0d788bca16a796cd9507fe5dd83","ref":"refs/heads/main","pushedAt":"2024-06-06T12:47:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Improve test resilience by wrapping updates around eventually statements (#3291)\n\n* Wrap PVC, PV and DataVolume updates inside an eventually statement to improve test resilience\n\nSigned-off-by: Alvaro Romero \n\n* Clean up Eventually statements\n\nSigned-off-by: Alvaro Romero \n\n* Ensure object consistency after update failures\n\nSigned-off-by: Alvaro Romero \n\n---------\n\nSigned-off-by: Alvaro Romero ","shortMessageHtmlLink":"Improve test resilience by wrapping updates around eventually stateme…"}},{"before":"8e4962322f6b45f142a5871050fa3eef9d1f2667","after":"3138701580996cbaf7da208a75c460c290b8b096","ref":"refs/heads/release-v1.59","pushedAt":"2024-06-05T11:59:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"forklift: rename API group (#3295) (#3305)\n\n* forklift: rename API group\n\nTo avoid collisions with forklift's CRDs\n\n\n\n* forklift: rename to forklift.cdi.kubevirt.io\n\n\n\n---------\n\nSigned-off-by: Benny Zlotnik ","shortMessageHtmlLink":"forklift: rename API group (#3295) (#3305)"}},{"before":"f8484ba343ca9e435e0ef3b068a090cbb4c9388e","after":"e662d51bda2a3db0ff8fae2bdeb5fa6ab3d73e09","ref":"refs/heads/main","pushedAt":"2024-06-05T02:55:36.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Add tests AfterEach nil checks (#3294)\n\nSigned-off-by: Arnon Gilboa ","shortMessageHtmlLink":"Add tests AfterEach nil checks (#3294)"}},{"before":"85f1383b10f49b204dc1278b607272ba4e405d8b","after":"f8484ba343ca9e435e0ef3b068a090cbb4c9388e","ref":"refs/heads/main","pushedAt":"2024-06-04T22:05:36.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"forklift: rename API group (#3295)\n\n* forklift: rename API group\n\nTo avoid collisions with forklift's CRDs\n\nSigned-off-by: Benny Zlotnik \n\n* forklift: rename to forklift.cdi.kubevirt.io\n\nSigned-off-by: Benny Zlotnik \n\n---------\n\nSigned-off-by: Benny Zlotnik ","shortMessageHtmlLink":"forklift: rename API group (#3295)"}},{"before":"073336b16cda8c3a605c8e702c69008a01210264","after":"85f1383b10f49b204dc1278b607272ba4e405d8b","ref":"refs/heads/main","pushedAt":"2024-06-04T16:47:36.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Revert LVMO storage CSI snapshot clone. (#3303)\n\nOn multi node LMVO this setting can cause issues\nso reverting to host assisted clone which works\neverywhere\n\nSigned-off-by: Alexander Wels ","shortMessageHtmlLink":"Revert LVMO storage CSI snapshot clone. (#3303)"}},{"before":"d365661e305bc67cb753cd963549b89b8a1f01e2","after":"073336b16cda8c3a605c8e702c69008a01210264","ref":"refs/heads/main","pushedAt":"2024-06-03T20:39:34.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Make upload client/server certs configurable (#3252)\n\n* Add client cert config to CDI resource\n\nSigned-off-by: Michael Henriksen \n\n* make client certs configurable\n\nSigned-off-by: Michael Henriksen \n\n* Create uploadserver.Config\n\nSigned-off-by: Michael Henriksen \n\n* uploadserver should read certs from files\n\nSigned-off-by: Michael Henriksen \n\n* make sure to not close doneChan when error occurs\n\ngenerally tighten up handling of \"done\" \"uploading\" and \"processing\"\n\nSigned-off-by: Michael Henriksen \n\n* add deadline support to uploadserver\n\nSigned-off-by: Michael Henriksen \n\n* Add deadline support to upload controller\n\nSigned-off-by: Michael Henriksen \n\n* clone controller should use configured client cert duration\n\nSigned-off-by: Michael Henriksen \n\n* make lint check happy\n\nSigned-off-by: Michael Henriksen \n\n* Extend existing func test to validate client certs configurable and will be rotated\n\nSigned-off-by: Michael Henriksen \n\n* Use deadline/rotation for clone pods as well\n\nForgot about the case where a source PVC may be in use. Bay be a big delay from when target pod is created and source.\n\nSigned-off-by: Michael Henriksen \n\n---------\n\nSigned-off-by: Michael Henriksen ","shortMessageHtmlLink":"Make upload client/server certs configurable (#3252)"}},{"before":"dd518e770bfc4131aec497386b2920c4a7d8cec9","after":"d365661e305bc67cb753cd963549b89b8a1f01e2","ref":"refs/heads/main","pushedAt":"2024-06-03T11:25:37.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Fix progress metric registration and parsing (#3292)\n\n* Fix progress metric registration and parsing\n\nUse default metric registration. We shouldn't use the controller-runtime\nregistration as we have no controller here and it will not register the\nmetric correctly.\n\nFix the metric parsing to match its new name. Otherwise the DV progress\nwill not be updated until its 100%.\n\nRegression introduced in #3254\n\nSigned-off-by: Arnon Gilboa \n\n* Add kubevirt_cdi_import_progress_total metric\n\nUse it in the importer instead of kubevirt_cdi_clone_progress_total and\nfix metric parsing accordingly.\n\nSigned-off-by: Arnon Gilboa \n\n* Move ProgressFromClaim to host-clone\n\nNobody else is using it.\n\nSigned-off-by: Arnon Gilboa \n\n* Add ProgressMetric interface\n\nProgressReader can now work with either import or clone progress metric.\nFIXME: consider removing the direct Add/Get and use only via interface.\n\nSigned-off-by: Arnon Gilboa \n\n* Refactor ProgressMetric interface\n\nSigned-off-by: Arnon Gilboa \n\n* Refactor progress parsing\n\nSigned-off-by: Arnon Gilboa \n\n* Refer metric names from the metrics package\n\nSigned-off-by: Arnon Gilboa \n\n---------\n\nSigned-off-by: Arnon Gilboa ","shortMessageHtmlLink":"Fix progress metric registration and parsing (#3292)"}},{"before":"c646bf72bf4d3a18deacef8a2e336e7798cc8679","after":"154d28f0e53e5df8256be6c2eb5b2956311997cf","ref":"refs/heads/release-v1.57","pushedAt":"2024-06-02T20:05:31.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"[release-v1.57] Add watch for DataImportCron-labeled PVCs deletion (#3302)\n\nManual backport of #3285\n\nWhen the DataImportCron last import DV is manually deleted, the\ncontroller reconciles, but due to k8s default background cascading\ndeletion, the PVC may still temporarily exist, so the controller will\nnot re-create the DV even after the PVC is deleted, unless it reconciles\ndue to other watched CR like DataSource. In the scenario of CNV-39688,\nsince we move from pvc source format to snapshot, the DataSource won’t\nupdate until a snapshot is created, which will never happen. To solve it\nwe add a watch for deletion of DataImportCron-labeled PVCs. The change\nwas tested locally, but since it required two storage classes it’s\ncurrently out of scope for the existing CI lanes, so it requires tier-2\ntest.\n\nSigned-off-by: Arnon Gilboa ","shortMessageHtmlLink":"[release-v1.57] Add watch for DataImportCron-labeled PVCs deletion (#…"}},{"before":"71493fedc267fca41799f048b3c187a026e7df8c","after":"dd518e770bfc4131aec497386b2920c4a7d8cec9","ref":"refs/heads/main","pushedAt":"2024-06-02T16:23:32.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Enable usestdlibvars linter (#3299)\n\n* Enable usestdlibvars linter\n\nThis linter recommends using standard library variables rather than\nmagic constants, so for example 200 is replaced by http.StatusOK.\n\nSigned-off-by: Edu Gómez Escandell \n\n* Replace magic values with standard library constants\n\nSigned-off-by: Edu Gómez Escandell \n\n---------\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Enable usestdlibvars linter (#3299)"}},{"before":"a7dc1e570d625a7a4a8a1a5820e7761a9f649928","after":"b98704052388739d1d0b3911daf6878a76f33534","ref":"refs/heads/release-v1.58","pushedAt":"2024-06-02T13:57:31.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"[release-v1.58] Add watch for DataImportCron-labeled PVCs deletion (#3301)\n\nManual backport of #3285\n\nWhen the DataImportCron last import DV is manually deleted, the\ncontroller reconciles, but due to k8s default background cascading\ndeletion, the PVC may still temporarily exist, so the controller will\nnot re-create the DV even after the PVC is deleted, unless it reconciles\ndue to other watched CR like DataSource. In the scenario of CNV-39688,\nsince we move from pvc source format to snapshot, the DataSource won’t\nupdate until a snapshot is created, which will never happen. To solve it\nwe add a watch for deletion of DataImportCron-labeled PVCs. The change\nwas tested locally, but since it required two storage classes it’s\ncurrently out of scope for the existing CI lanes, so it requires tier-2\ntest.\n\nSigned-off-by: Arnon Gilboa ","shortMessageHtmlLink":"[release-v1.58] Add watch for DataImportCron-labeled PVCs deletion (#…"}},{"before":"e9b2077ad4e1d6ad022fead8d9a34695ab444b26","after":"71493fedc267fca41799f048b3c187a026e7df8c","ref":"refs/heads/main","pushedAt":"2024-06-01T02:31:30.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"forklift: fix race between prime pvc and pod removal (#3280)\n\n* forklift: fix race between prime pvc and pod removal\n\nBecause there is time until deletionTimestamp is set on the PVC prime,\nand since the populator pod is deleted first we might reconcile again\nwhen the PVC is not yet marked for deletion, causing the populator pod\nto be recreated.\n\nThis change reorders the operation, to first wait until the PVC is\nmarked for deletion, and then delete the populator pod.\n\nSigned-off-by: Benny Zlotnik \n\n* forklift: add tests\n\nSigned-off-by: Benny Zlotnik \n\n---------\n\nSigned-off-by: Benny Zlotnik ","shortMessageHtmlLink":"forklift: fix race between prime pvc and pod removal (#3280)"}},{"before":"8bd9355fd1a76c6266809e60fe6293937de4dce2","after":"e9b2077ad4e1d6ad022fead8d9a34695ab444b26","ref":"refs/heads/main","pushedAt":"2024-05-30T19:53:33.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Reduce amount of logs fetched by tests that assert on control plane pod logs (#3184)\n\n* Replace raw 'kubectl logs' commands with time-bounded GetLogs API\n\nSigned-off-by: Edu Gómez Escandell \n\n* Specify container when pod has more than one of them\n\nThis pod has two containers:\n- importer\n- istio-proxy\n\nWe only need the logs from the first one.\n\nSigned-off-by: Edu Gómez Escandell \n\n* Remove PrintControllerLog\n\nSigned-off-by: Edu Gómez Escandell \n\n---------\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Reduce amount of logs fetched by tests that assert on control plane p…"}},{"before":"50563e2120b18b95ca484e712c800a63057f1bdd","after":"8bd9355fd1a76c6266809e60fe6293937de4dce2","ref":"refs/heads/main","pushedAt":"2024-05-30T14:29:34.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Enable Gosec linter (#3283)\n\n* Move gosec into golangci-lint\n\nRemove gosec target and scripts and use the golangci-lint linter\nThis ensures we stay up-to-date (so long as golangci-lint is up to date\ntoo).\n\nSigned-off-by: Edu Gómez Escandell \n\n* G601: Disable for-loop variable aliassing warning (not relevant fro Go>=1.22)\n\nSigned-off-by: Edu Gómez Escandell \n\n* G101: Ignore warning about plain-text credentials\n\nThey are false positives\n\nSigned-off-by: Edu Gómez Escandell \n\n* G102: Don't listen to all interfaces\n\nSigned-off-by: Edu Gómez Escandell \n\n* G107: Ignore potentially tainted GET requests\n\nThey are all in test code\n\nSigned-off-by: Edu Gómez Escandell \n\n* G109: Avoid integer overflows after parsing strings\n\nSigned-off-by: Edu Gómez Escandell \n\n* G110: Potential DoS vulnerability via decompression bomb\n\nSigned-off-by: Edu Gómez Escandell \n\n* G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured\n\nSigned-off-by: Edu Gómez Escandell \n\n* G114: Use of net/http serve function that has no support for setting timeouts\n\nSigned-off-by: Edu Gómez Escandell \n\n* G204: Subprocess launched with a potential tainted input or cmd arguments\n\nSigned-off-by: Edu Gómez Escandell \n\n* G305: File traversal when extracting zip/tar archive\n\nSigned-off-by: Edu Gómez Escandell \n\n* G306: Expect WriteFile permissions to be 0600 or less\n\nSigned-off-by: Edu Gómez Escandell \n\n* Bugfix: Misuse of file descriptor flags in file permission bits\n\nos.WriteFile always uses O_WRONLY|O_CREATE|O_TRUNC, the third argument\nis for the file's permission bits. This code is misleading, it will\ntruncate the file and not append to it. For that you'd need\nos.Openfile(path, os.O_APPEND, 0600)\n\nI also simplified the unnecessary []byte conversion.\n\nSigned-off-by: Edu Gómez Escandell \n\n* G401: Use of weak cryptographic primitive\n\nSigned-off-by: Edu Gómez Escandell \n\n* G402: Insecure TLS\n\nSigned-off-by: Edu Gómez Escandell \n\n* G404: Use of weak random number generator (math/rand instead of crypto/rand)\n\nSigned-off-by: Edu Gómez Escandell \n\n* G501: Blocklisted import crypto/md5: weak cryptographic primitive\n\nSigned-off-by: Edu Gómez Escandell \n\n* Run autoformatters\n\nUnrelated to the PR but this way we keep everything formatted\n\nSigned-off-by: Edu Gómez Escandell \n\n* Remove references to gosec in the makefile\n\nGosec has been moved into Golangci-lint\n\nSigned-off-by: Edu Gómez Escandell \n\n---------\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Enable Gosec linter (#3283)"}},{"before":"9a965239012287d324caf71a08d35f0febf551ff","after":"50563e2120b18b95ca484e712c800a63057f1bdd","ref":"refs/heads/main","pushedAt":"2024-05-30T04:14:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Update vendor dependencies (#3296)\n\n* Update vendor dependencies\n\nIn particular update the containers/image to 5.31.0\nwhich removes a dependency that has an incompatible\nlicense.\n\nSigned-off-by: Alexander Wels \n\n* Update gophercloud/utils to gophercloud/utils/v2\nwhich no longer uses hashicorp/go-uuid but instead\nuses gofrs/uuid/v5.\n\nSigned-off-by: Alexander Wels \n\n---------\n\nSigned-off-by: Alexander Wels ","shortMessageHtmlLink":"Update vendor dependencies (#3296)"}},{"before":"4a64af4bdb4d1215c71008dc6b9aca3c7e2c94f9","after":"9a965239012287d324caf71a08d35f0febf551ff","ref":"refs/heads/main","pushedAt":"2024-05-29T07:54:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Update makefile documentation (#3282)\n\n* Change help target to parse comments\n\nBy moving the documentation to the targets, it becomes less likely\nfor the descriptions to fall out of date.\n\nSigned-off-by: Edu Gómez Escandell \n\n* Add comments/helps messages to make targets\n\nSigned-off-by: Edu Gómez Escandell \n\n* Add support for sections\n\nLines starting with '##@' will define section headers\n\nSigned-off-by: Edu Gómez Escandell \n\n* Re-order targets and add section titles\n\nNote: the deleted/added targets were not modified, only moved around\n\nSigned-off-by: Edu Gómez Escandell \n\n* Update .PHONY targets\n\nAs far as I understand, all our targets are phony\n\nSigned-off-by: Edu Gómez Escandell \n\n* Split lines of .PHONY label and move it to then end of the Make file\n\nSigned-off-by: Edu Gómez Escandell \n\n---------\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Update makefile documentation (#3282)"}},{"before":"6d9c106f8f8a02449859977b9b0d20ccdbcb4470","after":"4a64af4bdb4d1215c71008dc6b9aca3c7e2c94f9","ref":"refs/heads/main","pushedAt":"2024-05-26T14:11:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"s390x rpm support (#3286)\n\nSigned-off-by: cfillekes ","shortMessageHtmlLink":"s390x rpm support (#3286)"}},{"before":"4342b3ca969e29720997f2f96a03bed5e9b42407","after":"6d9c106f8f8a02449859977b9b0d20ccdbcb4470","ref":"refs/heads/main","pushedAt":"2024-05-24T18:17:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Move golangci-lint timeout into the config file (#3288)\n\nIt's best to have all the config in the same place rather than having\nsome in the cli arguments and some in the config files.\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Move golangci-lint timeout into the config file (#3288)"}},{"before":"cd7c8b14a52b9e6a30a88d4beec4b93a341a6198","after":"4342b3ca969e29720997f2f96a03bed5e9b42407","ref":"refs/heads/main","pushedAt":"2024-05-23T10:13:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Add watch for DataImportCron-labeled PVCs deletion (#3285)\n\nWhen the DataImportCron last import DV is manually deleted, the\ncontroller reconciles, but due to k8s default background cascading\ndeletion, the PVC may still temporarily exist, so the controller will\nnot re-create the DV even after the PVC is deleted, unless it reconciles\ndue to other watched CR like DataSource. In the scenario of CNV-39688,\nsince we move from pvc source format to snapshot, the DataSource won’t\nupdate until a snapshot is created, which will never happen. To solve it\nwe add a watch for deletion of DataImportCron-labeled PVCs. The change\nwas tested locally, but since it required two storage classes it’s\ncurrently out of scope for the existing CI lanes, so it requires tier-2\ntest.\n\nSigned-off-by: Arnon Gilboa ","shortMessageHtmlLink":"Add watch for DataImportCron-labeled PVCs deletion (#3285)"}},{"before":"39cc977406927440b43f3e46b859bb89784b4d42","after":"c646bf72bf4d3a18deacef8a2e336e7798cc8679","ref":"refs/heads/release-v1.57","pushedAt":"2024-05-22T13:41:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Onboard Infinibox CSI driver (#3275)\n\nReference - https://github.com/Infinidat/infinibox-csi-driver/tree/develop/deploy/examples\nalongside some offline chats with recommendations.\n\nSigned-off-by: Alex Kalenyuk \nCo-authored-by: Alex Kalenyuk ","shortMessageHtmlLink":"Onboard Infinibox CSI driver (#3275)"}},{"before":"dd07461d2b82942249424b0b2c65f5632e84c583","after":"cd7c8b14a52b9e6a30a88d4beec4b93a341a6198","ref":"refs/heads/main","pushedAt":"2024-05-21T20:35:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Enable revive linter (#3241)\n\n* Enable revive linter\n\nSigned-off-by: Edu Gómez Escandell \n\n* Simplify cdi-func-test-proxy\n\nThis function had quite a bit of redundant code (caught by the linter).\nThe workgroup was never Done because all exit paths went through a\nlog.Fatal.\n\nSigned-off-by: Edu Gómez Escandell \n\n* Fix 'revive' linter warnings\n\nSigned-off-by: Edu Gómez Escandell \n\n* Fix tests that asserted on modified error messages\n\nSigned-off-by: Edu Gómez Escandell \n\n* Run make format\n\nThe formatted code has nothing to do with this PR but we may as well\ninclude it.\n\nSigned-off-by: Edu Gómez Escandell \n\n* Use lower-case variables and use built-in min function in vddk-datasource\n\nSigned-off-by: Edu Gómez Escandell \n\n* Use contexts in cdi-func-test-proxy\n\nThis added quite a bit of boilerplate per call, so I put everything in\na loop.\n\nSigned-off-by: Edu Gómez Escandell \n\n---------\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Enable revive linter (#3241)"}},{"before":"dde903cf50647108ee37caed89441086a81c20f3","after":"a7dc1e570d625a7a4a8a1a5820e7761a9f649928","ref":"refs/heads/release-v1.58","pushedAt":"2024-05-20T19:52:28.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Onboard Infinibox CSI driver (#3274)\n\nReference - https://github.com/Infinidat/infinibox-csi-driver/tree/develop/deploy/examples\nalongside some offline chats with recommendations.\n\nSigned-off-by: Alex Kalenyuk \nCo-authored-by: Alex Kalenyuk ","shortMessageHtmlLink":"Onboard Infinibox CSI driver (#3274)"}},{"before":"2b20854cb8cdae440493ec7892f0faee7ac73506","after":"dd07461d2b82942249424b0b2c65f5632e84c583","ref":"refs/heads/main","pushedAt":"2024-05-20T19:52:28.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"tests: correct check for containerdisk env vars eventually appearing as labels (#3263)\n\nSigned-off-by: Alex Kalenyuk ","shortMessageHtmlLink":"tests: correct check for containerdisk env vars eventually appearing …"}},{"before":"7b7dd7815fd8db9d31a24062b95bf5e8dfeeca1e","after":"6847b98f0b309acdeee54ca1c68daa343c605f21","ref":"refs/heads/release-v1.56","pushedAt":"2024-05-20T16:52:29.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Onboard Infinibox CSI driver (#3271) (#3276)\n\nReference - https://github.com/Infinidat/infinibox-csi-driver/tree/develop/deploy/examples\nalongside some offline chats with recommendations.\n\nSigned-off-by: Alex Kalenyuk ","shortMessageHtmlLink":"Onboard Infinibox CSI driver (#3271) (#3276)"}},{"before":"c82867b7688771d60b87439be8af507f96d2cc70","after":"ed95467fd6324d564d1ae328adcc5a22393fefec","ref":"refs/heads/release-v1.55","pushedAt":"2024-05-20T16:38:29.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Onboard Infinibox CSI driver (#3271) (#3277)\n\nReference - https://github.com/Infinidat/infinibox-csi-driver/tree/develop/deploy/examples\nalongside some offline chats with recommendations.\n\nSigned-off-by: Alex Kalenyuk ","shortMessageHtmlLink":"Onboard Infinibox CSI driver (#3271) (#3277)"}},{"before":"9ed29cd93846b6697c9f6c36eadc4258223f51c6","after":"2b20854cb8cdae440493ec7892f0faee7ac73506","ref":"refs/heads/main","pushedAt":"2024-05-20T12:08:28.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Onboard Infinibox CSI driver (#3271)\n\nReference - https://github.com/Infinidat/infinibox-csi-driver/tree/develop/deploy/examples\nalongside some offline chats with recommendations.\n\nSigned-off-by: Alex Kalenyuk ","shortMessageHtmlLink":"Onboard Infinibox CSI driver (#3271)"}},{"before":"6cbc5e7630b70b5fb6e6f1a2bfa3f9a255797f93","after":"9ed29cd93846b6697c9f6c36eadc4258223f51c6","ref":"refs/heads/main","pushedAt":"2024-05-18T02:50:26.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Apply monitoring linters (#3225)\n\n* Apply operator-observability rules linter\n\nApply operator-observability rules linter to ensure alerts and recording\nrules definitions are following the monitoring best practices.\n\nSigned-off-by: assafad \n\n* Apply monitoringlinter\n\nmonitoringlinter ensures that monitoring-related practices are\nimplemented within the pkg/monitoring directory using\noperator-observability.\n\nSigned-off-by: assafad \n\n---------\n\nSigned-off-by: assafad ","shortMessageHtmlLink":"Apply monitoring linters (#3225)"}},{"before":"a5cedb15559752e66566a467a840376fd3ad18c7","after":"6cbc5e7630b70b5fb6e6f1a2bfa3f9a255797f93","ref":"refs/heads/main","pushedAt":"2024-05-17T22:02:26.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Filter out generated files from coverage report (#3268)\n\nAs per the reference [1], the regex should be:\n\n^// Code generated .* DO NOT EDIT\\.$\n\nHowever, sometimes the final dot is dropped so I made it optional\n\n---\n[1] https://pkg.go.dev/cmd/go#hdr-Generate_Go_files_by_processing_source\n\nSigned-off-by: Edu Gómez Escandell ","shortMessageHtmlLink":"Filter out generated files from coverage report (#3268)"}},{"before":"7a49da9ad11833885f654ca98b5dc053a2b5a10c","after":"a5cedb15559752e66566a467a840376fd3ad18c7","ref":"refs/heads/main","pushedAt":"2024-05-17T12:02:25.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubevirt-bot","name":null,"path":"/kubevirt-bot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25687724?s=80&v=4"},"commit":{"message":"Progress metrics refactor and rename (#3254)\n\n* clone_progress metric refactor\n\nThe clone_progress metric is not in the monitoring package. The metric\nis with incorrect name, based on the kubevirt and Prometheus metrics\nnaming conventions. It's not documented and not located under\n/pkg/monitoring. After the code refactoring we should not have\nPrometheus metrics in other places in the code, other than the\n/monitoring/metrics package, and metrics should be registered using\noperator-observability package.\n\nSigned-off-by: Arnon Gilboa \n\n* openstack_populator_progress metric refactor\n\nSigned-off-by: Arnon Gilboa \n\n* clone_progress metric refactor CR fixes\n\nSigned-off-by: Arnon Gilboa \n\n* ovirt_progress metric refactor\n\nSigned-off-by: Arnon Gilboa \n\n* Align progress metrics names with linter rules\n\nAlso add the metrics to the doc and json generation tools.\n\nSigned-off-by: Arnon Gilboa \n\n* Remove redundant ListMetrics\n\nSigned-off-by: Arnon Gilboa \n\n---------\n\nSigned-off-by: Arnon Gilboa ","shortMessageHtmlLink":"Progress metrics refactor and rename (#3254)"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEXy8SygA","startCursor":null,"endCursor":null}},"title":"Activity · kubevirt/containerized-data-importer"}