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

Update docs and examples to use networking.k8s.io/v1 #1851

Merged
merged 11 commits into from
Aug 16, 2021

Conversation

lucacome
Copy link
Member

  • Updated docs, examples and tests to use networking.k8s.io/v1
  • Update edge pipeline to use 1.22.0

@lucacome lucacome requested a review from a team August 12, 2021 00:37
@lucacome lucacome self-assigned this Aug 12, 2021
@lucacome lucacome requested review from ciarams87 and soneillf5 and removed request for a team August 12, 2021 00:37
@github-actions github-actions bot added chore Pull requests for routine tasks documentation Pull requests/issues for documentation labels Aug 12, 2021
@lucacome lucacome force-pushed the chore/update-docs-examples-v1 branch from 3be1825 to 5680254 Compare August 12, 2021 17:10
@lucacome lucacome force-pushed the chore/update-docs-examples-v1 branch from 9d74294 to d74f6f1 Compare August 12, 2021 17:15
service:
name: tea-svc
port:
number: 80
- path: /coffee
pathType: ImplementationSpecific # default
Copy link
Contributor

Choose a reason for hiding this comment

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

it is mandatory, not default in v1, right? shall we remove the default comment?

I think we need to rewrite this doc a bit and merge New Features Available in Kubernetes 1.18 and Above into the rest of the doc, because with NIC 1.13, there will be only v1. I will create a PR for that

@lucacome lucacome enabled auto-merge (squash) August 13, 2021 16:47
@lucacome lucacome disabled auto-merge August 16, 2021 16:26
@lucacome lucacome merged commit 6a57a9a into master Aug 16, 2021
@lucacome lucacome deleted the chore/update-docs-examples-v1 branch August 16, 2021 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks documentation Pull requests/issues for documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants