-
Notifications
You must be signed in to change notification settings - Fork 1
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
154 vulnerability scans accessible through api or grafana #115
154 vulnerability scans accessible through api or grafana #115
Conversation
rx get vulnerability --application radix-test --copmonent frontend | ||
|
||
# Get a vulnerability scan results for job compute for an application radix-test, this also includes details about the vulnerability instead of a summary | ||
rx get vulnerability --application radix-test --job compute |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do not distinguish between component and job-component in another commands, maybe job
can be confusing with pipeline-job
. Can we use job-component
or just figure out this internally, using only component
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, we could by sending a few api calls extra, but the vulnerability api is seperate for jobs vs components?
cmd/getVulnerability.go
Outdated
os.Exit(2) | ||
} | ||
return nil | ||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary else
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With suggestions
Co-authored-by: Sergey Smolnikov <ssmol@equinor.com>
Co-authored-by: Sergey Smolnikov <ssmol@equinor.com>
…r vulnerability scans
removed job flag and kept only component |
…or vulnerability scans
… improved clarity
Summary is shown on get application and get environment, when component or job is specified, full list of vulnerabilities is shown.