Skip to content

First Release #1

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
Merged
merged 29 commits into from
Jul 3, 2017
Merged

First Release #1

merged 29 commits into from
Jul 3, 2017

Conversation

@pglombardo
Copy link
Contributor

@pglombardo pglombardo commented Jun 7, 2017 •
edited
Loading

This PR includes the changes required for the python debut.

  • Improved process detection & announce
  • Improved timer (as daemon) so processes exit in a timely fashion
  • Instana compatible IDs & Headers and conversion between the two
  • Slightly decouple from basictracer so our IDs are used
  • Add tests to validate IDs, Headers & conversion
  • Add tests to validate all opentracing interfaces
  • Instrument Django & add tests
    • Django middleware
  • WSGI Middleware
  • Fix snapshot reporting; delay send until process recognized
  • Fix delta reporting; report metric changes only
  • Write README
  • Add span buffering
  • Add registered span types; otherwise default to SDK span type reporting

pglombardo added 29 commits May 31, 2017 15:08
@pglombardo
Switch to psutil and update announce
b28b801
@pglombardo
Add travis test suite setup.
9047f7a
@pglombardo
Proper ID generation and conversion
9ed7b34
@pglombardo
Add tests to validate ID management
758f287
@pglombardo
Store timer + set daemon flag so procs exit timely
842cad3
@pglombardo
Basic tracer & span tests
7616d3b
@pglombardo
Fix hang, simplify finite state machine events
32c3ec8
@pglombardo
Linter fixes; Update fsm state check
bfa8827
@pglombardo
Include pid in log messages
1530b1c
@pglombardo
Set timer name/daemon for debug; linter improvements
b19e6d5
@pglombardo
Override start_span so Instana IDs are used via generate_id
d8595b1
@pglombardo
Set test env var
479de65
@pglombardo
Quick fix: delay sending snapshot
937e02f 
This allows process recognition to happen
before missing the snapshot were originally
sending on first metrics post (and then
again 5 minutes later).
@pglombardo
Proper delta reporting of metrics
2a8d0cf
@pglombardo
Unite all Data objects under a single namespace
c1fc36f
@pglombardo
More tests; use a single tracer
fb45cb5
@pglombardo
Remove unused imports.
1d36e66
@pglombardo
If no incoming context, use same id for s & t
a61125e
@pglombardo
Span buffering; SDK Span type support:
261410d 
* SDK span type definition and management
* Use a Queue to periodically batch send spans
* Add support for 'span.kind' tag and translation
* Add tests to validate SDK span types
* Add tests to validate proper span buffering
* Add functions to manage span queue
* Add Setup/Teardown functions for tests
@pglombardo
Better name for unified InstanaSpan module
936a600
@pglombardo
Pylint improvements
14f516f
@pglombardo
Migrate to class style test structure
1fa6863
@pglombardo
First draft for a repo README
1986bd1
@pglombardo
Rename package to stan; update README
5a6e165
@pglombardo
Expanded span.kind support & tests.
d5bc33f
@pglombardo
Allow only one sensor per process
15a5f24 
- Avoids duplicated, triplicated metric reporting
@pglombardo
Use a dedicated metric collection thread.
440ec9e
@pglombardo
Lowercase what fields we can for now.
396a279
@pglombardo
Rename package to instana
d265919
@pglombardo
Copy link
Contributor Author

pglombardo commented Jul 3, 2017

Going to put Django instrumentation into a separate PR so we can get started with running tests on Travis.

@pglombardo pglombardo merged commit 99cea9d into master Jul 3, 2017
@pglombardo pglombardo deleted the final_leg branch July 3, 2017 12:22
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.

2 participants

@pglombardo