search

cloud-native-toolkit / itzcli

Go Implementation of an IBM Technology Zone (https://techzone.ibm.com) CLI
MIT License
4 stars 1 forks source link

When itz is running on a RHEL VM with SELINUX enabled you need an additonal parm added in .itz/cli-config.yaml #25

Closed slipsibm closed 1 year ago

slipsibm commented 1 year ago

When running ocpi from within itz workspace ocpinstaller you need an extra parm if you are running on RHEL with SELINUX enabled. (Which is the default). Without the parm you get an error.

type ocpi to start... (app-root) [root@docker] OCPi # ocpi Traceback (most recent call last): File "/opt/app-root/bin/ocpi", line 33, in sys.exit(load_entry_point('OCPInstaller==1.0', 'console_scripts', 'ocpi')()) File "/usr/src/ocpnow/ocpi.py", line 23, in main init() File "/usr/src/ocpnow/ocpi.py", line 12, in init projectController = ProjectController() File "/usr/src/ocpnow/controllers/ProjectController.py", line 46, in init os.makedirs(PROJECTS_DIR, exist_ok = True) File "/usr/lib64/python3.9/os.py", line 225, in makedirs mkdir(name, mode) PermissionError: [Errno 13] Permission denied: './save/projects/'

The update necessary is in .itz/cli-config.yaml. You need to add :Z on to the end of the volumes: line itz: workspace: ocpinstaller: image: quay.io/ibmtz/ocpinstaller:stable local: true name: ocp-installer type: interactive volumes:

Instructions for running ocpinstaller under podman should also be modified to reflect the need for the parm when running on RHEL. 'podman run -it \ --name workspace \ --mount type=bind,source="$(pwd)"/save,target=/usr/src/ocpnow/save:Z \ quay.io/ibmtz/ocpinstaller'

Not sure what happens if you add the flag on a non-RHEL system. For now at least an update to the install doc warning RHEL users to add the flag.

nathanagood commented 1 year ago

This is a regression--it is functionality that was once there. It used to the default to having :Z "turned on" and then would roll to not have it in the file if it didn't work. I think it's best if we add a flag to itz doctor to fix this so the user doesn't have to be worried about fixing it manually in a YAML file.

nathanagood commented 1 year ago

This should now be resolved in the upcoming v.0.1.26 release. See PR https://github.com/cloud-native-toolkit/itzcli/pull/43

nathanagood commented 1 year ago

Included in merge to main https://github.com/cloud-native-toolkit/itzcli/pull/46

nathanagood commented 1 year ago

This was released in v0.1.26.