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.

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

add unset command #381

Merged
merged 2 commits into from
Jun 1, 2020
Merged

add unset command #381

merged 2 commits into from
Jun 1, 2020

Conversation

ffoysal
Copy link
Contributor

@ffoysal ffoysal commented May 30, 2020

Signed-off-by: Foysal Iqbal [email protected]

# remove the key from CR
qliksense config unset <key>

# remove the key from service inside configs/secrets of CR
qliksense config unset <service>.<key> 

# remove the service from inside configs/secrets of CR
qliksense config usnet <servcie> 

all of the above supports space separated multiple arguments

TODO: #380

Foysal Iqbal added 2 commits May 30, 2020 02:04
add unset command
b5d0570 
Signed-off-by: Foysal Iqbal <[email protected]>
add unset command
982bb83 
Signed-off-by: Foysal Iqbal <[email protected]>
@ffoysal ffoysal merged commit 9d0ac02 into master Jun 1, 2020
@ffoysal ffoysal mentioned this pull request Jun 1, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ashwathishiva ashwathishiva ashwathishiva approved these changes

@andriy-bulynko andriy-bulynko Awaiting requested review from andriy-bulynko

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ffoysal @ashwathishiva