Skip to content

Fix profiling config in randomized tests#1682

Merged
morrisonlevi merged 1 commit into
masterfrom
levi/profiling-randomized-tests
Aug 1, 2022
Merged

Fix profiling config in randomized tests#1682
morrisonlevi merged 1 commit into
masterfrom
levi/profiling-randomized-tests

Conversation

@morrisonlevi

Copy link
Copy Markdown
Collaborator

Description

This was still using zend_extension instead of extension for datadog-profiling.so.

Readiness checklist

  • Changelog has been added to the release document.
  • Tests added for this feature/bug. This is tests.

Reviewer checklist

  • Appropriate labels assigned.
  • Milestone is set.

@morrisonlevi morrisonlevi added dev/testing profiling Relates to the Continuous Profiler labels Aug 1, 2022
@morrisonlevi morrisonlevi added this to the 0.77.0 milestone Aug 1, 2022
@morrisonlevi
morrisonlevi requested a review from a team as a code owner August 1, 2022 19:54
@morrisonlevi morrisonlevi changed the title Fix profiler config in randomized tests Fix profiling config in randomized tests Aug 1, 2022

@labbati labbati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch 👍🏻

@morrisonlevi
morrisonlevi merged commit a5f696d into master Aug 1, 2022
@morrisonlevi
morrisonlevi deleted the levi/profiling-randomized-tests branch August 1, 2022 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev/testing profiling Relates to the Continuous Profiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants