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

feat: record metrics of conversion in task #192

Merged
merged 2 commits into from
Sep 18, 2023

Conversation

Desiki-high
Copy link
Collaborator

We have collected the metrics of conversion progress. Let's show them in the task. Add the SourceImageSize and TargetImageSize in the task and record them when the task is finished.

resolve #191.

@Desiki-high
Copy link
Collaborator Author

It seems some error happened in Concurrent, maybe the singleflight, I will fix it later.

@Desiki-high Desiki-high force-pushed the feat/task-metric branch 2 times, most recently from 4715090 to 4fe8a7b Compare September 14, 2023 13:00
@Desiki-high Desiki-high requested a review from imeoer September 14, 2023 13:17
@Desiki-high Desiki-high added the feat New feature or request label Sep 14, 2023
pkg/task/manager.go Outdated Show resolved Hide resolved
docs/development.md Outdated Show resolved Hide resolved
@Desiki-high Desiki-high force-pushed the feat/task-metric branch 3 times, most recently from f2515af to 4434483 Compare September 15, 2023 11:47
@Desiki-high Desiki-high requested a review from imeoer September 16, 2023 03:11
Copy link
Collaborator

@imeoer imeoer left a comment

Choose a reason for hiding this comment

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

other LGTM! Thanks!

We have collected the metrics of conversion progress. Let's show them in task.
Add the `SourceSize` and `TargetSize` in task and record them when
task finished.

Signed-off-by: Yadong Ding <[email protected]>
@Desiki-high Desiki-high requested a review from imeoer September 18, 2023 02:57
@imeoer imeoer merged commit eeaf124 into goharbor:main Sep 18, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature request: use metrics of conversion
2 participants