Skip to content

Update progress after GCS download finished#1698

Merged
danielmitterdorfer merged 1 commit intoelastic:masterfrom
danielmitterdorfer:fix-gcs-progress
Apr 3, 2023
Merged

Update progress after GCS download finished#1698
danielmitterdorfer merged 1 commit intoelastic:masterfrom
danielmitterdorfer:fix-gcs-progress

Conversation

@danielmitterdorfer
Copy link
Copy Markdown
Member

With this commit we ensure that the progress indicator is updated once again after a download from a GCS bucket has finished.

With this commit we ensure that the progress indicator is updated once
again after a download from a GCS bucket has finished.
@danielmitterdorfer danielmitterdorfer added bug Something's wrong :Track Management New operations, changes in the track format, track download changes and the like labels Mar 31, 2023
@danielmitterdorfer danielmitterdorfer added this to the 2.8.0 milestone Mar 31, 2023
@danielmitterdorfer danielmitterdorfer self-assigned this Mar 31, 2023
@danielmitterdorfer
Copy link
Copy Markdown
Member Author

I noticed this because on a new track that I've created I always saw this after the track data have been downloaded and I was puzzled why it wouldn't say 100%.

[INFO] Downloading track data (2.2 GB total size)                                 [ 98.9%]

With this change:

[INFO] Downloading track data (2.2 GB total size)                                [100.0%]

@dliappis dliappis self-requested a review March 31, 2023 09:45
Copy link
Copy Markdown
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM thanks for catching this

@danielmitterdorfer danielmitterdorfer merged commit 1d1e1f9 into elastic:master Apr 3, 2023
@danielmitterdorfer danielmitterdorfer deleted the fix-gcs-progress branch April 3, 2023 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something's wrong :Track Management New operations, changes in the track format, track download changes and the like

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants