Skip to content

For 1.20, drop requirement to have a Service associated with a pod #13717

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
Merged
merged 1 commit into from
Sep 5, 2023
Merged

For 1.20, drop requirement to have a Service associated with a pod #13717

merged 1 commit into from
Sep 5, 2023

Conversation

howardjohn
Copy link
Member

@howardjohn howardjohn requested a review from a team as a code owner August 10, 2023 20:23
@howardjohn howardjohn added the release-notes-none Indicates a PR that does not require release notes. label Aug 10, 2023
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 10, 2023
@howardjohn howardjohn added do-not-merge/hold Block automatic merging of a PR. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 10, 2023
@ericvn
Copy link
Contributor

ericvn commented Aug 11, 2023

Since you mention 1.20, we will need to wait to merge this until after we branch at the 1.19.0 release.

This was referenced Aug 11, 2023
Merged
Merged
ericvn
ericvn previously requested changes Aug 14, 2023
View reviewed changes
Copy link
Contributor

@ericvn ericvn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently the master branch for istio.io is for the 1.19 release (until we branch it on release day. Hold this PR until then.

@ericvn ericvn dismissed their stale review September 5, 2023 21:43

Main branch is now 1.20

@dhawton dhawton removed the do-not-merge/hold Block automatic merging of a PR. label Sep 5, 2023
@dhawton
Copy link
Member

dhawton commented Sep 5, 2023

Removed DNM as master is 1.20

@istio-testing istio-testing merged commit cb25017 into istio:master Sep 5, 2023
wilsonwu added a commit to wilsonwu/istio.io that referenced this pull request Sep 6, 2023
@wilsonwu
Sync istio#13717 into Chinese
b6e7d80
@wilsonwu wilsonwu mentioned this pull request Sep 6, 2023
Merged
10 tasks
istio-testing pushed a commit that referenced this pull request Sep 7, 2023
@wilsonwu @windsonsea
[zh] Sync #13717 into Chinese (#13826)
d8417b6 
* Sync #13717 into Chinese

* Update content/zh/docs/ops/deployment/requirements/index.md

Co-authored-by: Michael <haifeng.yao@daocloud.io>

---------

Co-authored-by: Michael <haifeng.yao@daocloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dhawton dhawton dhawton approved these changes

@ericvn ericvn ericvn approved these changes

Assignees
No one assigned
Labels
release-notes-none Indicates a PR that does not require release notes.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@howardjohn @ericvn @dhawton @istio-testing