lakectl doctor command to run a basic diagnose on lakeFS configuration#2948

Merged

idanovo merged 32 commits intomasterfrom 2226-Lakectl-doctor-basic

Feb 21, 2022

+278-4

Commits on Feb 14, 2022

New lakectl command 'doctor' to run a basic diagnosis of lakeFS configuration.

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed command outputs

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed command outputs

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed linter comments

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed linter comments

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed a bug of ignoring the '--config' flag

Idan Novogroder
authored and
Idan Novogroder
committed
Added tests for bad config files

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed tests to pass on runners

Idan Novogroder
authored and
Idan Novogroder
committed
changed IsValidAccessKeyID to be more readable

idanovo
and
itaidavid
authored
Fixed PR review

Idan Novogroder
authored and
Idan Novogroder
committed
Changed IsContainOnlyUpperLettersOrNumber func to work with regex

Idan Novogroder
authored and
Idan Novogroder
committed

Commits on Feb 15, 2022

Fixed a test to pass on runners

Idan Novogroder
authored and
Idan Novogroder
committed
Changed IsValidAccessKeyID to work with regex

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed tests and changed 'Lakectl' func to reuse 'LakectlWithParams' code

Idan Novogroder
authored and
Idan Novogroder
committed
Checking config file only if the basic command did not work

Idan Novogroder
authored and
Idan Novogroder
committed
Added unittests and changed IsValidAccessKeyID func to be more specific about acceptble conventions

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed linter errors

Idan Novogroder
authored and
Idan Novogroder
committed
Advising the user to use command on failure

Idan Novogroder
authored and
Idan Novogroder
committed
Apply style suggestions from code review

idanovo
and
arielshaqed
authored
Added a funnc that returnn what was the error we got while analyzing the list repo command, and add a recommendation to use the doctor command when command is dying

Idan Novogroder
authored and
Idan Novogroder
committed

Commits on Feb 16, 2022

Changed text for the default config error

Idan Novogroder
authored and
Idan Novogroder
committed
small styling change

Idan Novogroder
authored and
Idan Novogroder
committed
Return if no configuration issue was found

Idan Novogroder
authored and
Idan Novogroder
committed
Fixed a test

Idan Novogroder
authored and
Idan Novogroder
committed
removed unused file

Idan Novogroder
authored and
Idan Novogroder
committed
Added a new line on output

Idan Novogroder
authored and
Idan Novogroder
committed

Commits on Feb 17, 2022

Refcator for errors messages

Idan Novogroder
authored and
Idan Novogroder
committed

Commits on Feb 20, 2022

Commits on Feb 21, 2022

empty commit
idanovo
committed