-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
remove glusterfs references from the docs #37697
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
7448ae8
to
98febf7
Compare
/lgtm |
LGTM label has been added. Git tree hash: a44510df2df0040fe47240e26bbabdde997225ef
|
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 @humblec
I'd like to include glusterFS
somewhere in the volumes page, so that if people search the docs, they find a match. A single sentence explaining that Kubernetes v1.25 was the last minor release with in-tree support is enough.
Please don't update any of the pages with auto-generated: true
in their front matter. We'll cover those separately.
98febf7
to
843a391
Compare
@sftim indeed thats a good thought. I have added a note in volumes guide about the same.
sure |
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.
One more thing. Sorry.
nw @sftim 👍 , I committed those changes. ptal. thanks. |
sure @leonardpahlke . let me do one more rebase :) .. |
K8s v1.26 is scheduled for less than a week, so we need to get this done as soon as possible! 🙏 |
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
ffff118
to
4b7997a
Compare
@leonardpahlke @krol3 the PR has been rebased. we should be good to go . thanks ! |
@humblec Looks like the gate is not happy with the new commit? Could you pls try it again? |
@humblec It smells like a transient netlify build timeout issue. |
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
4b7997a
to
0da988b
Compare
/lgtm |
LGTM label has been added. Git tree hash: f4f189f879538b535bc6327e26d64da0095d22e9
|
/label tide/merge-method-squash |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tengqm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
we have removed glusterfs in-tree driver in this release (v1.26) via kubernetes/kubernetes#112749 .
This PR remove the references from the docs.