Skip to content
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

[YW] Running a backup sometimes causes TaskProgressStepbar to flicker on screen #3429

Closed
andrewc-dev opened this issue Jan 23, 2020 · 1 comment
Assignees
Labels
area/platform Yugabyte Platform priority/high High Priority

Comments

@andrewc-dev
Copy link
Contributor

Create a backup and go to the universe-level Tasks tab and see what happens visually. The TaskProgressStepbar appears above the Task History table and flickers briefly then goes away.

@andrewc-dev andrewc-dev self-assigned this Jan 23, 2020
@kmuthukk kmuthukk added the area/platform Yugabyte Platform label Jan 23, 2020
@rao-vasireddy rao-vasireddy added the priority/high High Priority label Jan 23, 2020
andrewc-dev pushed a commit to andrewc-dev/yugabyte-db that referenced this issue Jan 24, 2020
…se tables and remove border from TaskProgress

Summary:
Add additional check whether tables have been already fetched before calling API to fetch
tables. Remove border from TaskProgress because it was distracting and caused the spacing to look
strange.

Test Plan:
Create a backup (preferably one that takes a long time), and then navigate to the
universe-level Tasks tab and check what happens to the UI. Make sure that the stepbar does not do
any flickering.
*One symptom of the incorrect code was that a TON of network requests were being made and causing the UI to lag*

I recommend connecting local YW to customer's environment to repro. After tunneling, you can add the url to .env with `REACT_APP_YUGAWARE_API_URL=http://localhost:8080/api`

Example of fixed UI:
{F13140}

Reviewers: wesley, arnav, ram

Reviewed By: ram

Subscribers: kannan, rao, ui

Differential Revision: https://phabricator.dev.yugabyte.com/D7831
@andrewc-dev andrewc-dev reopened this Jan 24, 2020
andrewc-dev added a commit that referenced this issue Jan 24, 2020
…s and remove border from TaskProgress (#3440)

Summary:
Add additional check whether tables have been already fetched before calling API to fetch
tables. Remove border from TaskProgress because it was distracting and caused the spacing to look
strange.

Test Plan:
Create a backup (preferably one that takes a long time), and then navigate to the
universe-level Tasks tab and check what happens to the UI. Make sure that the stepbar does not do
any flickering.
*One symptom of the incorrect code was that a TON of network requests were being made and causing the UI to lag*

I recommend connecting local YW to customer's environment to repro. After tunneling, you can add the url to .env with `REACT_APP_YUGAWARE_API_URL=http://localhost:8080/api`

Example of fixed UI:
{F13140}

Reviewers: wesley, arnav, ram

Reviewed By: ram

Subscribers: kannan, rao, ui

Differential Revision: https://phabricator.dev.yugabyte.com/D7831
andrewc-dev added a commit that referenced this issue Jan 24, 2020
…s and remove border from TaskProgress (#3440)

Summary:
Add additional check whether tables have been already fetched before calling API to fetch
tables. Remove border from TaskProgress because it was distracting and caused the spacing to look
strange.

Test Plan:
Create a backup (preferably one that takes a long time), and then navigate to the
universe-level Tasks tab and check what happens to the UI. Make sure that the stepbar does not do
any flickering.
*One symptom of the incorrect code was that a TON of network requests were being made and causing the UI to lag*

I recommend connecting local YW to customer's environment to repro. After tunneling, you can add the url to .env with `REACT_APP_YUGAWARE_API_URL=http://localhost:8080/api`

Example of fixed UI:
{F13140}

Reviewers: wesley, arnav, ram

Reviewed By: ram

Subscribers: kannan, rao, ui

Differential Revision: https://phabricator.dev.yugabyte.com/D7831
@ramkumarvs
Copy link
Contributor

@yb-andrew can we close this issue?

carlos-username pushed a commit to carlos-username/yugabyte-db that referenced this issue Mar 11, 2020
…se tables and remove border from TaskProgress (yugabyte#3440)

Summary:
Add additional check whether tables have been already fetched before calling API to fetch
tables. Remove border from TaskProgress because it was distracting and caused the spacing to look
strange.

Test Plan:
Create a backup (preferably one that takes a long time), and then navigate to the
universe-level Tasks tab and check what happens to the UI. Make sure that the stepbar does not do
any flickering.
*One symptom of the incorrect code was that a TON of network requests were being made and causing the UI to lag*

I recommend connecting local YW to customer's environment to repro. After tunneling, you can add the url to .env with `REACT_APP_YUGAWARE_API_URL=http://localhost:8080/api`

Example of fixed UI:
{F13140}

Reviewers: wesley, arnav, ram

Reviewed By: ram

Subscribers: kannan, rao, ui

Differential Revision: https://phabricator.dev.yugabyte.com/D7831
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform Yugabyte Platform priority/high High Priority
Projects
None yet
Development

No branches or pull requests

4 participants