-
Notifications
You must be signed in to change notification settings - Fork 26
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.
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
RFC: create development dir - move [setup], [qm-settings-support] to development dir #430
Comments
Thanks @dougsland |
QM is not really a desktop tool, @alexlarsson is right, write it in a osbuild format so it's preconfigured on OSTree and non-OSTree images. Sometimes we write scripts to be run interactively because it's useful for demos. QM was written for Automotive, I wouldn't worry too much about use-cases outside AutoSD right now. |
I am fine with this change. The script should not be installed in the final system. |
On Thu, May 09, 2024 at 02:47:41AM -0700, Eric Curtin wrote:
QM is not really a desktop tool, ***@***.*** is right, write it in a
osbuild format so it's preconfigured on OSTree and non-OSTree images.
Sometimes we write scripts to be run interactively because it's useful for
demos.
QM was written for Automotive, I wouldn't worry too much about use-cases
outside AutoSD right now.
+1 for me with everything Eric said :)
|
Thanks everyone, going to prepare a patch and add it to a fresh release soon. |
I think qm could support other installation methods, like someone could have their own way to install an image that is not sample-images, and then install qm in that image. However, I don't think we should spend serious engineering manpower on supporting that. The main target for qm is the automotive usecase and sample-images, so lets focus on that. That said, for development, the setup script is nice, so we still want to keep it around in git. |
One question not answered here.
So based on the above, there is an answer for the quote, It is good to maintain setup script for non automotive deployments. @ericcurtin @alexlarsson @pypingou |
No strong opinions.
Yeah sure, the interactive script doesn't have to be purged from existence or anything if it's helpful. Keep in mind though the one built by osbuild is the one most accurate to the real-life use case and there's no reason we can't run osbuild in GitHub CI in fact I tested this only a few days ago.
Again, yes sure, feel free to de-duplicate but of course there's a balance between de-duplication and making things overly complex. osbuild manifests are just python scripts so they can run/call almost anything, but we have to be careful of putting square pegs in round holes also. |
This setup script is an unofficial solution designed to deploy QM in non-automotive environments. Fixes: #430 Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
@alexlarsson is looking for have a single solution for deployment (at least for now) and move the setup and qm-settings-support to a development dir and remove both from
rpm/qm.spec
. If users/developers would like to use it, need to clone the repo and run manually.Pros:
Possible Cons:
Example:
sample-images could care of software be installed/loaded into the "brain device of the drum" but how to solve cases where users need baremetal integration too like: "plug the drums into the PC to play and simulate the drums via (usb/usb-c)?
Could be: As soon the user connect the USB cable into the PC, udev rules triggers QM/podman (or similar solution) to bring the nested containers apps for each pad independently into the user PC." (sorry for the analogy, probably there many others examples out there but I am a drummer enthusiast). :)
Question raised now:
What's the point of keeping the packaged in Fedora/CentOS if all setup/settings will be executed in sample-images project only and the setup facilitator removed? Maybe QM should be a sub-project of sample-images only in this case? Keep the RPMs in CentOS Automotive repo for sample-images get during install still make sense but Fedora, EL9?
The text was updated successfully, but these errors were encountered: