mount error - Githubissues

zhoushengsheng-droid commented 2 years ago

Hi, I recently want to mount a network disk into the container of this project. I modified KVM Py file adds this parameter -- privileged. But after adding this parameter. My JNLP file cannot be opened. How to modify it？

IngoMeyer441 commented 2 years ago

Hey, for mounting, it is probably better to use the --mount or --volume option of the Docker command. This is the link to the documentation for an NFS mount: https://docs.docker.com/storage/volumes/#create-a-service-which-creates-an-nfs-volume. There are also other examples for network mounts.

Do you get an error message? You can run nojava-ipmi-kvm with the --debug flag to get more detailed output.

zhoushengsheng-droid commented 2 years ago

Not container error. Java in the container cannot open the JNLP file I obtained. If this parameter is not added, it can be accessed normally

IngoMeyer441 commented 2 years ago

Does it work when you replace --privilged with a --mount parameter? For direct mounts you don't need the --privileged flag.

zhoushengsheng-droid commented 2 years ago

I changed KVM directly Is the PY file OK? Will it affect other uses？

IngoMeyer441 commented 2 years ago

I think it is ok to change the Python file to test the mounting. It would be better to add a command line option to nojava-ipmi-kvm to make it available in general.

zhoushengsheng-droid commented 2 years ago

Hi, I have a new question. Nojava cannot be deleted continuously by holding down the backspace key. You can only use the backspace key to click continuously to delete. Can this be optimized?

IngoMeyer441 commented 2 years ago

Thanks for pointing me to this issue. This should be fixed in commit f7a60465eeed6f655c44f5fcfeda85b75ff6b178 and version v0.9.2. Please upgrade to the latest version to get the patch.

zhoushengsheng-droid commented 2 years ago

Hi. I have another problem. If I want to use the mount function in VNC, I need another JNLP file of VM. Can nojava download one more JNLP file specified by me internally and let me open it?

IngoMeyer441 commented 2 years ago

Couldn't you use another download link in the configuration for that case?

zhoushengsheng-droid commented 2 years ago

The main reason is that the supervisor in the image configuration has only one JNLP file open. Will he download two JNLP files and open them?

IngoMeyer441 commented 2 years ago

So, you would like to set a list of JNLP files in the central configuration file and then all the given JNLP files should be started in parallel? Sorry for all the questions, but I try to understand how this feature should work and why it is needed. Maybe there is another way of solving your problem.

zhoushengsheng-droid commented 2 years ago

VM on yes

KVM on yes

The attached image of this server is divided into two parts. I mean, can the program just define one more download address, and then it can download KVM files of two addresses and pull them into the container, so as to open the two JNLP files at the same time to complete the acquisition of this server.

IngoMeyer441 commented 2 years ago

The images are not visible in the GitHub issue tracker.

It would be possible to load multiple JNLP files, but this needs quite a lot of code changes and I have no system to test this. But I always welcome code contributions. I think these changes are necessary:

The config file needs to store either a string or a list as download endpoint. Since the config file is parsed by PyYaml, it would create either a Python string or list which is passed to the JavaHostConfig class and needs to be updated to accept a list type.
The download_endpoint attribute is read in the kvm.py module and then is passed to get_java_viewer.py in the Docker container.
The get_java_viewer.py needs to be updated to accept a list for the download endpoint and then needs to handle multiple download urls in the get_java_viewer function.
The supervisord configs files (for example this one) need to be updated to allow to start more than one JNLP file. Currently, this file is static. It probably needs to be generated dynamically to add as many javaws processes as needed.

zhoushengsheng-droid commented 2 years ago

HI In other words, I need to change the "KVM. Py" file, and then I need to change the two modules "get_java_viewer. Py" in the image to complete what I said, right.

Can you tell me in detail how "get_java_viewer. Py" and "KVM. Py" present this effect by command in the system. I need to know that he manually operates the manual process to make a code upgrade design. In this way, it is better to update the Python code.

THANKS

IngoMeyer441 commented 2 years ago

The code makes a login to the KVM web page, downloads the viewer and passes it to /usr/bin/javaws. The relevant code lines (and statements or commands) are referenced in my previous post.

IngoMeyer441 commented 2 years ago

Maybe the links are not shown in the GitHub mail. But they are visible in the GitHub comment box: https://github.com/sciapp/nojava-ipmi-kvm/issues/13#issuecomment-1090095499

zhoushengsheng-droid commented 2 years ago

Sorry, I'll delete it

zhoushengsheng-droid commented 1 year ago

Hi, I can successfully mount and attach the image by adding these two parameters. "-- CAP add = sys_admin", "-- CAP add = dac_read_search". Of course, if you add modules to nojava later, you'd better take another method.

------------------ 原始邮件 ------------------ 发件人: "sciapp/nojava-ipmi-kvm" @.>; 发送时间: 2022年3月28日(星期一) 下午5:49 @.>; 抄送: "DB @.**@.>; 主题: Re: [sciapp/nojava-ipmi-kvm] mount error (Issue #13)

I think it is ok to change the Python file to test the mounting. It would be better to add a command line option to nojava-ipmi-kvm to make it available in general.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

sciapp / nojava-ipmi-kvm

mount error #13