Salt api #1
Merged
Salt api #1
Conversation
This file contains hidden or 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
the wrapper isn't customized yet but contains the basic functionality we want
used Settings-Class in salt_api_wrapper to get access to external configuration
Customize the api_wrapper and document the functions
If the job is started async we need a way to get the results
The runner should use the api-wrapper and with it the salt-api
Use grains for os_family Use other modules directly in code There's still a problem with the net.ping which causes wrong results
Manual-Parsing on Ubuntu not working Net.traceroute is instable Processing of result is missing on proxy part
I don't know how you can do this properly right now with multiple salt-minions affected by the same command
Parsing NOT checked for Debian
Not tested; Won't work on IOS right now because the users module isn't implemented right now
Only IOS tested
…nted At the moment we can't implement a lot of the functions with the salt-napalm way. Until we found a good solution it's not possible to execute them
Tested only on IOS
Devices are'nt needed anymore because salt handles it by itself
Make _extractReturn static because the Runnerobject isnt used in this method and it's only a helper
Add the first few tests for the Runner class and some refactoring for it
Change the validator to use the python interface of pykwalify instead of starting a new process Return True if no exception False if an exception was raised
Add the validation anyway to be sure that the input file is correct
Remove imports of subprocess and Device because we don't need them anymore after the refactorings
Change jobresponse to taskresponse
The way how the comparison of "lists" was made was a bit complex actually we want to compare sets instead of lists
Add processing of the results before we return it because we are only interested in the hops and not the rtt
Change the way how we format the output
Added first parts to process the tests aysnc -> first create all jobs and collect the results after
…f multiple results The new classes will help to better differenciate between the results of different minions to the same test
Refactor the class attributes to become instance attributes
Except E501 line-length
When the test failed add the information which minion returned which value to the output
Remove unused import from salt-module
Add the example config file to the repository to have an example
Change the count because tracert takes to long Add a parameter with which it's possible to change the iteration count
Add some yml examples
Update readme to cover the latest changes
Add setup.py slightly changed from the official pypi repository that astalder and dmeister created. The setup it not tested yet
Change the formatting in the readme
Change the argument of iterations to m (for maximum)
Try to get yaml displayed correctly
Get yaml displayed correctly
Rename the class TestTestSuite to TestEvaluator because it contains tests for the class Evaluator
Handle the output over the logging mechanism because console and log mostly takes the same statements. Currently the color-characters are also written into the logfile.
Add hyperlinks
marcom4rtinez
added a commit
to marcom4rtinez/nuts
that referenced
this pull request
Oct 25, 2021
marcom4rtinez
pushed a commit
to marcom4rtinez/nuts
that referenced
this pull request
Nov 24, 2021
Resolve "Test Bundle Prototyp: VLAN Test" Closes network-unit-testing-system#1 See merge request nuts-2/code!1
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.
Enhancements to nuts: