Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cache race status #95

Draft
wants to merge 62 commits into
base: master
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
62 commits
Select commit Hold shift + click to select a range
0929620
fix cache race using kubernetes events
ilackarms Mar 5, 2020
18659a1
fix status update err/retry
ilackarms Mar 5, 2020
5a59050
have cache clean up files on start in test, fix test assert
ilackarms Mar 5, 2020
71b5502
fix fmt string
ilackarms Mar 5, 2020
9622657
fix retry logic for events
ilackarms Mar 5, 2020
a127547
make cache timeout configurble
ilackarms Mar 5, 2020
1a72308
fix timing of cache event
ilackarms Mar 5, 2020
8042e15
fix unit test
ilackarms Mar 5, 2020
4389e8f
fix bad assert
ilackarms Mar 5, 2020
528ec2f
add retry to gloo deployer
ilackarms Mar 5, 2020
deb23be
make the cache work with http
yuval-k Mar 19, 2020
2604814
use outbound cluster so its not localhost
yuval-k Mar 19, 2020
876fe83
when its outbound it doesnt make it there at all
yuval-k Mar 19, 2020
ec50afd
finally works
yuval-k Mar 19, 2020
d69f942
Merge remote-tracking branch 'origin/master' into fix-cache-race-status
yuval-k Mar 19, 2020
d9706b0
quay
yuval-k Mar 19, 2020
fb3f8b8
qauy
yuval-k Mar 19, 2020
c9d0438
deployment
yuval-k Mar 24, 2020
63a797e
compile
yuval-k Mar 24, 2020
7904f6e
Merge remote-tracking branch 'origin/master' into fix-cache-race-status
yuval-k Apr 28, 2020
62922a8
pr comments
yuval-k Apr 28, 2020
474371a
changelog
yuval-k Apr 28, 2020
ff2aa47
fix import cycle
yuval-k Apr 28, 2020
82ca0bb
fix test, add owner ref
yuval-k May 18, 2020
fdef179
build err
yuval-k May 18, 2020
e8ec806
don't tidy
yuval-k May 19, 2020
504aa40
pin version again
yuval-k May 19, 2020
01dfce5
pin go
yuval-k May 19, 2020
e0efc47
version error to warning
yuval-k May 19, 2020
b067d13
skv2
yuval-k May 19, 2020
34c1ccd
fix cli doc
yuval-k May 19, 2020
f28802f
fix
yuval-k May 19, 2020
f465d10
fix
yuval-k May 19, 2020
7af62c1
update istio
yuval-k May 20, 2020
f74bfca
istio artifact name
yuval-k May 20, 2020
50954e0
comment out
yuval-k May 20, 2020
f24f010
Adding changelog file to new location
Aug 12, 2020
fe7c52e
Deleting changelog file from old location
Aug 12, 2020
fd32bac
Adding changelog file to new location
Aug 13, 2020
d0e87da
Deleting changelog file from old location
Aug 13, 2020
8ffe195
Adding changelog file to new location
Aug 13, 2020
f52c3d2
Deleting changelog file from old location
Aug 13, 2020
1793734
Adding changelog file to new location
Aug 13, 2020
fb4126a
Deleting changelog file from old location
Aug 13, 2020
7101468
Adding changelog file to new location
Aug 14, 2020
a622a69
Deleting changelog file from old location
Aug 14, 2020
46694b0
Adding changelog file to new location
Aug 17, 2020
ddabcbc
Deleting changelog file from old location
Aug 17, 2020
9620581
Adding changelog file to new location
Oct 12, 2020
a73432a
Deleting changelog file from old location
Oct 12, 2020
f19b002
Adding changelog file to new location
Oct 21, 2020
8ae3eb1
Deleting changelog file from old location
Oct 21, 2020
2d4016d
Adding changelog file to new location
Oct 29, 2020
862b283
Deleting changelog file from old location
Oct 29, 2020
346bf94
Adding changelog file to new location
Oct 30, 2020
f025a6b
Deleting changelog file from old location
Oct 30, 2020
e8fde0b
Adding changelog file to new location
Nov 12, 2020
17afa4e
Deleting changelog file from old location
Nov 12, 2020
53d53ae
Adding changelog file to new location
Dec 7, 2020
ce95faf
Deleting changelog file from old location
Dec 7, 2020
66bd6c6
Adding changelog file to new location
Mar 31, 2021
58fdba6
Deleting changelog file from old location
Mar 31, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions .github/workflows/pull_request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
- name: Set up Go 1.14
uses: actions/setup-go@v2
with:
go-version: 1.14
go-version: 1.14.3
- uses: engineerd/[email protected]
with:
skipClusterCreation: "true"
Expand All @@ -41,10 +41,11 @@ jobs:
- name: Setup env
env:
KUBECTL: ${{ steps.kubectl.outputs.kubectl-path }}
ISTIO_VERSION: "1.6.0-rc.2"
run: |
curl -sSL https://get.helm.sh/helm-v3.2.0-linux-amd64.tar.gz | tar -xzf - linux-amd64/helm
curl -sSL https://github.com/istio/istio/releases/download/1.5.1/istio-1.5.1-linux.tar.gz | tar -xzf - istio-1.5.1/bin/istioctl
export PATH=$PWD/istio-1.5.1/bin:$PWD/linux-amd64:$(dirname $KUBECTL):$PATH
curl -sSL https://github.com/istio/istio/releases/download/${ISTIO_VERSION}/istio-${ISTIO_VERSION}-linux-amd64.tar.gz | tar -xzf - istio-${ISTIO_VERSION}/bin/istioctl
export PATH=$PWD/istio-${ISTIO_VERSION}/bin:$PWD/linux-amd64:$(dirname $KUBECTL):$PATH
go get -u github.com/onsi/ginkgo/ginkgo
make install-deps
./ci/setup-kind.sh
Expand Down
3 changes: 1 addition & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,7 @@ install-deps: mod-download
go get -v github.com/solo-io/[email protected]
go get -v github.com/golang/mock/[email protected]
go get -v golang.org/x/tools/cmd/[email protected]
go get -v github.com/cratonica/2goarray
go mod tidy
go get -v github.com/cratonica/2goarray@514510793eaa1ae2cc2217a9a743104312412f35



Expand Down
4 changes: 4 additions & 0 deletions changelog/v0.0.34/use-http-cache.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
changelog:
- type: FIX
description: Use HTTP cache instead of a file cache; this should solve file races and pod restart issues.
issueLink: https://github.com/solo-io/wasme/issues/104
3 changes: 2 additions & 1 deletion docs/content/reference/cli/wasme_deploy_istio.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ Note: currently only Istio 1.5.x is supported.


```
wasme deploy istio <image> --id=<unique name> [--config=<inline string>] [--root-id=<root id>] [--namespaces <comma separated namespaces>] [flags]
wasme deploy istio <image> --id=<unique name> [--config=<inline string>] [--root-id=<root id>] [--namespaces <comma separated namespaces>] [--labels deployment-label-key=value] [flags]
```

### Options
Expand All @@ -32,6 +32,7 @@ wasme deploy istio <image> --id=<unique name> [--config=<inline string>] [--root
--cache-repo string name of the image repository to use for the cache server daemonset (default "quay.io/solo-io/wasme")
--cache-tag string image tag to use for the cache server daemonset (default "dev")
--cache-timeout duration the length of time to wait for the server-side filter cache to pull the filter image before giving up with an error. set to 0 to skip the check entirely (note, this may produce a known race condition). (default 1m0s)
--dry-run print output any configuration changes to stdout rather than applying them to the target file / kubernetes cluster
-h, --help help for istio
--istio-namespace string the namespace where the Istio control plane is installed (default "istio-system")
-l, --labels stringToString labels of the deployment or daemonset into which to inject the filter. if not set, will apply to all workloads in the target namespace (default [])
Expand Down
9 changes: 5 additions & 4 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,11 @@ go 1.13
require (
github.com/avast/retry-go v2.4.3+incompatible
github.com/containerd/containerd v1.3.0
github.com/containerd/fifo v0.0.0-20200410184934-f15a3290365b // indirect
github.com/deislabs/oras v0.7.0
github.com/docker/cli v0.0.0-20191017083524-a8ff7f821017
github.com/docker/distribution v2.7.1+incompatible
github.com/docker/docker v1.13.1 // indirect
github.com/docker/docker v1.13.1
github.com/elazarl/goproxy v0.0.0-20190421051319-9d40249d3c2f // indirect
github.com/elazarl/goproxy/ext v0.0.0-20190421051319-9d40249d3c2f // indirect
github.com/envoyproxy/go-control-plane v0.9.6-0.20200401235947-be7fefdaf0df
Expand All @@ -28,14 +29,14 @@ require (
github.com/opencontainers/go-digest v1.0.0-rc1
github.com/opencontainers/image-spec v1.0.1
github.com/opencontainers/runc v1.0.0-rc9 // indirect
github.com/pkg/errors v0.8.1
github.com/pkg/errors v0.9.1
github.com/pseudomuto/protoc-gen-doc v1.3.0
github.com/pseudomuto/protokit v0.2.0
github.com/sirupsen/logrus v1.4.2
github.com/solo-io/gloo v1.3.5
github.com/solo-io/go-utils v0.15.2
github.com/solo-io/protoc-gen-ext v0.0.7
github.com/solo-io/skv2 v0.0.5
github.com/solo-io/protoc-gen-ext v0.0.9
github.com/solo-io/skv2 v0.2.2
github.com/solo-io/solo-kit v0.13.5
github.com/spf13/cobra v0.0.5
github.com/spf13/pflag v1.0.5
Expand Down
Loading