-
Notifications
You must be signed in to change notification settings - Fork 90
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
Test failure: test_running_real_assessment_job_ext_hms
#1616
Comments
❌ test_running_real_assessment_job_ext_hms: KeyError: 'ucx_ORS8' (25m36.24s)
Running from nightly #50 |
❌ test_running_real_assessment_job_ext_hms: KeyError: 'ucx_wWSV' (24m44.187s)
Running from nightly #51 |
❌ test_running_real_assessment_job_ext_hms: KeyError: 'ucx_D5jw' (25m26.539s)
Running from nightly #52 |
Running this integration test in isolation works on an up-to-date |
11 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
May 7, 2024
## Changes Similarly to the prefix catalogs with "C", tables with "T" and schemas with "S", also prefix random test group names with "G". ### Linked issues Hopefully it narrows the KeyError in: #1616 ### Functionality - [ ] added relevant user documentation - [ ] added new CLI command - [ ] modified existing command: `databricks labs ucx ...` - [ ] added a new workflow - [ ] modified existing workflow: `...` - [ ] added a new table - [ ] modified existing table: `...` ### Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [ ] manually tested - [ ] added unit tests - [ ] added integration tests - [ ] verified on staging environment (screenshot attached)
github-merge-queue bot
pushed a commit
that referenced
this issue
May 7, 2024
## Changes Similarly to the prefix catalogs with "C", tables with "T" and schemas with "S", also prefix random test group names with "G". ### Linked issues Hopefully it narrows the KeyError in: #1616 ### Functionality - [ ] added relevant user documentation - [ ] added new CLI command - [ ] modified existing command: `databricks labs ucx ...` - [ ] added a new workflow - [ ] modified existing workflow: `...` - [ ] added a new table - [ ] modified existing table: `...` ### Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [ ] manually tested - [ ] added unit tests - [ ] added integration tests - [ ] verified on staging environment (screenshot attached)
❌ test_running_real_assessment_job_ext_hms: KeyError: 'ucx_GIu4b' (28m11.568s)
Running from nightly #54 |
11 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
May 8, 2024
## Changes Assert if group name is in cluster policy ### Linked issues Does not resolve the issue, but asserts the key to avoid the key error in #1616 ### Functionality - [ ] added relevant user documentation - [ ] added new CLI command - [ ] modified existing command: `databricks labs ucx ...` - [ ] added a new workflow - [ ] modified existing workflow: `...` - [ ] added a new table - [ ] modified existing table: `...` ### Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [ ] manually tested - [ ] added unit tests - [ ] added integration tests - [ ] verified on staging environment (screenshot attached)
11 tasks
❌ test_running_real_assessment_job_ext_hms: AssertionError: Group ucx_Gfrhs not found in cluster policy (29m25.923s)
Running from nightly #55 |
github-merge-queue bot
pushed a commit
that referenced
this issue
May 9, 2024
## Changes (We think that) sometimes groups are not created before the test run, e.g. [this](https://github.com/databrickslabs/ucx/actions/runs/9002466231?pr=1532) and [that](#1616). This PR extends the `make_group` with an optional flag to wait for the groups to be created before continuing with the test. To limit impact on performance, we don't wait for the groups to be created by default. ### Linked issues Hopefully esolves #1616 ### Functionality - [ ] added relevant user documentation - [ ] added new CLI command - [ ] modified existing command: `databricks labs ucx ...` - [ ] added a new workflow - [ ] modified existing workflow: `...` - [ ] added a new table - [ ] modified existing table: `...` ### Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [x] manually tested - [ ] added unit tests - [ ] added integration tests - [ ] verified on staging environment (screenshot attached)
Not present anymore in latest test run |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
❌ test_running_real_assessment_job_ext_hms: KeyError: 'ucx_tnIh' (25m7.398s)
Running from nightly #49
The text was updated successfully, but these errors were encountered: