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.

Sign up for GitHub

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

Jump to bottom

Conflict in argument option --config-file with pytest 7.4.0 #565

Open
frenzymadness opened this issue Aug 9, 2023 · 3 comments · May be fixed by #566
Open

Conflict in argument option --config-file with pytest 7.4.0 #565

frenzymadness opened this issue Aug 9, 2023 · 3 comments · May be fixed by #566
Labels
SDK Issue pertains to the SDK itself and not specific to any service

Comments

@frenzymadness
Copy link

You have --config-file option in your conftest.py:

oci-python-sdk/tests/conftest.py

Line 16 in 31f38df

parser.addoption("--config-file", action="store", help="location of the config file",

and pytest 7.4.0 now has it as well: pytest-dev/pytest@4f3f36c

which results in:

argparse.ArgumentError: argument --config-file: conflicting option string: --config-file
danigm added a commit to danigm/oci-python-sdk that referenced this issue Aug 9, 2023
@danigm
test: Change --config-file to config
23874f6 
This patch changes the --config-file option to --config to avoid
conflict with pytest >= 7.4.0

Fix oracle#565
@danigm danigm linked a pull request Aug 9, 2023 that will close this issue
Open
@KartikShrikantHegde KartikShrikantHegde added the SDK Issue pertains to the SDK itself and not specific to any service label Aug 9, 2023
@jyotisaini
Copy link
Contributor

thanks for reporting the issue, we will work on updating the conftest.py

@frenzymadness
Copy link
Author

In Fedora Linux, I've solved the problem by simply renaming the option from --config-file to --config-file-path and it works fine for us as a workaround, see: https://src.fedoraproject.org/rpms/python-oci/blob/rawhide/f/python-oci.spec#_51

@paolabc
Copy link

paolabc commented Apr 20, 2024 •
edited
Loading

Hello! I would like to know if the issue has been fixed. I have the same error(pytest version 8.1.1). Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
SDK Issue pertains to the SDK itself and not specific to any service
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: Change --config-file to config danigm/oci-python-sdk
4 participants
@frenzymadness @jyotisaini @KartikShrikantHegde @paolabc