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

syz-manager: add common HTML header to every page #5495

Merged
merged 7 commits into from
Nov 8, 2024

Conversation

dvyukov
Copy link
Collaborator

@dvyukov dvyukov commented Nov 8, 2024

  • pkg/manager: add common page header type
  • pkg/manager: add common HTML page header
  • syz-manager: add pause function
  • pkg/manager: wrap stats page with the common header

Add common page header to every page.
This is similar to how we do in the dashboard app,
and it will allow to have a common HTML page header.
@dvyukov dvyukov changed the title dvyukov manager iface syz-manager: add common HTML header to every page Nov 8, 2024
pkg/manager/http.go Outdated Show resolved Hide resolved
pkg/manager/http.go Outdated Show resolved Hide resolved
pkg/manager/http.go Outdated Show resolved Hide resolved
pkg/manager/http.go Outdated Show resolved Hide resolved
pkg/html/pages/stats.go Outdated Show resolved Hide resolved
Add common HTML page header to every page.
The header contains common tabs, and expert mode switch.
The pause function is useful to make manager stop consuming CPU
for some time, e.g. when it runs on a shared machine,
and a user wants to use CPU for something else.
Previously one would need to kill the manager process and
restart later to achieve this, but the restart is costly,
and aborts all bug reproductions.
Add common manager HTML header to the stats page as well.
tarasmadan
tarasmadan previously approved these changes Nov 8, 2024
Copy link
Collaborator

@tarasmadan tarasmadan left a comment

Choose a reason for hiding this comment

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

Thanks!

@dvyukov
Copy link
Collaborator Author

dvyukov commented Nov 8, 2024

Need restamp after linter fix.

@dvyukov dvyukov added this pull request to the merge queue Nov 8, 2024
Merged via the queue into google:master with commit fd2284f Nov 8, 2024
17 checks passed
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