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

misc: updates and enhancments to host-side management tools #1027

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

efahl
Copy link
Contributor

@efahl efahl commented Oct 8, 2024

cleaner

  • add a tool to wipe out unused public/store/ entries

update_all_targets

  • extract configuration data from settings
  • add a mechanism to optionally flush the redis database
  • add a filtering mechanism so users can reduce footprint and bandwidth use on their local servers

More bits produced by my local server journey...

Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.55%. Comparing base (5e65dec) to head (d3fc006).
Report is 122 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1027      +/-   ##
==========================================
+ Coverage   80.75%   90.55%   +9.79%     
==========================================
  Files          15       14       -1     
  Lines         977     1080     +103     
==========================================
+ Hits          789      978     +189     
+ Misses        188      102      -86     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

cleaner
  - add a tool to wipe out unused public/store/ entries

update_all_targets
  - extract configuration data from settings
  - add a mechanism to optionally flush the redis database
  - add a filtering mechanism so users can reduce footprint
    and bandwidth use on their local servers

Signed-off-by: Eric Fahlgren <[email protected]>
@aparcar
Copy link
Member

aparcar commented Oct 9, 2024

Hi this is nice but I think a bit over engineered.

Regarding removal I use a simple find with -delete command which cleans up builds older than a week. It does the job reliably and I don't think it needs anything more advanced.

For reloading, this only happens once and is then triggered via web hooks. I think a solution for smaller installations would be to add a reload job and respond 202 while the information is still missing. Once available it either continues with the building or returns an error. The build status while 202 could be something like "updating_metadata"

@efahl
Copy link
Contributor Author

efahl commented Oct 9, 2024

Hi this is nice but I think a bit over engineered.

Of course! 😀 I should have labelled this RFC, we can kill it.

Most of this was really just me learning about how everything fit together and what happens when you change things.

The update script in that form is very useful to me (probably only me) because of the way I'm testing things. I use owut quite extensively to test changes to ASU and vice versa, I use the local server to test changes in owut. So, that means I restart the server a lot, I'm guessing I did over 50 cold restarts of my local server today alone (podman stop, wipe public/*, podman build and restart, run update script...).

@efahl
Copy link
Contributor Author

efahl commented Oct 9, 2024

triggered via web hooks

Can you point me to where the webhooks are fired off, from the buildbots or somewhere? I've dug through https://github.com/openwrt/openwrt/tree/main/.github/workflows but can't find anything that would fire off the update request...

@Neustradamus
Copy link

@efahl: I think that you can add 24.10 part?

@efahl
Copy link
Contributor Author

efahl commented Nov 8, 2024

add 24.10

That's in #1066

@Neustradamus
Copy link

@efahl: Oh yes, good!

This part must be reviewed/merged before ^^

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.

3 participants