Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Endpoint to list silo users #1235

Closed
david-crespo opened this issue Jun 20, 2022 · 1 comment · Fixed by #1261
Closed

Endpoint to list silo users #1235

david-crespo opened this issue Jun 20, 2022 · 1 comment · Fixed by #1261

Comments

@david-crespo
Copy link
Contributor

The existing /users endpoint lists built-in system users. I've been using it to test my work on role assignment in the console, but I will need IdP users for it to really work.

Because we know the silo from the authenticated user, I don't think we need the silo in the path. That makes me wonder if we should use /users for silo users and rename the system users endpoint (/system_users etc).
@david-crespo david-crespo mentioned this issue Jun 20, 2022
Closed
69 tasks
@davepacheco
Copy link
Collaborator

Yeah, I do think it makes sense to have /users be for Silo users and something else for the built-in users.

@david-crespo david-crespo mentioned this issue Jun 21, 2022
Open
@davepacheco davepacheco mentioned this issue Jun 23, 2022
Merged
@leftwo leftwo mentioned this issue Mar 29, 2024
Merged
leftwo added a commit that referenced this issue Mar 29, 2024
@leftwo
Update Propolis and Crucible (#5352)
04be445 
Propolis changes:
Add `IntrPin::import_state` and migrate LPC UART pin states (#669)
Attempt to set WCE for raw file backends
Fix clippy/lint nits for rust 1.77.0

Crucible changes:
Correctly (and robustly) count bytes (#1237)
test-replay.sh fix name of DTrace script (#1235)
BlockReq -> BlockOp (#1234)
Simplify `BlockReq` (#1218)
DTrace, cmon, cleanup, retry downstairs connections at 10 seconds.
(#1231)
Remove `MAX_ACTIVE_COUNT` flow control system (#1217)

Crucible changes that were in Omicron but not in Propolis before this commit.
Return *410 Gone* if volume is inactive (#1232)
Update Rust crate opentelemetry to 0.22.0 (#1224)
Update Rust crate base64 to 0.22.0 (#1222)
Update Rust crate async-recursion to 1.1.0 (#1221)
Minor cleanups to extent implementations (#1230)
Update Rust crate http to 0.2.12 (#1220)
Update Rust crate reedline to 0.30.0 (#1227)
Update Rust crate rayon to 1.9.0 (#1226)
Update Rust crate nix to 0.28 (#1223)
Update Rust crate async-trait to 0.1.78 (#1219)
Various buffer optimizations (#1211)
Add low-level test for message encoding (#1214)
Don't let df failures ruin the buildomat tests (#1213)
Activate the NBD server's psuedo file (#1209)

---------

Co-authored-by: Alan Hanson <alan@oxide.computer>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add API for listing silo users oxidecomputer/omicron
2 participants
@davepacheco @david-crespo