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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃敡 Baremetal: Support alternate installation paths #1022

Open
2 tasks done
Fen-nix opened this issue Dec 25, 2023 · 4 comments
Open
2 tasks done

馃敡 Baremetal: Support alternate installation paths #1022

Fen-nix opened this issue Dec 25, 2023 · 4 comments
Labels
enhancement New feature request/PR

Comments

@Fen-nix
Copy link

Fen-nix commented Dec 25, 2023

Is this feature missing in the latest version?

  • I'm using the latest release

Is your feature request related to a problem? Please describe.

I generally tend to keep /home/ reserved for real users of the system, rather than services, so it would be nice to have the ability to have an alternate installation path.

Unfortunately it's not as simple as just changing the INSTALLPATH setting in the arm.yaml file, as many many scripts have hardcoded paths and make assumptions about where files will be located. The existence of the INSTALLPATH setting made me consider treating this as a bug rather than an enhancement.

For example, despite setting INSTALLPATH in the arm.yaml, the logger for the ripper is set up to the hardcoded path /home/arm/logs

if file:
    # create file logger handler
    file_handler = logging.FileHandler('/home/arm/logs/arm.log')

This occurs in a number of other spots as well.

Describe the solution you'd like?

Ideally, everything that launches from arm should respect the INSTALLPATH defined in arm.yaml.

Describe alternatives you've considered?

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Fen-nix Fen-nix added the enhancement New feature request/PR label Dec 25, 2023
Copy link
Contributor

If youre having issues, please remember to read the wiki and follow the instructions carefully

@microtechno9000
Copy link
Collaborator

When you are running ARM you are doing it on bare metal?
Can you clarify, are you expecting the install path to be the ARM scripts folder '/opt/arm' or the user folders '/home/arm?

@Fen-nix
Copy link
Author

Fen-nix commented Dec 26, 2023

Yes, on bare metal.

I'm expecting the INSTALLPATH or some other user-controllable variable to be able to specify the location where all ARM code exists, especially since it's installed in the steps to a common known location (/etc/arm).

e.g.: I'd equally like to be able to set some version of:

ARM_LOCATION = "/usr/lib/arm"
ARM_LOCATION = "/var/arm"
ARM_LOCATION="/srv/arm"

Or wherever I want to put the application code.

@microtechno9000
Copy link
Collaborator

Had a look through the code, something like this would be possible, using something like a .env file to get the configuration.
However, noting the way ARM is currently setup and built, whilst not a large change the /opt/arm directory is called out in multiple places. To avoid breakage, sufficient testing would be required.
I have added this issue/feature request to the ARM backlog and the Devs will work towards it in 2024.

If you have any coding experience, you are welcome to implement the change yourself and submit a PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature request/PR
Projects
Status: 馃啎 New
Development

No branches or pull requests

2 participants