-
Notifications
You must be signed in to change notification settings - Fork 1
ROX-29531: include advisory in output #431
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
base: master
Are you sure you want to change the base?
Conversation
@janisz would you be able to help me out here? I don't know how to update the stackrox.model, so I don't know how to use the newer fields. Thanks |
@RTann it's described here https://github.com/stackrox/jenkins-plugin/blob/master/README.md#updating-api-schema |
Thanks! I missed this README and only saw the other one |
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.
This was generated by going to <staging environment>/main/apidocs
then pressing the Download
button. From there, I went to https://www.geeksforgeeks.org/json-to-yaml-converter/ and copy/pasted the JSON into the converter and saved the output to this file
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.
I also re-added the security stuff at the bottom of the file
21d1484
to
4f97d1c
Compare
Wondering if someone can help me test this. I can't seem to build locally. I haven't used Java in over 6 years, so I don't remember how to properly set this up 😄 |
Here you are a description how to run it: https://github.com/stackrox/jenkins-plugin?tab=readme-ov-file#setup |
I tested it with
You need to update js code to render new columns jenkins-plugin/stackrox-container-image-scanner/src/main/webapp/js/renderTables.js Line 1 in 5124135
|
Description