Skip to content

Conversation

@kcarmichael08
Copy link
Contributor

@kcarmichael08 kcarmichael08 commented Oct 11, 2024

PLEASE READ "ADDITIONAL INFORMATION"

Version(s):
4.6+

Issue

Links to docs previews:

83406--ocpdocs-pr.netlify.app/openshift-acs/latest/architecture/acs-architecture.html
83406--ocpdocs-pr.netlify.app/openshift-acs/latest/cloud_service/acscs-architecture.html
83406--ocpdocs-pr.netlify.app/openshift-acs/latest/operating/manage-vulnerabilities/scan-rhcos-node-host.html

QE review: ACS has no QE, approved by SME

  • QE has approved this change.

Additional information:

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 11, 2024

@kcarmichael08: This pull request references ROX-26400 which is a valid jira issue.

In response to this:

Version(s):
4.6+

Issue

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 11, 2024
@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 11, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 11, 2024

@kcarmichael08: This pull request references ROX-26400 which is a valid jira issue.

In response to this:

PLEASE READ "ADDITIONAL INFORMATION"

Version(s):
4.6+

Issue

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 11, 2024

@kcarmichael08: This pull request references ROX-26400 which is a valid jira issue.

In response to this:

PLEASE READ "ADDITIONAL INFORMATION"

Version(s):
4.6+

Issue

Links to docs previews:

83406--ocpdocs-pr.netlify.app/openshift-acs/latest/architecture/acs-architecture.html
83406--ocpdocs-pr.netlify.app/openshift-acs/latest/cloud_service/acscs-architecture.html
83406--ocpdocs-pr.netlify.app/openshift-acs/latest/operating/manage-vulnerabilities/scan-rhcos-node-host.html

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@kcarmichael08 kcarmichael08 changed the title ROX-26400: Update RHACS to use VEX ROX:26400: Update RHACS to use VEX Oct 11, 2024
@openshift-ci-robot openshift-ci-robot removed the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 11, 2024
@openshift-ci-robot
Copy link

@kcarmichael08: No Jira issue is referenced in the title of this pull request.
To reference a jira issue, add 'XYZ-NNN:' to the title of this pull request and request another refresh with /jira refresh.

In response to this:

PLEASE READ "ADDITIONAL INFORMATION"

Version(s):
4.6+

Issue

Links to docs previews:

83406--ocpdocs-pr.netlify.app/openshift-acs/latest/architecture/acs-architecture.html
83406--ocpdocs-pr.netlify.app/openshift-acs/latest/cloud_service/acscs-architecture.html
83406--ocpdocs-pr.netlify.app/openshift-acs/latest/operating/manage-vulnerabilities/scan-rhcos-node-host.html

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Copy link

@RTann RTann left a comment

Choose a reason for hiding this comment

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

left some comments. Biggest thing is I'm wondering if things might be clearer if we split the sources into separate StackRox Scanner and Scanner V4 sections

@kcarmichael08 kcarmichael08 force-pushed the rox-26400-scanner-vex branch 3 times, most recently from 6de4b88 to dd2c8ec Compare October 25, 2024 17:30
@kcarmichael08 kcarmichael08 force-pushed the rox-26400-scanner-vex branch 2 times, most recently from d40d105 to 646000c Compare October 28, 2024 20:48
@kcarmichael08 kcarmichael08 added this to the Continuous Release milestone Oct 31, 2024
@kcarmichael08 kcarmichael08 force-pushed the rox-26400-scanner-vex branch 2 times, most recently from 5333603 to 090255e Compare November 13, 2024 16:45
Copy link

@RTann RTann left a comment

Choose a reason for hiding this comment

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

a few comments. Some of the URLs changed very recently so I commented the new ones

@kcarmichael08 kcarmichael08 added the peer-review-needed Signifies that the peer review team needs to review this PR label Nov 26, 2024
@kowen-rh
Copy link
Contributor

/remove-label peer-review-needed
/label peer-review-in-progress

@openshift-ci openshift-ci bot added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Nov 26, 2024
Copy link
Contributor

@kowen-rh kowen-rh left a comment

Choose a reason for hiding this comment

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

This looks great! I only have one comment, otherwise this looks good to me. 👍

/remove-label peer-review-in-progress
/label peer-review-done

+
[NOTE]
====
{product-title-short} uses the OSV database available at link:https://osv.dev/[OSV.dev] under link:https://github.com/google/osv.dev/blob/master/LICENSE[this license].
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should name it as the Apache license here.

@openshift-ci openshift-ci bot added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Nov 26, 2024
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 26, 2024
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 26, 2024
[NOTE]
====
The {product-title-short} Scanner V4 uses the OSV database available at link:https://osv.dev/[OSV.dev] under link:https://github.com/google/osv.dev/blob/master/LICENSE[this license].
The {product-title-short} Scanner V4 uses the OSV database available at link:https://osv.dev/[OSV.dev] under the link:https://github.com/google/osv.dev/blob/master/LICENSE[Apache license].
Copy link

Choose a reason for hiding this comment

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

Suggested change
The {product-title-short} Scanner V4 uses the OSV database available at link:https://osv.dev/[OSV.dev] under the link:https://github.com/google/osv.dev/blob/master/LICENSE[Apache license].
The {product-title-short} Scanner V4 uses the OSV database available at link:https://osv.dev/[OSV.dev] under link:https://github.com/google/osv.dev/blob/master/LICENSE[Apache License 2.0].

+
[NOTE]
====
{product-title-short} uses the OSV database available at link:https://osv.dev/[OSV.dev] under the link:https://github.com/google/osv.dev/blob/master/LICENSE[Apache license].
Copy link

Choose a reason for hiding this comment

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

Suggested change
{product-title-short} uses the OSV database available at link:https://osv.dev/[OSV.dev] under the link:https://github.com/google/osv.dev/blob/master/LICENSE[Apache license].
{product-title-short} uses the OSV database available at link:https://osv.dev/[OSV.dev] under link:https://github.com/google/osv.dev/blob/master/LICENSE[Apache License 2.0].

@RTann
Copy link

RTann commented Nov 26, 2024

approved for faster turnaround, but two changes requested

@kcarmichael08
Copy link
Contributor Author

approved for faster turnaround, but two changes requested

Got them in and will merge after it rebuilds.

@openshift-ci
Copy link

openshift-ci bot commented Nov 26, 2024

@kcarmichael08: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@kcarmichael08 kcarmichael08 merged commit d3957cd into openshift:rhacs-docs-main Nov 26, 2024
2 checks passed
@kcarmichael08 kcarmichael08 deleted the rox-26400-scanner-vex branch November 26, 2024 23:25
@kcarmichael08
Copy link
Contributor Author

/cherrypick rhacs-docs-4.6

@openshift-cherrypick-robot

@kcarmichael08: new pull request created: #85532

In response to this:

/cherrypick rhacs-docs-4.6

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

peer-review-done Signifies that the peer review team has reviewed this PR rhacs-docs-4.6 size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants