Signed-off-by: Devansh3712 <[email protected]>
Reviewed-by: Kamoltat <[email protected]>

Signed-off-by: Devansh3712 <[email protected]>
Reviewed-by: Kamoltat <[email protected]>

The original pulpito used routes like /:run/:job_id. While we prefer the
more explicit routes, it'll be helpful if URLs are mostly backwards
compatible.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Med16-11 <[email protected]>

Problem: CI doesn't trigger on pull requests

Solution: rename deploy.yml to ci.yml and trigger CI on every PR.

Also Added a Linter to CI.

Signed-off-by: Kamoltat Sirivadhna <[email protected]>

Instead of using `formatDistanceToNow`, we came up with `timeSince` which basically utilizes Date.now() and used simple arithmetics to compute `started time`
Signed-off-by: Med16-11 <[email protected]>

pages/Job/index.js: fix job started time
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

Implement kill-run button
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

Revert "Implement kill-run button"
Reviewed-by Kamoltat Sirivadhna <[email protected]>
Reviewed-by Devansh Singh <[email protected]>

Signed-off-by: Med16-11 <[email protected]>

pages/Runs/index.tsx: fixed date filter box alignment
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

This required updating several other dependencies, and removing
@mui/styles

Signed-off-by: Zack Cerza <[email protected]>

npx @mui/codemod v5.0.0/jss-to-styled src

Signed-off-by: Zack Cerza <[email protected]>

It's deprecated.

Signed-off-by: Zack Cerza <[email protected]>

npm install --save-dev @typescript-eslint/parser \
  @typescript-eslint/eslint-plugin eslint typescript

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

Fixes errors mentioning "cannot be used as a JSX component."

Signed-off-by: Zack Cerza <[email protected]>

npm install @tanstack/react-query @tanstack/react-query-devtools
npm uninstall react-query

Signed-off-by: Zack Cerza <[email protected]>

Fixes "require is not defined"

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

The auto-reloads aren't great.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

This turned into a pretty major change. The breaking changes in the new
DataGrid API necessitated a change in how we were storing and updating
state. The reducers were dropped in favor of the use-query-params
library. As a bonus, filtering, pagination, and deep links all work
better than they did before.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

GHSA-c2qf-rxjj-qqgw

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

And use useDebounce so that we don't spam paddles

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

Tooling updates
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

Motivation:

We need a way to connect `pulpito-ng`
with `teuthology-api` + all other teuthology services.
Currently, we make use of docker containers to create
a development environment for `teuthology-api` and
other teuthology services.

Solution:

Created a Dockerfile and added a section in the README
file describing the process of connecting `pulpito-ng`
with otheer teuthology services such that we will be able
to containerize `pulpito-ng` and develop it along side
other services.

Signed-off-by: Kamoltat Sirivadhna <[email protected]>

Signed-off-by: Devansh3712 <[email protected]>

Also add a space between links.

Signed-off-by: Zack Cerza <[email protected]>

The "see similar runs" links we provide were worded poorly and spaced
weirdly.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Devansh3712 <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Vallari <[email protected]>

1. redirect to teuthology API's login
2. AppBar uses cookie data to show username

Signed-off-by: Vallari <[email protected]>

While trying to deploy in OpenShift I ran into many issues with Vite.
Some were permission-related, since Vite wants to write files during its
startup process. This should resolve all of those issues.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

This preserves the browser's history stack.
Also, add github icon to login button.

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Extract cookie related code from Login component
to lib/teuthologyAPi.ts

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Other minor additions:
1. In nodes table, add links in 'name' column
2. Add 'hideFooter' prop in DataGrid component

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

Signed-off-by: Vallari <[email protected]>

other minor fixes:
- DataGrid: show boolean true/false symbols in black color
- Drawer: "Nodes" route to /nodes?machine_type=smithi by default
- NodeList: sort by name + thin machine_type col + length desc
- StatsNodesJob: sort by name + display node's shortname
- StatsNodesLock: sort by owner

Signed-off-by: Vallari Agrawal <[email protected]>

- Node Page: use NodeList (instead of DataGrid)
- useNodes(): remove params because machine_type filters should
		be on handled on client side
- useNode(): return Node[] (instead of Node) so NodeList can be used

Signed-off-by: Vallari Agrawal <[email protected]>

Signed-off-by: Vallari Agrawal <[email protected]>

Done by adding 'pagingMode' prop to JobList component
because Run page uses client side paging and Node Page
needs server side paging

Signed-off-by: Vallari Agrawal <[email protected]>

Dockerfile: Run in production mode
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

This truncates the longer strings to 200 chars.

Signed-off-by: Zack Cerza <[email protected]>

JobList: Hover on status col shows failure reason
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

We could add path-browserify as a dependency and have access to Node.js'
path module, but that's 15kb and this is four lines.

Signed-off-by: Zack Cerza <[email protected]>

This is more helpful than linking directly to teuthology.log, which may
not even exist yet.

Signed-off-by: Zack Cerza <[email protected]>

And change the text to "Log Archive"

Signed-off-by: Zack Cerza <[email protected]>

Job{,List}: Link to job archive directories
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

Github Authentication
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

Other minor changes:
1. set retry=1 for teuthologyAPI useSession query
2. disable login button when teuthology-api cannot be pinged

Signed-off-by: Vallari Agrawal <[email protected]>

fix: default url for TEUTHOLOGY_API_SERVER
Reviewed-by: Kamoltat Sirivadhna <[email protected]>

We were rendering a blank page on all routes if the teuthology-api
endpoint wasn't configured.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>

They're not implemented as React hooks, so let's not name them as if
they are.

Signed-off-by: Zack Cerza <[email protected]>

Signed-off-by: Zack Cerza <[email protected]>