Skip to content

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.

@github-actions
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)
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

3 participants