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.

Sign up for GitHub

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

Jump to bottom

docs: Add upgrade note regarding custom ports #17975

Merged
merged 1 commit into from
Nov 23, 2021
Merged

docs: Add upgrade note regarding custom ports #17975

merged 1 commit into from
Nov 23, 2021

Conversation

errordeveloper
Copy link
Contributor

Close #15956

@errordeveloper errordeveloper requested a review from a team as a code owner November 23, 2021 11:42
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 23, 2021
@errordeveloper errordeveloper added area/daemon Impacts operation of the Cilium daemon. area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact. labels Nov 23, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 23, 2021
@errordeveloper errordeveloper mentioned this pull request Nov 23, 2021
Closed
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.0 Nov 23, 2021
@errordeveloper
Copy link
Contributor Author

cc @nathanjsweet @christarazi @michi-covalent FYI

@qmonnet qmonnet changed the title docs: Add upgrade note rearding custom ports docs: Add upgrade note regarding custom ports Nov 23, 2021
@borkmann
Copy link
Member

Travis failed with:

docker: Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit.

@errordeveloper
docs: Add upgrade note rearding custom ports
bf77826 
Towards: cilium#15956

Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
@errordeveloper
Copy link
Contributor Author

Travis failed with:

docker: Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit.

It's know infrastructure issues, we just need to re-run...

@errordeveloper errordeveloper added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 23, 2021
@borkmann borkmann merged commit 858b5e2 into cilium:master Nov 23, 2021
@qmonnet qmonnet mentioned this pull request Nov 26, 2021
Merged
12 tasks
@joestringer joestringer added the backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. label Dec 3, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport done to v1.11 in 1.11.0 Dec 3, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport done to v1.11 in 1.11.0 Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borkmann borkmann borkmann approved these changes

@qmonnet qmonnet qmonnet approved these changes

Assignees
No one assigned
Labels
area/daemon Impacts operation of the Cilium daemon. area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.11.0
Backport done to v1.11
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

expose VXLAN and health ports as a flags
4 participants
@errordeveloper @borkmann @qmonnet @joestringer