-
Notifications
You must be signed in to change notification settings - Fork 4
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
Syncing latest changes from upstream main for ramen #368
Commits on Oct 1, 2024
-
Refactor: Extract resource watching logic from drpc controller into d…
…rpc watcher - Moved all resource-watching related functions from drplacementcontrol_controller.go to a new file drplacementcontrol_watcher.go. - No functional changes introduced, purely a structural refactor. Signed-off-by: Benamar Mekhissi <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2d4b061 - Browse repository at this point
Copy the full SHA 2d4b061View commit details -
Watch for DRPolicy resource changes
- Added functionality to watch for changes in DRPolicy resources to trigger DRPC reconciliation when necessary. Signed-off-by: Benamar Mekhissi <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ada7056 - Browse repository at this point
Copy the full SHA ada7056View commit details -
Add external-snapshotter addon to kubevirt envs
Was forgetten when we replaced minikube volumesnapshot addons. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 75aa24b - Browse repository at this point
Copy the full SHA 75aa24bView commit details -
Reformat reconcileMissingVR comment for readability
The comment is still unclear, but at least easier to read. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for c8b3512 - Browse repository at this point
Copy the full SHA c8b3512View commit details -
Fix logs in reconcileMissingVR
- Replace "VR under deletion" with "deleted VR", matching the terminology used in the code. - Replace "detected as missing or deleted" with "is missing" when we know the VR does not exist. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b9552c3 - Browse repository at this point
Copy the full SHA b9552c3View commit details -
Extract VRGInstance.validateVRCompletedStatus()
Moved from validateVRStatus() to make room from checking VR VAlidated status. This also make the function easier to understand, keeping the same level of abstraction and getting rid of the uninteresting details. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for bd41cab - Browse repository at this point
Copy the full SHA bd41cabView commit details -
Rename msg to errorMsg and document that this is an error message, if the we could not get the condition value, because it is missing, stale, or unknown. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 830f770 - Browse repository at this point
Copy the full SHA 830f770View commit details -
Improve logging in delete VR flow
Log after important changes to the system in delete VR flow to make it easier to understand what the system is doing, and how ramen changed the system. New logs: - delete the VR resource - remove annotations from PV Improve logs: - remove annotations, labels, and finalizers from PVC Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7d170b5 - Browse repository at this point
Copy the full SHA 7d170b5View commit details -
Fix disable dr if VR failed validation
When deleting a primary VRG, we wait until the VR Completed condition is met. However if a VR precondition failed, for example using a drpolicy without flattening enabled when the PVC needs flattening, the VR will never complete and the vrg and drpc deletion will never complete. Since csi-addons 0.10.0 we have a new Validated VR condition, set to true if pre conditions are met, and false if not. VR is can be deleted safely in this state, since mirroring was not enabled. This changes modifies deleted VRG processing to check the new VR Validated status. If the condition exist and the condition status is false, validateVRStatus() return true, signaling that the VR is in the desired state, and ramen completes the delete flow. If the VR does not report the Validated condition (e.g. old csi-addon version) or the condition status is true (mirroring in progress), we continue in the normal flow. The VR will be deleted only when the Completed condition status is true. Tested with discovered deployment and vm using a pvc created from a volume snapshot. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 5dfc74a - Browse repository at this point
Copy the full SHA 5dfc74aView commit details -
Updating broken clusteradm link in the doc
Signed-off-by: Abhijeet Shakya <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 25061a9 - Browse repository at this point
Copy the full SHA 25061a9View commit details -
Add daily e2e job for refreshing the cache
To refresh the cache we need to checkout ramen source and run drenv cache with the environment files. Using a workflow for this make this job easy to implement and manage without accessing the runner directly. The job can also run manually from github UI. This is likely to work for people with write access. Signed-off-by: Nir Soffer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 29d6f22 - Browse repository at this point
Copy the full SHA 29d6f22View commit details