-
Notifications
You must be signed in to change notification settings - Fork 107
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
Removes use of pulpcore.app.settings from functional tests. #964
Conversation
Attached issue: https://pulp.plan.io/issues/7690 |
c7e0c2f
to
d036244
Compare
pulpcore/tests/functional/utils.py
Outdated
if "bin/python" in line.split()[4]: | ||
bin_dir = line.split()[4].rsplit("/", maxsplit=1)[0] | ||
dynaconf = "{}/dynaconf".format(bin_dir) | ||
stdout = cli_client.run((dynaconf, "list", "-k", setting_name)).stdout |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we wrap this in a pulpcore-manager subcommand?
pulpcore-manager list-settings
might be handy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i just noticed that there is a pulpcore-manager print_settings
command.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just realized that command is only available on our dev boxes and not on production installs.
84b5935
to
b41141a
Compare
pulpcore/tests/functional/utils.py
Outdated
@@ -60,3 +61,33 @@ def delete_orphans(): | |||
"""Delete orphans through bindings.""" | |||
response = OrphansApi(core_client).delete() | |||
monitor_task(response.task) | |||
|
|||
|
|||
def get_pulp_setting(cli_client, setting_name, python_type="str"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just wondering, whether this code should live in pulp-smash.
Other plugins might need it, too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree. I'll move this code there and release a new version of pulp-smash.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes please!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
fixes: #7690
https://pulp.plan.io/issues/7690