You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now the data-install lane already attempts to install datasets that are in the ready-for-reinstall status. The dataset reinstaller also attempts to install datasets that are in the ready-for-reinstall status.
I suggest that we clarify and simplify the dataset reinstallation process by doing the following:
Add a new InstallStatus enum member ready-for-cleanup which will indicate to the dataset-reinstaller that the dataset is able to be cleaned up and reinstalled
Alter the dataset-reinstaller to:
a. Uninstall datasets in the ready-for-cleanup status
b. Mark uninstalled datasets as ready-for-reinstall
c. Fire install events for the uninstalled datasets
This will remove the duplicated dataset installation logic and clarify the usages of the ready-for-reinstall status which is currently overloaded.
The text was updated successfully, but these errors were encountered:
Right now the data-install lane already attempts to install datasets that are in the
ready-for-reinstall
status. The dataset reinstaller also attempts to install datasets that are in theready-for-reinstall
status.I suggest that we clarify and simplify the dataset reinstallation process by doing the following:
ready-for-cleanup
which will indicate to the dataset-reinstaller that the dataset is able to be cleaned up and reinstalleda. Uninstall datasets in the
ready-for-cleanup
statusb. Mark uninstalled datasets as
ready-for-reinstall
c. Fire install events for the uninstalled datasets
This will remove the duplicated dataset installation logic and clarify the usages of the
ready-for-reinstall
status which is currently overloaded.The text was updated successfully, but these errors were encountered: