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.

Sign up for GitHub

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

Jump to bottom

fix(py): unwanted trace_id #627

Merged
merged 1 commit into from
Jun 28, 2024
Merged

fix(py): unwanted trace_id #627

merged 1 commit into from
Jun 28, 2024

Conversation

eeliu
Copy link
Collaborator

@eeliu eeliu commented Jun 28, 2024 •
edited
Loading

Describe your changes

Issue ticket number and link

#626

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics?
  • Will this be part of a product update? If yes, please write one phrase about this update.

@eeliu eeliu marked this pull request as draft June 28, 2024 05:13
@codecov Codecov
Copy link

codecov bot commented Jun 28, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.70%. Comparing base (c93e207) to head (3ad8f77).
Report is 22 commits behind head on dev.

Additional details and impacted files 
@@            Coverage Diff             @@
##              dev     #627      +/-   ##
==========================================
- Coverage   89.61%   80.70%   -8.91%     
==========================================
  Files          22       22              
  Lines        1569     1529      -40     
  Branches        0      165     +165     
==========================================
- Hits         1406     1234     -172     
+ Misses        163      162       -1     
- Partials        0      133     +133

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@eeliu eeliu force-pushed the patch-626 branch 4 times, most recently from 91cd49f to da09cea Compare June 28, 2024 07:55
@eeliu
fix(py): unwanted trace_id
3ad8f77 
- fix collector-agent compiled problem
- patch for pinpoint-apm#626
@eeliu eeliu marked this pull request as ready for review June 28, 2024 08:35
@eeliu eeliu merged commit 5d86c28 into pinpoint-apm:dev Jun 28, 2024
17 of 18 checks passed
@eeliu eeliu deleted the patch-626 branch June 28, 2024 08:36
@eeliu eeliu linked an issue Jul 1, 2024 that may be closed by this pull request
<built-in function add_clue> returned a result with an error set #626
Closed
eeliu added a commit to eeliu/pinpoint-c-agent that referenced this pull request Jul 9, 2024
@eeliu @mayingping-Bella
@dependabot @Evelyn0927
Ci upload assert (#66)
c05095f 
* support windows python (pinpoint-apm#592)

* feat(id): support unique id

- unique id in ms
- refactor common library
- remove dependence from shm
- php-agent 0.5.0
- python-agent v1.2.1

* feat(win32): support win32

- python-agent
- common library
- refactor common library
- fix bug when pooling
- cibuildwheel
- code style
- fix bug in endTrace

* fix bug in thread mode

* remove ambiguous API

* pipy 1.2.4 release test

* test char encoding

* fix bug in python

* hello packet, avoiding conflict

* fix bug in flask

* fix convert problem

* delete 'import MySQLdb' (pinpoint-apm#593)

Co-authored-by: eeliu <[email protected]>

* fix wheels bug

* intro docs

* feat:grpc-python (pinpoint-apm#588)

- api and example

* add docs for grpc

* Bump pymongo from 4.6.0 to 4.6.3 in /plugins/PY (pinpoint-apm#594)

Bumps [pymongo](https://github.com/mongodb/mongo-python-driver) from 4.6.0 to 4.6.3.
- [Release notes](https://github.com/mongodb/mongo-python-driver/releases)
- [Changelog](https://github.com/mongodb/mongo-python-driver/blob/master/doc/changelog.rst)
- [Commits](mongodb/mongo-python-driver@4.6.0...4.6.3)

---
updated-dependencies:
- dependency-name: pymongo
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(agent): sql v3 support (pinpoint-apm#595)

* feat(agent): sql v3 support

- remove metadata.go

* fix build script

* modify docs

* feat(py): PostgreSql (pinpoint-apm#599)

- psycopg2
- version :1.2.1

* fix import error

> PsycopgPlugins should follow the psycopg2

* feat(php): simple-php (pinpoint-apm#600)

- example for php

> hopes to help  pinpoint-apm#596

* fix(agent): status_code (pinpoint-apm#603)

- treats >=400  as an error

>pinpoint-apm#602

* correct the message format

* support collector-agent on Arm (pinpoint-apm#605)

* add arm

> pinpoint-apm#604

* docs(py): Guide for exclude url (pinpoint-apm#610)

> user path route

pinpoint-apm#609

* fix(py): redis plugin (pinpoint-apm#613)

- fix: Redis Collections Not release pinpoint-apm#612

* fix ci failed

- docs: python plugins

* --- (pinpoint-apm#616)

updated-dependencies:
- dependency-name: pymysql
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* --- (pinpoint-apm#615)

updated-dependencies:
- dependency-name: requests
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(py): fix redis connection (pinpoint-apm#617)

- use format_host
* add default value

* update test pinpointpy 1.3.4

* feat(collector-agent): error (pinpoint-apm#608)

- error analysis
- common,collector-agent  `EXP_V2`
- python new api `add_exception`

* Bump fastapi from 0.104.1 to 0.109.1 in /plugins/PY (pinpoint-apm#614)

Bumps [fastapi](https://github.com/tiangolo/fastapi) from 0.104.1 to 0.109.1.
- [Release notes](https://github.com/tiangolo/fastapi/releases)
- [Commits](fastapi/fastapi@0.104.1...0.109.1)

---
updated-dependencies:
- dependency-name: fastapi
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: eeliu <[email protected]>

* --- (pinpoint-apm#618)

updated-dependencies:
- dependency-name: requests
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: eeliu <[email protected]>

* Bump pymysql from 1.1.0 to 1.1.1 in /testapps/PY (pinpoint-apm#619)

Bumps [pymysql](https://github.com/PyMySQL/PyMySQL) from 1.1.0 to 1.1.1.
- [Release notes](https://github.com/PyMySQL/PyMySQL/releases)
- [Changelog](https://github.com/PyMySQL/PyMySQL/blob/main/CHANGELOG.md)
- [Commits](PyMySQL/PyMySQL@v1.1.0...v1.1.1)

---
updated-dependencies:
- dependency-name: pymysql
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: eeliu <[email protected]>

* build(go): go install (pinpoint-apm#622)

- support `go install`

* update for collector-agent

* fix(py): unwanted trace_id (pinpoint-apm#627)

- fix collector-agent compiled problem
- patch for pinpoint-apm#626

* support c level plugin in php (pinpoint-apm#623)

* test c-level plugins

* test pinpoint_join_cut

* use zend_string_release

- fix bug in pinpoint_interceptor_handler_entry

* add try_end

* test pinpoint_join_cut

* test pdo

* ci(versions): test php versions (#63)

- covered all phpt by docker compose
- pdo,redis,curl
- fix CI

* remote php5 support

* support ci for php7

* add php8.0 testCI

* add php8.3

* rename internal API

- pinpoint_start_trace => _pinpoint_start_trace

* update pinpoint_php module version

* add testcase for simplephp

* update docs

* Feat join cut get caller (#64)

* feat(php): pinpoint_get_caller_arg
* pinpoint_get_caller_arg and phpt
* update pinpoint_php module version 0.5.2
* - remove `fastapi==0.109.1` , it crashes CI
* simple_php for curl/redis/memcache/mysql

* test pack_php_module

* test wordpress

* enable logger

* test phpmyadmin

* fix phpmyadmin

* test(php): more popular project (#65)

- php wordpress, phpmyadmin
- java_call_app

* fix simplephp

* for wilderness test

* change package naming

* update docs

* fix extends bug in pinpoint-php

* support flarum

* support cachethq

* update docs

* update CN docs

* PHP DOC KR version update (pinpoint-apm#638)

* update docs

* update script for pinpoint_php (pinpoint-apm#639)

* fix version problem

* fix(php): yii2

- fix pinpoint-apm#640 yii2 plugins

* fix docs

* script for automatically uploading assert

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: mayingping-Bella <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evelyn0927 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

<built-in function add_clue> returned a result with an error set
1 participant
@eeliu