-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Cut 1.0.1 #242
Cut 1.0.1 #242
Conversation
Is this a cherry-pick from master? If yes could you please link it here? If not could you please first fix the bug on master branch? |
@piosz yes this is a fix that is already in master. How do you want me to link to it, issue, PR or commit (or all)? |
sgtm |
@piosz do you want me to link in the changelog or just in the PR description? |
Just in PR description. |
@piosz updated the PR description to include the issue where it was reported and the fixing PR. |
@brancz thanks! |
Shall we merge this since all comments addressed. |
/assign @piosz |
/lgtm |
merge based on comment above. ping @brancz @piosz @loburm for building and pushing images. |
Backporting this as it's going to take a bit of time until we release
v1.1.0
, as there are few more things to figure out for that. But to ensure people have an official release and don't get nil panics I wanted to just backport this.The fix that is being backported is #240, which was originally reported in #239.
@fabxc @andyxning @matthiasr
I'll coordinate publishing the images with @loburm / @piosz .
/cc @smarterclayton
This change is