https://bugzilla.wikimedia.org/show_bug.cgi?id=65549

--- Comment #1 from Bryan Davis <[email protected]> ---
[13:15]  <    bd808>     Tin gets the deployment::target define for scap via
mediawiki::sync which is how all of the other nodes get it too.
[13:16] Ryan_Lane     nods
[13:19]  <    bd808>     Off the top of my head I can't think of a clean way to
keep Deployment::Target['scap'] from applying on tin.
[13:19]  <    bd808>     How horrible would it be to handle this edge case in
trebuchet itself?
[13:20]  <Ryan_Lane>     hm. it's not a simple edge case to workaround
[13:20]  <Ryan_Lane>     it's doable using compound matching for targeting
[13:20]  <Ryan_Lane>     you could use "scap and not deployment server" I
believe
[13:21]  <    bd808>     Ryan_Lane: Where would that go?
[13:22]  <Ryan_Lane>     inside of the runner code
[13:22]  <Ryan_Lane>     the runner code looks up the grain, then does a salt
call via the salt client api
[13:23]  <    bd808>     Ryan_Lane: Ok. `grain = "deployment_target:" + grain`
.... in deploy.py
[13:24]  <Ryan_Lane>     yep
[13:24]  <Ryan_Lane>     and client.cmd(grain, cmd, expr_form='grain', arg=arg,
...
[13:24]  <Ryan_Lane>    
http://docs.saltstack.com/en/latest/ref/clients/index.html#salt.client.LocalClient.cmd
[13:25]  <Ryan_Lane>     expr_form= <-- that would need to be changed to
compound
[13:25]  <    bd808>     yup.
[13:25]  <Ryan_Lane>    
http://docs.saltstack.com/en/latest/topics/targeting/compound.html

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to