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

symbolication: reduce error logs for local symbol upload [PROF-10143] #39

Merged
merged 2 commits into from
Jul 10, 2024

Conversation

Gandem
Copy link
Member

@Gandem Gandem commented Jul 9, 2024

Reduce error logs for local symbol upload by:

  • Avoiding logging an error for short-lived executables
  • Splitting symbol copy and symbol upload timeouts to give local upload more chance to succeed

Best reviewed commit by commit!

@DataDog/profiling-full-host

@Gandem Gandem force-pushed the nayef/symbol-upload branch 3 times, most recently from afdc567 to 85756b8 Compare July 9, 2024 16:59
@Gandem Gandem changed the title symbolication: separate extraction from upload timeouts symbolication: reduce error logs for local symbol upload [PROF-10143] Jul 9, 2024
@Gandem Gandem force-pushed the nayef/symbol-upload branch 3 times, most recently from 3596058 to 8a5165f Compare July 10, 2024 05:37
@Gandem Gandem marked this pull request as ready for review July 10, 2024 07:40
@Gandem Gandem requested a review from a team as a code owner July 10, 2024 07:40
we're seeing sporadic timeouts on internal fleets, splitting
timeouts to give both the extraction and the upload a bit more
headroom
@Gandem Gandem force-pushed the nayef/symbol-upload branch from 8a5165f to 85fc1b0 Compare July 10, 2024 07:40
short lived executables might no longer exist by the time we want
to copy symbols, this hopefully reduces the amount of error logs
that the uploader outputs for those executables
@Gandem Gandem force-pushed the nayef/symbol-upload branch from 85fc1b0 to d0d80b6 Compare July 10, 2024 08:06
@Gandem Gandem merged commit e3277a2 into main Jul 10, 2024
15 checks passed
@Gandem Gandem deleted the nayef/symbol-upload branch July 10, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants