Skip to content

PMM-14109-improve-advisor-ux - #4478

Draft
ademidoff wants to merge 22 commits into
v3from
PMM-14109-improve-advisor-ux
Draft

PMM-14109-improve-advisor-ux#4478
ademidoff wants to merge 22 commits into
v3from
PMM-14109-improve-advisor-ux

Conversation

@ademidoff

@ademidoff ademidoff commented Jul 16, 2026

Copy link
Copy Markdown
Member

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6767/

@Percona-Lab Percona-Lab deleted a comment from JNKPercona Jul 16, 2026
@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6775/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6790/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have failed: https://pmm.cd.percona.com/job/pmm3-api-tests/6858/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6860/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6861/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6863/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6883/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6905/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/6936/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/7065/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/7071/

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/7172/

Copy link
Copy Markdown
Contributor

The red @fb-alerting|@fb-settings check reproduced — it's a pmm-qa test issue in the PMM-14109 companion branch, not a problem with the server change. getSubpageTooltips() in pmmSettingsPage.js still lists the Advisors tooltip under /settings/advanced-settings, but percona/pmm#5656 moves AdvisorsForm to the dedicated /settings/advisors tab, so PMM-T1227 waits for [data-testid=advisors-label-description] on a page that no longer renders it.

Fix: percona/pmm-qa#1258, opened against PMM-14109-improve-advisor-ux (not main, where the current mapping is still correct). Verified on a throwaway VM with your FB image perconalab/pmm-server-fb:PR-4478-1cb361d: failing before, 1 passed after, and the spec's other @fb-settings scenarios stay green.

Merge that into your pmm-qa branch and the next FB run should be clean.


Generated by Claude Code

@JNKPercona

Copy link
Copy Markdown
Collaborator

Looks like there are outdated source branches.
Please update them and restart the job
percona/pmm#5656

@JNKPercona

Copy link
Copy Markdown
Collaborator

@JNKPercona

Copy link
Copy Markdown
Collaborator

API tests have succeded: https://pmm.cd.percona.com/job/pmm3-api-tests/7185/

Copy link
Copy Markdown
Contributor

Your red FB Tests run (33154828980) was looked at — the single failing check, @service-account / PMM-T1883, is a pmm-qa test bug and not your change. I reproduced it on a throwaway VM at your exact FB build (3.10.0-PMM-14109-improve-advisor-ux-b42a0448d): the test opens MySQL Instances Overview over from=now-1m right after starting its write load, but the URL carries no refresh param and that dashboard ships with refresh disabled, so Grafana queries the panels once — over a window that predates the load — and the 300s waitForGraphsToHaveData then just re-counts a frozen render. While the unfixed test sat there showing empty panels, the panel's own query returned 0.41 and the InnoDB counters were advancing in 25 of 30 consecutive 5s scrapes; the data was there, nothing was re-reading it. The metrics themselves are fine, so there's no product bug here.

Fix: percona/pmm-qa#1271 (adds refresh=10s; 3/3 first-attempt passes vs a first-attempt failure before). Note your FB run pins PMM_QA_BRANCH=PMM-14109-improve-advisor-ux, so it'll pick the fix up once that pmm-qa branch (#1083) is rebased on main after #1271 lands. Since it's a race rather than a hard failure, a re-run of the failed job may also come back green in the meantime.


Generated by Claude Code

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.

3 participants