Commit Graph

4 Commits

Author SHA1 Message Date
Casper Nielsen e27f5befb0
Fix/81 dapr http endpoint (#107)
* Fix: #81 by implementing custom tool for wrapping HTTP calling

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Ensure _base_url is passed if FQDN

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Explicit exclude the other tools from type checking so we can check http tool

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct arg-type for url

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ruff formatting

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Update deps

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Feat: First draft impl. of OTel

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: import

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct import and pass var

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Instantiation instead of model

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Circular ref

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Remove specific tracer as lib should pick it up & add logger

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Attempt to aquire tracer and use logger

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Set the logger

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Allow passing custom endpoint per provider

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Set tmp correct logger port

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct sending json encoded

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Ensure requests client always run http

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct tmp port for logger

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Feat: Ensure /v1/[traces|metrics|logs] always in otlp_endpoint

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Don't capitalize

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Ensure we check for v1 and set if not

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ensure http always

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Transition otel to tools.utils

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct import

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Feat: Clean init of provider with validator func

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: conform to new validator

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Remove not needed endpoint pass to logger client

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F821

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Remove redundant check

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Improve validator func for less code

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Remove unused import

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Feat: Refine checks and ensure resiliency on url creation

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add parsing of DAPR_AGENTS_OTEL_ENABLED to disable OTel from client

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Convert to reusable function passing the http verb

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct string to upper rather than capitalize

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Bump python version on build agent as 3.9 don't contain switch statement

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Pass version as str

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Move otel into dapr_agents.agent.telemetry

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Include errors from dapr_agents.agent.telemetry

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct import of DaprAgentsOTel

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Include type-check on http

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Proper name of DaprAgentsOTel

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct imports

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Raising ToolError results in workflow breaking rather than reiterating the tool request

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: .json() not guaranteed to hold a value

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Remove unused import

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Decorate the tool

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Feat: Better naming convention for docstring

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Feat: Export tool decoration for consumption

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Revert

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Import & formatting

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

---------

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>
Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
2025-05-02 09:04:29 -07:00
Casper Nielsen f129754486
Fix/30 add linter action (#95)
* Fix: Fix Setup lint GitHub action #30

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Remove branch filter on PR and remove on push

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Remove on mergequeue

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add tox.ini file

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Return on push

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: tox -e ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore .ruff_cache

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Update tox file

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Add mypy.ini

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore if line is too long

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Set the ignore in command instead

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W503

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: 541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W503

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ignore F401, unused imports as __init__ files has them

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Return linebreak as tox -e ruff yields that

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore W503 as ruff introduces it

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E203

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W293

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E203

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E203

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ruff want's the space before :

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore space before :

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add dev-requirements.txt

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct python version

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Ref dev-requirements.txt

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add mypy cache dir

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Update mypy version

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Exclude cookbook and quicstarts

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Remove unused import

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Add specific sub module ignore on error for future smaller fixing

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Reintroduce branches filter on push and pull_request

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: ruff formatting

* Chore: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: E401

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: E711

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

---------

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>
2025-04-23 22:58:48 -07:00
Yaron Schneider f5dc9372e7
Revert "Fix: Setup lint GitHub action #30 (#85)" (#88)
This reverts commit 199fcf9d02.
2025-04-18 15:39:26 -07:00
Casper Nielsen 199fcf9d02
Fix: Setup lint GitHub action #30 (#85)
* Fix: Fix Setup lint GitHub action #30

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Remove branch filter on PR and remove on push

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Remove on mergequeue

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add tox.ini file

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Return on push

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: tox -e ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore .ruff_cache

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Update tox file

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Add mypy.ini

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore if line is too long

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Set the ignore in command instead

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W503

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: 541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W503

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Ignore F401, unused imports as __init__ files has them

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Return linebreak as tox -e ruff yields that

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore W503 as ruff introduces it

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E203

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W293

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E203

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E203

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F841

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: W291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F811

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: F541

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ruff want's the space before :

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Ignore space before :

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: E291

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add dev-requirements.txt

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Correct python version

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Ref dev-requirements.txt

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Add mypy cache dir

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Update mypy version

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Fix: Exclude cookbook and quicstarts

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Remove unused import

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Chore: Add specific sub module ignore on error for future smaller fixing

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* Reintroduce branches filter on push and pull_request

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

* chore: Ruff

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>

---------

Signed-off-by: Casper Guldbech Nielsen <scni@novonordisk.com>
2025-04-15 18:20:34 -07:00