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(worker): handle transient faults (#625) #626

Merged
merged 3 commits into from Sep 28, 2018
Merged

fix(worker): handle transient faults (#625) #626

merged 3 commits into from Sep 28, 2018

Conversation

jeehwancho
Copy link
Contributor

@jeehwancho jeehwancho commented Sep 10, 2018

feat(worker): handle transient faults (#625)
fix(worker): handle JSON.parse exception caused by transient faults in
watch
feat(worker): support KUBECONFIG (feedback from #596)

fix(worker): handle JSON.parse exception caused by transient faults in
watch
feat(worker): support KUBECONFIG (feedback from #596)
@technosophos
Copy link
Contributor

The code looks good, but I think we need to run a manual test.

@jeehwancho
Copy link
Contributor Author

@technosophos Any clue on when this might be tested in order to get merged? I hope to see this available in the next release if possible.

@technosophos
Copy link
Contributor

If @radu-matei has no additional concerns, I think we can mark this as good and merge it.

@technosophos
Copy link
Contributor

Ping @radu-matei

@radu-matei
Copy link
Contributor

Oups, sorry..
Yes, this looks good to me!

@technosophos technosophos merged commit d6f2a59 into brigadecore:master Sep 28, 2018
@jeehwancho jeehwancho deleted the feat-worker-retry branch October 4, 2018 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants