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

Change show kube command default value of insecure key to True #2517

Merged
merged 2 commits into from
Nov 30, 2022

Conversation

lixiaoyuner
Copy link
Contributor

@lixiaoyuner lixiaoyuner commented Nov 22, 2022

Signed-off-by: Yun Li yunli1@microsoft.com

What I did

Based on our new design about k8s project, change k8s related show commands default value.

How I did it

  1. Change show command insecure key default value from False to True

How to verify it

Execute "show kube server config" inside SONiC device, if insecure key real value is empty, should display True

Previous command output (if the output of a command-line utility has changed)

ip                 port            insecure          disable
----------         ----------      ------------      ---------
x.x.x.x            6443            False             False

New command output (if the output of a command-line utility has changed)

ip                 port            insecure          disable
----------         ----------      ------------      ---------
x.x.x.x            6443            True              False

Signed-off-by: Yun Li <yunli1@microsoft.com>
@lixiaoyuner lixiaoyuner marked this pull request as ready for review November 22, 2022 09:38
@losha228
Copy link

reviewing it now

Copy link

@losha228 losha228 left a comment

Choose a reason for hiding this comment

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

LGTM

@losha228 losha228 self-requested a review November 30, 2022 09:56
@lixiaoyuner lixiaoyuner merged commit 91bd6de into sonic-net:master Nov 30, 2022
yxieca pushed a commit that referenced this pull request Dec 1, 2022
* Change show kube command default value of insecure key to True

Signed-off-by: Yun Li <yunli1@microsoft.com>

* Add test cases

Signed-off-by: Yun Li <yunli1@microsoft.com>
preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Dec 6, 2022
…-net#2517)

* Change show kube command default value of insecure key to True

Signed-off-by: Yun Li <yunli1@microsoft.com>

* Add test cases

Signed-off-by: Yun Li <yunli1@microsoft.com>
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Dec 6, 2022
Update sonic-utilities submodule pointer to include the following:
* ca9a020 [generate_dump] [Mellanox] Fix the duplicate dfw dump collection problem by adding symlinks ([sonic-net#2536](sonic-net/sonic-utilities#2536))
* 92c7001 [config] Add check in config interface ip command to block if the interface is portchannel member ([sonic-net#2539](sonic-net/sonic-utilities#2539))
* e8130f5 [system-health] Improve code structure of system health CLIs ([sonic-net#2453](sonic-net/sonic-utilities#2453))
* 00c01b3 Transceiver eeprom dom CLI modification to show output from TRANSCEIVER_DOM_THRESHOLD table ([sonic-net#2535](sonic-net/sonic-utilities#2535))
* 42f51c2 sonic-utilities: Update config reload() to verify formatting of an input file ([sonic-net#2529](sonic-net/sonic-utilities#2529))
* a5e1e2b [GCU] Add RemoveCreateOnlyDependency Validator/Generator ([sonic-net#2500](sonic-net/sonic-utilities#2500))
* 6411b52 [QoS] Introduce delay to the qos reload flow ([sonic-net#2503](sonic-net/sonic-utilities#2503))
* fce7ec3 Use github code scanning instead of LGTM ([sonic-net#2530](sonic-net/sonic-utilities#2530))
* 91bd6de Change show kube command default value of insecure key to True ([sonic-net#2517](sonic-net/sonic-utilities#2517))
* c44c584 Add db_migrator_constants.py script to setup.py ([sonic-net#2534](sonic-net/sonic-utilities#2534))
* 6a3238e [drop counters] Fix CLI script for unconfigured PGs ([sonic-net#2518](sonic-net/sonic-utilities#2518))
* 263810b Update vrf add, del commands for duplicate/non-existing VRFs ([sonic-net#2467](sonic-net/sonic-utilities#2467))
* addae73 Port 202012 DB migration changes to newer branches ([sonic-net#2515](sonic-net/sonic-utilities#2515))
* 2af8cfa [VXLAN]Fixing traceback in show remotemac when mac moves during command execution ([sonic-net#2506](sonic-net/sonic-utilities#2506))

Signed-off-by: dprital <drorp@nvidia.com>
StormLiangMS pushed a commit that referenced this pull request Dec 30, 2022
* Change show kube command default value of insecure key to True

Signed-off-by: Yun Li <yunli1@microsoft.com>

* Add test cases

Signed-off-by: Yun Li <yunli1@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants