Skip to content

Cap the spacesPlugin wait at Kibana 9.5.3 - #137

Closed
coutoPL wants to merge 1 commit into
developfrom
fix/spaces-plugin-wait-cap
Closed

coutoPL wants to merge 1 commit into
developfrom
fix/spaces-plugin-wait-cap

Conversation

@coutoPL

@coutoPL coutoPL commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

What is broken

changeTenancyAndAwaitSpaces intercepts */bundles/plugin/spaces/1.0.0/spaces.chunk* and waits for
it on every Kibana version >= 9.4.0. On Kibana 9.5.4 that bundle no longer loads on a tenancy
switch, so the intercept never fires and cy.wait('@spacesPlugin') times out after 10s.

This fails the sanity check ro kibana access and sanity check ro_strict kibana access specs on
every e2e run against ES/KBN 9.4.x and 9.5.4 since readonlyrest_kbn added 9.4.7/9.5.4 support
(RORDEV-2228/2230, readonlyrest_kbn#1043).

The fix

Cap the wait at Kibana <= 9.5.3. This mirrors the fix already applied in readonlyrest_kbn's local
automatic-tests/ copy of this same file (readonlyrest_kbn#1043) — that fix never made it into this
repo, which is the actual source cloned by CI (scripts/e2eTestsLib.sh in readonlyrest_kbn points
here), so every run kept hitting the unfixed code.

Follow-up

Same fix needs to land on master too, since a manual/dispatch run with no matching branch falls
back to develop then master — I'll open a companion PR there.

Kibana 9.5.4 no longer loads the spaces.chunk bundle on a tenancy
switch, so the intercept never fires and cy.wait('@spacesPlugin')
times out. Mirrors the fix already applied in readonlyrest_kbn's
automatic-tests copy (PR #1043).
@coderabbitai

This comment was marked as resolved.

@coutoPL

coutoPL commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator Author

Consolidating into a single PR against master (RORDEV-2229) — closing this one.

@coutoPL coutoPL closed this Sep 17, 2026
@coutoPL
coutoPL deleted the fix/spaces-plugin-wait-cap branch September 17, 2026 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants