*: add back service account

[AlexNPavel]

This adds service account support back to the sdk, as well as
updates some documentation and fixes the dep warning when creating
a new project

[estroz]

LGTM

[hasbro17]

/cc @shawn-hurley just a heads up that we'll now have a service account by default for the ansible-operator as well.

[shawn-hurley]

+1 thank you for updating this file

[shawn-hurley]

This library did not work for me. I did not see a diff with different colors. it would be easier if we just printed out the mismatched lines IMO. Is there some section of a user guide that I have not updated?

[AlexNPavel]

It does seem a bit iffy. I think my macbook wasn't showing different colors, but it seems to be working right now on my thinkpad and in travis. It might have something to do with the way that the terminals handle escapes and color. This is the the way it's handled in diffmergepatch: https://github.com/sergi/go-diff/blob/master/diffmatchpatch/diff.go#L1183

If we want to keep using this library, we can either keep it like this or change to DiffToDelta, which would have an output like this instead (the numbers after =, -, and + are characters to keep, remove, or add respectively):

--- FAIL: TestGopkgtoml (0.00s)
        gopkgtoml_test.go:33: expected vs actual differs.
                =1548   -3      =72     +    unused-packages = false%0A

[hasbro17]

@AlexNPavel Is there any way to print out only the lines that differ for expected vs actual?
The above output is a little hard to comprehend.
The colorized diff works for me but like you pointed out its dependent on the terminal settings.

[AlexNPavel]

It doesn't look like the diffmergepatch library has any tools for printing lines that differ or outputting a more diff-like output. So we're kinda stuck with the above two options unless we move to a new library or make our own helper functions, which may not be that simple.

[joelanford]

If it helps, I've been playing with this same library to get diffs out of the helm operator. Here's where I've ended up so far:

func diff(a, b string) string {
	dmp := diffmatchpatch.New()

	wSrc, wDst, warray := dmp.DiffLinesToRunes(a, b)
	diffs := dmp.DiffMainRunes(wSrc, wDst, false)
	diffs = dmp.DiffCharsToLines(diffs, warray)
	var buff bytes.Buffer
	for _, diff := range diffs {
		text := diff.Text

		switch diff.Type {
		case diffmatchpatch.DiffInsert:
			_, _ = buff.WriteString(prefixLines(text, "+"))
		case diffmatchpatch.DiffDelete:
			_, _ = buff.WriteString(prefixLines(text, "-"))
		case diffmatchpatch.DiffEqual:
			_, _ = buff.WriteString(prefixLines(text, " "))
		}
	}
	return buff.String()
}

func prefixLines(s, prefix string) string {
	var buf bytes.Buffer
	lines := strings.Split(s, "\n")
	ls := regexp.MustCompile("^")
	for _, line := range lines[:len(lines)-1] {
		buf.WriteString(ls.ReplaceAllString(line, prefix))
		buf.WriteString("\n")
	}
	return buf.String()
}

[AlexNPavel]

@joelanford that works very well. Thanks. Since this diff work is unrelated to this PR, I'll leave this PR as is and make a new PR to change that replaces the diffs in the current tests with your methods, plus color as that can improve readability in terminals that support it. The lines based method is much easier to read, especially if you're only modifying a couple characters in a line, which can become really messy with the DiffPrettyText method.

[hasbro17]

Yeah let's add in Joe's diff changes in a new PR.

[hasbro17]

could not find the manifest deploy/service_account.yaml

[hasbro17]

LGTM after nit