Skip to content

Conversation

@kdevops-kpd
Copy link

@kdevops-kpd kdevops-kpd bot commented Apr 21, 2025

Pull request for series with
subject: Add uniquifier to storage pool path
version: 2
url: https://patchwork.kernel.org/project/kdevops/list/?series=953185

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 21, 2025

Upstream branch: 23c9fca
series: https://patchwork.kernel.org/project/kdevops/list/?series=953185
version: 2

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 21, 2025

Upstream branch: ef5f294
series: https://patchwork.kernel.org/project/kdevops/list/?series=953185
version: 2

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 7ce773b
series: https://patchwork.kernel.org/project/kdevops/list/?series=953185
version: 2

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 1199224
series: https://patchwork.kernel.org/project/kdevops/list/?series=953185
version: 2

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 4a0beb6
series: https://patchwork.kernel.org/project/kdevops/list/?series=953185
version: 2

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 292c2a5
series: https://patchwork.kernel.org/project/kdevops/list/?series=953185
version: 2

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 24, 2025

Upstream branch: 76b19c2
series: https://patchwork.kernel.org/project/kdevops/list/?series=956655
version: 4

@kdevops-kpd kdevops-kpd bot added V4 and removed V2 labels Apr 24, 2025
@kdevops-kpd kdevops-kpd bot force-pushed the series/951631=>main branch from 709b435 to 904ab5e Compare April 24, 2025 17:23
@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 24, 2025

Upstream branch: 370657c
series: https://patchwork.kernel.org/project/kdevops/list/?series=956655
version: 4

I would like to reuse this variable name. Two places define it,
currently, but otherwise it appears to be unused.

Clean up: Remove the places that define the variable.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Prepare to replace the "kdevops" string in the storage pool path
with the name of the user account running kdevops. First step is to
define a Kconfig variable for the storage pool user. For the moment,
the storage pool user name defaults to the same string it is
currently.

I would love for there to be a way to provide an actual pathname in
CONFIG_KDEVOPS_STORAGE_POOL_PATH. I haven't found a way so far.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
I'd like to be able to run more than one instance of kdevops per
physical host. Currently the kdevops guestfs set-up steers all
storage pool activity into ${STORAGE_POOL}/kdevops/guestfs, which
means there's a good chance that two different logged-in users will
create virtual machines whose names (and block devices) conflict.

So far I haven't been able to get the other storage pool-related
settings to add sufficient uniqueness to prevent this conflict.

Instead, optionally replace the "kdevops" string in the storage pool
path with the name of the user account running kdevops. Default
behavior is to continue using "kdevops".

Suggested-by: Luis Chamberlain <mcgrof@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
@kdevops-kpd kdevops-kpd bot force-pushed the series/951631=>main branch from 904ab5e to e600c64 Compare April 24, 2025 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants