You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2023/07/24 14:39:08 UTC

[kyuubi] branch branch-1.7 updated (01910087c -> 6e4eebf1c)

This is an automated email from the ASF dual-hosted git repository.

chengpan pushed a change to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


 discard 01910087c [KYUUBI #5086] [KYUUBI # 5085] Update config section of deploy on kubernetes
     new 6e4eebf1c [KYUUBI #5085] Update config section of deploy on kubernetes

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (01910087c)
            \
             N -- N -- N   refs/heads/branch-1.7 (6e4eebf1c)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:


[kyuubi] 01/01: [KYUUBI #5085] Update config section of deploy on kubernetes

Posted by ch...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

chengpan pushed a commit to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/kyuubi.git

commit 6e4eebf1c5ce9288cc3bba4e9e41b16af538240f
Author: mans2singh <ma...@yahoo.com>
AuthorDate: Sun Jul 23 19:39:14 2023 +0800

    [KYUUBI #5085] Update config section of deploy on kubernetes
    
    ### _Why are the changes needed?_
    Updated [kyuubi on kubernetes config section](https://kyuubi.readthedocs.io/en/master/deployment/kyuubi_on_kubernetes.html#config) to state <code> Kyuubi **does** not recommend using this way on Kubernetes</code>
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request
    
    Closes #5086 from mans2singh/ISSUE-5085.
    
    Closes #5086
    
    5faf0df2e [mans2singh] [KYUUBI # 5085] Update config section based on review comments
    df9f62f36 [mans2singh] [KYUUBI # 5085] Update config section of deploy on kubernetes
    
    Authored-by: mans2singh <ma...@yahoo.com>
    Signed-off-by: Cheng Pan <ch...@apache.org>
    (cherry picked from commit ecc3b4af6149ff49faff5f5360b67a6459d84856)
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 docs/deployment/kyuubi_on_kubernetes.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/deployment/kyuubi_on_kubernetes.md b/docs/deployment/kyuubi_on_kubernetes.md
index 8bb1d88c3..11ffe8e48 100644
--- a/docs/deployment/kyuubi_on_kubernetes.md
+++ b/docs/deployment/kyuubi_on_kubernetes.md
@@ -90,7 +90,7 @@ See more related details in [Using RBAC Authorization](https://kubernetes.io/doc
 
 ## Config
 
-You can configure Kyuubi the old-fashioned way by placing kyuubi-default.conf inside the image. Kyuubi do not recommend using this way on Kubernetes.
+You can configure Kyuubi the old-fashioned way by placing `kyuubi-defaults.conf` inside the image. Kyuubi does not recommend using this way on Kubernetes.
 
 Kyuubi provide `${KYUUBI_HOME}/docker/kyuubi-configmap.yaml` to build Configmap for Kyuubi.