Skip to content
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

Adding report url instead of just ID #531

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Adding report url instead of just ID #531

merged 1 commit into from
Nov 9, 2023

Conversation

ArneTR
Copy link
Member

@ArneTR ArneTR commented Nov 9, 2023

Instead of this:

####################################################################################
Please access your with the ID c0a0a5c8-efbf-4b1c-9549-1bbb1d3c260d
####################################################################################

The GMT now reports this:

####################################################################################
Please access your report on the URL http://metrics.green-coding.internal:9142/stats.html?id=c0a0a5c8-efbf-4b1c-9549-1bbb1d3c260d
####################################################################################

This makes it easier for very unexperienced users to understand that a website is attached to the GMT and where it was configured.

Copy link

github-actions bot commented Nov 9, 2023

Eco-CI Output:

Label 🖥 avg. CPU utilization [%] 🔋 Total Energy [Joules] 🔌 avg. Power [Watts] Duration [Seconds]
Total Run 20.8956 2128.46 2.85699 753
Measurement #1 20.9799 2128.46 2.85699 747

📈 Energy graph:

 
 6.55 ┤                                                                       ╭──╮                 ╭╮
 6.06 ┤        ╭─────╮                                                        │  ╰╮      ╭─╮    ╭╮╭╯│     ╭───╮╭─╮             ╭╮
 5.58 ┤      ╭╮│     │   ╭─╮                                        ╭╮       ╭╯   ╰╮ ╭───╯ │    │╰╯ ╰───╮ │   ╰╯ │             ││                                                                                                                                                                                                                                                                                                              ╭╮                            ╭╮            ╭╮                                             ╭╮
 5.10 ┤      │╰╯     │   │ │                  ╭─╮   ╭╮              ││       │     │╭╯     │    │       │ │      │        ╭╮ ╭╮││                            ╭╮                                                                                                                                                                             ╭╮                                                                                                 ││                            ││            ││              ╭╮                             ││                                                                                                                                                                                          ╭╮
 4.62 ┤     ╭╯       │╭──╯ ╰───╮╭───────────╮╭╯ ╰─╮ │╰──────────────╯│╭╮     │     ╰╯      │    │       │ │      │       ╭╯│ │││╰─╮                          ││╭╮           ╭╮            ╭╮          ╭╮╭╮         ╭╮╭╮          ╭╮╭╮           ╭╮            ╭╮           ╭╮          ╭╮╭╮           ╭╮           ╭╮                       ││╭╮          ╭───╮                       ╭─╮╭╮                        ╭╮╭╮                        ││╭╮                          ││╭╮          ││             ╭╯│╭╮                           ││╭╮                         ╭╮╭╮           ╭╮          ╭╮╭╮          ╭╮           ╭╮          ╭╮╭╮                              ╭╮                                                     ╭─╮╭╯│╭╮                         ╭╮
 4.14 ┤     │        ╰╯        ╰╯           ││    ╰╮│                │││     │             │    │       │╭╯      │       │ │╭╯││  │          ╭╮          ╭╮ ╭╯╰╯│          ╭╯│          ╭─╯│          │╰╯│         ││││          ││││           ││          ╭─╯│         ╭─╯│          │╰╯│           ││          ╭╯│          ╭──╮         ││││          │   │                       │ ╰╯╰╮                      ╭╯╰╯│                   ╭╮ ╭─╯╰╯│         ╭╮              ╭╯│││          ││             │ ╰╯╰╮           ╭╮        ╭╮ ╭─╯╰╯│          ╭╮╭╮        ╭╮ │╰╯│          ╭╯│          │╰╯│         ╭╯│          ╭╯│          │╰╯╰╮         ╭─╮╭╮             ╭╮││          ╭╮            ╭──╮                       ╭─╯ ││ ╰╯│          ╭╮             ││
 3.66 ┤     │                               ╰╯     ╰╯                ╰╯│╭╮  ╭╯             │    │       ╰╯       │       │ ╰╯ ╰╯  │          ││╭╮        ││ │   │         ╭╯ │          │  │         ╭╯  │         │╰╯╰╮         │╰╯│          ╭╯╰╮         │  │         │  │          │  │         ╭─╯│          │ ╰╮         │  │         │╰╯╰╮         │   │                    ╭╮ │    │                      │   │                   ││ │    │         │╰╮             │ ╰╯╰╮        ╭╯│          ╭╮ │    │         ╭╮││        ││ │    │          ││││        ││ │  │          │ │         ╭╯  │         │ ╰╮         │ │         ╭╯   │         │ │││╭╮          ╭╯│││          ││            │  │        ╭─╮            │   ╰╯   │          ││             ││
 3.18 ┤    ╭╯                                                          ╰╯│╭─╯              │╭╮ ╭╯                │       │        │        ╭─╯│││        ││ │   │         │  │         ╭╯  │         │   │         │   │         │  ╰╮        ╭╯  │         │  │         │  ╰╮         │  │         │  ╰╮         │  │         │  │         │   │         │   ╰╮                  ╭╯│╭╯    │                  ╭─╮╭╯   ╰╮        ╭╮        ││ │    │         │ │         ╭─╮╭╯    │        │ │╭╮        ││ │    │        ╭╯│││        ││ │    │         ╭╯│││        ││ │  ╰╮         │ │         │   │         │  │         │ │         │    │         │ ╰╯│││          │ ╰╯│         ╭╯│            │  │        │ ╰─╮          │        ╰╮        ╭╯│         ╭─╮╭╯╰─╮       ╭
 2.70 ┤    │                                                             ││                ╰╯│ │                 ╰╮      │        │        │  │││        ││ │   │         │  ╰╮        │   │         │   │         │   │         │   │        │   │         │  │         │   │        ╭╯  ╰╮        │   │        ╭╯  │         │  │         │   │        ╭╯    │        ╭─╮       │ ││     │         ╭╮       │ ││     │        ││        ││ │    │        ╭╯ ╰─╮       │ ││     │       ╭╯ │││        ││ │    │        │ ╰╯│        ││ │    │        ╭╯ │││        ││ │   │        ╭╯ │         │   │         │  │         │ ╰╮        │    │        ╭╯   │││          │   ╰╮       ╭╯ │╭╮         ╭╯  │        │   │          │         │       ╭╯ │╭╮       │ ││   │       │
 2.22 ┤   ╭╯                                                             ╰╯                  │ │                  │      │        │        │  │││       ╭╯│ │   ╰╮        │   │        │   ╰╮        │   │         │   │         │   │        │   │        ╭╯  │         │   │        │    │     ╭╮ │   │        │   │        ╭╯  │         │   │        │     │        │ │       │ ││     │         ││       │ ││     │        ││        ││ │    │        │    │       │ ││     │       │  ╰╯│       ╭╯│╭╯    │        │   │        ││ │    ╰╮       │  │││        ││ │   │        │  ╰╮        │   │         │  │        ╭╯  │        │    │        │    │││        ╭╮│    │     ╭╮│  │││         │   │        │   │        ╭╮│         │       │  │││       │ ││   │       │
 1.73 ┼───╯                                                                                  ╰─╯                  ╰──────╯        ╰────────╯  ╰╯╰───────╯ ╰─╯    ╰────────╯   ╰────────╯    ╰────────╯   ╰─────────╯   ╰─────────╯   ╰────────╯   ╰────────╯   ╰─────────╯   ╰────────╯    ╰─────╯╰─╯   ╰────────╯   ╰────────╯   ╰─────────╯   ╰────────╯     ╰────────╯ ╰───────╯ ╰╯     ╰─────────╯╰───────╯ ╰╯     ╰────────╯╰────────╯╰─╯    ╰────────╯    ╰───────╯ ╰╯     ╰───────╯    ╰───────╯ ╰╯     ╰────────╯   ╰────────╯╰─╯     ╰───────╯  ╰╯╰────────╯╰─╯   ╰────────╯   ╰────────╯   ╰─────────╯  ╰────────╯   ╰────────╯    ╰────────╯    ╰╯╰────────╯╰╯    ╰─────╯╰╯  ╰╯╰─────────╯   ╰────────╯   ╰────────╯╰╯         ╰───────╯  ╰╯╰───────╯ ╰╯   ╰───────╯
                                                                                                                                                                                                                                                                                                                                                                                    Watts over time

