Skip to content

Conversation

@ArneTR
Copy link
Member

@ArneTR ArneTR commented Dec 27, 2023

Since PR #632 was merged we now can add the reading of the body in the GMT on error case in the API.

This was not possible before as the stream was consumed and not available anymore after initial parsing of the middleware

@github-actions
Copy link

Eco-CI Output:

Label 🖥 avg. CPU utilization [%] 🔋 Total Energy [Joules] 🔌 avg. Power [Watts] Duration [Seconds]
Total Run 10.4156 1916.02 2.56495 754
Measurement #1 10.4692 1916.02 2.56495 749

📈 Energy graph:

 
 7.86 ┤                                                                                   ╭╮
 7.25 ┤                                                                                   ││                             ╭╮
 6.64 ┤                                                                                   ││                             ││
 6.03 ┤                                                                     ╭╮            ││                        ╭─╮  │╰╮
 5.42 ┤                                                                     ││            │╰╮                       │ ╰╮ │ ╰─╮
 4.81 ┤                                                                     ││           ╭╯ │  ╭╮ ╭╮               ╭╯  │ │   │
 4.20 ┤        ╭╮    ╭╮ ╭╮     ╭─╮         ╭╮╭─╮    ╭─╮╭╮                   ││           │  │╭─╯╰─╯│               │   ╰╮│   │        ╭╮    ╭─╮                                 ╭╮                                                                                   ╭────╮                                                                                               ╭╮                          ╭╮                         ╭╮                         ╭╮                            ╭╮                            ╭╮                            ╭╮                                                                                                                                                                                          ╭╮
 3.60 ┤      ╭─╯╰────╯╰─╯╰───╮╭╯ ╰─────────╯││ │    │ ╰╯╰───────────────────╯╰───────────╯  ╰╯     ╰╮              │    ╰╯   │        ││╭╮╭╮│ ╰╮          ╭╮╭╮        ╭╮╭─╮╭──╮╭╯│╭──╮╭──╮╭╮         ╭╮╭─╮         ╭╮╭─╮         ╭──╮         ╭╮╭╮         ╭╮╭─╮╭─╮╭─╯    ╰───╮╭╮ ╭╮╭─╮╭─╮╭─╮         ╭╮╭╮         ╭╮╭─╮         ╭──╮         ╭╮╭╮         ╭╮╭─╮         ╭╯│╭╮         ╭╮        ╭╮ ╭─╯╰─╮                  ╭╮ ╭─╯│╭╮                   ╭╮ ╭╯│╭─╮         ╭─╮         ╭╮ ╭╯│╭─╮         ╭─╮         ╭╮ ╭╯│╭╮          ╭╮╭╮        ╭╮ ╭╯│╭─╮         ╭─╮         ╭╮ ╭╮╭─╮          ╭─╮         ╭╮╭╮           ╭╮           ╭╮         ╭─╮╭─╮         ╭─╮╭╮╭╮          ╭───╮          ╭╮           ╭╮╭─╮          ╭╮           ╭────╯│╭╮          ╭╮          ╭╮ ╭─
 2.99 ┤      │               ││             ││ │╭╮  │                                               │              │         │       ╭╯││╰╯││  │          ││││       ╭╯││ ╰╯  ││ ││  ││  │││         │╰╯ │         │││ │         │  │         │││╰╮        │││ ││ ╰╯          ││╰╮│││ ││ ││ │        ╭╯│││         │││ │         │  │         │╰╯│         │││ │         │ ││╰╮        ││        ││ │    │         ╭╮       │╰╮│  ╰╯│         ╭╮        ││╭╯ ││ │         │ ╰╮        ││ │ ╰╯ │         │ ╰╮        ││ │ ││╰╮         │╰╯│        ││ │ ││ │         │ ╰╮        ││ │││ │          │ │         │││╰╮          │╰╮        ╭╮│╰╮        │ ││ │         │ ││╰╯│          │   │          ││╭╮         │╰╯ │          ││╭╮         │     ╰╯│        ╭╮│╰─╮        ││ │
 2.38 ┤      │               ╰╯             ╰╯ ╰╯│  │                                               │             ╭╯         │       │ ││  ││  │        ╭╮││││       │ ││     ╰╯ ╰╯  ╰╯  ╰╯╰╮        │   │         │╰╯ │         │  │         │╰╯ │        │╰╯ ││             ╰╯ ││││ ╰╯ ││ │        │ ╰╯╰╮        │││ │        ╭╯  │         │  │         │╰╯ │         │ ╰╯ │        ││        ││ │    │         ││       │ ││    ╰╮        ││       ╭╯││  ╰╯ │       ╭╮│  │        ││ │    │       ╭╮│  │        ││ │ ╰╯ │       ╭╮│  │        ││ │ ╰╯ │       ╭╮│  │        ││ │╰╯ │         ╭╯ │        ╭╯╰╯ │        ╭─╯ │        │╰╯ │        │ ╰╯ │       ╭╮│ ││  │        ╭╮│   │        ╭╮│╰╯│         │   │        ╭─╯│││         │       ╰╮       │││  │        ││ │
 1.77 ┼──────╯                                   ╰──╯                                               ╰─────────────╯          ╰───────╯ ╰╯  ╰╯  ╰────────╯╰╯╰╯╰───────╯ ╰╯                   ╰────────╯   ╰─────────╯   ╰─────────╯  ╰─────────╯   ╰────────╯   ╰╯                ╰╯╰╯    ╰╯ ╰────────╯    ╰────────╯╰╯ ╰────────╯   ╰─────────╯  ╰─────────╯   ╰─────────╯    ╰────────╯╰────────╯╰─╯    ╰─────────╯╰───────╯ ╰╯     ╰────────╯╰───────╯ ╰╯     ╰───────╯╰╯  ╰────────╯╰─╯    ╰───────╯╰╯  ╰────────╯╰─╯    ╰───────╯╰╯  ╰────────╯╰─╯    ╰───────╯╰╯  ╰────────╯╰─╯   ╰─────────╯  ╰────────╯    ╰────────╯   ╰────────╯   ╰────────╯    ╰───────╯╰╯ ╰╯  ╰────────╯╰╯   ╰────────╯╰╯  ╰─────────╯   ╰────────╯  ╰╯╰─────────╯        ╰───────╯╰╯  ╰────────╯╰─╯
                                                                                                                                                                                                                                                                                                                                                                                     Watts over time

@ArneTR ArneTR merged commit d8de41c into main Dec 27, 2023
@ArneTR ArneTR deleted the starlette-non-blocking branch December 27, 2023 09:13
ArneTR added a commit that referenced this pull request Jan 1, 2024
* main:
  Command fix. Must be list append
  Refactorings
  Moved tinyproxy out of if clause
  Refactoring for error messages and security fix for path echoing (#636)
  GMT color via own commit hash (#634)
  Hotfix for branch not main
  Non-Blocking starlette body read (#633)
  Bump fastapi from 0.105.0 to 0.108.0 (#632)
  Updated XGBoost submodule
  Bump pydantic from 2.5.2 to 2.5.3 (#628)
  Added stddev to timeline (#627)
ArneTR added a commit that referenced this pull request Jan 1, 2024
* main:
  Stderr is now by default UTF-8 (#624)
  Refactored kill/killpg mechanism to be unified and actually fail on n… (#625)
  Command fix. Must be list append
  Refactorings
  Moved tinyproxy out of if clause
  Refactoring for error messages and security fix for path echoing (#636)
  GMT color via own commit hash (#634)
  Hotfix for branch not main
  Non-Blocking starlette body read (#633)
  Bump fastapi from 0.105.0 to 0.108.0 (#632)
  Updated XGBoost submodule
  Bump pydantic from 2.5.2 to 2.5.3 (#628)
  Added stddev to timeline (#627)
  Disable tinyproxy systemd service (#623)
ArneTR added a commit that referenced this pull request Jan 1, 2024
* main: (23 commits)
  System check providers running (#619)
  Stderr is now by default UTF-8 (#624)
  Refactored kill/killpg mechanism to be unified and actually fail on n… (#625)
  Command fix. Must be list append
  Refactorings
  Moved tinyproxy out of if clause
  Refactoring for error messages and security fix for path echoing (#636)
  GMT color via own commit hash (#634)
  Hotfix for branch not main
  Non-Blocking starlette body read (#633)
  Bump fastapi from 0.105.0 to 0.108.0 (#632)
  Updated XGBoost submodule
  Bump pydantic from 2.5.2 to 2.5.3 (#628)
  Added stddev to timeline (#627)
  Disable tinyproxy systemd service (#623)
  Text change
  Value formatting on status page
  Normalized URL for machines endpoint
  Less confusing error messages
  Status has now a waiting time (#599)
  ...
ArneTR added a commit that referenced this pull request Jan 2, 2024
* main:
  Dev mode (#637)
  Hotfix: resolutions -> resolution
  Adds skip_checks to providers (#566)
  Bump pytest from 7.4.3 to 7.4.4 (#639)
  Disabled second PSU provider for VM tests
  System check providers running (#619)
  Stderr is now by default UTF-8 (#624)
  Refactored kill/killpg mechanism to be unified and actually fail on n… (#625)
  Command fix. Must be list append
  Refactorings
  Moved tinyproxy out of if clause
  Refactoring for error messages and security fix for path echoing (#636)
  GMT color via own commit hash (#634)
  Hotfix for branch not main
  Non-Blocking starlette body read (#633)
  Bump fastapi from 0.105.0 to 0.108.0 (#632)
  Updated XGBoost submodule
  Bump pydantic from 2.5.2 to 2.5.3 (#628)
  Added stddev to timeline (#627)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants