-
Notifications
You must be signed in to change notification settings - Fork 29
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
Users/inakypg/fixes 0601 #50
Open
inakypg
wants to merge
17
commits into
master
Choose a base branch
from
users/inakypg/fixes-0601
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Only query passwords once, unless we do an special answer (ask) and then it splits to per-server Signed-off-by: Inaky Perez-Gonzalez <[email protected]>
Unify the handling of "how many subprocesses" to use in the servers code with a utility to guess how many we shall use. This is fed into run_fn_on_each_server() so it can take care of the guessing for the caller (who can also override it if they know). Signed-off-by: Inaky Perez-Gonzalez <[email protected]>
… inventory If we have downloaded only partial inventory for performance, we might lack the interfaces section, so watch out for that.
…n interface requested When pulling targets from inventory, only request the interface information if the target is requested to have interface information. Signed-off-by: Inaky Perez-Gonzalez <[email protected]>
Instead of printing a raw Python list.
Shall be more scalable for big deployments
…I framework Cleans up a wee bit the output in the case of multiple servers so it is not confusing.
inakypg
force-pushed
the
users/inakypg/fixes-0601
branch
from
June 2, 2023 01:57
9e62f6c
to
93f743a
Compare
Fixes an issue with KEY=VALUE (needed data passed as json = data)
…rride --all In some commands we want to force if we see or not disabled targets
Signed-off-by: Inaky Perez-Gonzalez <[email protected]>
- Remove tcfl/util.py, which had old healthcheck code - Move tcf healtcheck code to tcfl/healthcheck.py - Run under tcfl.ui_cli.run_fn_on_each_targetspec(), so it can support multiple targets, unifying arguments, etc
Fix early copything date; this started in 2014, open sourced 2016? 2017?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.