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

Fixes nil map bug on creation #1743

Merged
merged 4 commits into from
Nov 16, 2020
Merged

Fixes nil map bug on creation #1743

merged 4 commits into from
Nov 16, 2020

Conversation

kensipe
Copy link
Member

@kensipe kensipe commented Nov 13, 2020

Signed-off-by: Ken Sipe kensipe@gmail.com

It appears no one tested creating a new instance with this release of kudo. I don't see how it could have worked.
This was tested working on konvoy 1.6-rc4 with kudobuilder/controller:dev which was dev build with the solution.

We will look at making sure we have a test solution after we get this bug fix out.

Fixes #1742

Signed-off-by: Ken Sipe <kensipe@gmail.com>
@kensipe
Copy link
Member Author

kensipe commented Nov 13, 2020

we need to look at this... the e2e-test is failing on the bison install... but it isn't clear why.

kensipe and others added 3 commits November 13, 2020 18:19
Signed-off-by: Ken Sipe <kensipe@gmail.com>
Signed-off-by: Ken Sipe <kensipe@gmail.com>
Signed-off-by: Alena Varkockova <varkockova.a@gmail.com>
@kensipe kensipe merged commit 8ba2b02 into main Nov 16, 2020
@kensipe kensipe deleted the ken/nil-map-bug branch November 16, 2020 14:33
@kensipe
Copy link
Member Author

kensipe commented Nov 16, 2020

thanks @alenkacz !

kensipe added a commit that referenced this pull request Nov 16, 2020
Co-authored-by: Alena Varkockova <varkockova.a@gmail.com>
Signed-off-by: Ken Sipe <kensipe@gmail.com>
kensipe added a commit that referenced this pull request Nov 16, 2020
Co-authored-by: Alena Varkockova <varkockova.a@gmail.com>
Signed-off-by: Ken Sipe <kensipe@gmail.com>
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.

Nil Map Panic
2 participants