Stephen Smith [Fri, 21 Jun 2024 18:18:30 +0000 (14:18 -0400)]
21925: Don't pass full resource to progress bar useEffect to stop request spam
* Changes progress bar fetch to accept UUID
* Split polling flag into 2 for process and project
* Fetch method now only returns polling flag for projects
* The previous changes for project support switched to updating isRunning
in the fetch action where it would use the store for processes and progress
counts for projects
* This resulted in polling not starting if a draft process is already open
when started since it would only update if already polling
* This new approach uses the store (including WS updates) for processes and
polling-based updates for projects
* Splitting these up also allows eliminating an extra refresh when project
polling stops while keeping a final refresh when websocket updates a process
to stopped
* Added comments and better type guards
Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>
The immediate bug this is fixing is: on Red Hat, if you installed the
package before a web server, it would not install /etc/arvados (because
it didn't know who should own it), but then tell you to edit
/etc/arvados/config.yml.
This commit improves that reporting, so it gives you a more specific
reason that configuration stopped, and next steps for the administrator
to take. Admittedly, I did quite a bit of work to get there, but here we
are.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Brett Smith [Tue, 18 Jun 2024 11:19:15 +0000 (07:19 -0400)]
21906: Improve WWW_OWNER handling on Red Hat systems
On Red Hat systems, if you install arvados-api-server without a web
server already installed, no user account exists that we can use to set
ownership for configuration files and writable directories. In this
case, explicitly set WWW_OWNER="" and use that to gate filesystem
setup.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Zoë Ma [Mon, 17 Jun 2024 03:30:04 +0000 (11:30 +0800)]
In arv-copy command, fix name error in "except" clause
In arv_copy.py, when getting or putting (as part of "arv-copy"
operation), the exception-handling clause had undefined names in
themselves. This would not only throw a hard error at the user but also
prevent subsequent cleanup of the queues.
Fixed by binding the variable name to the exception.
Arvados-DCO-1.1-Signed-off-by: Zoë Ma <zoe.ma@curii.com>
Lucas Di Pentima [Thu, 13 Jun 2024 19:21:21 +0000 (16:21 -0300)]
21832: Allows to customize the initial RDS storage allocation.
Defaults to 20% of the default max storage allocation.
Also, don't ignore changes in password, as they don't require an RDS recreation
like username does.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>
Lucas Di Pentima [Tue, 11 Jun 2024 19:44:18 +0000 (16:44 -0300)]
21832: Fixes variable & output parameters naming.
The user created on RDS is not the service admin user (we don't have access
to it, actually), so we'll just use its credentials for setting up Arvados.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>
Lucas Di Pentima [Fri, 7 Jun 2024 19:15:17 +0000 (16:15 -0300)]
21832: Adds 'use_rds' TF var to create additional VPC resources.
Even though an RDS instance supposedly can be set up as single-AZ,
it requires to be related to a DB Subnet Group resource, which in turn
requires at least 2 subnets from different AZs.
So, when enabling 'use_rds' this code creates a new subnet on a different
AZ from the one that private_subnet is in.
It also supports specifying a custom additional subnet id.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>
Peter Amstutz [Wed, 12 Jun 2024 20:42:10 +0000 (16:42 -0400)]
20943: Ensure that #destroy deletes outbound permission links
Inbound links are destroyed by ActiveRecord's has_many
relationship (e.g. destroying the record will also destroy link
records with that "head_uuid"). However, that isn't the case for
outbound links (tail_uuid). Make sure they get cleaned up.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Brett Smith [Tue, 11 Jun 2024 21:09:32 +0000 (17:09 -0400)]
21572: Write CLI setup instructions with complete package directions
Fobbing the reader off to individual tool pages for install instructions
is unfriendly and more choice than they need. Just give them a set of
commands that installs everything.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Brett Smith [Tue, 11 Jun 2024 13:39:08 +0000 (09:39 -0400)]
21572: Remove GPG fingerprint
This information is not useful to users if we don't tell them how to
verify the fingerprint themselves.
And, that's nontrivial to do on Red Hat distros because yum downloads
and stores the key automatically.
And, there's not much reason for users to trust the fingerprint that
comes from https://doc.arvados.org more than they trust the key itself
from https://public.arvados.org.
So just remove this.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Stephen Smith [Fri, 7 Jun 2024 18:03:11 +0000 (14:03 -0400)]
21225: Allow multi select in project WF runs tab
* Adds forceMultiSelectMode to preserve existing behavior
* Consolidates negative margin styles to subToolbar wrapper element
* Wrapper element allows hiding toolbar without collapsing space
* Add subToolbar hiding when nothing selected and single select allowed
* Remove unnecessary route check now that styles are on the toolbar wrapper
Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>
Stephen Smith [Mon, 3 Jun 2024 14:48:34 +0000 (10:48 -0400)]
21225: Add project workflow runs tab.
* Moves loadMissingProcessInformation to project panel run middleware
* Removes processes from data tab
* Removes processes from data tab filters
* Adds process type filter serializer for runs tab
* Removes process-related columns from data tab
Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>
Stephen Smith [Fri, 17 May 2024 15:20:19 +0000 (11:20 -0400)]
21225: Rename project panel middleware to data middleware and separate data explorer
Also removes unnecessary dependencies on the middleware like references to
column names from other DataExplorers, shared getFilters that won't be able to
be shared in the future,
Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>
Brett Smith [Fri, 7 Jun 2024 16:11:34 +0000 (12:11 -0400)]
21363: Pin SALT_VERSION to the latest minor version
The current Salt bootstrap script seems to want this: starting from
3007, it expects to be able to find what it's looking for under a
`minor/` subdirectory, and the subdirectories of that all follow
`300X.Y/` format.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>