@ArneTR
Copy link
Member Author

ArneTR commented Nov 9, 2023

@dan-mm Can you double confirm that you are not using a string matching anywhere on this ID in either tests or in the Github Actions / Gitlab setup?

@ArneTR ArneTR merged commit 520b5a1 into main Nov 9, 2023
4 checks passed
@ArneTR ArneTR deleted the report-url branch November 9, 2023 10:06
@dan-mm
Copy link
Contributor

dan-mm commented Nov 9, 2023

I can confirm - I don't check this particular string in tests

ArneTR added a commit that referenced this pull request Nov 10, 2023
* main: (110 commits)
  Adding report url instead of just ID (#531)
  Makes error optional except if there is data we need (#530)
  Bump uvicorn[standard] from 0.23.2 to 0.24.0.post1 (#526)
  Adding current clocksource in linux according to timing problems outlined here: https://www.brendangregg.com/blog/2021-09-26/the-speed-of-time.html (#524)
  Bump orjson from 3.9.9 to 3.9.10 (#519)
  Bump pandas from 2.1.1 to 2.1.2 (#520)
  Update Dockerfile - Removed playwright version pinning (#523)
  Bump fastapi from 0.104.0 to 0.104.1 (#522)
  Casts everything to bigint in hog api
  Qol Updates (#521)
  build containers workflow: removed conditional from job scope and moved to step
  added options for build containers manual run to specify which container to build
  fix file permissions  for build-containers script
  Bump playwright/python in /docker/auxiliary-containers/gcb_playwright (#513)
  Bump pytest from 7.4.2 to 7.4.3 (#515)
  Added lshw to hardware_info_root (#514)
  Tag dockerhub containers (#518)
  Update XGBoost model
  Update README.md
  Bump pylint from 3.0.1 to 3.0.2 (#512)
  ...
ArneTR added a commit that referenced this pull request Nov 21, 2023
* main:
  New depencies for XGboost added
  add workflow to build codespace container
  Bump scipy from 1.11.3 to 1.11.4 (#548)
  Bump psycopg[binary] from 3.1.12 to 3.1.13 (#549)
  Fomantic update to 2.9.3 (#547)
  Allow digits in env var keys (#541)
  Hardware info root mac (#544)
  Trying if ref works
  Bump pydantic from 2.5.0 to 2.5.1 (#539)
  Adds the citation file (#537)
  HOG returned after mutliple loops
  Info button now works for page 2+ task coalitions
  Bump pydantic from 2.4.2 to 2.5.0 (#538)
  TDP typo
  Bump pandas from 2.1.2 to 2.1.3 (#534)
  Makes the root_hardware_info root writable only
  Hog API Fix for broken payload (#532)
  Adding report url instead of just ID (#531)
  Makes error optional except if there is data we need (#530)
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