LGTM with a test tweak
https://codereview.appspot.com/58490043/diff/1/state/apiserver/client/client_test.go File state/apiserver/client/client_test.go (right):
https://codereview.appspot.com/58490043/diff/1/state/apiserver/client/client_test.go#newcode1718 state/apiserver/client/client_test.go:1718: c.Assert(script, gc.Not(jc.Contains), "apt-get") Please add the opposite check to the test above this one to ensure apt-get still gets inserted in the script when DisablePackageCommands is false
https://codereview.appspot.com/58490043/
« Back to merge proposal
LGTM with a test tweak
https:/ /codereview. appspot. com/58490043/ diff/1/ state/apiserver /client/ client_ test.go /client/ client_ test.go (right):
File state/apiserver
https:/ /codereview. appspot. com/58490043/ diff/1/ state/apiserver /client/ client_ test.go# newcode1718 /client/ client_ test.go: 1718: c.Assert(script, jc.Contains) , "apt-get") ommands is
state/apiserver
gc.Not(
Please add the opposite check to the test above this one to ensure
apt-get still gets inserted in the script when DisablePackageC
false
https:/ /codereview. appspot. com/58490043/