Comments (5)
This appears to be a deficiency in RunnerClient.cmd_sync()
and RunnerClient.cmd_async()
which are the methods that salt-api calls. RunnerClient.cmd()
works correctly which is what the salt-run
CLI command uses.
>>> import salt.config
>>> import salt.runner
>>> opts = salt.config.master_config('/etc/salt/master')
>>> runner = salt.runner.RunnerClient(opts)
>>> runner.cmd('test.arg', ['foo', 'bar=Bar', 'baz=True'])
{'args': ('foo',), 'kwargs': {'bar': 'Bar', 'baz': True}}
>>> runner.cmd_sync({'fun': 'test.arg', 'args': ['foo', 'bar=Bar', 'baz=True'], 'username': 'saltd
ev', 'password': 'saltdev', 'eauth': 'auto'})
{'args': ['foo', 'bar=Bar', 'baz=True'], 'kwargs': {}}
It looks like the two methods above are not processing the input through salt.utils.args.parse_input() which both LocalClient
and RunnerClient.cmd()
use.
@jfroche does this bug sound like something you can tackle from here? My available Salt time is limited at the moment. If not we can tag the core Salt team on the issue and they can add it to their sprint backlog.
from pepper.
Er. That shell session may not be 100% accurate. Pepper may be sending slightly differently low data when using --client=runner
. Must verify what Pepper is sending first, but parse_input()
is still the likely fix.
from pepper.
+1
from pepper.
So the PR with the fix is merged into Salt's develop
branch. Let's wait for its release. ;)
from pepper.
@ashald thank you very much for fixing that!
from pepper.
Related Issues (20)
- [BUG] Can't send pillar data in kwarg HOT 2
- using pepper with tokens does not honor profiles HOT 1
- Add changelog HOT 1
- Release 0.7.6 breaks --fail-any-none flag HOT 1
- Server error on jobs.lookup_jid with runner client HOT 3
- saving the response to file on MacOS add random characters HOT 4
- pepper.cmd wrapper not included in pip package
- No way to pass tgt_type to API HOT 1
- Should provide a logout function
- Unable to use --state-output=mixed --state-verbose=false with pepper HOT 2
- TLS issues are always ignored HOT 1
- Migrate Travis CI jobs to GitHub Actions HOT 1
- Installing salt-pepper 0.5.5-1 on ubuntu 20.04 failing: cli.py:474: SyntaxWarning: "is not" with a literal. Did you mean "!="?
- Makefile not working on ubuntu 18.04 dev machine
- ~/.pepperc not referred on latest 0.7.6 HOT 1
- Using pepper to get list of registered salt-minions ? HOT 2
- Fix failing test suite HOT 1
- Pepper no longer respecting --out flag HOT 5
- [CI] Remove Python 3.7 in favor of 3.11
- pytest.helpers.remove_stale_minion_key not registered
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from pepper.