fix(users): Online Users 'All time' filter returns empty (#28145) [1.13]

[sonika-shah]

Cherry-pick of #28145 to 1.13.

Summary

Closes #27993 on the 1.13 release branch.

The /v1/users/online endpoint computed the SQL threshold as now - timeWindow*60*1000. When the UI sent timeWindow=0 for the All time dropdown option, the threshold collapsed to now, producing lastActivityTime > now — which matches zero rows. The page therefore rendered empty whenever a user selected All.

The fix skips the time predicate entirely when timeWindow <= 0, so the filter falls through to "all non-bot users".

Conflict resolution

UserResourceIT.java import block — kept both the existing SubjectCache/SubjectContext imports (already on 1.13) and the new HttpMethod/RequestOptions imports the new test needs.

Test plan

• mvn -pl openmetadata-service,openmetadata-integration-tests -am test-compile passes locally
• CI green
• Manual: in the UI, open Settings → Members → Online Users, select All time — list populates

[gitar-bot]

Code Review ✅ Approved

Cherry-pick of the fix for the 'All time' online users filter, which now skips the time predicate when the window is zero to correctly return all non-bot users. No issues found.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[github-actions]

The Python checkstyle failed.

Please run make py_format and py_format_check in the root of your repository and commit the changes to this PR.
You can also use pre-commit to automate the Python code formatting.

You can install the pre-commit hooks with make install_test precommit_install.

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ingestion'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

🔴 Playwright Results — 13 failure(s), 16 flaky

✅ 3756 passed · ❌ 13 failed · 🟡 16 flaky · ⏭️ 90 skipped

Shard	Passed	Failed	Flaky	Skipped
✅ Shard 1	290	0	0	4
🟡 Shard 2	710	0	4	7
🔴 Shard 3	705	13	2	12
🟡 Shard 4	730	0	5	24
✅ Shard 5	623	0	0	35
🟡 Shard 6	698	0	5	8

Genuine Failures (failed on all attempts)

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/SearchSeparation/searchSeparationSuite.ts › live indexing produces searchable separation for all four facets (shard 3)

TypeError: (0 , _admin.createAdminApiContext) is not a function

❌ Features/Tasks/TaskNavigation.spec.ts › clicking task notification while on entity task tab refreshes the task list (shard 3)

�[31mTest timeout of 180000ms exceeded.�[39m

❌ Features/Tasks/TaskNavigation.spec.ts › two sessions: admin on Columns tab creates task, assignee sees refresh on notification click (shard 3)

�[31mTest timeout of 180000ms exceeded.�[39m

🟡 16 flaky test(s) (passed on retry)

• Features/BulkEditEntity.spec.ts › Glossary (shard 2, 1 retry)
• Features/ColumnBulkOperations.spec.ts › should keep latest search results when responses arrive out of order (shard 2, 1 retry)
• Features/Glossary/GlossaryP3Tests.spec.ts › should handle multiple rapid API calls (shard 2, 1 retry)
• Features/Glossary/GlossaryWorkflow.spec.ts › should start term as Draft when glossary has reviewers (shard 2, 1 retry)
• Features/Permissions/GlossaryPermissions.spec.ts › Team-based permissions work correctly (shard 3, 1 retry)
• Flow/PersonaFlow.spec.ts › Set default persona for team should work properly (shard 3, 1 retry)
• Pages/DataContracts.spec.ts › Create Data Contract and validate for Store Procedure (shard 4, 1 retry)
• Pages/DataContracts.spec.ts › Create Data Contract and validate for Api Collection (shard 4, 1 retry)
• Pages/DataContractsSemanticRules.spec.ts › Validate Owner Rule Is_Set (shard 4, 1 retry)
• Pages/Domains.spec.ts › Rename domain with owners and experts preserves assignments (shard 4, 1 retry)
• Pages/Domains.spec.ts › Multiple consecutive domain renames preserve all associations (shard 4, 1 retry)
• Pages/Lineage/DataAssetLineage.spec.ts › verify create lineage for entity - Dashboard (shard 6, 1 retry)
• Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
• Pages/Lineage/LineageRightPanel.spec.ts › Verify custom properties tab IS visible for supported type: searchIndex (shard 6, 1 retry)
• Pages/ServiceEntity.spec.ts › Inactive Announcement create & delete (shard 6, 1 retry)
• Pages/Users.spec.ts › Permissions for table details page for Data Consumer (shard 6, 1 retry)

📦 Download artifacts

How to debug locally

# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace