Skip to content

Ensure that states are set correctly based on remaining vuln reqs#388

Merged
md2119 merged 4 commits intomasterfrom
md/unsnooze-fix
Jan 20, 2022
Merged

Ensure that states are set correctly based on remaining vuln reqs#388
md2119 merged 4 commits intomasterfrom
md/unsnooze-fix

Conversation

@md2119
Copy link
Contributor

@md2119 md2119 commented Jan 20, 2022

Description

see title

Checklist

  • Investigated and inspected CI test results
  • Unit test and regression tests added
  • Evaluated and added CHANGELOG entry if required
  • Determined and documented upgrade steps

If any of these don't apply, please comment below.

Testing Performed

Unit test+ Manual
create request for same cve for diff images
image

verify that the cves are assoc with correct reqs:
image
image

Unsnooze one scope and verify:
image

image

image

@md2119 md2119 requested a review from theencee January 20, 2022 20:29
@md2119 md2119 enabled auto-merge (squash) January 20, 2022 20:29
@ghost
Copy link

ghost commented Jan 20, 2022

Tag for build #126683 is 3.68.x-8-g04fd8604db.

💻 For deploying this image using the dev scripts, run the following first:

export MAIN_IMAGE_TAG='3.68.x-8-g04fd8604db'

📦 You can also generate an installation bundle with:

docker run -i --rm stackrox/main:3.68.x-8-g04fd8604db central generate interactive > bundle.zip

🕹️ A roxctl binary artifact can be downloaded from CircleCI.

@md2119 md2119 disabled auto-merge January 20, 2022 21:08
@md2119 md2119 added this to the 68.0-rc.2 milestone Jan 20, 2022
@md2119 md2119 enabled auto-merge (squash) January 20, 2022 21:24
"activeState(query: String): ActiveState",
"vulnerabilityState: String!",
"effectiveVulnerabilityRequest: VulnerabilityRequest!",
"effectiveVulnerabilityRequest: VulnerabilityRequest",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

have to do this since the earlier #384 removes the restriction of non-nil response

@md2119 md2119 merged commit 750601d into master Jan 20, 2022
@md2119 md2119 deleted the md/unsnooze-fix branch January 20, 2022 21:39
vikin91 pushed a commit that referenced this pull request Jan 21, 2022
* Ensure that states are set correctly based on remaining vuln reqs

(cherry picked from commit 750601d)
RTann pushed a commit that referenced this pull request Apr 6, 2022
* Ensure that states are set correctly based on remaining vuln reqs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants