chainhelen chainhelen Just want to get better. http://blog.hacking.pub/

issue closedenvoyproxy/envoy

data-plane-api lack tracing.operation_name: ingress

Title: data-plane-api lack tracing.operation_name: ingress

Description:

When I send Ads to all local envoy dynamiclly,I find that lacking tracing.operation_name: ingress

Repro steps:

import (  
        ...
    http_conn_manager "github.com/envoyproxy/go-control-plane/envoy/config/filter/network/http_connection_manager/v2"
        ...
)
    listenFilterHttpConn.Tracing = &http_conn_manager.HttpConnectionManager_Tracing{
        OperationName:  http_conn_manager.INGRESS,  
        RandomSampling: &_type.Percent{Value: 1.0},
    }

    listenFilterHttpConnConv, err := util.MessageToStruct(listenFilterHttpConn)

Config dump: I expect

    tracing:
        operation_name: ingress
        random_sampling: 1.0

but in fact, as below

    tracing:
        random_sampling: 1.0

Guess: I guess, maybe

OperationName HttpConnectionManager_Tracing_OperationName `protobuf:"varint,1,opt,name=operation_name,json=operationName,proto3,enum=envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager_Tracing_OperationName" json:"operation_name,omitempty"`

omitempty cause this promble, becase http_conn_manager.INGRESS is 0 in actually.

closed time in a day

chainhelen

issue commentenvoyproxy/envoy

data-plane-api lack tracing.operation_name: ingress

@htuch ok, get it.Thx for your reply

chainhelen

comment created time in a day

issue commentenvoyproxy/envoy

data-plane-api lack tracing.operation_name: ingress

@htuch Thx for your reply.

I expect

    tracing:
        operation_name: ingress
        random_sampling: 1.0

but in fact, as below

    tracing:
        random_sampling: 1.0

En, I just care,does this affect the final effect for envoy?

chainhelen

comment created time in 2 days

issue openedenvoyproxy/envoy

data-plane-api lack tracing.operation_name: ingress

Title: data-plane-api lack tracing.operation_name: ingress

Description:

When I send Ads to all local envoy dynamiclly,I find that lacking tracing.operation_name: ingress

Repro steps:

import (  
        ...
    http_conn_manager "github.com/envoyproxy/go-control-plane/envoy/config/filter/network/http_connection_manager/v2"
        ...
)
    listenFilterHttpConn.Tracing = &http_conn_manager.HttpConnectionManager_Tracing{
        OperationName:  http_conn_manager.INGRESS,  
        RandomSampling: &_type.Percent{Value: 1.0},
    }

    listenFilterHttpConnConv, err := util.MessageToStruct(listenFilterHttpConn)

Config dump: I expect

    tracing:
        operation_name: ingress
        random_sampling: 1.0

but in fact, as below

    tracing:
        random_sampling: 1.0

Guess: I guess, maybe

OperationName HttpConnectionManager_Tracing_OperationName `protobuf:"varint,1,opt,name=operation_name,json=operationName,proto3,enum=envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager_Tracing_OperationName" json:"operation_name,omitempty"`

omitempty cause this promble, becase http_conn_manager.INGRESS is 0 in actually.

created time in 9 days

push eventchainhelen/go-control-plane

chainhelen

commit sha 138272bc2e8f9a3a74ab4b18b7a0919441fec543

Update http_connection_manager.pb.go

view details

push time in 21 days

fork chainhelen/go-control-plane

Go implementation of data-plane-api

fork in 22 days

startedjaegertracing/jaeger

started time in a month

startedMisterBooo/LeetCodeAnimation

started time in a month

issue commentgo-delve/delve

Problem: Installation(repo) error on windows

@Bazis007 Something wrong with git, not dlv. Just like this about exit status 128, or it

Bazis007

comment created time in a month

push eventchainhelen/godbg

chainhelen

commit sha a8e4356b556aafda20667c84ee7be052d68a306a

Feature: list registers

view details

push time in 2 months

startedmbinary/USTC-CS-Courses-Resource

started time in 2 months

issue commentgo-delve/delve

dlv failed to find lib when debug consul

@varnson The source code are compiled failed,maybe you can try go build to find reason.

varnson

comment created time in 2 months

startedMicropoor/Micro8

started time in 2 months

issue commentgo-delve/delve

dlv throws "could not launch process: could not find external debug info file"

// ErrNoDebugInfoFound is returned when Delve cannot find the external debug information file.
var ErrNoDebugInfoFound = errors.New("could not find external debug info file")

It means there is not any debug symbols.

openstacks

comment created time in 2 months

startedfatedier/frp

started time in 2 months

pull request commentgo-delve/delve

service/debugger: Add tryMatchRelativePathByProc

Done, thx for your guidance.

chainhelen

comment created time in 2 months

push eventchainhelen/delve

chainhelen

commit sha d07c22336a41da83c2e0c7b60758609a572807c3

service/debugger: support relative paths in location expressions If the user specifies a relative path in a location expression try to match it relative to the path of the executable. Fixes #1474

view details

push time in 2 months

PR opened go-delve/delve

service/debugger: Add tryMatchRelativePathByProc

This change try to match relative path according to the directory of ptraced executable program when loc.FileMatch fail.

Fixes #1474

En, If you don't support this pr, I will close it.

+6 -1

0 comment

1 changed file

pr created time in 2 months

create barnchchainhelen/delve

branch : relativepath

created branch time in 2 months

push eventchainhelen/delve

chainhelen

commit sha ea54a6b2aff14b91d2054838bba708233f58fcce

cmd,service: remove temporary def of `Server interface` in func Use the defination of Server interface in service package, instead of temporary in func.

view details

Russ Magee

commit sha 4e4ed0294807c7be65513112bde18dd70c0f165f

config,terminal: Add source-list-line-color configuration option This change adds a config flag to specify the foreground color of line numbers for the source list command.

view details

aarzilli

commit sha 74c98bc9616ad4ee324eb4d31717b61f4a6ab8ce

proc: support position independent executables (PIE) Support for position independent executables (PIE) on the native linux backend, the gdbserver backend on linux and the core backend. Also implemented in the windows native backend, but it can't be tested because go doesn't support PIE on windows yet.

view details

chainhelen

commit sha 143cf6aebfc0a23a57fb61420e92bd2dfe46b9b1

pkg/proc: extend conversion about `string` for array/str Extend `string()` 1.convert `byte/rune array`(not only `slice`) to string. 2.convert string to string(itself), just like `string(str)` in go language.

view details

aarzilli

commit sha d2904322faaa569162164ac06614ffd86ddb1903

proc: add flag to disable escape checking in function calls Fix escape checking in function calls and add a flag to disable it.

view details

aarzilli

commit sha b59032516eef15e65ff7cc6d01677712dc3baf6a

proc,service,terminal: eval expressions in the scope of a deferred call Add ability to evaluate variables on the scope of a deferred call's argument frame.

view details

aarzilli

commit sha d7ea38b89a858c5820827c50042dcf4ac39a3c99

Documentation: rename "where" parameter of checkpoint to "note" The name "where" may confuse users into thinking that this parameter actually does something where in fact it's just arbitrary text used to identify the checkpoint. Fixes #1373

view details

aarzilli

commit sha f72c48c034f8a876421a6a4b2bc2cebb3b65391c

proc: Continue should always work after CallFunction Continue did not resume execution after a call to CallFunction if the point where the process was stopped, before the call CallFunction, was a breakpoint. Fixes #1374

view details

aarzilli

commit sha b5cf58456418a76902aa551ed5b022c90cd76fa7

proc/gdbserial: backward continue should stop at start of process ContinueOnce didn't detect the way RR signals that it has reached the start of the process and would never finish. Fixes #1376

view details

Derek Parker

commit sha c9206300824da892a617c98071f53526db03cd74

cmd: Fix trace bug when no package specified Fix a bug where the regexp would be used as a package string incorrectly. This would manifest when trying to execute the trace subcommand as `dlv trace <regexp>` without specifying a package identifier. The `<regexp>` would be passed to the Go build command causing an error.

view details

Derek Parker

commit sha ff760d87aaa718e2d2aa9293371c5b0c8714ed81

cmd: Add --exec and --test flags for trace sub cmd Adds an --exec flag for the trace subcommand allowing users to specify a pre-compiled binary to exec and trace. Also adds a --test flag as a convienance for compiling and tracing a test binary. Fixes #1073

view details

aarzilli

commit sha d0f0a872a7ef545977dc7bc58c032b446a94a51b

Documentation,Makefile: better error when gencert.sh fails Fixes #1378

view details

aarzilli

commit sha 4db99398458dabd17c76c648a88ca343c94c8c90

proc/native: Mask MS_VC_EXCEPTION [windows] Some libraries (for example steam_api64.dll) will send this exception code to set the thread name on Microsoft VisualC. In theory it should be fine to send the exception back to the target, which is responsible for setting a handler for it, in practice in some cases (steam_api64.dll) this will crash the program. So we'll mask it instead. Fixes #1383

view details

Derek Parker

commit sha 3129aa73304be77b9787701c6f8a9e4ad9a2ce0c

*: Show return values on CLI trace This patch allows the `trace` CLI subcommand to display return values of a function. Additionally, it will also display information on where the function exited, which could also be helpful in determining the path taken during function execution. Fixes #388

view details

aarzilli

commit sha ea3428550d3ff55fd569640fc01e5aaefe37bc18

proc/native,proc/core: deduplicate linux register handling code The linux version of proc/native and proc/core contained largely overlapping implementations of the register handling code, deduplicate it by moving it into proc/linutil.

view details

Derek Parker

commit sha 810b22a6331b9839d75772396d235b2b8e9fb78f

pkg/dwarf: Remove exec bit from reader.go For some reason the Go file pkg/dwarf/reader/reader.go had the exec bit set on it. This patch removes that.

view details

aarzilli

commit sha 73d636f7d7adf159e26f35f37aba514f647492ce

tests: rename _fixtures/vendor to _fixtures/internal Some tests used a fake vendor directory placed inside _fixtures to import some support packages. In go.mod mode vendor directory are only supported on the root of the project, which breaks some of our tests. Since vendor directories outside the root of the project are so rare anyway it's possible that a future version of go will stop supporting it even in GOPATH mode. Also it was weird and unnecessary in the first place anyawy.

view details

aarzilli

commit sha 7d0d2cb3e60f4e6041f3f809bb0f0c421297d270

pkg/core: make code in core.go OS/arch agnostic Make code in core.go OS and architecture agnostic in preparation for adding Windows minidump support.

view details

Derek Parker

commit sha 4927bc6c7a2045ba52f9f77633c867d39cafbbe5

*: Switch to using Go modules

view details

Derek Parker

commit sha 2f746e1d3cc73d45557bca71844a5c297dd1750c

pkg/config: Clean up godoc lint issues

view details

push time in 2 months

push eventchainhelen/godbg

chainhelen

commit sha d0fa37a45a1623b6b1795b4bca1a22853a05ab1a

Finish: Findlocation and SetBreakPoint

view details

push time in 3 months

push eventchainhelen/godbg

chainhelen

commit sha 8fa45f69b3b22e8d4745b1db768d63d6160c32b1

Feature: add log functions

view details

push time in 3 months

push eventchainhelen/godbg

chainhelen

commit sha 675f8ab0462a77b9a3c6edc550c5240714ae3fd1

Feature: add some description into README.md

view details

push time in 3 months

push eventchainhelen/godbg

chainhelen

commit sha 0737eb51ba67b6c07c8b7b62c2ef184d38d605ed

Feature: scan variables from .dwarf

view details

push time in 3 months

issue commentgo-delve/delve

Setting a value does not work on function calls

@chainhelen unfortunately it doesn't solve the problem but thanks for noticing the issue.

I mean some wrong with your description, maybe be 404 here, not 22?

dlsniper

comment created time in 3 months

issue commentgo-delve/delve

Setting a value does not work on function calls

@dlsniper image

Maybe 404, not 22??

dlsniper

comment created time in 3 months

pull request commentgin-gonic/gin

Keep the logic of handling errors in render

Ha, I found #1739 include the same modification. so my commit can be closed.

I just can't understand why you merge his commit which include the same modification as mine, but you don't support my suggestion?

chainhelen

comment created time in 3 months

more