Skip to content

Commit

Permalink
Removes Istio bootstrap (#178)
Browse files Browse the repository at this point in the history
The maintenance and side-effects of Istio bootstrap integration have
proven to be the bulk of effort in maintaining this project, and the
bulk of its size. This removes the feature to free us to focus on Envoy,
which better aligns with the primary charter of the project.

Before:
```bash
$ du -k build/bin/*/amd64/getenvoy
77924	build/bin/darwin/amd64/getenvoy
66692	build/bin/linux/amd64/getenvoy
```

Now:
```bash
$ du -k build/bin/*/amd64/getenvoy
20768	build/bin/darwin/amd64/getenvoy
18428	build/bin/linux/amd64/getenvoy
```

Signed-off-by: Adrian Cole <adrian@tetrate.io>
  • Loading branch information
codefromthecrypt authored Apr 21, 2021
1 parent a0de163 commit 46fc4c1
Show file tree
Hide file tree
Showing 26 changed files with 122 additions and 2,552 deletions.
4 changes: 2 additions & 2 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,11 @@ linters:
- "prealloc"
- "unconvert"
- "unparam"
- "gomoddirectives"

disable:
- "gochecknoinits"
- "gochecknoglobals"
- "gomoddirectives" # Istio 1.7 dependency conflicts prevent enabling this
- "gochecknoinits"

# all available settings of specific linters
linters-settings:
Expand Down
4 changes: 0 additions & 4 deletions .licenserignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,3 @@ go.sum
.bingo
.dockerignore
.idea

# imported file from istio
envoy_bootstrap.json
configs.yaml
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ GO_TEST_EXTRA_OPTS ?=

# TODO(yskopets): include all packages into test run once blocking issues have been resolved, including
# * https://github.com/tetratelabs/getenvoy/issues/87 `go test -race` fails
COVERAGE_PKG_LIST ?= $(shell go list ./pkg/... | grep -v -e github.com/tetratelabs/getenvoy/pkg/binary/envoy/controlplane -e github.com/tetratelabs/getenvoy/pkg/binary/envoy/debug)
COVERAGE_PKG_LIST ?= $(shell go list ./pkg/... | grep -v -e github.com/tetratelabs/getenvoy/pkg/binary/envoy/debug)
GO_COVERAGE_OPTS ?= -covermode=atomic -coverpkg=./...
GO_COVERAGE_EXTRA_OPTS ?= -p 1

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
| File | Description | Purpose |
| ----------------- | ------------------------ | ----------------------------------------------------------------------- |
| `example.yaml` | `Example` descriptor | Describes runtime requirements, e.g. a specific version of `Envoy` |
| `envoy.tmpl.yaml` | `Envoy` bootstrap config | Provides `Envoy` config that demoes extension in action |
| `envoy.tmpl.yaml` | `Envoy` bootstrap config | Provides `Envoy` config that demos extension in action |
| `${EXTENSION_CONFIG_FILE_NAME}` | `Extension` config | Provides configuration for extension itself |

## Components
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
| File | Description | Purpose |
| ----------------- | ------------------------ | ----------------------------------------------------------------------- |
| `example.yaml` | `Example` descriptor | Describes runtime requirements, e.g. a specific version of `Envoy` |
| `envoy.tmpl.yaml` | `Envoy` bootstrap config | Provides `Envoy` config that demoes extension in action |
| `envoy.tmpl.yaml` | `Envoy` bootstrap config | Provides `Envoy` config that demos extension in action |
| `${EXTENSION_CONFIG_FILE_NAME}` | `Extension` config | Provides configuration for extension itself |

## Components
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
| File | Description | Purpose |
| ----------------- | ------------------------ | ----------------------------------------------------------------------- |
| `example.yaml` | `Example` descriptor | Describes runtime requirements, e.g. a specific version of `Envoy` |
| `envoy.tmpl.yaml` | `Envoy` bootstrap config | Provides `Envoy` config that demoes extension in action |
| `envoy.tmpl.yaml` | `Envoy` bootstrap config | Provides `Envoy` config that demos extension in action |
| `${EXTENSION_CONFIG_FILE_NAME}` | `Extension` config | Provides configuration for extension itself |

## Components
Expand Down
66 changes: 2 additions & 64 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -6,42 +6,18 @@ go 1.16
require (
bitbucket.org/creachadair/shell v0.0.6
github.com/Masterminds/semver/v3 v3.1.1
github.com/Microsoft/go-winio v0.4.17-0.20210211115548-6eac466e5fa3 // indirect
github.com/StackExchange/wmi v0.0.0-20210224194228-fe8f1750fd46 // indirect
github.com/andybalholm/brotli v1.0.1 // indirect
github.com/asaskevich/govalidator v0.0.0-20210307081110-f21760c49a8d
github.com/containerd/cgroups v0.0.0-20210114181951-8a68de567b68 // indirect
github.com/containerd/containerd v1.4.4
github.com/containerd/continuity v0.0.0-20210208174643-50096c924a4e // indirect
github.com/deislabs/oras v0.11.1
github.com/docker/go-metrics v0.0.1 // indirect
github.com/envoyproxy/go-control-plane v0.9.8-0.20201019204000-12785f608982
github.com/envoyproxy/protoc-gen-validate v0.5.1 // indirect
github.com/go-ole/go-ole v1.2.5 // indirect
github.com/gogo/protobuf v1.3.2 // indirect
github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect
github.com/golang/snappy v0.0.3 // indirect
github.com/hashicorp/errwrap v1.1.0 // indirect
github.com/imdario/mergo v0.3.11 // indirect
github.com/klauspost/compress v1.11.13 // indirect
github.com/klauspost/pgzip v1.2.5 // indirect
github.com/lunixbochs/vtclean v1.0.0 // indirect
github.com/envoyproxy/go-control-plane v0.9.9-0.20201210154907-fd9021fe5dad
github.com/manifoldco/promptui v0.8.0
github.com/mattn/go-colorable v0.1.8 // indirect
github.com/mattn/go-isatty v0.0.12
github.com/mattn/go-runewidth v0.0.12 // indirect
github.com/mattn/go-shellwords v1.0.11
github.com/mholt/archiver/v3 v3.5.0
github.com/mitchellh/go-homedir v1.1.0
github.com/moby/term v0.0.0-20201216013528-df9cb8a40635 // indirect
github.com/opencontainers/image-spec v1.0.1
github.com/openshift/api v3.9.1-0.20191008181517-e4fd21196097+incompatible // indirect
github.com/otiai10/copy v1.5.1
github.com/pierrec/lz4/v4 v4.1.4 // indirect
github.com/pkg/errors v0.9.1
github.com/prometheus/client_golang v1.10.0 // indirect
github.com/prometheus/common v0.20.0 // indirect
github.com/russross/blackfriday/v2 v2.1.0 // indirect
github.com/schollz/progressbar/v3 v3.7.6
github.com/shirou/gopsutil/v3 v3.21.3
github.com/spf13/cobra v1.1.3
Expand All @@ -50,45 +26,7 @@ require (
github.com/tetratelabs/multierror v1.1.0
// Match data/extension/init/templates/tinygo/*/default/go.mod_ See RATIONALE.md for why
github.com/tetratelabs/proxy-wasm-go-sdk v0.1.1
github.com/tklauser/go-sysconf v0.3.5 // indirect
github.com/ulikunitz/xz v0.5.10 // indirect
go.opencensus.io v0.23.0 // indirect
golang.org/x/crypto v0.0.0-20210322153248-0c34fe9e7dc2 // indirect
golang.org/x/net v0.0.0-20210405180319-a5a99cb37ef4 // indirect
golang.org/x/sync v0.0.0-20210220032951-036812b2e83c // indirect
golang.org/x/sys v0.0.0-20210403161142-5e06dd20ab57 // indirect
golang.org/x/term v0.0.0-20210406210042-72f3dc4e9b72 // indirect
google.golang.org/genproto v0.0.0-20210406143921-e86de6bf7a46 // indirect
google.golang.org/protobuf v1.26.0
gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b
istio.io/api v0.0.0-20210205171209-1744f93fc684
istio.io/istio v0.0.0-20210308183630-97e10d79b8b5
k8s.io/apiextensions-apiserver v0.19.9 // indirect
k8s.io/klog/v2 v2.4.0 // indirect
k8s.io/utils v0.0.0-20201110183641-67b214c5f920 // indirect
rsc.io/letsencrypt v0.0.3 // indirect
gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c
sigs.k8s.io/yaml v1.2.0
)

// Resolve import problems caused by using istio, currently istio/istio@1.8.4
// See https://github.com/istio/istio/blob/1.8.4/go.mod and go.sum
replace (
// istio 1.8.4 deps
github.com/envoyproxy/go-control-plane => github.com/envoyproxy/go-control-plane v0.9.8-0.20201019204000-12785f608982

google.golang.org/grpc => google.golang.org/grpc v1.33.3
istio.io/api => istio.io/api v0.0.0-20210205171209-1744f93fc684
istio.io/istio => istio.io/istio v0.0.0-20210308183630-97e10d79b8b5

// Latest patch of k8s version included in istio 1.8.4
k8s.io/api => k8s.io/api v0.19.9
k8s.io/apiextensions-apiserver => k8s.io/apiextensions-apiserver v0.19.9
k8s.io/apimachinery => k8s.io/apimachinery v0.19.9
k8s.io/client-go => k8s.io/client-go v0.19.9
)

// Handle ambiguous import due istio imports
exclude (
github.com/Azure/go-autorest v10.8.1+incompatible
github.com/hashicorp/consul v1.3.1
)
Loading

0 comments on commit 46fc4c1

Please sign in to comment.