-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Refactoring a bit to keep the CLI fast #33997
Conversation
@@ -1714,6 +1712,7 @@ class Caller(object): | |||
''' | |||
def __init__(self, c_path=os.path.join(syspaths.CONFIG_DIR, 'minion'), | |||
mopts=None): | |||
import salt.minion |
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.
Maybe add a comment here for the late load as well?
@cachedout Woo! This will be nice. I had two tiny suggestions about adding comments in some of the places you've moved imports to later in the code. Also, it looks like the SSH tests that are failing are related to this. The other two (gem test and wheel test) are currently failing on develop. |
This is an ongoing attempt to avoid the loader in the CLI.
Awesome! |
* upstream/develop: (24 commits) salt-cloud will use list_floating_ips for Openstack (saltstack#34280) Update Azure ARM driver to support 2.0rc5 (saltstack#34251) Pylint fix Refactoring a bit to keep the CLI fast (saltstack#33997) file.find module: fix handling of broken symlinks Fixed typo in filtering LDAP's potential_ous Port boto_rds to Boto3 version 1.3.1 Minor lint fix. External auth can return dynamically generated ACL from auth call. Fixes saltstack#34135, Fixes saltstack#34137 some cleanup and renaming better way to check for openSUSE Leap Fix for openSUSE Tumbleweed fixes for fopen mock and some os_release_map for SLES11SP3 unit tests for SUSE os grains detection Normalization of osfullname grain for openSUSE one clause to set OS grain from CPE_NAME Test fixed: get OS grain from /etc/os-release if possible fix: osarch_mock osarch mock for unit test ... # Conflicts: # salt/modules/boto_rds.py
My ongoing battle to keep the CLI at least somewhat lightweight.