forked from openSUSE/open-build-service
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Buildinfo UI - initial design and first iteration #9
Closed
ncounter
wants to merge
17
commits into
hennevogel:feature/buildinfo-ui
from
ncounter:ncounter-buildinfo-ui
Closed
Buildinfo UI - initial design and first iteration #9
ncounter
wants to merge
17
commits into
hennevogel:feature/buildinfo-ui
from
ncounter:ncounter-buildinfo-ui
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
houndci-bot
reviewed
Jun 13, 2022
houndci-bot
reviewed
Jun 13, 2022
d09b083
to
f62bbb7
Compare
houndci-bot
reviewed
Jun 13, 2022
f62bbb7
to
90be4af
Compare
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
Repository owner
deleted a comment from
houndci-bot
Jun 13, 2022
90be4af
to
68fc617
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
openSUSE#11694
Initial rendering of all the
build-info
available at_buildinfo?debug=1
route.Everything is computed and rendered after parsing the raw data served at that route.
This PR presents the data sliced by the following criteria:
The data are normalized and manipulated from the
buildinfo.js
(for the sake of this exercise), as they are already available in the content of the HTML. If this initial iteration will have a follow-up or a spin-off, we should first start thinking about providing data in a different shape on the backend side, and evaluating about serving the data maybe async or even in a lazy-lading behavior for the different pieces.Short-term: the dependency relationship table could optionally be represented as a tree starting from the direct dependency triggered by the
spec
file, and indenting by levels of requirements the dependencies. A robust decoding of the names is required for that, as of now the debug build info comes in the shape of:At the moment the reverse logic is computed and in the page we have the data in the shape of:
Here
(direct):
stands for it is a direct requirement triggered from thespec
file,perl:
instead means an intermediate dependency is occurring:perl
-->libc.so6()(64bit)
-->glibc
, something that could be represented as a tree:Long-term: due to the fact dependencies cannot be purely represented as a tree (even if it could already be a valuable representation) but it is more like a graph, an additional d3.js implementation could be a very nice to have, something like https://www.jasondavies.com/d3-dependencies/
Below some screenshot:
Summary
![build-info-1](https://user-images.githubusercontent.com/7080830/173353963-7c1478ab-d54b-4c03-afb4-2f43f805af03.png)
Expanded project sources
![build-info-2](https://user-images.githubusercontent.com/7080830/173353971-de14bc44-7ee8-458e-890d-f73ad8889b11.png)
Dependency relationship
![build-info-3](https://user-images.githubusercontent.com/7080830/173353973-b86c7978-8038-4e76-8c57-db32d013a65f.png)
Accessible raw data
![build-info-4](https://user-images.githubusercontent.com/7080830/173353978-f46b757f-50f7-4c81-bf34-6e7232aa7abb.png)