dweeezil commented on this pull request.

Changed event when removing a log device.

> +      */
+       vd->vdev_removing = B_TRUE;
+
+       vdev_dirty_leaves(vd, VDD_DTL, *txg);
+       vdev_config_dirty(vd);
+
+       spa_history_log_internal(spa, "vdev remove", NULL,
+           "%s vdev %llu (log) %s", spa_name(spa), vd->vdev_id,
+           (vd->vdev_path != NULL) ? vd->vdev_path : "-");
+
+       spa_vdev_config_exit(spa, NULL, *txg, 0, FTAG);
+
+       *txg = spa_vdev_config_enter(spa);
+
+       sysevent_t *ev = spa_event_create(spa, vd, NULL,
+           ESC_ZFS_VDEV_REMOVE_AUX);

Previously, removing a log device would post a VDEV_REMOVE_DEV event but now it 
posts a VDEV_REMOVE_AUX.  Is this intentional?  If so, event consumers may need 
to be updated appropriately.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/482#pullrequestreview-86763071
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/Teafc5ffabd7f916f-M9a1db399766b0fa271f1e059
Powered by Topicbox: https://topicbox.com

Reply via email to