Skip to content

Conversation

@sjd78
Copy link
Member

@sjd78 sjd78 commented May 29, 2025

Resolves: https://issues.redhat.com/browse/MTA-5181

Historically and for performance reasons, the application inventory page tried to not always refresh in the background. The approach never really consistently worked, and if hub has a large number of applications, the refresh query logic would almost always run anyway.

The refresh logic has been removed and the page will refresh the applications every 5 seconds.

Performance issues may emerge in a large environment and #2350 was written to capture what is known and have some ideas about how to approach a solution.

Resolves: https://issues.redhat.com/browse/MTA-5181

For performance reasons, the application inventory page
does not always refresh in the background.  When there
are no active tasks or imports, the application query
does not auto-refresh.  At the point where the app goes
from having active tasks to no active tasks (e.g. running
an analysis), it is possible that the auto-refresh is
disabled before updates can be fetched.  This is at most
a 5 second window.

To avoid this, when the tasks dashboard query goes from having
active tasks to no active tasks, the application query is
invalidated forcing the query to refresh its data.

Signed-off-by: Scott J Dickerson <[email protected]>
@sjd78 sjd78 requested review from ibolton336 and rszwajko as code owners May 29, 2025 19:09
@sjd78 sjd78 added this to the v0.7.1 milestone May 29, 2025
@sjd78 sjd78 added the cherry-pick/release-0.7 This PR should be cherry-picked to release-0.7 branch label May 29, 2025
@sjd78 sjd78 changed the title 🐛 when active tasks go inactive, invalidate the application query 🐛 always refresh the application query on application inventory Jun 2, 2025
@sjd78 sjd78 force-pushed the applicaiton-table-refresh branch from 66c2a39 to ca6b23c Compare June 2, 2025 14:26
Copy link
Collaborator

@rszwajko rszwajko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good decision!

@sjd78 sjd78 merged commit a024e87 into konveyor:main Jun 2, 2025
10 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 2, 2025
Resolves: https://issues.redhat.com/browse/MTA-5181

Historically and for performance reasons, the application inventory page
tried to not always refresh in the background. The approach never really
consistently worked, and if hub has a large number of applications, the
refresh query logic would almost always run anyway.

The refresh logic has been removed and the page will refresh the
applications every 5 seconds.

Performance issues may emerge in a large environment and #2350 was
written to capture what is known and have some ideas about how to
approach a solution.

---------

Signed-off-by: Scott J Dickerson <[email protected]>
Signed-off-by: Cherry Picker <[email protected]>
sjd78 pushed a commit that referenced this pull request Jun 2, 2025
… (#2353)

Resolves: https://issues.redhat.com/browse/MTA-5181

Historically and for performance reasons, the application inventory page
tried to not always refresh in the background. The approach never really
consistently worked, and if hub has a large number of applications, the
refresh query logic would almost always run anyway.

The refresh logic has been removed and the page will refresh the
applications every 5 seconds.

Performance issues may emerge in a large environment and #2350 was
written to capture what is known and have some ideas about how to
approach a solution.

---------

Signed-off-by: Scott J Dickerson <[email protected]>
Signed-off-by: Cherry Picker <[email protected]>
@sjd78 sjd78 deleted the applicaiton-table-refresh branch June 18, 2025 04:09
sshveta pushed a commit to sshveta/tackle2-ui that referenced this pull request Oct 31, 2025
…eyor#2346)

Resolves: https://issues.redhat.com/browse/MTA-5181

Historically and for performance reasons, the application inventory page
tried to not always refresh in the background. The approach never really
consistently worked, and if hub has a large number of applications, the
refresh query logic would almost always run anyway.

The refresh logic has been removed and the page will refresh the
applications every 5 seconds.

Performance issues may emerge in a large environment and konveyor#2350 was
written to capture what is known and have some ideas about how to
approach a solution.

---------

Signed-off-by: Scott J Dickerson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick/release-0.7 This PR should be cherry-picked to release-0.7 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants