-
Notifications
You must be signed in to change notification settings - Fork 7
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
chore: update api-server to 2.2.0 #100
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have built and run the image. I saw this:
_daemon_@ac591692f9e4:/$ /bin/apiserver --config=/config --sampleconfig=/config/sample_config.json -logtostderr=true
I0620 19:05:20.802271 9 client_manager.go:170] Initializing client manager
I0620 19:05:20.802379 9 config.go:57] Config DBConfig.MySQLConfig.ExtraParams not specified, skipping
F0620 19:11:45.211662 9 error.go:425] dial tcp: lookup mysql on 172.31.0.2:53: no such host
I think the server tries to start and fails because it is waiting for mysql, so I think we're good. Thanks @orfeas-k ! I left a comment that is not super critical for this PR.
Updated since I realized I had probably compared wrong files and thought there were no changes introduced so I brought another change that was missing. |
I just pushed one more commit due to this error during rock build (wasn't here one day ago :P)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tiny comment, other than that LGMT
Co-authored-by: Daniela Plascencia <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @orfeas-k !
Changes: I also did some refactoring based on #93.
Closes #82