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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow more mount options #1337

Closed
jskacel opened this issue Nov 15, 2022 · 1 comment · Fixed by #1342
Closed

Allow more mount options #1337

jskacel opened this issue Nov 15, 2022 · 1 comment · Fixed by #1342
Assignees
Labels
bug Researched, reproducible, committed to fix new New issues and PRs to triaged

Comments

@jskacel
Copy link

jskacel commented Nov 15, 2022

ISSUE TYPE
  • Bug Report
SUMMARY

Not all correct podman mountpoint options are not supported (like :O).

ANSIBLE-NAVIGATOR VERSION
ansible-navigator 2.1.0
CONFIGURATION

N/A

LOG FILE
$ ansible-navigator inventory -i snow.yml --eev /etc/pki/ca-trust:/etc/pki/ca-trust:O
Warning: Issues were found while applying the settings.
   Hint: Command provided: 'inventory -i snow.yml --eev /etc/pki/ca-trust:/etc/pki/ca-trust:O'

  Error: The following execution-environment.volume-mounts entry could not be parsed: /etc/pki/ca-trust:/etc/pki/ca-trust:O (Command
         line). Errors were found: Unrecognized option: 'O', available options include 'Z' and/or 'z'.
   Hint: Try again with format <source-path>:<destination-path>:<options>'. Note: options is optional.

   Note: Configuration failed, using default log file location. (/home/testuser/ansible-navigator.log) Log level set to debug
   Hint: Review the hints and log file to see what went wrong.
STEPS TO REPRODUCE

Try to use --eev /etc/pki/ca-trust:/etc/pki/ca-trust:O'.

EXPECTED RESULTS

To pass the mountpoint how it was in version 1.1.0.

ACTUAL RESULTS

End up in error.

ADDITIONAL INFORMATION

Looks like problem is here:

@jskacel jskacel added bug Researched, reproducible, committed to fix new New issues and PRs to triaged labels Nov 15, 2022
@shatakshiiii
Copy link
Contributor

Thanks @jskacel for logging the issue here! We'll look forward adding support for other VolumeMount options.
This will be super helpful :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Researched, reproducible, committed to fix new New issues and PRs to triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants