* Cleanup usage of go.rice in favor of go:embed

The usage of go.rice predates the availability of go:embed, but we
should switch to using go:embed instead to ship specific assets like
config files that we need.

go.rice is also incompatible with Go 1.19 and while it might see a fix
in the future, it seems better to go with the recommended Go approach
that is available these days.

Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>

* Move vtctld to also use `go embed` instead of go.rice

Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>

* Remove last rice-box related comments

Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>

* Remove config moving

This right now breaks building the actual tests since the tests might
also end up loading the regular code which has a `go embed` and refers
to the package with the config embeds.

This doesn't mean that the config isn't properly included in the
binaries. Also with using `go embed` we have a build time dependency on
the files and we always know the latest is included, so we don't have
the issue of potentially outdated files either.

All in all, it seems simplest to remove this logic and trust that Go
itself works as advertised.

Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

This reverts commit 76159fd.

Signed-off-by: Priya Bibra <pbibra@slack-corp.com>
Signed-off-by: 'Priya Bibra' <pbibra@slack-corp.com>

…vitess into upgrade-go-slack-vitess-r14.0.5

Signed-off-by: 'Stanislav Maksimov' <smaksimov@slack-corp.com>

* Empty commit to trigger CI

Signed-off-by: Eduardo J. Ortega U <5791035+ejortegau@users.noreply.github.com>

* `slack-vitess-r14.0.5-dsdefense`: use larger runner

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* Use runner group instead

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* Rename group

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* Move more jobs to runner group

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* use vitess-ubuntu20 runner group

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* Rever change of runner type for e2e ERS PRS new features heavy test

Signed-off-by: Eduardo J. Ortega U <5791035+ejortegau@users.noreply.github.com>

---------

Signed-off-by: Eduardo J. Ortega U <5791035+ejortegau@users.noreply.github.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Co-authored-by: Tim Vaillancourt <tim@timvaillancourt.com>

…_heavy. (#116)

We do this because the test is taking a lot longer & failing in some cases with
the larger runner.

Signed-off-by: Eduardo J. Ortega U <5791035+ejortegau@users.noreply.github.com>

* `slack-vitess-r14.0.5`: use dedicated larger runner

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* fix fileNameFromPosition

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* empty commit

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* empty commit

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

---------

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* `slack-vitess-r14.0.5`: allow conn overrides in consul topo

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

* fix e2e test

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

---------

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

…itess-r14.0.5