Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

ribalba
Copy link
Member

@ribalba ribalba commented Sep 2, 2025

Something that annoyed me all the time is that when I do install_linux.sh the install script just overwrites my config files. Especially if I have set up something not trivial and then I switch branches and install to get the right libs everything is gone. Now I just save the config files to a .backup so I can copy paste the things I need.

@ribalba ribalba requested a review from ArneTR September 2, 2025 13:26
Copy link

github-actions bot commented Sep 2, 2025

Eco CI Output - Old Energy Estimation

Eco CI Output [RUN-ID: 17405005416]:

🌳 CO2 Data:
City: , Lat: , Lon:
IP:
CO₂ from energy is: 1.540902720 g
CO₂ from manufacturing (embodied carbon) is: 0.388169436 g
Carbon Intensity for this location: 321 gCO₂eq/kWh
SCI: 1.929072 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #118.36874800.323.531360.50
Total Run18.374800.323.531360.50
Additional overhead from Eco CIN/A16.304.543.59

Copy link

github-actions bot commented Sep 2, 2025

Eco CI Output - Old Energy Estimation

Eco CI Output [RUN-ID: 17405053110]:

🌳 CO2 Data:
City: Washington, Lat: 38.7095, Lon: -78.1539
IP: 20.42.8.150
CO₂ from energy is: 1.816907950 g
CO₂ from manufacturing (embodied carbon) is: 0.416592397 g
Carbon Intensity for this location: 365 gCO₂eq/kWh
SCI: 2.233500 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #117.93834977.833.411460.12
Total Run17.944977.833.411460.12
Additional overhead from Eco CIN/A15.344.123.72

Copy link
Member

@ArneTR ArneTR left a comment

Choose a reason for hiding this comment

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

I do that always manually, but do agree that this is insider knowledge and nice to have as QoL.

To my understandig these files also need to be in .gitignore:

  • docker/nginx/block.conf
    
  • docker/nginx/frontend.conf

Can you confirm and if so add. Then please merge from your side

@ribalba
Copy link
Member Author

ribalba commented Sep 3, 2025

After thinking about this I decided to add *.backup as a general gitignore. This could be debated. I think this is cleaner and quite clear. But could also go for the option of naming each file.

@ArneTR
Copy link
Member

ArneTR commented Sep 3, 2025

Please explicitely name the files. Makes it also clearer where they are created.

Catch-Alls should at least be directory restricted IMHO unless they are a standard (like .DS_Store etc.)

Copy link

github-actions bot commented Sep 3, 2025

Eco CI Output - Old Energy Estimation

Eco CI Output [RUN-ID: 17425884754]:

🌳 CO2 Data:
City: Des Moines, Lat: 41.6015, Lon: -93.6127
IP: 135.119.239.34
CO₂ from energy is: 2.108809080 g
CO₂ from manufacturing (embodied carbon) is: 0.288691924 g
Carbon Intensity for this location: 518 gCO₂eq/kWh
SCI: 2.397501 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #125.32274071.064.021011.84
Total Run25.324071.064.021011.84
Additional overhead from Eco CIN/A11.374.122.76

@ArneTR
Copy link
Member

ArneTR commented Sep 6, 2025

@ribalba ping

@ribalba
Copy link
Member Author

ribalba commented Sep 16, 2025

done

Copy link

Eco CI Output [RUN-ID: 17763595356]:

🌳 CO2 Data:
City: Phoenix, Lat: 33.4532, Lon: -112.0748
IP: 172.182.225.134
CO₂ from energy is: 1.601524450 g
CO₂ from manufacturing (embodied carbon) is: 0.344901597 g
Carbon Intensity for this location: 335 gCO₂eq/kWh
SCI: 1.946426 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #123.78494780.673.951208.85
Total Run23.784780.673.951208.85
Additional overhead from Eco CIN/A13.414.163.22

@ArneTR ArneTR merged commit b170583 into main Sep 16, 2025
1 check passed
@ArneTR ArneTR deleted the config-backup branch September 16, 2025 16:52
ArneTR added a commit that referenced this pull request Sep 19, 2025
* main: (66 commits)
  Remove network from sci (#1352)
  Updated Cloud Energy
  Relaxed test due to noise of host OS network management. See #1322 for details
  tcpdump will report errors for now instead of except
  Remaned wait_time to proper name idle_time
  Bump psutil from 7.0.0 to 7.1.0 (#1349)
  Refactor logs structure to json (#1341)
  Removed proxy passing into docker gunicorn container
  Changed jobs endpoint to deliver http 202
  (fix): After moving to /lib folder the __main__ check was not relevant anymore. Checking for tty now which should not conflict when running in cluster mode
  fixes the problem with updates not being installed (#1346)
  Bump fastapi[standard] from 0.116.1 to 0.116.2 (#1342)
  Saves the existing config files on install (#1311)
  Fixes sudo bug when creating test env (#1310)
  Bump pydantic from 2.11.7 to 2.11.9 (#1340)
  Bump pytest-randomly from 4.0.0 to 4.0.1 (#1339)
  Bump actions/github-script from 7.0.1 to 8.0.0 in /.github/workflows (#1338)
  Bump actions/create-github-app-token in /.github/workflows (#1337)
  Added powermetrics carbon entries for config.js
  Added more key metrics to frontend display tests
  ...
ArneTR added a commit that referenced this pull request Sep 19, 2025
* main: (84 commits)
  Allow to stream stdout connected output (#1353)
  Catching exception in tcpdump
  Showing test duration
  Remove network from sci (#1352)
  Updated Cloud Energy
  Relaxed test due to noise of host OS network management. See #1322 for details
  tcpdump will report errors for now instead of except
  Remaned wait_time to proper name idle_time
  Bump psutil from 7.0.0 to 7.1.0 (#1349)
  Refactor logs structure to json (#1341)
  Removed proxy passing into docker gunicorn container
  Changed jobs endpoint to deliver http 202
  (fix): After moving to /lib folder the __main__ check was not relevant anymore. Checking for tty now which should not conflict when running in cluster mode
  fixes the problem with updates not being installed (#1346)
  Bump fastapi[standard] from 0.116.1 to 0.116.2 (#1342)
  Saves the existing config files on install (#1311)
  Fixes sudo bug when creating test env (#1310)
  Bump pydantic from 2.11.7 to 2.11.9 (#1340)
  Bump pytest-randomly from 4.0.0 to 4.0.1 (#1339)
  Bump actions/github-script from 7.0.1 to 8.0.0 in /.github/workflows (#1338)
  ...
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.