Peter Amstutz [Wed, 23 Apr 2025 14:59:04 +0000 (10:59 -0400)]
22680: Add credential_scopes
I'm certain that we will want this eventually (I want to have a way to
look up what credentials to use based on the resource being accessed),
so it should just be included from the beginning.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Peter Amstutz [Fri, 18 Apr 2025 17:17:18 +0000 (13:17 -0400)]
21074: Only apply strict validation to linked workflow collections
If you have a collection of type: workflow, strict validation only
applies if it is actually linked. This accomodates legacy collections
which have type: workflow but don't have all the newly defined fields.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Brett Smith [Thu, 17 Apr 2025 15:10:58 +0000 (11:10 -0400)]
22792: Update CUDA tasks for version 570 driver
Version 570 introduces a udev rule to load the driver when needed.
`/etc/modules-load.d/nvidia.conf` is gone, and we no longer need to
arrange to load it dynamically.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Peter Amstutz [Wed, 2 Apr 2025 18:19:05 +0000 (14:19 -0400)]
21074: Linking workflow record to collection
(I have squashed a bunch of commits because of an issue of
accidentally including and then subsequently removing some unrelated
database migrations.)
Tests filling in workflow record from collection
Test for synchronization of workflow record with collection
Hide workflows where the collection is trashed
Validate properties on collection
Test destroying workflow collection
Use dependent association instead of after_destroy hook
Added include=collection_uuid to group contents API
Add collection.properties.blah table join filters
Documentation for linking workflow to collection
Figuring out manifest_text in response
Fix behavior & add tests for include=collection_uuid
Specifically need special behavior for manifest_text so it is not
returned by default and can't be selected (because Controller won't
sign it) but unsigned_manifest_text is ok.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Peter Amstutz [Mon, 7 Apr 2025 16:26:26 +0000 (12:26 -0400)]
null values fall back to zero when computing read_length
found refs #21074, original bugfix refs #22232
Null is not the same as zero, in SQL (null + a number) is null, not
the number. As a result a single null value would cause the entire
row to be considered null (and Ruby would assign it zero size)
regardless of the size of the other columns in the row. Use coalese()
to get the intended behavior.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Peter Amstutz [Wed, 9 Apr 2025 21:29:29 +0000 (17:29 -0400)]
Filter the development- prefix before sorting, refs #22767
Unfortunately 'development' sorts before numbers, so the
development-3.0.0 tag was continuing to show up at the top (resulting
in it assigning 3.1.0~dev versions even after 3.1.0 had been released
and tagged) when it should have been picking up 3.1.0 as the most
recent version. Tweak the logic to get rid of the development prefix
before sorting. With this, it correctly generates the versions as
3.2.0~dev.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Brett Smith [Tue, 15 Apr 2025 15:57:35 +0000 (11:57 -0400)]
22788: Fix getting dispatch public key from private key content
As commented, the way we tried to pass the private key to ssh-keygen via
stdin unfortunately doesn't work. ssh-keygen really wants to work with a
file, so write a tempfile to do that.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Brett Smith [Thu, 10 Apr 2025 14:37:07 +0000 (10:37 -0400)]
22617: Remove partition encryption dependency on docker.socket
Otherwise:
systemd[1]: docker.socket: Found ordering cycle on arvados-ensure-encrypted-partitions.service/start
systemd[1]: docker.socket: Found dependency on basic.target/start
systemd[1]: docker.socket: Found dependency on sockets.target/start
systemd[1]: docker.socket: Found dependency on docker.socket/start
systemd[1]: docker.socket: Job arvados-ensure-encrypted-partitions.service/start deleted to break ordering cycle
As long as we start before the actual service, that's sufficient. Just
remove the relationship with the socket.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Brett Smith [Wed, 2 Apr 2025 15:57:08 +0000 (11:57 -0400)]
22698: Bugfix incorrect shift
If no argument was passed, there will be nothing to shift, and this will
cause an error exit. Arguments after mandatory ones need to be handled
by position. Closes #22698.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Brett Smith [Tue, 1 Apr 2025 13:26:17 +0000 (09:26 -0400)]
Add trailing slash to COPY destination
According to the Dockerfile reference and the `docker build` error
message I got, this is required to copy multiple files to a destination
directory. No issue #.
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
Peter Amstutz [Mon, 31 Mar 2025 20:48:26 +0000 (16:48 -0400)]
22581: Added a "initial_path" field to published_ports
This is a relative path (and fragment, and query parameters) that
should be added to the URL when constructing the clickable link in
Workbench.
I thought of this as I was working on branch for the client side
support. For a given container service we might want to start at a
page somewhere other than the root, or we might want to provide a
query parameter to the service. This gives a way to do that.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
Stephen Smith [Sun, 30 Mar 2025 23:34:01 +0000 (19:34 -0400)]
22395: Use toggleResourceTrashed everywhere. Also detect isTrashed using items.
Technically we probably shouldn't allow a mixed group if isTrashed like before,
and with batching, it seems better to present an error for double trash/untrash
instead of performing multiple operations at once. In the future maybe we can
better tie isTrashed to the logic in the ToggleTrashAction component.
Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>