fix: No panic on watch. Fixes #3411 #3426

alexec · 2020-07-08T18:57:54Z

Checklist:

Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
I've signed the CLA.
I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
My builds are green. Try syncing with master if they are not.
My organization is added to USERS.md.

Fixes #3411

alexec · 2020-07-08T18:59:25Z

@sarabala1979 I'd like to back-port to v2.9.3 if you can expedite review please?

fix: No panic on watch. Fixes argoproj#3411

5e0ae76

alexec requested a review from sarabala1979 July 8, 2020 18:58

alexec added the type/bug label Jul 8, 2020

alexec marked this pull request as ready for review July 8, 2020 19:17

sarabala1979 approved these changes Jul 8, 2020

View reviewed changes

alexec merged commit 910f636 into argoproj:master Jul 8, 2020

alexec deleted the fix-3411 branch July 8, 2020 19:25

alexec added a commit that referenced this pull request Jul 8, 2020

fix: No panic on watch. Fixes #3411 (#3426)

dcc09c9

Provide feedback