-
Notifications
You must be signed in to change notification settings - Fork 75
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
Fix installation issue when upgrading from an older version of the tool and improve logs #740
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #740 +/- ##
=======================================
Coverage 79.43% 79.44%
=======================================
Files 42 42
Lines 4411 4412 +1
Branches 819 819
=======================================
+ Hits 3504 3505 +1
Misses 693 693
Partials 214 214 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add tests to auto-merge
@@ -192,7 +192,7 @@ def _remote_folder_object(self) -> workspace.ObjectInfo: | |||
def _installed_query_state(self): | |||
if not self._state.dashboards: | |||
for k, v in self._state_pre_v06().items(): | |||
prefix, suffix = k.split(":", 2) | |||
prefix, suffix = [*k.split(":"), None][:2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add tests, as suggested by codecov bot.
* Corrected unit test setup as per PR [740](#740) * Fixed corrupted state warning
* Added `databricks labs ucx repair-run --step ...` CLI command for repair run of any failed workflows, like `assessment`, `migrate-groups` etc. ([#724](#724)). * Added `databricks labs ucx revert-migrated-table` command ([#729](#729)). * Allow specifying a group list when group match options are used ([#725](#725)). * Fixed installation issue when upgrading from an older version of the tool and improve logs ([#740](#740)). * Renamed summary panel from Failure Summary to Assessment Summary ([#733](#733)). * Retry internal error when getting permissions and update legacy table ACL documentation ([#728](#728)). * Speedup installer execution ([#727](#727)).
* Added `databricks labs ucx repair-run --step ...` CLI command for repair run of any failed workflows, like `assessment`, `migrate-groups` etc. ([#724](#724)). * Added `databricks labs ucx revert-migrated-table` command ([#729](#729)). * Allow specifying a group list when group match options are used ([#725](#725)). * Fixed installation issue when upgrading from an older version of the tool and improve logs ([#740](#740)). * Renamed summary panel from Failure Summary to Assessment Summary ([#733](#733)). * Retry internal error when getting permissions and update legacy table ACL documentation ([#728](#728)). * Speedup installer execution ([#727](#727)).
* Corrected unit test setup as per PR [740](#740) * Fixed corrupted state warning
* Added `databricks labs ucx repair-run --step ...` CLI command for repair run of any failed workflows, like `assessment`, `migrate-groups` etc. ([#724](#724)). * Added `databricks labs ucx revert-migrated-table` command ([#729](#729)). * Allow specifying a group list when group match options are used ([#725](#725)). * Fixed installation issue when upgrading from an older version of the tool and improve logs ([#740](#740)). * Renamed summary panel from Failure Summary to Assessment Summary ([#733](#733)). * Retry internal error when getting permissions and update legacy table ACL documentation ([#728](#728)). * Speedup installer execution ([#727](#727)).
* Corrected unit test setup as per PR [740](#740) * Fixed corrupted state warning
* Added `databricks labs ucx repair-run --step ...` CLI command for repair run of any failed workflows, like `assessment`, `migrate-groups` etc. ([#724](#724)). * Added `databricks labs ucx revert-migrated-table` command ([#729](#729)). * Allow specifying a group list when group match options are used ([#725](#725)). * Fixed installation issue when upgrading from an older version of the tool and improve logs ([#740](#740)). * Renamed summary panel from Failure Summary to Assessment Summary ([#733](#733)). * Retry internal error when getting permissions and update legacy table ACL documentation ([#728](#728)). * Speedup installer execution ([#727](#727)).
Fixes #730: