Project

General

Profile

Actions

Support #22285

closed

Update architecture page

Added by Peter Amstutz 16 days ago. Updated 16 days ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Documentation
Due date:
Story points:
-
Release:
Release relationship:
Auto

Subtasks 1 (0 open1 closed)

Task #22287: Review 22285-architecture-pageResolvedTom Clegg11/05/2024Actions
Actions #1

Updated by Peter Amstutz 16 days ago

  • Status changed from New to In Progress
Actions #3

Updated by Peter Amstutz 16 days ago

  • Release set to 70
Actions #4

Updated by Tom Clegg 16 days ago

Big improvements, thanks.

Only nitpicks, not blockers:
  • It would be nice if all the arrowheads were the same size (ditto text in boxes)
  • I think "TCP connections" would be better than "outgoing connections"
  • arv-ws should be arvados-ws (or "websocket server"?) -- arv-ws is our Python websocket client
Actions #5

Updated by Tom Clegg 16 days ago

"init" description could be: create an initial configuration file for a new cluster. (it's not really specific to "boot")

A couple of instances of "This program does X" could be abbreviated to "do X" to match the rest.

Actions #6

Updated by Brett Smith 16 days ago

Tom Clegg wrote in #note-4:

  • I think "TCP connections" would be better than "outgoing connections"

IMO "TCP connections" is overly detailed and technical compared to the rest of the document. What if we start supporting HTTP/3 in the future? At some level an admin might need to know and care about that of course, but I don't think the architecture diagram needs to be the place we explain it.

Actions #7

Updated by Peter Amstutz 16 days ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by Peter Amstutz 16 days ago

I agree with Brett, saying "TCP Connections" is overly specific.

Fixed the rest of the notes from Tom and merged to main.

Actions

Also available in: Atom PDF