The original check in ScanSA assumed the service account on the pod spec
was not blank. It can be blank and when it is kubernetes will default
to using the 'default' service account. This change extends the check
logic to also include that.
* bugfix for port forward costum containers not showing
#1844
* Adjust the port forward test key structure
Co-authored-by: Eugene <eugene.berger@dell.com>
* feat: add noLatestRevCheck flag to configure whether k9s should fetch the latest rev from the Github repo.
* refactor: rename "noLatestRevCheck" to more appropriate "skipLatestRevCheck"
Add the same policy view to ServiceAccounts as users/groups.
This also works under the xray view. This commit also fixes the
issue with role/rolebindings not having the same name not showing
up in Policy view.
When checking permissions to (un)suspend cronjobs, a user will always be
considered as unauthorized, because of the wrong arguments order when
calling the `CanI` function.
For `CanI` definition, see:
2f72441bac/internal/client/client.go (L138)
```
func (a *APIClient) CanI(ns, gvr string, verbs []string) (auth bool, err error)
```
* Update CronJob version to v1 (from v1beta1, unavaliable from 1.25 onwards).
* Try both batch/v1beta1 (for legacy) and batch/v1
* Revert previous commit. Use c.GVR() to find the correct version. Make
cronjob (un)suspend version aware.
* Simply changes by dropping support for deprecated batch/v1beta1 version
* Fix path when viewing ScreenDump
* Log save should use `CurrentCluster` iso `CurrentContext`
* Remove sanitizing of dir in ScreenDump dao
* Switch `CurrentCluster` to `CurrentContext` for SD
* Revert log saving to `CurrentContext`
* Save tables to `CurrentContext` iso `CurrentCluster`
Toggling any of the log commands on and off causes the log indicator to
dance around - I assume this is because the line is centered, and when
on it is N characters long, when off N+1 characters, and so on
centering, the x-offset gets rounded.
This commit just adds an extra space after each "On" string so that
it occupies the same width as the "Off" string ... and the when
furiously toggling, the line remains blissfully static.
* Replace clipboard package
Fixes#868
* Raise error when clipboard.Write fails
* Call clipboard.Write when Init successes
* Flash error rather than panic
* Refactor copying into clipboard
* Refactor cpCmd to remove wrappers
* Use clipboardWrite instead of cpCmd
On second thought, I'll accept your PR and will update the code to use `all` as I am getting ready for a new drop and would like to include your changes. Thanks Anthony!
`var x int64` means the value is 0 rather than the intended nil
(default).
`Delete` doesn't provide a way to configure the grace period anyway so
remove the variable and only override on a `force`
Co-authored-by: Phil Kates <phil.kates@cartacapitalmarkets.com>
* working get values and get all valeus
* less verbose variable naming
* addressing comments and refactored to have a single view that has toggleable values
* better returns and logging
* returning event instead of nil
Co-authored-by: Joshua Ward <joshua.l.ward@leidos.com>
Fernand Galiana
Ruben Rodriguez
Dan Ramich
Eugene Berger
Mohamed Messaad
Audun V. Nes
Samuel Demirdjian
Grzegorz Burzyński
SammyA
fredericvl
Richard Hull
Takumi Sue
Anthony
Phil Kates
muicoder
Maurizio Pasquinelli
이태형
Tuan Anh Pham
Aleksei Romanenko
Ben Blackmore
Aleksei Romanenko
Christian Provenzano
Bryan A. S
Gabriel Simmer
Joshua Ward
tyzbit
Léo