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
cleanup/ipam: Remove hostscope-legacy IPAM option #12984
Conversation
eff3310
to
a1c85a2
Compare
test-me-please |
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.
Minor docs typos/nits, otherwise LGTM
a1c85a2
to
ef9d91d
Compare
Jenkins jobs are completed successfully, force push to address review comments related to docs. |
test-me-please |
Failure seems related to #13056
|
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.
The changes in the PR looks good to me. But I wonder with the changes in the PR we don't need any documentation related to hostscope
IPAM as well?
Like removing - https://github.com/cilium/cilium/blob/master/Documentation/concepts/networking/ipam/hostscope.rst
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.
LGTM assuming everyone else's comments will be addressed.
Good catch, thanks a lot. Let me scan the docs one more time 🧐 |
Force pull for docs update, full CI is not required git diff
|
ef9d91d
to
17bb9d1
Compare
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.
LGTM 💯
To remove hostscope-legacy IPAM option Fixes: cilium#12974 Signed-off-by: Tam Mach <sayboras@yahoo.com>
17bb9d1
to
00ba9f0
Compare
test-me-please |
To remove hostscope-legacy IPAM option
Fixes: #12974
Signed-off-by: Tam Mach sayboras@yahoo.com
TODO: