tracing disabled in pprof facility

Bug #1742955 reported by Junien Fridrick on 2018-01-12
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju
Wishlist
Anastasia

Bug Description

Hi,

It's not possible to have a trace with the current pprof facility.
Comments at the top of worker/introspection/pprof/pprof.go say :

// - For compatability with Go 1.2.1, support for obtaining trace data
// has been removed.

But we don't really care about Go 1.2.1, do we ?

Thanks

AIUI, pprof is a sampling tool (wakes up every 100ms and checks the call
stack). I didn't think it was possible to operate in tracing mode and have
it actually record all call stacks all the time.

On Fri, Jan 12, 2018 at 6:23 PM, Junien Fridrick <<email address hidden>
> wrote:

> Public bug reported:
>
> Hi,
>
> It's not possible to have a trace with the current pprof facility.
> Comments at the top of worker/introspection/pprof/pprof.go say :
>
> // - For compatability with Go 1.2.1, support for obtaining trace data
> // has been removed.
>
> But we don't really care about Go 1.2.1, do we ?
>
> Thanks
>
> ** Affects: juju
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are subscribed to juju.
> Matching subscriptions: juju bugs
> https://bugs.launchpad.net/bugs/1742955
>
> Title:
> tracing disabled in pprof facility
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju/+bug/1742955/+subscriptions
>

Anastasia (anastasia-macmood) wrote :

@Junien Fridrick,

Could you please explain why you need it in 'trace" mode?

Changed in juju:
status: New → Incomplete
Junien Fridrick (axino) wrote :

I think it could help diagnosing bugs such as bug 1733708.

Changed in juju:
status: Incomplete → New
Changed in juju:
status: New → Triaged
importance: Undecided → Wishlist
Anastasia (anastasia-macmood) wrote :

@Junien Fridrick (axino) ,

I have proposed against develop (heading into 2.4): https://github.com/juju/juju/pull/8358

Could you please have a look and take it for a spin before I land? Does this answer your needs?

Changed in juju:
status: Triaged → In Progress
assignee: nobody → Anastasia (anastasia-macmood)
Changed in juju:
status: In Progress → Fix Committed
milestone: none → 2.4-beta1
Changed in juju:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers