Skip to content

Disable tinyproxy systemd service #623

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.

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

Merged
merged 1 commit into from
Dec 23, 2023
Merged

Disable tinyproxy systemd service #623

merged 1 commit into from
Dec 23, 2023

Conversation

ArneTR
Copy link
Member

@ArneTR ArneTR commented Dec 23, 2023

Tinyproxy was started as a service. We do not want that on our machines.

Copy link

Eco-CI Output:

Label 🖥 avg. CPU utilization [%] 🔋 Total Energy [Joules] 🔌 avg. Power [Watts] Duration [Seconds]
Total Run 10.6724 1668.06 2.55837 659
Measurement #1 10.7338 1668.06 2.55837 654

📈 Energy graph:

 
 7.54 ┤                                     ╭╮
 6.96 ┤                                     ││                 ╭╮
 6.38 ┤        ╭╮                           │╰╮           ╭╮   ││╭╮
 5.81 ┤        ││                           │ │           ││  ╭╯│││
 5.23 ┤       ╭╯╰╮                          │ │         ╭─╯╰─╮│ ╰╯│
 4.65 ┤       │  │    ╭╮                    │ ╰──────╮  │    ││   │            ╭╮                                                                                                                                                                                                                                                                                              ╭╮                                                                                   ╭╮
 4.08 ┤     ╭─╯  ╰────╯╰───────╮╭───────╮  ╭╯        │  │    ╰╯   │        ╭╮╭╮│╰╮                        ╭─────────────╮           ╭╮╭╮         ╭╮╭╮          ╭─╮          ╭─╮          ╭─╮╭╮╭───────╮  ╭─╮ ╭╮╭─╮╭╮          ╭─╮          ╭─╮           ╭╮          ╭─╮         ╭╮╭╮         ╭───╮                      ╭─╮╭╮                      ╭──╮                      ╭╯│                          ╭──╮                        ╭──╮                     ╭╮ ╭╯│╭╮                     ╭╮ ╭╮╭╮          ╭╮         ╭╮╭╮          ╭─╮          ╭╮          ╭╮╭╮         ╭╮              ╭──╮         ╭╮            ╭─╮         ╭╮            ╭────╮                     ╭╮ ╭─
 3.50 ┤    ╭╯                  ╰╯       ╰─╮│         ╰╮ │         │        │╰╯╰╯ ╰╮        ╭──╮        ╭╮ │             ╰─╮         │╰╯│         │╰╯│         ╭╯ │         ╭╯ │         ╭╯ ╰╯╰╯       ╰──╯ ╰─╯╰╯ ╰╯│         ╭╯ ╰╮         │ ╰╮        ╭─╯│         ╭╯ │         │╰╯│         │   │         ╭╮        ╭╮ │ ╰╯│                  ╭╮ ╭╯  │                   ╭╮ │ ╰─╮        ╭─╮         ╭╮ ╭╯  │         ╭╮          ╭╮╭╯  │         ╭╮╭╮        ││ │ ╰╯│        ╭─╮╭╮        ││ │╰╯│         ╭╯│         │╰╯╰╮         │ │         ╭╯│         ╭╯╰╯│        ╭╯╰─╮╭╮          │  │        ╭╯│           ╭╯ │        ╭╯│           ╭╯    ╰╮        ╭─╮         ││ │
 2.92 ┤    │                              ││          │ │         ╰╮       │      │       ╭╯  │        ││ │               │         │  │         │  │         │  │         │  │         │                          ╰╮        │   │         │  │        │  │         │  │         │  │         │   │         ││        ││ │   │         ╭╮       │╰╮│   ╰╮        ╭╮        ││ │   │        │ │         │╰╮│   │        ╭╯│╭╮       ╭╯││   ╰╮        │╰╯│        ││ │   │        │ │││        ││ │  │         │ │         │   │         │ │         │ ╰╮        │   │        │   │││       ╭╮ │  │        │ │           │  │        │ │           │      │        │ │         ││ │
 2.35 ┤    │                              ││          │ │          │      ╭╯      │       │   │        ││ │               │         │  │         │  ╰╮        │  │         │  │         │                           │        │   │         │  │        │  ╰╮        │  │         │  │         │   │         ││       ╭╯│ │   ╰╮        ││       │ ││    │        ││        ││ │   │        │ ╰─╮       │ ││   ╰╮       │ ╰╯│       │ ││    │       ╭╯  │        ││ │   │        │ │││       ╭╯│╭╯  │         │ ╰╮        │   │        ╭╯ │         │  │        │   │        │   │││       ││ │  │        │ ╰╮          │  │        │ ╰╮        ╭╮│      │       ╭╯ ╰╮        ││ │
 1.77 ┼────╯                              ╰╯          ╰─╯          ╰──────╯       ╰───────╯   ╰────────╯╰─╯               ╰─────────╯  ╰─────────╯   ╰────────╯  ╰─────────╯  ╰─────────╯                           ╰────────╯   ╰─────────╯  ╰────────╯   ╰────────╯  ╰─────────╯  ╰─────────╯   ╰─────────╯╰───────╯ ╰─╯    ╰────────╯╰───────╯ ╰╯    ╰────────╯╰────────╯╰─╯   ╰────────╯   ╰───────╯ ╰╯    ╰───────╯   ╰───────╯ ╰╯    ╰───────╯   ╰────────╯╰─╯   ╰────────╯ ╰╯╰───────╯ ╰╯   ╰─────────╯  ╰────────╯   ╰────────╯  ╰─────────╯  ╰────────╯   ╰────────╯   ╰╯╰───────╯╰─╯  ╰────────╯  ╰──────────╯  ╰────────╯  ╰────────╯╰╯      ╰───────╯   ╰────────╯╰─╯
                                                                                                                                                                                                                                                                                                                                     Watts over time

@ArneTR ArneTR merged commit c3f7663 into main Dec 23, 2023
@ArneTR ArneTR deleted the disable-tinyproxy branch December 23, 2023 07:03
ArneTR added a commit that referenced this pull request Dec 23, 2023
* main: (26 commits)
  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)
  Run ID is now accessible even after fail and thus can be sent via ema… (#601)
  Switched from cmd to command (#615)
  Hotfix for check on frequency provider
  Tests run_until must be guard-claused with cleanup routine (#616)
  Fix check if stderr is empty (#613)
  Bump uvicorn[standard] from 0.24.0.post1 to 0.25.0 (#612)
  Fxing the network provider stderror
  Branch and filename are now always not null (#602)
  Adds a more elaborate depends_on test
  Support reading notes from services (#590)
  docker build command in tests now checks reason for docker build failure. If it is a permission issue with the cache, it will continue the rest of the workflow (#576)
  Use depends_on for container startup order (refactored) (#593)
  Bump psycopg[binary] from 3.1.15 to 3.1.16 (#610)
  Added powercap info to hardware_info (#609)
  ...
ArneTR added a commit that referenced this pull request Dec 23, 2023
* main:
  Disable tinyproxy systemd service (#623)
@@ -117,9 +117,13 @@ git submodule update --init
print_message "Installing needed binaries for building ..."
if lsb_release -is | grep -q "Fedora"; then
sudo dnf -y install lm_sensors lm_sensors-devel glib2 glib2-devel tinyproxy lshw
sudo systemctl stop tinyproxy
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ArneTR why do you issue the same command in Fedora and else. Couldn't you only have it once outside of the if?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true, this could be simplified. I changed it on main now.

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)
  ...
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